Problème de ThreadRésolu

Question

Bonjour,
Bon je suis pas encore tout &#224; fait au point avec les Threads. La question est certainement pas tr&#232;s compliqu&#233;e.
J'ai une forme qui contient un bouton "search". Quand on clique dessus, le programme commence &#224; faire une recherche qui peut durer un certain temps (tout d&#233;pend).
Avant, tout mon code (Recherche + Affichage de la recherche) se faisait dans l'event click de mon bouton "search". Le probl&#232;me &#233;tait &#233;videmment que si la recherche prennait du temps, rien ne se passait pendant plusieurs secondes (et la fenetre de recherche &#233;tait "bloqu&#233;e", on ne peut plus la bouger).


J'ai donc cr&#233;ee une m&#233;thode Search, que je d&#233;marre dans l'event click de mon bouton avec un thread, et avant &#231;a je met un petit message dans un statusBar pour avertir l'utilsateur que la recherche a commenc&#233;e, &#231;a donne qqch comme :


// Search recherche et affiche le r&#233;sultat
myStatusBar.Text = "Recherche en cours... patientez...";
Thread threadSearch = new Thread(new ThreadStart(Search));
threadSearch.Start();


Le probl&#232;me c'est que le code suivant me retourne une erreur, il me dit qu'un contr&#244;l cr&#233;ee par un thread ne peut pas &#234;tre parent&#233; &#224; un control d'un thread different. C'est normal, puisque dans la m&#233;thode search, je recherche et ensuite j'affiche ma recherche (dans un datagrid). Donc je me dis que dans ma m&#233;thode Search, je dois juste faire la recherche, et le reste dans l'event de mon bouton "search" :


// Cette fois, search ne fait que la recherche !
myStatusBar.Text = "Recherche en cours... patientez...";
Thread threadSearch = new Thread(new ThreadStart(Search));
threadSearch.Start();
AfficheLesResultats();


J'arrive &#224; un autre probl&#232;me : AfficheLesResultats se sert &#233;videmment d'une variable globale (une collection) qui contient les r&#233;sultats. Mais cette variable n'est bien s&#251;re pas initialis&#233;e tant que la m&#233;thode Search() n'a pas &#233;t&#233; execut&#233;e. Donc j'ai un null pointer exception. J'ai essay&#233; un threadSearch.Join() juste avant la m&#233;thode AfficheLesResultats(), la &#231;a ne plante plus, mais je n'ai rien de plus qu'au d&#233;but (la form est fig&#233;e, on peut plus la bouger et c'est &#233;videmment pas le but).


J'ai d&#233;j&#224; rencontr&#233; ce probl&#232;me pleins de fois, et j'ai jamais r&#233;ussi &#224; r&#233;gler &#231;a proprement, quelqu'un peut-il m'aider? Merci !

[Pub] http://www.csharpfr.com/auteurdetail.aspx?ID=13319 [\Pub]
C# forever

MorpionMx · Accepted Answer

Voila comment je proc&#232;de. 

Contexte : Un splashScreen qui active / desactive des fonctionnalit&#233;s de l'appli selon certaines conditions



Je lance mon thread :

            t = new Thread(new ThreadStart(Start));

            t.IsBackground = true;

            t.SetApartmentState(ApartmentState.STA);

            t.Start();



Dans la m&#233;thode Start, j'ai &#231;a (Condition : connexion a la bdd a reussie ):

 if (cp.Connect())

{

    BeginInvoke(new EnableControl(mainform.SwitchInterface), new object[] { true });

    ...

}

else

{

    BeginInvoke(new EnableControl(mainform.SwitchInterface), new object[] { false });

    ...

}



le d&#233;l&#233;gu&#233; EnableControl :

        private delegate void EnableControl(bool enable);



et la m&#233;thode SwitchInterface dans ma classe MainForm :

        public void SwitchInterface(bool enable)

        {

            ...

         }



Donc j'imagine que tu pourrais faire pareil, en passant comme parametre
qqchose comme un DataSet contenant le resultat de ta recherche.

Ou alors tu as reussi d'une autre fa&#231;on ?



Mx

TheSaib · Answer

As tu regard&#233; du c&#244;t&#233; de l'asynchrone , c'est ce qu'il y a de plus simple pour g&#233;rer ce genre de cas.

::|The S@ib|::
MVP C#.NET

Lutinore · Answer

Pour eviter que ta forme "freeze" tu peux faire plus simple:



private void searchButton_Click( object sender, System.EventArgs args )
{
 searchButton.Enabled = false; // Pour ne pas entrer a nouveau dans la fonction


 while ( !found )
 {
  // Ici tu recherches etape par etape pas tout d'un coup


  Application.DoEvents( ); // Traite les evenements
 }



 searchButton.Enabled = true;
}


Sinon pour faire patienter un thread utilise l'expression "lock"

MorpionMx · Answer

J'ai aussi souvent eu ce prob de modifications inter-thread interdites.

Pour le moment, j'ai pu resoudre ces probs via  BeginInvoke().



Fais nous part de comment tu as pu resoudre ton souci, ca peut toujours servir 

Mx

cs_Bidou · Answer

D&#232;s que j'ai r&#233;solu le probl&#232;me, je te fais part de ma solution !

[Pub] http://www.csharpfr.com/auteurdetail.aspx?ID=13319 [\Pub]
C# forever

Lutinore · Answer

Je persiste a dire que tu pouvais te passer d'un second thread en utilisant la methode "DoEvents" a intervalles reguliers pendant la recherche pour ne pas freezer ta forme . Sinon as tu essaye d'initialiser ta collection avec des valeurs bidons ( a zero ) avant de lancer ton second thread ( sans la methode join ) puis de reactualiser ta collection avec les vrais valeurs une fois que ton thread est stoppe ( ThreadState.Stopped ).

Si tu as un probleme de synchronisation alors utilise le mot cle "lock" pour empecher un second thread de modifier une partie de code avant que le premier thread est fini son boulot, mais la tu vas encore avoir un probleme de blocage de ta forme.


Tu peux aussi utiliser la fonction "join" avec un parametre de temps "join( 20 )" et la tu affiches tes resultats petis a petits, pas tout d'un coup ! comme la commande Rechercher de windows dans le menu demarrer !


Le mot cle "volatile" peut aussi etre utile dans une application multithread.

cs_Bidou · Answer

Je n'en doute pas, mais l'autre solution me para&#238;t beaucoup plus propre tout de m&#234;me.
Pour faire cet appel asynchrone, on fait comment exactement?

Quand on clique sur Ok :
{
 ...
myDataGrid.BeginInvoke(myDelegates);
...
}

// Appel&#233; par le delegates
public void Test()
{
   // on cherche et on affiche ?
}

[Pub] http://www.csharpfr.com/auteurdetail.aspx?ID=13319 [\Pub]
C# forever

cs_Bidou · Answer

A vrai dire je n'ai pas encore r&#233;ussi 
Petite question sur ton code : A quel object appartient ta m&#233;thode BeginInvoke ?
(et accessoirement, t.IsBackground et t.SetApartmentState(ApartmentState.STA) servent &#224; quoi exactement ?).

[Pub] http://www.csharpfr.com/auteurdetail.aspx?ID=13319 [\Pub]
C# forever

MorpionMx · Answer

Le BeginInvoke est appel&#233; depuis la form qui sert de SplashScreen, donc j'imagine que c'est une m&#233;thode de la classe Control



Pour ce qui est du IsBackground, c'est pour sp&#233;cifier que c'est un
thread d'arriere plan, donc qui n'empeche pas un processus de s'arreter.

Et le SetApartmentState (qui est une m&#233;thode 2.0 en fait, mais dans la
version 1.1, on peut acceder a la propri&#233;t&#233; ApartmentState), bah je te
renvoies a l'aide du framework parce que comme ca c'est dur a resumer
(et a vrai dire, ca reste un peu flou pour moi)

Mx

cs_Bidou · Answer

Merci pour l'aide et l'explication !
Bon j'ai enfin r&#233;ussi &#224; faire plus ou moins marcher mon truc. Voici ma solution (bas&#233;e sur la tienne): 


private delegate void SearchDelegate(MoviesCollection mCol);


// Clique effectu&#233; sur le bouton de recherche
private void btnSearch_Click(object sender, System.EventArgs e)
{
 this.statusBarSearch.Text = "Recherche en cours... patientez...";
 // D&#233;marre la recherche dans un nouveau thread
 Thread t = new Thread(new ThreadStart(Searching));
 t.IsBackground = true;
 t.Start();
}


// Commence la recherche
private void Searching()
{
 // La collection qu'il faut remplir et qui va &#234;tre affich&#233;e dans le datagrid
 MoviesCollection moviesCol = null;
 // J'ai ici un object Search que j'ai con&#231;u qui me permet d'effectuer mes
 // recherches selon certains crit&#232;res
 Search s = new Search(_variable);
 /* Ensuite un code assez long pour rechercher, voici un aper&#231;u simplifi&#233; */
 moviesCol = s.Go();
 // On passe la collection (qui est la r&#233;ponse de la recherche) au delegate
 // que j'ai cr&#233;e, qui prend justement une telle collection en argument :-)
 object[] mCol = { moviesCol };
 myDataGrid.BeginInvoke(new SearchDelegate(Invocation), mCol);
}


// Appel asynchrone, on re&#231;oit en argument l'object pass&#233; lors de l'appel de BeginInvoke
private void Invocation(MoviesCollection moviesCol)
{
 // Ici je fais qqe traitement sur ma collection
 // Puis je remplis mon datagrid avec ma collection
}


Ceci dit il reste encore un miniscule probl&#232;me : dans la m&#233;thode Invocation, selon le r&#233;sultat de la recheche, je change la taille de ma form ou se trouve le datagrid (je l'agrandis si y'a beaucoup de r&#233;sultat, pour que ce soit plus conviviale). Ca semble marcher.
Maintenant, puisque tout est fait comme je le souhaite, je peux bouger ma forme comme j'en ai envie pendant la recheche, &#231;a bloque plus comme avant (c'&#233;tait le but :-) ). Si je relache la forme AVANT que le code passe par la m&#233;thode Invocation, pas de probl&#232;me, par contre si je bouge ma forme &#224; ce moment, alors il semble que le code ne soit pas ex&#233;cuter (la forme ne s'agrandit pas).


En tout cas, merci du coup de main.

[Pub] http://www.csharpfr.com/auteurdetail.aspx?ID=13319 [\Pub]
C# forever

Problème de Thread

10 réponses

Votre réponse

Discussions similaires