Problème de réception de données via QTcpSocket

Version imprimable

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:

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:

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

16/04/2011, 12h40
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.

Citation:

Envoyé par 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.

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:

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:

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:

1 2 3 4 erreur2: erreur2: erreur2: erreur2:
Vraiment je comprends pas

16/04/2011, 17h58
Amnell

Citation:

Envoyé par bilou_2007

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:

in >> tailleMsg;

D'où la modification nécessaire de :

Code:

if (socket->bytesAvailable() < tailleMsg)

En :

Code:

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

Bonne continuation,
Amnell.
16/04/2011, 18h40
bilou_2007
Citation:

Envoyé par Amnell

Vous faites :

Code:

in >> tailleMsg;

D'où la modification nécessaire de :

Code:

if (socket->bytesAvailable() < tailleMsg)

En :

Code:

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:

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

ça a donné ceci:
Code:

1 2 3 4 erreur2: erreur2: erreur2: erreur2:
Merci pour votre patience :oops:
PS: j'ai oublié peut etre de dire que tailleMsg est quint16

Votre code, remaké en un peu plus joli :

Code:

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.

Citation:

Envoyé par Amnell

Votre code, remaké en un peu plus joli :

Code:

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:

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

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

1 2 QString message; in >> message;
Par :
Code:

1 2 3 4 QString message; in >> message; qDebug() << "Message obtenu";
Qu'obtenez-vous ?

Citation:

Envoyé par Amnell

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

Code:

1
2
    QString message;
    in >> message;

Par :

Code:

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

Qu'obtenez-vous ?

Ça donne:

Code:

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

16/04/2011, 20h02
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 ?

Citation:

Envoyé par 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 ?

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

Code:

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);
        }
    }
}

16/04/2011, 20h35
Amnell
Tant qu'à faire, autant avoir des logs plus détaillés :

Code:

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

À remplacer par :
Code:

1 2 qDebug() << "Error: socket->bytesAvailable() < tailleMessage, with byteAvailable() = " << socket->bytesAvailable() << " and tailleMsg = " << tailleMessage;
16/04/2011, 20h45
bilou_2007
Citation:
Envoyé par Amnell

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

Code:

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

À remplacer par :

Code:

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:

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 :) )