Discussion :

[iFelix]

Bonjour,

Voici la structure de mon programme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
void AuClicSurQLabel() {
Arecuperer = Uneautreclasse::algotreslong();
MAJ(Arecuperer); //Met à jour le QLabel ave le QLabel avec les infos de Arecuperer
}

Le problème c'est que l'algo est long, et du coup je me retrouve avec une fenêtre qui ne répond pas...
Je sais qu'il y a une histoire de thread mais je ne n'arrive pas à maitriser le concept...
AuClicSurQLabel() est une méthode héritée d'un QWidget donc elle doit être executé par le thread GUI ? J'aimerai libérer ce thread le temp de l'algo pour que la fenêtre ne bug pas, comment s'y prendre ?

Merci beaucoup pour votre aide !

Félix.

Edit : J'utilise Qt si ça peut simplifier des choses...

[Médinoc]

Ce qu'il faut faire, c'est un truc comme ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
void AuClicSurQLabel() {
	DemarrerUnNouveauThread(Uneautreclasse::algotreslong())
}
 
void AlgoTresLongFini(Arecuperer) {
	MAJ(Arecuperer); //Met à jour le QLabel ave le QLabel avec les infos de Arecuperer
}

Ensuite, j'ignore ce que QT offre pour qu'un thread (GUI ou non) prévienne un thread GUI.
L'API Windows offre PostMessage()/SendMessage(), .Net possède Control::Invoke()/BeginInvoke(), QT possède probablement un truc similaire à l'un ou l'autre.

[iFelix]

Merci !!
J'aimerais pouvoir essayer pour dire si ça marche mais j'ai encore une erreur...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

     std::thread t1(pointeurSurFenetre->lanceAlgo);

ou

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

std::thread t1(MainWindow::lanceAlgo);

Me renvoient

erreur : C3867: &'MainWindow::lanceAlgo'ÿ: liste d'arguments manquante dans l'appel de fonctionÿ; 
utilisez 'MainWindow::lanceAlgo' pour créer un pointeur vers membre

Alors que lanceAlgo ne prend pas d'arguments...

(j'ai suivi le modèle donné ici par ex : http://www.baptiste-wicht.com/2012/0...start-threads/ )

Edit : Bon ça c'est résolu... Il faut mettre

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

std::thread t1(&MainWindow::lanceAlgo, pointeurSurFenetre);

On va voir si ça marche pour le reste... Merci !

[iFelix]

Malheureusement, ça ne marche pas...

L'algo a le temps de tourner un tout petit peu puis j'ai une alert "abort has been called" comme si arrivé à la fin de la fonction ClicSurLabel il mettait fin à tous les threads ?

[Ehonn]

Bonjour

Ton thread est détruit avant qu'il finisse son calcul. Tu peux utiliser la méthode .join() pour attendre la fin du thread. (Selon ton code, tu auras peut être besoin de déclarer ton thread dans le bloc d'avant.)

[iFelix]

Merci pour ta réponse,

Le problème c'est que je ne veux justement pas synchroniser (mettre un join()), je récapitule la structure actuelle :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
class qui gère laffichage {
   void auClic() {
   thread t1(&MainWindow::lanceAlgo, pointeurSurFenetre);
   // Je veux sortir au plus vite de cette fonction car tant qu'on reste dedans : sablier de chargement puis ne répond pas.
   }
 
   void lanceAlgoFini() {
  // MAJ l'affichage...
   }
};

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
class MainWindow {
   void lanceAlgo() {
   // des calculs...
   Affichage->lanceAlgofini()
   }
};

Si je rajoute t1.join() juste après sa déclaration ça revient exactement à ce que je fais avant sans les threads

[Ehonn]

(Selon ton code, tu auras peut être besoin de déclarer ton thread dans le bloc d'avant.)

Apparemment tu es dans ce cas là. Dans ton exemple, tu dois déclarer le thread comme attribut de ta classe.
Tu devras faire le .join() sur le thread uniquement au dernier moment (dès que tu seras obligé d'attendre la fin du thread afin de faire autre chose).

[Médinoc]

Peut-être ne devrais-tu pas mettre ton objet thread sur la pile.

[Ehonn]

Je pense que le thread est très bien sur la pile (mais dans un bloc plus haut).
Si ton thread est autonome, tu peux aussi utiliser .detach().