Ca rame sévère !Résolu

Question

Voila, g fait un programme serveur en utilisant les NetworkStream et je c pas pourquoi, d&#232;s qu'il y a 3 clients minimum, le serveur se met a ramer s&#233;v&#232;re. Alors je voulais savoir ce que vous pensez de mon code. Est-ce que c a cause de mon code ?

Voila mon code comment&#233; (merci d'avance pour les lecteurs) :


using System;

using System.Collections;

using System.IO;

using System.Net;

using System.Net.Sockets;

using System.Threading;


namespace Serveur_NetworkStream_Objets
{

//On cr&#233;&#233; un objet client
//Ainsi, on cr&#233;&#233; un tableau pour tous les clients connect&#233;s
class Client
{

public 
int Numero;

public 
bool Connecte;
//Bool&#233;en pour savoir si le client est connect&#233;
public NetworkStream NS; 
//Contient le flux r&#233;seau du client avec le serveur
public StreamReader SR; 
//Flux pour &#233;couter le client
public StreamWriter SW; 
//Flux pour parler au client

//Constructeur de l'objet client
public Client(
int New_Numero, NetworkStream New_NS)
{
Numero = New_Numero;
Connecte = 
true;

NS = New_NS;
SR = 
new StreamReader(NS);
SW = 
new StreamWriter(NS);
}



//Fonction qui &#233;coute le client et poste les messages re&#231;us sur la console
public 
void Ecouter()
{

string Message_Recu;
Console.WriteLine("Ecoute du client {0}",Numero);

while (Connecte)
//Tant que le client reste connect&#233;
{

try
   {

   //On attend un message dans le flux pour &#233;couter
   //et d&#232;s qu'il y a quelque chose, on le met dans Message_Recu
   Message_Recu=SR.ReadLine();

   Console.WriteLine("Client {0} : {1}",Numero,Message_Recu);

   }
//En cas d'erreur dans le flux, on d&#233;connecte le client
catch(Exception e)
   {
   Console.WriteLine("Erreur avec le client {0}, d&#233;connection du client !",Numero);
   Connecte = 
false;
   }
}
//Fin de tant qu'il est connecte (Connecte=true)
//Client d&#233;connect&#233;, on ferme tous les flux
SR.Close();
SW.Close();
NS.Close();
}





//Fonction qui permet d'envoyer un message au client
public 
void Envoyer(String Message)
{
SW.WriteLine(Message);
SW.Flush();
}


}



class Serveur
{

[STAThread]

static 
void Main(
string[] args)
{

int Port = 1313; 
//Num&#233;ro du port
TcpListener TCP_Serveur = 
new TcpListener(Port); 
//TCP du serveur
//R&#233;cup&#233;ration du tableau contenant les IPs du serveur (normalement une seule)
IPAddress[] Tableau_IP = (Dns.GetHostByName(Dns.GetHostName())).AddressList;
TcpClient TCP_Client; 
// TCP du client
NetworkStream NS;
Client[] Clients=
new Client[50];
//Cr&#233;ation d'un tableau de client
int Numero_Client = 0;
TCP_Serveur.Start(); 
//Lancement du TCP du serveir
Console.WriteLine("Serveur d&#233;marr&#233; sur {0}:{1}",Tableau_IP[0],Port);



while(
true)
   {

   //On attend la connexion d'un client
   TCP_Client = TCP_Serveur.AcceptTcpClient();
   NS = TCP_Client.GetStream();

   //Cr&#233;ation du client connecte
   Clients[Numero_Client]= 
new Client(Numero_Client,NS);

   //Lancement d'un thread client
   Thread T = 
new Thread(
new ThreadStart(Clients[Numero_Client].Ecouter));
   T.Start();
   Numero_Client++;
}
}
}
}

Je pensais que le pb venait du fait que je fasse un nouveau new alors que g d&#233;j&#224; cr&#233;&#233; le tableau, mais si je le fais pas, il compile pas !!!

Merci pour vos r&#233;ponses

Billou_13
Bask En Force

MorpionMx · Accepted Answer

Si tu penses que c'est le tableau qui gene, utilises plutot un Arraylist, pour lequel tu utiliseras la m&#233;thode Add().
Well, sinon, je vais essayer de regarder ca plus en profondeur.



a++
Mx

MorpionMx · Answer

J'ai test&#233; ton code, ouvert une dizaine de clients qui envoient en
boucle le meme message chaque seconde, et sinc&#233;rement, j'ai pas de
ramage :/

La seule modification que j'ai fait, c'est que dans la classe Serveur,
j'ai sorti du Main() tout le code pour le mettre dans un constructeur,
juste parce que ca fait plus beau. Mais ca m'&#233;tonnerait beaucoup que ce
soit le probleme. Et aussi, j'ai chang&#233; l'addresse du serveur par
127.0.0.1 pour tester.



D'ailleurs, il serait pr&#233;f&#233;rable que tu fasses ceci :



TcpListener TCP_Serveur = new TcpListener(Tableau_IP[0], Port); 



Car le constructeur que tu utilises pour le TCPListener est d&#233;pr&#233;ci&#233;e dans la version 1.1 du framework



Sinon, y'a rien de sp&#233;cial il me semble

Comment interviennent tes "ramages" ? Comment se font-ils ressentir ?


Mx

billou_13 · Answer

Merci beaucoup a toi !!!

Ca fait deux fois que tu r&#233;pond a mes questions, si tu continues, les autres vont se plaindre !!!
En tout cas, merci beaucoup de t'&#234;tre mis sur mon code, j'esp&#232;re que c'&#233;tait pas trop chiant !
Pour tes suggestions, je les met en oeuvre de suite.

En fait, je d&#233;veloppe sur le serveur d'une soci&#233;t&#233;, donc je pense que c les programmes du serveur qui font ramer. Faudrait que je teste sur mon ordi .
Ce que je trouve bizarre, c que on doit cr&#233;er un new tableau de 50 objets et apres on doit recr&#233;er un new objet des qu'on cr&#233;&#233; un client, c'est pas ANORMAL ca ??? (je croyais que c'etait ca qui fesait ramer)

Et une ptite question, est ce qu'on peut cr&#233;&#233; un tableau d'objet dynamique ???

Merci a toi en tout cas pour tout ce que tu as fait pour moi (et surement pour les autres)

       Bonne soir&#233;e

Billou_13
Bask En Force

MorpionMx · Answer

Pour ce qui est du tableau d'objet dynamique, c'est a ca que sert
l'ArrayList (tu n'es pas oblig&#233; de sp&#233;cifier le nombre d'objets que tu
auras dedans )



et pour ta question sur la recreation d'un objet apres la creation d'un
tableau de 50 objets, le bout de code qui suit revient au meme, et ca
te semblera peut etre plus clair



Client[] Clients=new Client[50]; // La tu alloues de la m&#233;moire pour 50 objets de type Client

...



Client c = new Client(Numero_Client,NS); // puis tu cr&#233;es un client
Clients[Numero_Client] = c; // que tu ins&#233;res dans le tableau


Ce que tu fais en faisant  Clients[Numero_Client]= new
Client(Numero_Client,NS);  est une version contract&#233;e des 2
dernieres lignes de l'exemple



Bon courage et bonne soir&#233;e a toi aussi 

Tiens nous au courant.


Mx

billou_13 · Answer

Oki, g compris, je croyais que qd je fesais :
Client[] Clients=new Client[50];
je cr&#233;&#233; d&#233;j&#224; mes objets
 , donc je trouvais bizarre de cr&#233;&#233; des objets par la suite. Mais g compris la maintenant, en fait, tu cr&#233;&#233; que l'emplacement pour le tableau.

Merci, par contre, en ce qui concerne les ArrayList, g essayer et je n'arrive pas &#224; le faire !
ArrayList Clients = new ArrayList,
ca marche mais apr&#232;s je peux pas ins&#233;rer des objets, il me dit que c pas compatible. Alors je vois pas trop la syntaxe.

        Je v tester alors, en tout cas merci a toi pour ton aide, je te tiendrais au courant !

          PS: arr&#234;te de r&#233;pondre aussi vite, je suis plus lol  (j'd&#233;conne, merci beaucoup de r&#233;pondre aussi vite !!!)

              Bonne soir&#233;e, moi je vais essayer de faire de la place sur mon ordi pour installer Visual Studio 2003. Mais ca craint, je ve pas effacer HL&#178; LOL, c un compromis !

                   @+

Billou_13
Bask En Force

MorpionMx · Answer

Pour ajouter ton objet a un Arraylist, tu fais juste tonArrayList.Add(tonObjet);

A priopri, y'a pas de souci. Ce qui donne :

            ArrayList Clients=new ArrayList();



...

                //Cr&#233;ation du client connecte

                Client c = new Client(Numero_Client,NS);

                Clients.Add(c);

                //Lancement d'un thread client

                Thread T = new Thread(new ThreadStart(c.Ecouter));

Mx

billou_13 · Answer

ok je fesais pas comme ca !!!
Merci beaucoup, je v mettre ca en place d&#232;s demain.

      Bonne soir&#233;e, je v installer Visual Studio sur mon ordi, g trouver de la place ( sans virer HL&#178; LOL)

          A demain, je te tiens au courant !!!!

Billou_13
Bask En Force

MorpionMx · Answer

a+, et bon courage. (Et merci d'avoir epargn&#233; Gordon, il a deja assze souffert le pauvre ^^ )

Mx

TheSaib · Answer

Plus rapide qu'une arraylist , utilise une hastable

::|The S@ib|::
MVP C#.NET

MorpionMx · Answer

C'est vrai que dans ce cas la, le HashTable pourrait aussi etre
pratique, comme ca tu pourrais retrouver n'importe quel objet client a
partir de son num&#233;ro (qui servirait de cl&#233;).

Et puis si c'est plus rapide, alors ce serait impec ^^

Mx

billou_13 · Answer

Merci les gars , je vais tester de suite !

Billou_13
Bask En Force

billou_13 · Answer

G fais ce que vous m'avez dit: g utiliser Hashtable et ca marche, merci !

Hashtable Clients = new Hashtable();

 

Client c = new Client(Numero_Client,NS);

Clients.Add(c,Numero_Client);





Seulement j'aimerais savoir comment parcourir tous mes objets et
envoyer un message a chacun ( une fois qu'un client se connecte par
exemple !) Je batailles dessus et je trouve pas comment acc&#232;der &#224; ma
fonction envoi a partir de la Hashtable.



je pense avec un foreach() mais apr&#232;s je c pas trop quoi mettre



        Merci d'avance,


Billou_13
Bask En Force

TheSaib · Answer

un foreach (plus lent qu'un for ou un decalage de bit) sur le values de ta HashTable

foreach(object o in hash.Values)

::|The S@ib|::
MVP C#.NET

billou_13 · Answer

Merci je teste !!! et je te dis 


Billou_13
Bask En Force

MorpionMx · Answer

Deja, a ta place je mettrais :



Clients.Add(Numero_Client, c) ;

(Num&#233;ro_client est ta cl&#233;, c est ton object).



Comme ca, si tu veux acceder au client 3, tu auras juste a faire



Client c = (Client)Clients[3]



Autrement, pour ta question d'envoyer un message a tout le monde , je
pense que ceci marche (je peux pas tester pour etre sur, mais a priori
y'a pas de souci)



foreach(client c in Clients)

    c.Envoyer(tonMessage);



Et si le foreach marche pas, tu utilises une boucle for basique, et tu recupere tes clients avec l'exemple un peu plus haut.



a++

Mx

billou_13 · Answer

Ca marche pas, il me propose pas mieux !

J'ai aussi teste avec:

foreach(object o  in Clients.Keys) //car les objets sont dans la key non ?

{

   o. // et la il me propose Equals, GetHashCode, Gettype ,
Tostring alors que c'est un object de ma table et je veux utiliser la
m&#233;thode de l'object Envoyer (pour envoyer un message a tout le monde).
Avec Values c la m&#234;me chose.

   o.Envoyer("nouveau client entr&#233;")

}



D&#233;sol&#233;,

Billou_13
Bask En Force

billou_13 · Answer

oki je teste Mx

Billou_13
Bask En Force

TheSaib · Answer

Non les cl&#233;s sont les identifiants , les objets sont dans values

::|The S@ib|::
MVP C#.NET

TheSaib · Answer

Tu cr&#233;es des evenements

public delegate void MessageReceptionEventHandler(Object sender, EventArgs e);

public event MessageReceivedEventHandler MessageRecevived;

Tu abonnes tes clients &#224; cette fonctions que tu d&#233;clares dans ta classe principale

ex :

Quand tu cr&#233;es un nouveau client dans ta classe:

monclient.MessageRecevived += new MessageReceptionEventHandler(this.TraiteMessage);

LA fonction traitemessage sera lanc&#233; a chaque fois qu'un client recevra un message, donc tu y fait ce que tu veux 

J'espere avoir &#233;t&#233; un peu clair 

PS : il me saoule ce Richtextbox 
::|The S@ib|::
MVP C#.NET

TheSaib · Answer

Il faut que tu dises aussi quand cet evenement est declanch&#233;, dans les fonctions d'ecoute tu rajoute (si l'ecoute est dans le client)

MessageRecevived(this,"le message");

::|The S@ib|::
MVP C#.NET

Ca rame sévère !

29 réponses

Votre réponse

Discussions similaires