Discussion :

[petitclem]

Bonjour à tous,

dans mon application avec QtCreator, je démarre un QThread quand j'appui sur un bouton et ça pas de problèmes mais dans ma méthode run() de mon QThread, je démarre un timer et j'ai fait appel à un slot lorsque le timer est fini.
Lorsque le timer est fini, l'appel au slot ne se fait pas !!!

Méthode run() :

Code :

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

1
2
3
4
5
6
7
8
9
 
void ThreadSaveData::run()
{
TimerSave = new QTimer(this);
TimerSave->setSingleShot(false);
connect(TimerSave, SIGNAL(timeout()), this, SLOT(RecupDataToSave()));
UpdateSave = StringListRequeteCom.at(0).toDouble() * 2000;
TimerSave->start(UpdateSave); // Xms timeout
}

[Troudhyl]

Et le slot ThreadSaveData::RecupDataToSave() que tu connectes, il existe ?

[petitclem]

oui, il existe !

Code :

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

1
2
3
4
5
 
void ThreadSaveData::RecupDataToSave()
{
...
}

Je l'ai déclaré dans private slots dans mon .h .

[Troudhyl]

Le code est pas long, tu pourrais t'en sortir seul étant donné que y'a pas de souci particulier avec un connect dans un QThread.

StringListRequeteCom.at(0).toDouble() renvoie une bonne valeur ? Est-ce que ça marche si tu mets une valeur en dur à la place de UpdateSave (qu'il faudrait vérifier) ?

[petitclem]

J'ai mis une valeur en dur (5000) donc quand le timer est fini au bout de 5 secondes ça devrait appeler mon slot "RecupDataToSave()" ?
Cela ne change rien !!!

[gbdivers]

Lorsque le timer est fini, l'appel au slot ne se fait pas

Comment sais tu que le timer est fini ? Juste le temps écoulé ou tu as un autre slot qui te prévient ?
Comment testes tu que le slot n'est pas appelé ? Avec un qDebug() ou équivalent ?

Pour les threads, il est bien de donner aussi le code de déclaration du ThreadSaveData et de la création de l'objet.

Sinon, en première lecture, il y a 2 problèmes dans ton code :
- ThreadSaveData appartient au thread principal donc mettre this comme parent du QTimer fait que QTimer appartient aussi au thread principal
- tu lances pas exec() dans run() donc tu lances jamais ta boucle d'event du thread secondaire
En d'autres mots, ton thread ne sert à rien

De plus, pour ce genre de chose, tu devrais regarder du côté de QtConcurrent, ça devrait simplifier ta vie (si utilisable)

Pour terminer, il existe des tutos sur les threads de Qt en français, il suffit de cliquer sur le gros bouton "Tutoriels Qt" en haut de cette page...

[Troudhyl]

Sinon, en première lecture, il y a 2 problèmes dans ton code :
- ThreadSaveData appartient au thread principal donc mettre this comme parent du QTimer fait que QTimer appartient aussi au thread principal
- tu lances pas exec() dans run() donc tu lances jamais ta boucle d'event du thread secondaire
En d'autres mots, ton thread ne sert à rien

ThreadSaveData c'est son QThread, il a du faire "start()" depuis le thread principal, ce qui exécute le run(). Pour ton premier point, c'était le but de ma question sur l'existence de la méthode, vu que les débutant font souvent l'erreur de faire connect avec "this" en récepteur (copier-coller malheureux).
Je pense donc que son implémentation de thread fonctionne (même si elle est déconseillée).

[petitclem]

Bonjour,

J'ai bien lu la FAQ et j'ai rajouté exec() et maintenant tout fonctionne !
Merci pour votre aide.

Je pense donc que son implémentation de thread fonctionne (même si elle est déconseillée).

J'aimerai quand même comprendre qu'est ce qui ne va pas dans l'implémentation de mon thread et ce que vous me conseillerai.

[gbdivers]

ThreadSaveData c'est son QThread, il a du faire "start()" depuis le thread principal, ce qui exécute le run(). Pour ton premier point, c'était le but de ma question sur l'existence de la méthode, vu que les débutant font souvent l'erreur de faire connect avec "this" en récepteur (copier-coller malheureux).
Je pense donc que son implémentation de thread fonctionne (même si elle est déconseillée).

Pour les connexions signaux-slots entre threads, les signaux passent par la boucle d'event donc il faut que celle ci soit tourne, donc il ne faut pas oublier le exec()

J'aimerai quand même comprendre qu'est ce qui ne va pas dans l'implémentation de mon thread et ce que vous me conseillerai.

Pour terminer, il existe des tutos sur les threads de Qt en français, il suffit de cliquer sur le gros bouton "Tutoriels Qt" en haut de cette page...

[petitclem]

OK, merci beaucoup à tous pour votre aide.