Discussion :

[Chatbour]

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 :

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

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 :

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

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.

[yan]

Salut


ca compile ton code???

Je te conseille de commencer par un petit tour sur la FAQ
http://qt.developpez.com/faq/?page=Thread

[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.

[Chatbour]

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 :

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

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" ^^).

[superjaja]

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 :

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

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 :

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

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

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

Code :

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

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.

[Chatbour]

je vois..
merci beaucoup superjaja

[yan]

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