Methode write des sockets n'envoie rien

**geek-59600** · 17/06/2009, 00h12

Bonjour,

Je suis entrain de développer un petit client réseau.

Dans ce client j'ai un slot qui est appelé à chaque pression sur un bouton

Code :

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

1
2
3
4
5
6
7
8
9
QByteArray paquet;
QDataStream out(&paquet, QIODevice::WriteOnly);
QString messageAEnvoyer = (m_lineedit2->text());
    out << (quint16) 0;
    out << messageAEnvoyer;
    out.device()->seek(0);
    out << (quint16) (paquet.size() - sizeof(quint16));
    m_socket->write(paquet); // On envoie le paquet
    m_lineedit2->clear();

J'ai presque recopier mot pour mot l'exemple du sdz j'ai juste changer le nom de mon socket et la provenance du message à envoyer

J'ai également connecter le slot readyRead de mon socket a une méthode qui m'affiche "data" dans un QtextEdit, pour le moment, je ne veux pas interpréter ce que le serveur m'envoie je m'en sers pour voir si le serveur me répond.

Quand je me connecte a un serveur ftp en local, le serveur m'envoie le message de bienvenue, mais a chaque fois que je lui envoie une trame rien ne me revient, bien sur en temps normal le ftp me renvoie une trame meme si il n'a pas compris la mienne (testé avec telnet).

Edit : Solution trouvé, la trame envoyé était sérialisé donc le serveur ne comprenait pas je pense j'ai résolu mon souci en procédent comme ca :

Code :

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

1
2
3
QByteArray paquet = (m_lineedit2->text().toAscii()+"\n");
char* message = paquet.data();
m_socket->write(message,(quint64) (strlen(message)));

Désolé de vous avoir dérangé pour si peu.

**yan** · 17/06/2009, 09h18

Code :

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

out << messageAEnvoyer;

sérialise bien une QString et non le directement le texte contenue.

Le plus propre est bien

Code :

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

out << messageAEnvoyer.toLatin1();

ou une autre fonction de conversion. Tout dépend de l'encodage utilisé.

Par contre, comme on peut déjà utiliser QByteArray, évite de passer par char*

Tu n'est pas et ne sera pas le seul à avoir eu ce problème
Merci d'avoir donné la solution

Pense au bouton

**geek-59600** · 17/06/2009, 12h14

Si j'applique .toLatin1(). ma trame n'est pas bonne :/ même avec .toAscii()

Exemple avec l'user "monnom.monprenom"

\000\032\000\000\000\026user monnom.monprenom

voici le code :

Code :

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

1
2
3
4
5
6
7
8
9
QByteArray paquet;
QDataStream out(&paquet, QIODevice::WriteOnly);
QString messageAEnvoyer = (m_lineedit2->text()+"\n");
out << (quint16) 0;
out << messageAEnvoyer.toAscii();
out.device()->seek(0);
out << (quint16) (paquet.size() - sizeof(quint16));
m_socket->write(paquet); // On envoie le paquet
m_lineedit2->clear();

J'ai essayé avec le "\n" et sans -> même résultat.

**nouknouk** · 17/06/2009, 13h17

salut,

le souci vient à mon avis de l'emploi de la fonction 'seek' qui n'est valable que pour le QIODevice permettant l'accès aléatoire:

Envoyé par QtDoc

For random-access devices, this function sets the current position to pos, returning true on success, or false if an error occurred. For sequential devices, the default behavior is to do nothing and return false.

Or le QIODevice représenté par une socket est un device de type séquentiel. La fonction seek() ne peut donc pas être utilisée. Pour s'en persuader:

1/ Teste la valeur de retour de seek ; à tous les coups elle doit retourner 'false'

2/ Même principe en testant la valeur de retour de 'isSequential()' qui si elle retourne 'true' implique que la fonction 'seek' ne peut pas être utilisée.

L'idée serait donc de générer un QByteArray intermédiaire (ou un buffer dans ce style) pour préparer les données à envoyer. Une fois celles-ci construites, tu envoies en une seule fois l'ensemble de ton bloc de données.

EDIT: un petit tour dans le code source de QAbstractSocket.cpp finit de nous convaincre qu'une socket est bien un device de type séquentiel.

EDIT: non, j'avais mal lu, tu fais un seek sur un QDataStream basé sur un QByteArray, et pas sur la socket. Oublie mon post

**yan** · 17/06/2009, 13h30

à oui tien

out << messageAEnvoyer.toAscii();

serialise le QByteArray et non le contenue.
Tu peut faire ceci

Code :

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

1
2
3
4
5
6
7
8
    QByteArray paquet;
    QDataStream out(&paquet, QIODevice::WriteOnly);
    QString messageAEnvoyer = "monnom.monprenom";
    out << (quint16) 0;
    paquet += messageAEnvoyer.toAscii();
    out.device()->seek(0);
    out << (quint16) (paquet.size() - sizeof(quint16));