Problème de thread. Fenêtre et thread externe.Résolu

Question

Bonjour tout le monde !

J'aimerais vous relater un probl&#232;me qui m'arrive dans mon projet en ce moment.

J'ai une classe externe (Socket_Serveur) et j'ai ma fen&#234;tre principale.
Dans cette fen&#234;tre, j'ai une ListBox o&#249; j'aimerais afficher des infos concernant le serveur, genre : conect&#233;, envoie du message, etc ...
En respectant le principe d'une faible couplage, j'aimerais que ma classe puisse afficher ces m&#233;thodes pour n'importe quelle ListBox. Vous me suivez jusque l&#224; ?
Donc dans ma classe (Socket_Serveur), j'ai une m&#233;thode qui &#233;coute en continu si un client demande une connexion et donc veut envoyer un message au serveur. Pour ne pas surcharger le processeur, je l'ai mise dans un thread.
Donc nous voyons bien qu'il y a deux threads, un pour afficher la fen&#234;tre et son ensemble (&#233;coute d'&#233;v&#233;nements) et mon thread d'ecoute du port d&#233;sign&#233;. Suis-je compr&#233;hensible ?
Enfin, pour utiliser l'afficher sur n'importe quelle listbox, je d&#233;clare un nouveau membre qui est une listbox. A l'instanciation, je met la listbox de ma fen&#234;tre. (L'instanciation se fait bien sur dans le code de la fen&#234;tre), 
Genre : Socket_Serveur serv = new Socket_Serveur(port, this.ListeBoxe);
Et bien d&#232;s que je veux afficher, j'ai une exception : du genre crossthreading interdit. Il me dit exactement : Le thread o&#249; la listbox a &#233;t&#233; instanci&#233;e n'est pas le m&#234;me que celui o&#249; vous l'utilisez.

Est-ce assez clair, je l'esp&#232;re ...

Voil&#224;, sur cela, je vous laisse.
Malgr&#233; la longueur du message, j'esp&#232;re avoir &#233;t&#233; clair et attends une r&#233;ponse rapide de votre part (c'est assez crucial pour mon projet). 
Merci d'avance. Ivanoff.

cs_coq · Accepted Answer

exemple : 


Thread m_thread;

private void ThreadProc()
{
MessageDelegate message = new MessageDelegate(OnMessage);
object[] objs = new object[1];

for ( int i=0; i <10; i++ )
{
// ajout du message au listbox
objs[0] = i.ToString();
listBox.Invoke(message, objs);

// l&#233;g&#232;re attente
Thread.Sleep(500);
}
}

// delegate qui sera utilis&#233; pour l'"envoi" du message au controle
private delegate void MessageDelegate ( string message );

// la m&#233;thode qui fera l'insertion du message
private void OnMessage ( string message )
{
listBox.Items.Add("Message recu : " + message );
}

private void button1_Click(object sender, System.EventArgs e)
{
m_thread.Start();
}

sans oublier dans le constructeur du Form : 

m_thread = new Thread(new ThreadStart(ThreadProc));

Cocoricoooooooo !!!!
coq
MVP Visual C#

Arthenius · Answer

ben l'exception a l'air clair...
tu genere une listbox a partir de ton thread d'ecoute qui doit etre gerer par le thread d'affichage et il aime pas ca....

donc le mieux serait de t'arranger pour que ton thread d'ecoute passe le message a ton thread d'affichage du type "ca y est genere ton listbox..."

en gros...


Arthenius

 "Il n'y a pas de mauvais d&#233;veloppeurs,... 
      mais uniquement de mauvais utilisateurs..."

cs_coq · Answer

ce sujet parle du m&#234;me probl&#232;me : 
http://www.csharpfr.com/forum.v2.aspx?ID=356593
tu n'as plus qu'a adapter :-)

Cocoricoooooooo !!!!
coq
MVP Visual C#

ivanoff1 · Answer

Merci d'avoir répond aussitôt ! Le problème c'est que ça ne repond pas exactement a mon problème. Certes, cela y ressemble, mais le problème est que tout se gère dans le fenêtre. Or je veux que ce soit ma deuxième classe (Socket_Serveur) qui envoie une message à la listbox pour afficher à chaque fois que le serveur reçoit une demande de connexion un message dans la listbox. Pour être plus clair, je vais vous donner une partie de mon code : Pour la classe socket_serveur : ///

/// Permet de traiter chaque client en attente ///

protected void Sockets_Connectes() { while ( true) { this.Liste_De_Lecture.Clear(); for ( int i = 0; i < this.Ensemble_Des_Clients.Count; i++) { this.Liste_De_Lecture.Add(( Socket) this.Ensemble_Des_Clients[i]); } if ( this.Liste_De_Lecture.Count > 0) { /* DÃ©termine le statut des sockets dans la liste * Socket.Select( this.Liste_De_Lecture, null, null, 1000); /* Pour chacun d'entre eux */ for ( int i = 0; i < this.Liste_De_Lecture.Count; i++) { string Message = ""; /* Tant que l'on reÃ§oit des donnÃ©es, on les traite while ((( Socket) this.Liste_De_Lecture[i]).Available > 0) { byte[] donnees = new byte[ this.Taille_Tampon]; (( Socket) this.Liste_De_Lecture[i]).Receive(donnees); Message += Encoding.Default.GetString(donnees); } /* C EST ICI QUE JE VEUX FAIRE : * this.malistbox.items.add("Message recu : " + Message); */ } } /* On ne surcharge pas trop le processeur */ Thread.Sleep(10); } } Et dans la fenêtre Fserveur, lors du chargement de la fenêtre, on a : public Fserveur() { Socket_Serveur serv = new (numPort, tailleDonnees, this.listBoxe); } Puis lors de l'ouverture, on a juste : private void Form1_Load( object sender, EventArgs e) { serv.CommencerEcoute(); } Et dans commencer ecoute, on a la méthode citée plus haut, celle qui pose problème. Je suis désolé mais je en vois pas comment adapter la source ... En effet la méthode que je vous ai présentée est placée dans une boule infinie, donc c'est à chaque "tour", donc dès qu'un client demande une connexion, qu'on affiche un message dans la listbox. Merci d'avance, pour vos réponses, et en espérant une nouvelle réponse de votre part. Ivanoff

ivanoff1 · Answer

Merci le coq !!

Vraiment merci beaucoup !!
Je te serai eternellement reconnaissant de mavoir aider !

Commen savai tu cela ?

Ivanoff, heureux

ivanoff1 · Answer

Le bouton d'acceptation de la r&#233;ponse ne marche pas !
Pourquoi ?

ivanoff1 · Answer

Non c'est bon sa fonctionne, d&#233;sol&#233;.

cs_coq · Answer

euh ba au fur et &#224; mesure on apprend pas mal de choses lol

pour le bouton accepter si il fonctionne : regle ton anti-popup pour qu'il les laisse passer sur ce site ^^

Cocoricoooooooo !!!!
coq
MVP Visual C#

ivanoff1 · Answer

Je suis d&#233;sol&#233; de revenir &#224; la charge mais j'ai un nouveau probl&#232;me pour les threads.

Vu que je devais le r&#233;p&#233;ter dans bon nombre de classes,j'ai d&#233;cid&#233; de faire une classe sp&#233;cifique pour ce lien vers la listbox.

J'ai cr&#233;&#233; une classe m&#232;re qui est g&#233;n&#233;rale pour tout objet. Et j'ai cr&#233;&#233; une classe fille h&#233;ritant de cette sur-classe fonctionnant pour une listbox (et j'ai l'intention de le faire pour la combo, ...)

Voila je la cr&#233;e, je l'instancie dans ma classe Socket_Serveur, et l&#224; : Toujours le m&#234;me probl&#232;me d'illegal threads....

Je vous file le code :
Ma classe sp&#233;cifique &#224; l'affichage : 


public 
abstract 
class 
Controleur_Affichage 


{


/* D&#195;&#169;l&#195;&#169;gu&#195;&#169; permettant l'affichage



protected 
delegate 
void 
MessageDelegue(
string message);


/* Handle du d&#195;&#169;l&#195;&#169;gu&#195;&#169;



protected 
Controleur_Affichage.
MessageDelegue MessageD;



private 
object Controle;



public 
Controleur_Affichage(
object controle)


{



this.Mon_Controle = controle;


}



/// 



/// M&#195;&#169;thode permettant l'affichage dans le controle voulu



/// 



/// "message">Message &#195;  affiche



protected 
abstract 
void OnMessage(
string message);


 


 


/* ****************************************************************************** */


/* ************************** ACCESSEURS **************************************** */


/* ****************************************************************************** */



/// 



/// Accesseur du controle



/// 



/// 



protected 
object Mon_Controle


{



get


{



return 
this.Controle;


}



set


{



this.Controle = 
value;


}


}


}



/// 



/// Classe permettant de g&#195;&#169;rer l'affichage dans une listbox entre un thread et une For



/// 



public 
class 
ControleurAffichage_ListBox : 
Controleur_Affichage


{



/// 



/// Constructeur de la classe d&#195;&#169;riv&#195;



/// 



/// "lb">ListBox o&#195;&#185; l'on va afficher les message



public 
ControleurAffichage_ListBox(
ListBox lb) : 
base( (
object)lb)


{



base.Mon_Controle = lb;



base.MessageD = 
new 
MessageDelegue(
this.Ajouter_Message);


}



public 
void Ajouter_Message(
string Message)


{



object[] objs = 
new 
object[1];


objs.SetValue(Message, 0);



this.Ma_ListeBox.Invoke(
base.MessageD, objs);


}



/// 



/// M&#195;&#169;thode permettant l'affichage dans le controle voulu



/// 



/// "message">Message &#195;  affiche



protected 
override 
void OnMessage(
string message)


{



this.Ma_ListeBox.Items.Add(message);


}


 


 


/* ****************************************************************************** */


/* ************************** ACCESSEURS **************************************** */


/* ****************************************************************************** */



/// 



/// Permet de r&#195;&#169;cup&#195;&#169;rer la listeb



/// 



/// 



private 
ListBox Ma_ListeBox


{



get


{



return ((
ListBox) 
base.Mon_Controle);


}


}


}


}

Et voici son utilisation dans la classe Socket_Serveur : 

while(true)
{
   .....
   .....
   


this.Mon_Afficheur.Ajouter_Message(
"Essai" + i.ToString());



}

Mon afficheur est une instance de la classe fille de gestion d'affichage.
Il est instanci&#233; dans la classe Socket_Serveur (dans le constructeur).

Merci d'avance de votre aide ! 
Et encore d&#233;sol&#233; de n'&#234;tre qu'un novice dans les threads.

Pourrais-je avoir des explications quant &#224; l'explication de l'utilisation obligatoire de invoke et des d&#233;l&#233;gu&#233;s.

Merci beaucoup.

Ivanoff, inquiet....

cs_coq · Answer

Invoke permet d'effectuer l'action (dans notre cas l'ajout d'item) dans le contexte du thread auquel "appartient" le controle, et non dans notre thread.

Cocoricoooooooo !!!!
coq
MVP Visual C#

mschoum · Answer

Bonjour j'ai un petit probl&#232;me.

Voil&#224; j'ai un TreeView qui reconstruit l'arborescence de mon FileSystem.
J'ai activ&#233; l'option checkboxes du treeView.
lorsque je s&#233;lectionne un node, je stocke le chemin de ce node dans une arraylist.

j'ai &#233;galement un watcher qui &#233;coute et d&#233;tecte lorsqu'un r&#233;pertoire est cr&#233;&#233;, renomm&#233; ou supprim&#233;.

Je voudrais donc &#224; chaque fois remettre &#224; jour mon treeview. pour &#231;a j'ai suivi la m&#233;thode propos&#233; coq ci-dessus.
Le treeview se met &#224; jour sans probl&#232;me.
Le probl&#232;me est que lorsque je parcours le nouveau treeview, je fais un test du style
foreach(string s in listeRepertoireEcoute){
if(node.fullPath() == s){
    node.checked = true;
}
}

afin de cocher tous les nodes qui &#233;taient coch&#233;s avec de faire la mise &#224; jour du treeview.

Mais cela ne fonctionne pas, aucun node n'est s&#233;l&#233;ctionn&#233;. pourtant il passe bien dans la condition et passe &#233;galement sur le node.checked = true.

Quelqu'un aurait-il une id&#233;e 

Merci d'avance

Problème de thread. Fenêtre et thread externe.

11 réponses

Votre réponse

Discussions similaires