Boucle qui ne s'arrête pas

**Yazuka2.0** · 02/11/2022, 16h59

Bonjour,

J'ai une boucle While qui fonctionne cependant je veux exécuter plusieurs commandes.
Le script finit par bloquer à une commande et ne lance pas celle d'après. En plus, une fois les commandes lancées, je ne trouve pas la condition qui pourrait faire que la boucle s'arrête.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
        char b[] = "ZynqMP";
        while (serial.waitForReadyRead() && rest != 0)
        {
            QByteArray data = serial.readAll();;
            rest = strcmp(data, b);
            qDebug() << data.toStdString().c_str();
            serial.write("123\n");
            QByteArray readLine;
            serial.write("setenv ipaddr 192.168.0.222\n");
            serial.write("setenv serverip 192.168.0.36\n");
            serial.write("saveenv\n"); 
            serial.write("tftpboot 0x10000000 boot.scr\n");
         }

Merci à tous ceux qui prendront le temps de m'aider.

**kaitlyn** · 03/11/2022, 12h18

Salut,

Essaye en changeant l'ordre d'évaluation de ta condition while:
while (rest != 0 && serial...

**Yazuka2.0** · 03/11/2022, 12h26

Merci de ta réponse.
Ça n'a malheureusement pas changé le problème.

En plus les commandes "saveenv" et "tftpboot" continue de fonctionner. Mais si "saveenv" est lancé, "tftpboot" ne démarrera jamais. Un peu comme si elles effaçaient tout ce qui était écrit après elles.

J'ai tenté avec une variable indépendante mais soit ça n'arrête pas le programme soit ça ne le lance même pas. Comme si une variable supplémentaire était prioritaire sur les commandes de "serial.write" peu importe l'ordre d'écriture.

**kaitlyn** · 03/11/2022, 12h42

Je n'utilise pas Qt mais essaye en remplaçant la ligne 5 par rest= qstrcmp(data.data(), b);

**Yazuka2.0** · 03/11/2022, 12h52

Ça n'a rien changé.

C'est pas grave. Merce d'avoir essayé de m'aider.

**kaitlyn** · 03/11/2022, 13h15

Fournis un code un peu plus étoffé, ce qu'il est supposé lire et faire sur le port série.
Sinon, les fonctions membres suivantes devraient t'aider:
https://doc.qt.io/qt-6/qiodevice.html#canReadLine
https://doc.qt.io/qt-6/qiodevice.html#readLine

**Yazuka2.0** · 03/11/2022, 13h40

Voilà globalement le code. En espèrent que ça aide un peu plus à le débloquer.

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
 
int main(int argc, char *argv[])
{
    int a, rest = 1;
    char b[] = "ZynqMP";
    cout << "A quelle étape voulez vous vous rendre ? 0 pour toutes les faire.\n";
    cin >> a;
    QCoreApplication app(argc, argv);
    QSerialPort serial("/dev/ttyUSB0");
    serial.setBaudRate(115200);
    serial.setDataBits(QSerialPort::DataBits::Data8);
    serial.setParity(QSerialPort::Parity::NoParity);
    serial.setStopBits(QSerialPort::StopBits::OneStop);
    serial.setFlowControl(QSerialPort::FlowControl::NoFlowControl);
 
    if (serial.open(QIODevice::ReadWrite))
    {
        while (serial.waitForReadyRead() && rest != 0)
        {
            QByteArray data = serial.readAll();;
            rest = qstrcmp(data.data(), b);
            qDebug() << data.toStdString().c_str();
            serial.write("123\n");
            QByteArray readLine;
            if (a == 0 || a == 2)
            {
                serial.write("setenv ipaddr 192.168.0.222\n");
                serial.write("setenv serverip 192.168.0.36\n");
                serial.write("saveenv\n"); 
                serial.write("tftpboot 0x10000000 boot.scr\n");
                serial.write("printenv ipaddr\n");
        }
    }
    else
    {
        qDebug() << QString("Error %1").arg(serial.errorString());
    }
 
    QSerialPortInfo description("/dev/ttyUSB0");
 
    serial.setPortName("ttyUSB0");
    return 0;
}

Il y a d'autre commande à faire mais si ça ne fait pas le "printenv" ça n'avancera pas peu importe la commande suivante je pense.

**Mat.M** · 03/11/2022, 13h46

Envoyé par Yazuka2.0

J'ai une boucle While qui fonctionne ce pendant je veux exécuter plusieurs commandes.

dans le code donné il n'y a aucune instruction qui permette de vérifier si les commandes sur le port série se déroulent bien.
Le code donné est insuffisant il faut tester une valeur de retour pour savoir si les instructions marchent bien.
Par exemple QSerialPort::setBaudRate retourne un booléen donc vérifier à chaque fois si les commandes s'exécutent bien.
Une fois que vous avez fait cela c'est un problème de synchronisation en apparence.

**Yazuka2.0** · 03/11/2022, 13h51

Envoyé par Mat.M

dans le code donné il n'y a aucune instruction qui permette de vérifier si les commandes sur le port série se déroulent bien.
Le code donné est insuffisant il faut tester une valeur de retour pour savoir si les instructions marchent bien.
Par exemple QSerialPort::setBaudRate retourne un booléen donc vérifier à chaque fois si les commandes s'exécutent bien.
Une fois que vous avez fait cela c'est un problème de synchronisation en apparence.

Je n'ai pas tout compris. Quand je lance le programme les commandes ont l'air de bien fonctionner puisque dans le terminal je vois les commandes se lancer.

Comment fonctionne QSerial::setBaudRate exactement ?

**kaitlyn** · 03/11/2022, 14h18

Dans quel objectif ont été écrites les lignes 18 à 31 ?
Qu'est supposé faire ce bout de code ou plutôt qu'en en attends-tu ?

**Yazuka2.0** · 03/11/2022, 14h20

Envoyé par kaitlyn

Dans quel objectif ont été écrites les lignes 18 à 31 ?
Qu'est supposé faire ce bout de code ou plutôt qu'en en attends-tu ?

Je lance le programme, il va lire ce qu'envoi la carte et écrire le mot de passe pour être connecté en série.
Ensuite il écrit les commandes qui permettent de configurer la carte.

Normalement il y a plusieurs "if" en fonction du nombre de commande à fournir mais pour l'instant je n'ai mis que le début.

**kaitlyn** · 03/11/2022, 15h25

Concentre-toi d’abord sur la lecture avant de chercher à écrire sur le port série. Je vais regarder pour te trouver un début de piste.

**Yazuka2.0** · 03/11/2022, 15h30

La partie lecture fonctionne. Je l'ai déjà vérifié plusieurs fois.

**kaitlyn** · 03/11/2022, 15h30

sous réserve,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
bool keep_reading= true;
while (keep_reading) {
    if (serial.waitForReadyRead() && serial.canReadLine()) {
        QByteArray line= serial.readLine();
        qDebug()<< "line: "<< line<< '\n';
    }
 
    else {
        qDebug()<< "waiting for data...\n";
    }
}

Envoyé par Yazuka2.0

La partie lecture fonctionne. Je l'ai déjà vérifié plusieurs fois.

Oui mais il faut que le code se comporte correctement sinon tu inondes le "buffer" du port série.

**Yazuka2.0** · 03/11/2022, 15h35

Tu veux que je remplace par ce code à partir de la ligne 16 ?

Si c'est ça, le programme ne fait qu'attendre. Il ne s'arrête pas contrairement à ce que j'ai tenté mais il ne bouge pas non plus.

**Yazuka2.0** · 03/11/2022, 15h38

Envoyé par kaitlyn

Oui mais il faut que le code se comporte correctement sinon tu inondes le "buffer" du port série.

Je ne suis pas non plus une très grande professionnelle la dedans. Je viens a peine de commencer. Qu'est-ce que tu veux dire par inonder le "buffer" ? Et comment vérifier que cette partie du code se comporte correctement alors que la lecture a l'air de se faire correctement ?

**kaitlyn** · 03/11/2022, 15h44

Non à partir de la ligne 18, ta boucle while. Parce que là ton code écrit de manière inconditionnelle sur le port série. Ensuite pour arrêter ta boucle, il faut une condition et il faut la spécifier, ce peut être un mot envoyé par la carte par exemple.

**Yazuka2.0** · 03/11/2022, 15h50

Je réfléchis depuis plusieurs jour à ce que pourrait être la condition de sortie mais je ne trouve rien.

Le problème du code que tu viens de m'envoyer c'est qu'il ne fait que lire à coup de "line : ....."
Ce n'est pas pratique et en plus je ne vois à aucun moment le texte indiquant qu'il faut mettre le mot de passe.

**kaitlyn** · 03/11/2022, 15h57

Qu'est ce qui est affiché exactement ?
Pourquoi faire une boucle de lecture si ce que t'envoie ta carte ne t’intéresse pas ?

**Yazuka2.0** · 03/11/2022, 16h08

Ce que renvoi la carte m'intéresse. Mais avec le code que tu as proposé je me suis perdu dans ce qu'envoi la carte.

Avec ton code on a ça :

Code :

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

1
2
3
4
5
6
7
line:  "Destination Device is PL, changing LoadAddress\r\n" 
 
line:  "XPFW: Calling ROM PWRUP Handler..Done\r\n" 
 
line:  "XPFW: Calling ROM Isolation Handler..Done\r\n" 
 
line:  "Non authenticated Bitstream download to start now\r\n"

Et de mon côté ce que j'obtiens habituellement ressemble à ça :

Code :

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

1
2
3
4
5
Destination Device is PL, changing LoadAddress
XPFW: Calling ROM PWRUP Handler..
Done
XPFW: Calling ROM Isolation Handler..Done
Non authenticated Bitstream download to start now

C'est la même chose mais pour retrouver une ligne en particulier c'est plus compliqué puisqu'elles démarrent toutes pareilles et que je n'arrive pas à insérer ma commande pour mettre le mot de passe. (Le petit "serial.write("123\n");")

Boucle qui ne s'arrête pas

Plateformes

Discussions similaires

Partager

Partager