Discussion :

[rs485]

Salut!

Je fais un petit programme avec Qt, ayant un thread principal (pour l'UI), et un thread pour le calcul.

Le problème est que mon thread dédié aux calculs manipule un tableau (via son pointeur), et que le thread principal doit afficher périodiquement les valeurs à l'écran.


Si j'utilise un QMutex dans le thread du calcul, puis-protéger les donnés liées aux pointeur?

voici le code du thread dédié aux calculs pour mieux vous expliquer mon problème:

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
26
#define MAX 10
class Thread : public QThread
{
    Q_OBJECT
 
public:
    Thread(){for(int i=0; i<MAX; i++) x[i]=0;}
 
private:
    int x[MAX];
    QMutex mutex;
 
signals:
    sendP(int*, int);
 
protected:
    void run(){
        for(;;)
        {
            QMutexLocker locker(&mutex); // Est-ce que x[0] à x[10] sont protégé?
            for(in i=0; i<MAX; i++)
                x[i]+=3;
        }
        emit sendP(x, MAX)
    }
};

et voici une portion de code de l'UI:

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
 
MainWidget::MainWidget(QWidget * parent)
    :QWidget(parent)
{
    //... intialisation
    thread = new Thread;
    connect(thread, SIGNAL(sendP(int*, int)), this, SLOT(receiveP(int*, int)));
    thread->start();
    //...
}
 
MainWidget::receiveP(int*x, int s)
{
    for(int i=0; i<s; i++)
        qDebug() << x[i] 
    // je dois avoir à l'écran 10 valeurs identiques
}

Et si j'enlève le mutex de la classe Thread pour le mettre dans le slot receiveP(int*, int), cela marchera?

Merci

[atttchoum]

Bonjour,

Code :

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

1
2
3
4
5
6
7
8
9
10
 
    void run(){
        for(;;)
        {
            QMutexLocker locker(&mutex); // Est-ce que x[0] à x[10] sont protégé?
            for(in i=0; i<MAX; i++)
                x[i]+=3;
        }
        emit sendP(x, MAX)
    }

Sa n'as pas trop de sens car ton emit sendP(x, MAX) ne sera jamais exécuté.

Donc je suppose que sa doit plutôt ressembler a sa :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void run(){
    for(;;)
    {
	{
	    QMutexLocker locker(&mutex); // Est-ce que x[0] à x[10] sont protégé?
	    for(in i=0; i<MAX; i++)
		x[i]+=3;
	}
	emit sendP(x, MAX);
    }
}

de plus :

MainWidget::receiveP(int*x, int s)
{
for(int i=0; i<s; i++)
qDebug() << x[i]
// je dois avoir à l'écran 10 valeurs identiques
}

Tu ne lock pas le mutex ici ! Il n'attendra donc pas que tout le tableau soit incrémenté pour affiché et je suppose que tu obtient parfois des valeurs différentes (sans compter les risques d'accès concurrents).

tu doit rajouter ceci :

Code :

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

1
2
3
4
5
6
7
8
MainWidget::receiveP(int*x, int s)
{
    QMutexLocker locker(&thread->mutex); 
    for(int i=0; i<s; i++)
        qDebug() << x[i];
    // je dois avoir à l'écran 10 valeurs identiques
}

Pour terminer je te conseil d'utiliser un QVector plutôt qu'un tableau à la C.

[yan]

Salut,
c'est pas comme cela qu'il faut faire.
Regarde
http://qt.developpez.com/doc/latest/...emaphores.html
http://qt.developpez.com/doc/latest/...onditions.html

Si c'est juste pour envoyer à l'affichage quelques valeurs de temps en temps, un simple signal devrais suffire. Pas besoin de mutex ou autre. Faut juste ne pas utiliser un pointeur brut comme cela.

Si tu peut expliquer un peu plus ton besoin.