Mon thread s’exécute séquentiellement et pas en parallèle

Version imprimable

Bonjour à tous,

j'essai de réaliser ce qui suit : lorsque je clique sur un bouton, celui-ci devient gris (setEnabled(false)), puis un traitement sera effectué, et enfin le bouton redevient accessible (setEnabled(true)) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 void MainWindow::onSketchingBtnClick() { // Classe qui dérive de QThread class OnSketchingBtnClick : public QThread { public: void run() { // Traitement processing(); // Rendre le bouton accessible de nouveau button->setEnabled(true); exec(); } }; // Rendre le bouton inaccessible button->setEnabled(false); // Lancer le traitement dans un thread OnSketchingBtnClick *thread = new OnSketchingBtnClick(this); thread->start(); cout << "done.." << endl; }
Le problème c'est que je note pas l’exécution de la méthode setEnabled(true). C'est comme si l’exécution était séquentielle : le message de cout n'est affiché qu’après la fin du traitement tout comme le code suivant :
Code:

1 2 3 4 5 6 7 // Rendre le bouton inaccessible button->setEnabled(false); // Traitement processing(); cout << "done.." << endl; // Rendre le bouton accessible de nouveau button->setEnabled(true);
Quelqu'un a-t-il une explication ? Merci d'avance,
Chatbour.

10/11/2010, 09h48
yan

Salut

:koi:
ca compile ton code???

Je te conseille de commencer par un petit tour sur la FAQ ;)
http://qt.developpez.com/faq/?page=Thread
10/11/2010, 12h47
superjaja

Comme Yan je comprend pas du tout ce que tu as fais...
Avec un peu plus d'info et un code plus propre on devrait pouvoir t'aider.

désolé en fait dans le code que j'ai fourni j'ai supprimé quelques informations afin de rendre le code plus clair, mais appariement j'ai causé le contraire. Voici le code original :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
void MainWindow::onSketchingBtnClick() {
        // Classe qui dérive de QThread
	class OnSketchingBtnClick : public QThread {
                MainWindow *window;
	public:
                OnSketchingBtnClick (MainWindow *window) {
                        this->window = window;
		}
		void run() {
                        // Traitement
                        windows->processing();
                        // Rendre le bouton accessible de nouveau
			windows->button->setEnabled(true);
			exec();
		}
	};
 
        // Rendre le bouton inaccessible
	button->setEnabled(false);
 
        // Lancer le traitement dans un thread
	OnSketchingBtnClick *thread = new OnSketchingBtnClick(this);
	thread->start();
        cout << "done.." << endl;
}

ce que je fais c'est déclarer une classe qui dérive de QThread afin d'y mettre le code que je veux exécuter en parallèle (esprit "javaïste" ^^).

en gros pour moi un truc propre serait de bien deja declarer un thread dans un .h et un Cpp.

De plus au niveau du code de ta fonction je ferais :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
void MainWindow::onSketchingBtnClick() 
{
        // Rendre le bouton inaccessible
	button->setEnabled(false);
 
        // Lancer le traitement dans un thread
       //Attention ici tu as une fuite mémoire il faut mettre la variable thread en variable membre
	OnSketchingBtnClick *thread = new OnSketchingBtnClick(this);
        //Ici je ferai un connect avec un slot perso genre updateBtn() comme ci dessous
connect(.....);
	thread->start();
 
}

Slot personalisé :

Code:

1
2
3
4
void MainWindow::updateBtn() 
{
button->setEnabled(true);
}

Pour ce qui est de la methode run je la definirai comme ceci :

Code:

1
2
3
4
5
6
7
void run() {
                        // Traitement
                        processing();
//signal qui devra etre connecté au slot updateBtn() avant la lancement du thread 
emit refreshBtn();
 
}

Je ne sais pas si ce code compile en l’état c'est juste en gros le raisonnement que j'aurai si je devais faire ton truc.

10/11/2010, 13h29
Chatbour

je vois..
merci beaucoup superjaja :)
10/11/2010, 14h15
yan

Citation:

Envoyé par Chatbour

esprit "javaïste" ^^

je m'en doutais ^^
Juste pour compléter ce qu'as dit superjaja, il ne faut surtout pas utiliser des méthodes IHM dans un thread
http://qt.developpez.com/faq/?page=Thread#ihm-thread