Discussion :

[bilou_2007]

Salut à tous,
J'ai une classe client et une classe serveur qui s'échangent des données de types QString, du coté du serveur, aucun probleme au niveau de la reception car j'arrive à voir les données emises par le client. Par contre du coté de celui la j'ai un souci:
ce test est bon :

Code :

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

1
2
3
4
5
6
7
8
if (socket->bytesAvailable() < (int)sizeof(quint16))
        {//ça rentre pas ici donc on est dans les normes
            std::cout<<erreur1.toStdString()<<":"<<std::endl;
            return;
        }
        //la taille du message recu est bonne
        //on la stocke dans une variable
        in >> tailleMsg;

Mais pas celui ci:

Code :

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

1
2
3
4
5
6
 
if (socket->bytesAvailable() < tailleMsg)
    {
        std::cout<<erreur2.toStdString()<<":"<<std::endl;
        return;
    }

Sur la console s'affiche le message erreur2 donc socket->bytesAvailable() est bien inférieure à tailleMsg.
À quoi tient ce probleme et pourquoi j'ai pas ce probleme au niveau de la classe serveur?
Merci d'avance

[Amnell]

Bonjour,

Dit comme ça, c'est comme si je demandais "pourquoi j'ai un arbre devant ma fenêtre", non pas parce que c'est évident, mais parce que c'est quelque chose qui a été établi, et dont la raison n'est pas déterminable avec les informations données.

Toutefois, j'aurais tendance à dire que dans le deuxième cas, tailleMsg = 0, ce qui fait que forcément, on entre dans la fonction. Dans ce cas, il serait bon de tester la valeur de tailleMsg.

Bonne continuation,
Amnell.

[bilou_2007]

Bonjour,

Dit comme ça, c'est comme si je demandais "pourquoi j'ai un arbre devant ma fenêtre", non pas parce que c'est évident, mais parce que c'est quelque chose qui a été établi, et dont la raison n'est pas déterminable avec les informations données.

Toutefois, j'aurais tendance à dire que dans le deuxième cas, tailleMsg = 0, ce qui fait que forcément, on entre dans la fonction. Dans ce cas, il serait bon de tester la valeur de tailleMsg.

Bonne continuation,
Amnell.

Merci Amnell pour ta réponse.
J'ai affiché le contenu de la tailleMsg ainsi que celui de socket->bytesAvailable(), les deux sont égaux à 6 durant l'exécution. Ce qui est vraiment étrange!
Demo : Le slot chargé de traité les données recues:

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
void maClasse::receptionDonnées//SLOT
{
    QString erreur1="erreur1";
    QString erreur2="erreur2";
    std::cout<<testtest.toStdString()<<":"<<std::endl;
    QDataStream in(socket);
    if (tailleMessage == 0)
    {
        if (socket->bytesAvailable() < (int)sizeof(quint16))
        {
            std::cout<<erreur1.toStdString()<<":"<<std::endl;
            return;
        }
        in >> tailleMsg;
        QString t1=QString::number(tailleMessage);
        QString t2=QString::number(socket->bytesAvailable());
        std::cout<<t1.toStdString()<<":"<<std::endl;
        std::cout<<t2.toStdString()<<":"<<std::endl;
    }
//****JE METS CETTE PARTIE "probleme" EN COMMENTAIRE
    //if (socket->bytesAvailable() < tailleMsg)
    //{
      //  std::cout<<erreur2.toStdString()<<":"<<std::endl;
        //return;
    //}
    // Si on arrive jusqu'à cette ligne, on peut récupérer le message entier
    QString str;
 
    in >> str;
 str+="*";
str="*"+str;
    std::cout<<str.toStdString()<<":"<<std::endl;
 
 
//    
    tailleMsg = 0;
}

Affichage console:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
6
6
*0*
6
6
*0*
6
6
*3*
6
6
*8*

Et si je "décommente" la pratie que j'ai appelée probleme

Code :

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

1
2
3
4
erreur2:
erreur2:
erreur2:
erreur2:

Vraiment je comprends pas

[Amnell]

J'ai affiché le contenu de la tailleMsg ainsi que celui de socket->bytesAvailable(), les deux sont égaux à 6 durant l'exécution. Ce qui est vraiment étrange!

Vous faites :

Code :

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

in >> tailleMsg;

D'où la modification nécessaire de :

Code :

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

if (socket->bytesAvailable() < tailleMsg)

En :

Code :

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

if (socket->bytesAvailable() <= tailleMsg)

Bonne continuation,
Amnell.

[bilou_2007]

Vous faites :

Code :

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

in >> tailleMsg;

D'où la modification nécessaire de :

Code :

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

if (socket->bytesAvailable() < tailleMsg)

En :

Code :

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

if (socket->bytesAvailable() <= tailleMsg)

Bonne continuation,
Amnell.

Moi je ne veux pas qu'il rentre dans la condition,or le programme rentre quand meme bien que la condition ne soit pas satisfaite. De toutes facons j'ai modifié mon if et ça na pas donné le résultat souhaité:

Code :

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

if (socket->bytesAvailable() <= tailleMsg)

ça a donné ceci:

Code :

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

1
2
3
4
erreur2:
erreur2:
erreur2:
erreur2:

Merci pour votre patience
PS: j'ai oublié peut etre de dire que tailleMsg est quint16

[Amnell]

Votre code, remaké en un peu plus joli :

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 /* Votre slot de réception de données */
{
    QDataStream in(socket);
 
    if (tailleMessage == 0)
    {
        if (socket->bytesAvailable() < (int)sizeof(quint16))
        {
            qDebug() << "Error: socket->bytesAvailable() < (int)sizeof(quint16)"
            return;
        }
        in >> tailleMessage;
    }
 
    if (socket->bytesAvailable() < tailleMessage)
    {
        qDebug() << "Error: socket->bytesAvailable() < tailleMessage";
        return;
    }
 
    QString message;
    in >> message;
 
    tailleMessage = 0;
}

Il faut savoir que la réception ne se fait pas d'un coup, d'où le fait qu'il y ait de tels tests dans le code. Mais à un moment ou à un autre, vous devez franchir les deux conditions.

[bilou_2007]

Votre code, remaké en un peu plus joli :

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 /* Votre slot de réception de données */
{
    QDataStream in(socket);
 
    if (tailleMessage == 0)
    {
        if (socket->bytesAvailable() < (int)sizeof(quint16))
        {
            qDebug() << "Error: socket->bytesAvailable() < (int)sizeof(quint16)"
            return;
        }
        in >> tailleMessage;
    }
 
    if (socket->bytesAvailable() < tailleMessage)
    {
        qDebug() << "Error: socket->bytesAvailable() < tailleMessage";
        return;
    }
 
    QString message;
    in >> message;
 
    tailleMessage = 0;
}

Il faut savoir que la réception ne se fait pas d'un coup, d'où le fait qu'il y ait de tels tests dans le code. Mais à un moment ou à un autre, vous devez franchir les deux conditions.

Ça donne ce résultat:

Code :

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

1
2
3
4
5
Error: socket->bytesAvailable() < tailleMessage 
Error: socket->bytesAvailable() < tailleMessage 
Error: socket->bytesAvailable() < tailleMessage 
Error: socket->bytesAvailable() < tailleMessage 
Error: socket->bytesAvailable() < tailleMessage

[Amnell]

C'est tout à fait normal. Et maintenant, si vous remplacez :

Code :

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

1
2
    QString message;
    in >> message;

Par :

Code :

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

1
2
3
4
    QString message;
    in >> message;
 
    qDebug() << "Message obtenu";

Qu'obtenez-vous ?

[bilou_2007]

C'est tout à fait normal. Et maintenant, si vous remplacez :

Code :

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

1
2
    QString message;
    in >> message;

Par :

Code :

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

1
2
3
4
    QString message;
    in >> message;
 
    qDebug() << "Message obtenu";

Qu'obtenez-vous ?

Ça donne:

Code :

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

1
2
3
4
5
Error: socket->bytesAvailable() < tailleMessage 
Error: socket->bytesAvailable() < tailleMessage 
Error: socket->bytesAvailable() < tailleMessage 
Error: socket->bytesAvailable() < tailleMessage 
Error: socket->bytesAvailable() < tailleMessage

[Amnell]

Dans ce cas, c'est sans doute la partie serveur qui pose problème. Puis-je voir la partie serveur où vous envoyez les données, s'il vous plait ?

[bilou_2007]

Dans ce cas, c'est sans doute la partie serveur qui pose problème. Puis-je voir la partie serveur où vous envoyez les données, s'il vous plait ?

Oui bien sur, voici la méthode qui permet d'envoyer

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 FenServeur::envoyerATous(const QString &message,QTcpSocket* emetteur)
{
    // Préparation du paquet
    QByteArray paquet;
    QDataStream out(&paquet, QIODevice::WriteOnly);
    out << (quint16) 0; //On écrit 0 au début du paquet pour réserver la place pour écrire la taille
    out << message;
    out.device()->seek(0); //On se replace au début du paquet
    //On écrase le 0 qu'on avait réservé par la longueur du message
    out << (quint16) (paquet.size() - sizeof(quint16));
    // Envoi du paquet préparé à tous les clients connectés au serveur
    for (int i = 0; i < clients.size(); i++)
    {
        if(clients[i]!=emetteur)
        {
            //ecrire sur la fenetre serveur ce qu'il envoie
            etatServeur->setText(message);
            clients[i]->write(paquet);
        }
    }
}

[Amnell]

Tant qu'à faire, autant avoir des logs plus détaillés :

Code :

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

qDebug() << "Error: socket->bytesAvailable() < tailleMessage";

À remplacer par :

Code :

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

1
2
qDebug() << "Error: socket->bytesAvailable() < tailleMessage, with byteAvailable() = "
    << socket->bytesAvailable() << " and tailleMsg = " << tailleMessage;

[bilou_2007]

Tant qu'à faire, autant avoir des logs plus détaillés :

Code :

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

qDebug() << "Error: socket->bytesAvailable() < tailleMessage";

À remplacer par :

Code :

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

1
2
qDebug() << "Error: socket->bytesAvailable() < tailleMessage, with byteAvailable() = "
    << socket->bytesAvailable() << " and tailleMsg = " << tailleMessage;

J'avais pensé à ça comme derniere recours et j'ai obtenu:

Code :

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

1
2
3
4
5
200  <  36852
210  <  36852
220  <  36852
230  <  36852
240  <  36852

Donc j'ai pensé à initialisé tailleMessage à 0 au début de la fonction et ça marché, ceci dit, j'aimerais comprendre pourquoi au niveau de la meme fonction du serveur yavais pas ce probleme alors que dans ma classe client j'ai du faire cette initialisation dont je ne suis pas sur de la fiabilité.(ceci dit je suis content d'avoir trouvé quelque chose )