Discussion :

[Matthieu76]

Bonjour à tous,

j'ai un problème très simple et pourtant je n'ai trouvé nulle part une solution.
J'ai une fenêtre Qt avec un bouton qui lance une fonction de traitement qui prends plusieurs minutes à s’exécute, le problème est que lorsque je ferme la fenêtre avant la fin du traitement la fonction ne s’arrête pas et le traitement continue en arrière plan, comment faire pour empêcher cela ?
En gros, je voudrais que fermer la fenêtre arrête la totalité de l’exécution du code.
Avez-vous déjà eu ce problème ?

Merci d'avance pour l'aide,
Matthieu

[ymoreau]

Tu lances ton traitement dans un autre thread peut être ? Si oui il faudra le stopper et attendre qu'il ait terminé avant de quitter.
Dans tous les cas tu dois stopper ton traitement quand tu détectes la fermeture de ta fenêtre (ou de l'appli). Tu peux faire ça dans void QWidget::closeEvent(QCloseEvent * event) (de ta QMainWindow), ou bien en utilisant le signal void QCoreApplication::aboutToQuit() mais comme le dit la doc c'est vraiment au dernier moment que tu vas exécuter ce code là et il n'est pas possible d'avoir une interaction utilisateur.
De manière générale il est préférable de gérer l'arrêt de ton traitement au plus près du code qui le lance, ne serait-ce que pour l'accès à l'objet qui gère le thread par exemple.

[Matthieu76]

Merci de ta réponse et désolé pour cette réponse tardive.

En gros dans mon code j'ai plein de boucle for() qui s’enchaîne quand je clique sur des boutons et donc du coup je dois créer un boolean de fermeture de ma fenêtre et le vérifier dans chaque boucle, c'est un peu "relou" à faire, non ? (mon code est assez conséquent vue que je développe une IA)

Je pensais vraiment que Qt avais une fonction qui aurait permis de kill la totalité du code lancé par une fenêtre (en gros, fermer le processus sans aller dans le gestionnaire de tache) .

Cordialement,
Matthieu

Ps: Mon thread est effectué automatiquement grâce à la fonction QObject::connect() qui relis ma fonction à l'appuis du bouton.

[ymoreau]

On peut toujours tout tuer, c'est juste qu'en général on préfère terminer proprement (en libérant des ressources, en écrivant dans des fichiers etc). Selon la tâche on peut vouloir enregistrer l'état courant et proposer de la reprendre plus tard.

C'est sûr que selon ton code tester un booléen venant du thread principal est plus ou moins pénible, il faut décider de "paliers" réguliers dans la procédure de calcul où tester le booléen et interrompre les calculs. Pour ça tu peux envisager une petite fonction qui teste la valeur et quitte le thread, pas besoin de copier coller le code du test donc, juste à appeler ta fonction à chaque palier.

Sinon tu dois pouvoir faire ce que tu veux avec void QThread::terminate().

[Matthieu76]

Merci pour ton aide.
J'ai testé ça, mais ça ne fonctionne pas.

Code :

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

1
2
3
4
5
 
void MainWindow::closeEvent(QCloseEvent *event)
{
    ui->button_start->thread()->terminate();
}

Je ne comprends pas pourquoi quelque chose d'aussi simple est aussi dur à faire, j'ai l'impression de louper un truc

[stephane78l]

Bonjour ce que tu fais avec ton code precedent c'est que tu termines le thread principal de ton programme,car le thread de ton bouton donc d'un element de l'UI est le thread d'UI donc le thread principal.
Donc comme solution,il faut que tu conserves un pointeur sur le thread secondaire que tu crees et que tu lances dans ta classe Mainwindow (comme variable membre de Mainwindow) et que tu appelles dans la methode precedente MainWindow::closeEvent la methode terminate de ce pointeur .(m_pthread->terminate(); si tu as mis le pointeur sur le thread secondaire dans m_pthread)