Socket, C++ et buffer !

Bonjour à tous !

J'avoue que je ne comprends plus (pourtant ça fait deux jours que je cherche :oops:).

J'essaie de faire une appli ftp (client) pour me faire la main avec le C++ sous GNU/Linux (j'ai quelques connaissances en Java).

Le problème se situe exactement ici :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 const int MAX_REPLY_CHARS = 5000; char buffer_reply_chr[MAX_REPLY_CHARS]; void fonction() { //===début if(recv(sockfd, &buffer_reply_chr, MAX_REPLY_CHARS, 0) == -1) { perror("recv"); exit(1); } traiter_recv(buffer_reply_chr); //===fin } void Connect::traiter_recv(char* buffer) { for(int i = 0; i<=MAX_REPLY_CHARS; i++) { cout << buffer[i]; } }
Ce qu'il faut savoir c'est que le socket se crée normalement. Mon problème se trouve avec la gestion des tableaux et surtout avec la variable buffer_reply_chr ! Si j'ai bien compris (d'après les specifications UNIX), je suis sensé donner un &buffer_reply_char dans la fonction recv(...). Ce que je fais.

Après, j'ai essayé plusieurs manières pour récupérer le message renvoyé par le serveur mais j'ai une sortie sur la console atroce (sauf la première ligne qui est lisible et qui ne contient aucun signes bizarres).

Citation:

220 ProFTPD 1.3.0rc1 Server [xxx.xxx.xx.xxx]
��C��(��hY��{��|&2��t�t��|�� C�� Y��{��6��@��W��u7��W��0��8��F~�� lZ��{��F~��8~��Z��{��F~��4Y��g��8~��PY��g��@W��
��|&��C�� [��{��Y��Y��r��C��^��p ^�)D^�)D`�)D��^��[��5}��Y��Y��_��]��Y��xZ��]��]��]��]��4 ��xZ��Y��``��]��Z��[��]��] ��]��]��V6��[��Z��[��xZ��u7�4��^��*��4�]��F~��| [��8~��]��]��]��6��F~��|[��Z��[��u7��^��1�� ]��\��]��6��]��]��]��|[��u7��^��1��]��& ��^��ｽ��C��!��$\��1��6��]�� 翃��#��`��\��@��\�� \��]��#��ﾕ��\��6��#�� ^��q��`��'��.��\��x��P��``�� x��P��`�`��`�ﾕ�� g��8g�� g��8g�� B��@��Db��2��`��
��x`��@�R��[�� YC��
_��1��`��0��翃��C��\ _��1��C��|_��1��V��1�� 翃��y:<_R��4`��@��t��_�� 1��_��z�� z��y:<``��_R��#��z��H��C��|`��1��C��`��1��V�� 翃��<�n��Ta��@��8
��a�� >� Y��a��a��@��Pa��<��a��n�� -��Pa��
��`a��
>� �a��Y��a��a��B��Tö·⎽ ┼Ûë·Ü-Õ·ôæ·À…æ·°¶Ô·¨▒‰¿ÂÚ·À…æ·œ
ö·¸¡0™¸▒‰¿ ìð·À…æ·œ
ö·è▒‰¿$ñ·€Uö·œ
ö·¸¡0™ø▒‰¿ %ù·Œ-ö·œ
ö·¸¡0™ø▒‰¿½ñ·¸¡œ
ö·
C⎺┼┼␊␌├␋⎺┼ ⎽┤␌␌␊⎽⎽°┤┌┌≤ ␊⎽├▒␉┌␋⎽␤␊␍. R␊▒␍≤ °⎺⎼ ␌⎺└└▒┼␍⎽.

et à la console, je ne peux plus que taper des caractères tout aussi étranges !

Bon, visiblement j'ai manqué un truc (ou pas compris) ! Si une âme charitable pouvait me sauver, ça serait sympa :mrgreen: !

(ah oui, ce que je veux faire c'est afficher le message reçu proprement... visiblement il n'y a qu'une ligne de message ? est-ce possible ? comment le savoir ?)

Merci de m'avoir lu en tous cas :).

Je pense que tu devrais allouer la mémoire nécessaire à ton buffer:
Code:

1 2 const unsigned short BUFFER_SIZE = 100; char buffer_reply_chr[BUFFER_SIZE];
aprés pour ton appel recv:

Code:

recv(sockfd, buffer_reply_chr, BUFFER_SIZE, 0)

Citation:
Envoyé par MatRem
Je pense que tu devrais allouer la mémoire nécessaire à ton buffer:
Code:

1 2 const unsigned short BUFFER_SIZE = 100; char buffer_reply_chr[BUFFER_SIZE];
aprés pour ton appel recv:

Code:

recv(sockfd, buffer_reply_chr, BUFFER_SIZE, 0)
Merci beaucoup pour cette réponse :). J'avais malheureusement oublié deux lignes dans le bout de code que j'ai fourni dans le premier post, celles-ci :
Code:

1 2 3 const int MAX_REPLY_CHARS = 5000; char buffer_reply_chr[MAX_REPLY_CHARS];
Désolé.

Par contre comme tu le suggères, je ferais mieux de supprimer le sizeof(buffer_reply_chr) pour le remplacer par la constante MAX_REPLY_CHARS... c'est plus rapide/propre et peut-être aussi plus correct (je teste ça).

Marche pas. J'obtiens toujours une sortie console dans le même style qu'au dessus ! Mis à part l'esthétique relative, on ne peut rien en tirer...:(

Dans la fonction recv(...), ne vaut-il mieux pas passer par référence le tableau buffer_reply_chr ?? Parce que s'il devient trop gros, ça risque de poser problème non ? (J'ai testé les deux et visiblement ça fonctionne identiquement donc je laisse mon petit &).

bien le bonsoir,

Citation:

Envoyé par agent007se

Marche pas. J'obtiens toujours une sortie console dans le même style qu'au dessus ! Mis à part l'esthétique relative, on ne peut rien en tirer...:(

es-tu sûr que ton envoi de données est fait correctement ?

Citation:

Dans la fonction recv(...), ne vaut-il mieux pas passer par référence le tableau buffer_reply_chr ?? Parce que s'il devient trop gros, ça risque de poser problème non ?

non, ne pas passer de référence à un tableau. La fonction recv est faite pour prendre un pointeur sur une zone mémoire (un void*)

Citation:

Envoyé par khayyam90

bien le bonsoir,

es-tu sûr que ton envoi de données est fait correctement ?

non, ne pas passer de référence à un tableau. La fonction recv est faite pour prendre un pointeur sur une zone mémoire (un void*)

1) hmmm, je n'envoie strictement aucune donnée au début. Lorsque j'initie la connection via :
Code:

1 2 3 4 5 6 if(connect(sockfd, (struct sockaddr *)&dest_addr, sizeof(struct sockaddr)) == -1) { perror("connect"); exit(1); }
le serveur FTP renvoie un message (220 (=Service ready for new user) dans ce cas-ci...). Ce que j'aimerais faire c'est recevoir tout le message quelqu'il soit et l'afficher correcter pour tout serveur ftp... (eh oui, je n'en suis qu'au début :roll:).

2) d'accord pour le pointeur mais en C++ j'ai lu qu'un tableau était justement un pointeur vers le premier élément du tableau ! donc si je passe en référence mon tableau (&tableau), c'est un pointeur finalement non ? (référence vers un pointeur)

=> je dis ceci mais ce n'est qu'une hypothèse newbiste sûrement fausse mais je tente simplement de poser les questions telles qu'elles me viennent pour bien comprendre la notion de pointeur/référence (qui est la grosse, grosse différence avec Java...) (ps: j'ai lu pas mal de tutos mais ça ne veut toujours pas entrer en pratique, j'aimerais aussi une confirmation ou infirmation de mon "hypothèse")

Sinon j'ai compris, un pointeur (!) pour la fonction recv comme indiqué dans la définition de la fonction. Donc je retire mon &. Merci pour cette précision :).

Ok, je me répond à moi-même avant d'aller dormir :D.
Code:

1 2 octets = read(sockfd, buffer_reply_chr, MAX_REPLY_CHARS); write(STDOUT_FILENO, buffer_reply_chr, octets);
voilà, deux malheureuses lignes à ajouter (write écrit parfaitement, ce n'est pas la peine de réimplémenter la roue :D mais j'irai quand même fouiller dans les sources pour voir comment elle a été implémentée !).

Merci à tous (je mettrai en résolu dès qu'on me précise pour ce qui concerne les pointeur ci-dessus !).

Un tableau est implicitement convertible en pointeur.
Si tu tiens à utiliser &, l'équivalence d'écriture est la suivante:

Code:

(tab+offset) == &tab[offset]

Avec &tab, tu obtenais un pointeur sur ta variable tableau, pas sur le premier élément du tableau.

Citation:

for(int i = 0; i<=MAX_REPLY_CHARS; i++)

C'était <, pas <=
Mais ça va pas du tout, parce que tu ne lis pas MAX_REPLY_CHARS à chaque fois, il faut utiliser la valeur de retour de read() ou recv().

Citation:

Avec &tab, tu obtenais un pointeur sur ta variable tableau, pas sur le premier élément du tableau.

Un tableau est contigu.
Un pointeur vers un tableau et un pointeur vers son premier élément ont donc la même valeur, ce qui change c'est le type et donc l'arithmétique qui en dépend, mais de toutes façons c'est converti en void*.

Donc on peut faire
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 const int MAX_REPLY_CHARS = 5000; char buffer_reply_chr[MAX_REPLY_CHARS]; void fonction() { int n = recv(sockfd, &buffer_reply_chr, MAX_REPLY_CHARS, 0); if(n < 0) { perror("recv"); exit(1); } traiter_recv(buffer_reply_chr, n); } void Connect::traiter_recv(char* buffer, int n) { for(int i = 0; i<n; i++) { cout << buffer[i]; } }

Tu ne devrais pas initialiser ton buffer de caractères à \0 ?
Peut-être que ta fonction traiter_recv() affiche plus qu'il n'en faut du buffer ;)

loufoque > merci beaucoup !! Pour le tableau j'avais donc "raison" pour la partie "c'est pareil, le pointeur pointe vers la même chose" mais étant donné que les types des pointeurs entrent en jeux... (ici j'ai eu du bol car comme tu l'as si bien souligné, c'est convertit en void*).

Pour la fonction j'utilise write qui fait ça très bien :-). Je voulais simplement afficher sur la console.

Merci encore à tous ;).