vector<byte> n'est pas d'accord.

Bonjour !

Voici le code en question:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

/* le contexte */ // int len // int bytes // struct Packet { vector<byte> bytes; }; // char* substr(char*, int, int); -> cf. manuel PHP. Pareil ^^   XServer::Packet PacketFactory::CreatePacket(char *s) { len = strlen(s); bytes = floor((float)len/2)-floor((float)len/6); for(int i=1; i<=bytes; ++i) { cout << "Pushing " << substr(s, (i-1)*3, 1) << endl; Packet.bytes.push_back(substr(s, (i-1)*3, 1)); } Packet.index = 0; cout << "Created packet : " << Packet.bytes.size() << endl; for(int i=0; i<bytes; ++i) { cout << "byte " << i << Packet.bytes[i] << endl; } return Packet; }

---

Résultat (En utilisant s = "01 00 01 00 00 00 00 00 00 00 00 00 00 00")
Pushing 1
Pushing 0
Pushing 1
[..] Plein de Pushing 0
Created Packet 14
Byte 0 :(
Byte 1
Byte 2 :(
Byte 3
Byte 4
[..] tout est vide ? Rien ne s'affiche

(Et oui, les smiley sont sur ma console ! Je suis serieux xD)

De plus, j'ai essayé d'utiliser Packet.bytes.at(i), cela ne marche pas non plus !

Merci d'avance.

Pourquoi ne pas créer dans ta fonction une string du C++ initialisée avec ton s, puis travailler avec cette string du C++ et utiliser la fonction membre substr de la classe string, tu pourrais sans doute contrôler alors un peu mieux ce que tu fais à l'aide des outils du C++, non ?
(et aussi : pourquoi utiliser des float au lieu de double ?)
D'autre part qu'est-ce que c'est ce Packet.index qui d'après tes premières lignes en commentaire ne semble tout simplement pas exister ?
Et encore : pourquoi donner le même nom à deux variables nommées toutes deux 'bytes' - l'une de type int l'autre de type vector<bytes> - bien distinctes, le type même de la chose à éviter, en principe...

Bonsoir.

Pour commencer, pourquoi utiliser un char* et créer une fonction qui retourne une sous-chaîne ? (je serais curieux de voir con code, d'ailleurs...)
En C++, le type std::string est adapté à tous les usages de chaîne de caractères (avec un transtypage implicite char* :arrow: std::string, à condition que la chaîne se termine par la caractère nul)
La fonction membre std::string::substr(size_type, size_type) fait déjà ce que tu peux en attendre.

Ensuite, tu fais des transtypages à la C.
Si tu veux faire du C++, va jusqu'au bout !
F.A.Q. C++: Comment effectuer une conversion de type explicite (cast) ?
Et puis pour le cas qui nous intéresse, un simple floor(len / 2.) suffirait ; toutefois le résultat de la division devient un double.
Quoique, en y repensant, une division entière (donc sans floor) est parfaitement adaptée à ton cas.

Autre chose, tu fais une boucle sur l'intervalle [1, bytes] et une autre sur l'intervalle [0, bytes[.
Ce n'est pas très cohérent. :?
De plus, si tu commences à 0 pour ta première boucle, ça t'évitera d'avoir à faire une soustraction...

Pour finir, il y a quelque chose qui me surprend beaucoup.
Si j'ai bien suivi, substr(s, (i-1)*3, 1) est censée ne retourner qu'un caractère.
En l'occurrence, le premier caractère de chaque paire.
Alors j'ai du mal à comprendre comment tu peux transformer tes "01" en 1 ; normalement, seul le '0' devrait être lu.

Au fait, on peut créer un byte à partir d'un char* :question: :roll:

Grilled... :mrgreen:

Citation:

---

Envoyé par ptyxs

D'autre part qu'est-ce que c'est ce Packet.index qui d'après tes premières lignes en commentaire ne semble tout simplement pas exister ?

---

Bien vu, je ne l'avais pas remarqué... :oops:

Citation:

---

Envoyé par ptyxs

Et encore : pourquoi donner le même nom à deux variables nommées toutes deux 'bytes' - l'une de type int l'autre de type vector<bytes> - bien distinctes, le type même de la chose à éviter, en principe...

---

Elle ne partagent pas le même espace de nom.
Alors où est le problème ? :?

Citation:

---

Envoyé par Steph_ng8

Elle ne partagent pas le même espace de nom.
Alors où est le problème ? :?

---

En effet, pas de clash, mais il me semble que c'est chercher à ajouter un zest de confusion possible bien inutile....
:P

Ok.
vu comme ça... ;) :ccool:

Salut,

Citation:

---

Envoyé par ptyxs

Pourquoi ne pas créer dans ta fonction une string du C++ initialisée avec ton s, puis travailler avec cette string du C++ et utiliser la fonction membre substr de la classe string, tu pourrais sans doute contrôler alors un peu mieux ce que tu fais à l'aide des outils du C++, non ?
(et aussi : pourquoi utiliser des float au lieu de double ?)
D'autre part qu'est-ce que c'est ce Packet.index qui d'après tes premières lignes en commentaire ne semble tout simplement pas exister ?
Et encore : pourquoi donner le même nom à deux variables nommées toutes deux 'bytes' - l'une de type int l'autre de type vector<bytes> - bien distinctes, le type même de la chose à éviter, en principe...

---

Citation:

---

Envoyé par Steph_ng8

Bonsoir.

Pour commencer, pourquoi utiliser un char* et créer une fonction qui retourne une sous-chaîne ? (je serais curieux de voir con code, d'ailleurs...)
En C++, le type std::string est adapté à tous les usages de chaîne de caractères (avec un transtypage implicite char* :arrow: std::string, à condition que la chaîne se termine par la caractère nul)
La fonction membre std::string::substr(size_type, size_type) fait déjà ce que tu peux en attendre.

Ensuite, tu fais des transtypages à la C.
Si tu veux faire du C++, va jusqu'au bout !
F.A.Q. C++: Comment effectuer une conversion de type explicite (cast) ?
Et puis pour le cas qui nous intéresse, un simple floor(len / 2.) suffirait ; toutefois le résultat de la division devient un double.
Quoique, en y repensant, une division entière (donc sans floor) est parfaitement adaptée à ton cas.

Autre chose, tu fais une boucle sur l'intervalle [1, bytes] et une autre sur l'intervalle [0, bytes[.
Ce n'est pas très cohérent. :?
De plus, si tu commences à 0 pour ta première boucle, ça t'évitera d'avoir à faire une soustraction...

Pour finir, il y a quelque chose qui me surprend beaucoup.
Si j'ai bien suivi, substr(s, (i-1)*3, 1) est censée ne retourner qu'un caractère.
En l'occurrence, le premier caractère de chaque paire.
Alors j'ai du mal à comprendre comment tu peux transformer tes "01" en 1 ; normalement, seul le '0' devrait être lu.

Au fait, on peut créer un byte à partir d'un char* :question: :roll:

---

Attention, si le char est en réalité destiné à renvoyer des valeurs numérique (comprises entre 0 et 255), comme on peut raisonnablement le penser, la solution ne passe pas par la

Code:

---

std::string.

---

En effet, la std::string ne peut afficher que les caractères ayant une valeur supérieure à32 (le caractère représenté par l'espace).

De toute évidence, si on essaye de pousser une valeur "01" (qui est en fait 0x01 et qui correspond simplement... à 1 ;) ) c'est que l'on n'est pas dans le cas où la std::string est utile.

Dans ce cas, il faut bel et bien travailler avec des tableaux de char, et même de unsigned char ;)

@ DakM : j'aimerais en savoir plus sur les valeurs que tu reçois, et surtout sur le protocole qui te permet de les recevoir.

En effet, tu semble vouloir utiliser de byte, c'est à dire des (unsigned) char pour la valeur qu'ils représentent comprise entre 0 et 255 (car '1' ne vaut pas ... 1 :aie:).

Si, comme je le pense du moins, c'est le cas, c'est tout ton système de récupération des valeurs et de remplissage du packet qu'il faudra revoir ;)

Le problème, c'est que je ne pars que de suppositions peut etre fausses, et il est donc nécessaire d'en savoir plus ;)

Hum...
Tel que présenté, j'ai l'impression que le char * est utilisé comme une simple chaîne de caractères.
Donc std::string paraît tout à fait approprié, quitte à utiliser std::string::c_str().

Sauf si la chaîne d'entrée est en réalité "0x01 0x00 0x01 0x00...".
Dans ce cas-là, d'accord.
Quoique rien ne lui interdit d'utiliser std::basic_string<unsigned char>. ;)

Sinon, encore une fois, je suis curieux de voir le code de substr...

Citation:

---

Envoyé par Steph_ng8

Hum...
Tel que présenté, j'ai l'impression que le char * est utilisé comme une simple chaîne de caractères.
Donc std::string paraît tout à fait approprié, quitte à utiliser std::string::c_str().

Sauf si la chaîne d'entrée est en réalité "0x01 0x00 0x01 0x00...".
Dans ce cas-là, d'accord.
Quoique rien ne lui interdit d'utiliser std::basic_string<unsigned char>. ;)

Sinon, encore une fois, je suis curieux de voir le code de substr...

---

Le problème, c'est qu'à voir la série à obtenir, j'aurais tendance à dire qu'il s'agit de byte, et donc de séries de bits...

D'autant plus que, si l'on regarde correctement le code, il ne prend que l'équivalent d'un caractère sur trois, si, du moins substr agit bel et bien comme on peut l'envisager avec un prototype proche de std::string substr(std::string const & in, int start, int count).

De plus, std::basic_string<unsigned char> posera de toute manière exactement le même problème dans le sens où... elle considérera les unsigned char comme devant être affichés sous la forme de leur représentation graphique dans la table ASCII et non sous la forme de leur valeur.

En définitive, si les (unsigned ) char doivent, effectivement etre considérés comme des valeurs numérique, il ne reste qu'une solution cohérente : l'usage d'un std::vector< (unsinged) char> ;)

Oui, mais le malheur est que sur toutes ces questions nous en somme réduits à des suppositions, l'auteur n'ayant pas expliqué de façon suffisamment précise ce qu'il essayait de faire vraiment... La balle est maintenant dans son camp...

Citation:

---

Envoyé par koala01

D'autant plus que, si l'on regarde correctement le code, il ne prend que l'équivalent d'un caractère sur trois, si, du moins substr agit bel et bien comme on peut l'envisager avec un prototype proche de std::string substr(std::string const & in, int start, int count).

---

C'est bien ça qui me chiffonne... :?

Citation:

---

Envoyé par koala01

De plus, std::basic_string<unsigned char> posera de toute manière exactement le même problème dans le sens où... elle considérera les unsigned char comme devant être affichés sous la forme de leur représentation graphique dans la table ASCII et non sous la forme de leur valeur.

---

Ah...
Et il se passe quoi pour les caractères dont le code ASCII est compris entre 0 et 32 (exclus) ?
Je veux dire, tant qu'on ne cherche pas à les afficher ?
Au fait, de mémoire, les codes ASCII de '\n', '\r' et '\t' sont compris entre 0 et 32... :roll:

Citation:

---

Envoyé par koala01

En définitive, si les (unsigned ) char doivent, effectivement etre considérés comme des valeurs numérique, il ne reste qu'une solution cohérente : l'usage d'un std::vector< (unsinged) char> ;)

---

À mon avis (mais ce n'est qu'une autre supposition ;)), le but du problème est justement de transformer des données d'entrée (clavier ? fichier ?) en tableau de byte...

Citation:

---

Envoyé par Steph_ng8

C'est bien ça qui me chiffonne... :?

---

C'est bien pour cela que j'ai pris les disposition d'usage en disant "si c'est bien ce qu'on croit :D

Citation:

---

Ah...
Et il se passe quoi pour les caractères dont le code ASCII est compris entre 0 et 32 (exclus) ?
Je veux dire, tant qu'on ne cherche pas à les afficher ?
Au fait, de mémoire, les codes ASCII de '\n', '\r' et '\t' sont compris entre 0 et 32... :roll:

---

Oui, et tu as aussi le bell, le esc, le '\b', et tous les autres...

Essaye de les introduire dans une std::string, et de les afficher... tu verra le résultat :D... mais prévois des boulles quies :D

Citation:

---

À mon avis (mais ce n'est qu'une autre supposition ;)), le but du problème est justement de transformer des données d'entrée (clavier ? fichier ?) en tableau de byte...

---

Non, pour autant que j'en sache (grace aux différentes discussions récentes, le but est de récupérer les données depuis un socket :P

Citation:

---

Envoyé par koala01

Essaye de les introduire dans une std::string, et de les afficher... tu verra le résultat :D... mais prévois des boulles quies :D

---

Ah donc c'est bien ce que je pensais, c'est lorsqu'on veut les afficher qu'il y a un problème...

Citation:

---

Envoyé par koala01

Non, pour autant que j'en sache (grace aux différentes discussions récentes, le but est de récupérer les données depuis un socket :P

---

Ah oui, c'est vrai. :)

Bon, je crois que l'on peut arrêter les suppositions et attendre que DakM vienne mettre d'autres informations... :roll:

Citation:

---

Envoyé par Steph_ng8

Ah donc c'est bien ce que je pensais, c'est lorsqu'on veut les afficher qu'il y a un problème...

---

Et pour cause: la table ASCII est, justement, une table utilisée pour l'affichage :D

Ceci dit, il risque aussi d'y avoir des problèmes à l'interprétation, purs et simples : '\0' met classiquement fin à la chaine de caractères (tout caractère qui suit le '\0' est ignoré, fut-il affichable :D), alors qu'une valeur nulle est tout à fait possible dans un paquet de données, sans pour autant représenter la fin du paquet :D

Citation:

---

Envoyé par koala01

Ceci dit, il risque aussi d'y avoir des problèmes à l'interprétation, purs et simples : '\0' met classiquement fin à la chaine de caractères (tout caractère qui suit le '\0' est ignoré, fut-il affichable :D), alors qu'une valeur nulle est tout à fait possible dans un paquet de données, sans pour autant représenter la fin du paquet :D

---

Ah oui, exact... :)

bonjour !
Whoa tellement de réponses !

Alors:
Fonction substr():

Code:

---

1 2 3 4

int substr(char *s, int start, int length) { char t[2] = {s[start], s[(start+length)]}; return (int)strtol(t, NULL, 16); }

---

(Hé oui, elle est spécifique)

Pourquoi ?

Le dialogue client/Server se fait avec ceci: (exemple)
Bonjour: 01 00 00 01
Bonjour moi c'est le serveur: 01 00 01 01 00 01
tu vas bien ?: 02 00 00 05
moi c'est developpez.com: 05 ff 03 64 6f

Donc mon substr

Citation:

---

Envoyé par DakM

bonjour !
Whoa tellement de réponses !

Alors:
Fonction substr():

Code:

---

1 2 3 4

int substr(char *s, int start, int length) { char t[2] = {s[start], s[(start+length)]}; return (int)strtol(t, NULL, 16); }

---

(Hé oui, elle est spécifique)

Pourquoi ?

---

Parce qu'elle permet de lever un doute sur son utilisation ;)

Une petite remarque en passant: Si tu renvoie d'office un char[2], la taille n'a strictement pas lieu d'être utilisée (tu peux supprimer l'argument et utiliser directement char t[2] = {s[start], s[(start+1)]} d'autant plus que, telle qu'elle est écrite, elle ment sur l'utilisation de l'argument length, qui semble indiquer une taille, alors qu'il indique plutot un nombre de caractères à passer entre le premier caractère qui sera pris (et qui se trouve à la position start) et celui qu'il faudra placer en deuxième position :aie:
Amuse toi par exemple à observer la sortie de ce tout petit programme..

Code:

---

1 2 3 4 5 6 7 8 9 10 11

int main() { char *str="Bonjour\0"; int i = 0; while(str[i]!='\0') { std::cout<<substr(str,0,i)<<std::endl; ++i } return 0; }

---

Tu verra, ca vaut le coup, et je ne crois pas que ce soit ce à quoi tu t'attendais :D

Citation:

---

Le dialogue client/Server se fait avec ceci: (exemple)
Bonjour: 01 00 00 01
Bonjour moi c'est le serveur: 01 00 01 01 00 01
tu vas bien ?: 02 00 00 05
moi c'est developpez.com: 05 ff 03 64 6f

Donc mon substr

---

Oui, mais on ne sait toujours pas si 01, 00, 02,05,ff,03,64 et 6f sont envoyés sous la forme de chaine ou sous la forme de valeur!

Et tout le problème vient de là!!!

En effet, si c'est sous forme de chaine de caractères (qu'on les manipule à la mode C ou à la mode C++, cela n'a pas énormément d'importance ;) ), il faut prendre deux caractères et les "triturer" pour récupérer une valeur qui entre dans un byte (qui, classiquement correspond à la taille d'un caractère), alors que si c'est sous la forme de valeur (représentée ici en format hexadécimal), il y a peut etre des choses plus intéressantes à faire ;)

Je recois ca par un socket, donc le buffer est un char *buf = "01 01 01 01 01 01"!

Citation:

---

Envoyé par DakM

Je recois ca par un socket, donc le buffer est un char *buf = "01 01 01 01 01 01"!

---

Ben, tu dis cela comme si cela allait de soi, mais cela aurait pu etre un groupe de 6 bytes (donc de 6 ensemble de 8 bits :D) et 01 aurait pu représenter... la valeur hexadécimale reçue

Pour info, tu peux recevoir par socket des informations issues de port COM et similaires qui sont en réalité ... des ensembles de 8 bits, ayant des valeurs correspondant à des caractères non affichables de la table ASCII.

Mais, dans ce cas, je te conseillerais fortement de virer au plus tot ton char* pour utiliser la classe string, accessible dans l'espace de noms std par inclusion du fichier <string>

Ta fonction pourrait alors ressembler à quelque chose comme

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

XServer::Packet PacketFactory::CreatePacket(std::string const & buf) { std::vector<std::string> prepare; // XServer::Packet serait un typedef? std::stringstram ss(buf) /* on commence par séparer chaque paire de caractères représentatifs   while(ss)   {   std::string temp;   ss>>temp;   prepare.push_back(temp);   }   /* allez, amusons nous à les afficher */ for(size_t i=0;i<prepare.size();++i) { std::cout<<"elemen ["<<i<<"] :"<<prepare[i]<<std:endl; } /* si XServer::Packet n'est pas un typedef de std::vector<std::string>   * on convertit chaque chaine présente dans prepare ;)   */ return /* ce qui soit prepare, soit le résulat de la conversion */ }

---

Oui mais je recois en tant que char*, n'est-il pas couteux de faire une convertion vers un string au lieu de rester sur du char* et de bosser avec ca ?

Citation:

---

Envoyé par DakM

Oui mais je recois en tant que char*, n'est-il pas couteux de faire une convertion vers un string au lieu de rester sur du char* et de bosser avec ca ?

---

Pas outre mesure : il existe un constructeur de string prenant un char * en paramètre qui te permet meme de passer directement ton char* à la fonction ;)

[EDIT]En outre, il faut vérifier, mais si tu suis mon conseil de te tourner vers boost asio pour la gestion de tes socket, je serais surpris que tu ne puisse pas récupérer les informations directement sous la forme de std :: string ;)

Etant donné que je ne suis pas bon, je preferes pour l'instant ne pas utiliser de grosses biblio lourdes et dont je n'utiliserais pas 1% des fonctionnalités !

Salut,

Citation:

---

Envoyé par DakM

Code:

---

1 2	cout << "byte " << i << Packet.bytes[i] << endl;

---

---

Si bytes est de type char, la sortie n'est pas numérique mais le charactère correspondant, c'est à dire probablement quelque chose de farfelu si hors de la plage des caractères ASCII affichables. Forces le type numérique pour voir la valeur :

Code:

---

cout << "byte " << i << std::hex<< static_cast<unsigned>(Packet.bytes[i]) << endl;

---

S'il s'agit d'octets reçus et à traiter en tant que tels, il vaut mieux utiliser un type non signé : struct Packet{std::vector<unsigned char> bytes};

Citation:

---

Envoyé par DakM

Etant donné que je ne suis pas bon, je preferes pour l'instant ne pas utiliser de grosses biblio lourdes et dont je n'utiliserais pas 1% des fonctionnalités !

---

C'est à peu près l'inverse qui est recommandé : utilise des bibliothèques éprouvées si tu débutes, tu bénéficiera de facto de l'expérience de ceux qui l'ont construite. Faire par soi même quand on ne maîtrise pas, c'est la meilleure façon de faire le plus de bug possible.

Citation:

---

Envoyé par DakM

Code:

---

// char* substr(char*, int, int); -> cf. manuel PHP. Pareil ^^

---

---

Citation:

---

Envoyé par DakM

Fonction substr():

Code:

---

1 2 3 4

int substr(char *s, int start, int length) { char t[2] = {s[start], s[(start+length)]}; return (int)strtol(t, NULL, 16); }

---

(Hé oui, elle est spécifique)

---

Hum...
Tu te contredis... :?

J'ajouterais juste aux remarques de koala01 que la fonction strtol attend une chaîne de caractères terminée par un caractère nul en premier paramètre.
Et vu comment tu l'appelles, il n'y a aucun moyen de le garantir, ni même de le prévoir.
Ce problème serait résolu en écrivant quelque chose comme ceci (modulo les modifications adéquates, cf. remarques de koala01) :

Code:

---

char t[] = {s[start], s[(start+length)], '\0'};

---

Vraiment c'est étrange le C++. Il y a un truc que j'capte pas ici.

Voici un screen de VS durant le debug!

http://img847.imageshack.us/img847/5863/wtfffffn.png

On voit le char* buffer, avec 01 00 01 ...
Et on voit packet, avec 1 ' ', 0, 1 ' ', 0,0,0,0,0 ....
C'est quoi ce blanc ? (Il correspond aux smiley qui s'affichent sur l'écran au lieu des 0 ...)

Sinon eh bien, mes chars ne vont toujours pas dans mon vector, (enfin ils y vont, mais je n'arrivent pas à les lires!

voici substr, mais changée:

Code:

---

1 2 3 4

char getbyte(char *s, int at) { char t[3] = {s[at], s[(at+1)], '\0'}; return (char)strtol(t, NULL, 16); }

---

voici là ou je construit mon vector<unsigned char> (changé de byte!)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

XServer::Packet PacketFactory::CreatePacket(char *s) { len = strlen(s); bytes = floor((float)len/2)-floor((float)len/6); for(int i=1; i<=bytes; ++i) { cout << "Pushing " << getbyte(s, (i-1)*3) << endl; Packet->bytes.push_back(getbyte(s, (i-1)*3)); } Packet->index = 0; cout << "Created packet : " << Packet->bytes.size() << endl; for(int i=0; i<bytes; ++i) { cout << "byte " << i << Packet->bytes.at(i) << endl; } return *Packet; }

---

Rien que sur le premier cout, j'obtient à l'écran
Pushing :)
Pushing
Pushing :)
Pushing
...

Citation:

---

Envoyé par koala01

@ DakM : j'aimerais en savoir plus sur les valeurs que tu reçois, et surtout sur le protocole qui te permet de les recevoir.

---

Il me semble, au point où nous en sommes, que, pour toi, la chose essentielle à faire pour avancer est de tout mettre en œuvre pour répondre à cette question !! (ou bien doit-on comprendre que tu traites des données dont tu ne connais pas vraiment la nature ??)

Il me semble avoir deja répondu à ca.

Le client & le serveur communique en "hexadecimal".

Exemple, pour que le client s'identifie, il enverra:

01 04 6e 69 63 6f 0a 6d 6f 74 64 65 70 61 73 73 65
01 = header du packet qui dira au server que le client essaye de s'identifier.
04 = longueur de l'identifiant.
6e 69 63 6f = identifiant (nico)
0a = longueur du MDP.
6d 6f 74 64 65 70 61 73 73 65 = motdepasse

Et le serveur, en fonction des verifications aupres de la BD. répondra
01 01 (connecté!)
ou
01 02 (acces refusé!)

Et ca continue !

Juste histoire d'être sûr...
Désolé d'en remettre une couche, mais je pense qu'il vaut mieux passer suffisamment de temps pour bien comprendre ça, plutôt que de ne pas partir sur les mêmes bases et s'en rendre compte dans 3 ans...

strlen("01 04 6e 69 63 6f 0a 6d 6f 74 64 65 70 61 73 73 65") vaut 17 ou 50 ?

Citation:

---

Envoyé par DakM

Il me semble avoir deja répondu à ca.

Le client & le serveur communique en "hexadecimal".

---

Non, tu n'avais jamais répondu à cela, autrement, nous n'aurions pas insisté pour que tu nous le dise :D

Maintenant, nous allons pouvoir t'aider ;)

Exemple, pour que le client s'identifie, il enverra:

01 04 6e 69 63 6f 0a 6d 6f 74 64 65 70 61 73 73 65
01 = header du packet qui dira au server que le client essaye de s'identifier.
04 = longueur de l'identifiant.
6e 69 63 6f = identifiant (nico)
0a = longueur du MDP.
6d 6f 74 64 65 70 61 73 73 65 = motdepasse

Et le serveur, en fonction des verifications aupres de la BD. répondra
01 01 (connecté!)
ou
01 02 (acces refusé!)

Et ca continue ![/QUOTE]Bon, maintenant, nous pouvons te proposer une solution.

La première chose à faire, c'est bien de récupérer ce que le serveur envoie sous la forme de unsigned char non pas dans un unsigned char * mais dans un std::vector< unsigned char>.

Tu n'auras alors même plus à t'inquiéter de la conversion, parce que ce que tu obtiens est déjà... un tableau de byte!

Et cela t'évitera un tas de problème du fait des caractères non affichables (parce que tu ne considère pas ce que tu obtiens comme l'index de la table ASCII mais bien comme... une valeur numérique de petite taille).

Tous tes problèmes viennent de là!

Comme je te l'ai dit plus haut '1' ne vaut pas 1 !!!

1 en tant que valeur (en tant qu'index dans la table ASCI) est est caractère non affichable SOH (Start Of Header) alors que '1' en tant que représentation graphique se trouve en réalité à l'index... 49 (décimal) ou, en hexadécimal 0x31.

Si j'insiste là dessus, c'est parce qu'il est important que tu comprennes la différence.

Pour faire un parallèle, '1' sera ce que tu lira en ouvrant un fichier texte avec notepad (ou autre) alors que 1 sera ce que tu obtiens en lisant un fichier binaire.

Ici, ce que tu obtiens, c'est un flux équivalent à un fichier binaire, tu ne peux donc pas le gérer comme si c'était du texte!

Citation:

---

Envoyé par Steph_ng8

Juste histoire d'être sûr...
Désolé d'en remettre une couche, mais je pense qu'il vaut mieux passer suffisamment de temps pour bien comprendre ça, plutôt que de ne pas partir sur les mêmes bases et s'en rendre compte dans 3 ans...

strlen("01 04 6e 69 63 6f 0a 6d 6f 74 64 65 70 61 73 73 65") vaut 17 ou 50 ?

---

Eh bien si je recois dans mon char* buffer, strlen=50.
Ensuite pour récuperer le nombre de bytes, je fais:

Code:

---

floor((float)strlen(buf)/2)-floor((float)strlen(buf)/6)

---

ensuite je lit chaque byte avec:

Code:

---

1 2 3 4 5

for(int i=1; i<=bytes; ++i) { cout << "Pushing " << getbyte(s, (i-1)*3) << endl; Packet->bytes.push_back(getbyte(s, (i-1)*3)); }

---

Donc pour le byte 1, je vais ajouter a ma pile le charactere (i-1)*3 et celui qui suit. Et transformer ca en hex (ce que doit faire ma fonction getbyte.)

Citation:

---

Envoyé par koala01

Non, tu n'avais jamais répondu à cela, autrement, nous n'aurions pas insisté pour que tu nous le dise :D

Maintenant, nous allons pouvoir t'aider ;)

[..]
Bon, maintenant, nous pouvons te proposer une solution.

La première chose à faire, c'est bien de récupérer ce que le serveur envoie sous la forme de unsigned char non pas dans un unsigned char * mais dans un std::vector< unsigned char>.

Tu n'auras alors même plus à t'inquiéter de la conversion, parce que ce que tu obtiens est déjà... un tableau de byte!

Et cela t'évitera un tas de problème du fait des caractères non affichables (parce que tu ne considère pas ce que tu obtiens comme l'index de la table ASCII mais bien comme... une valeur numérique de petite taille).

Tous tes problèmes viennent de là!

Comme je te l'ai dit plus haut '1' ne vaut pas 1 !!!

1 en tant que valeur (en tant qu'index dans la table ASCI) est est caractère non affichable SOH (Start Of Header) alors que '1' en tant que représentation graphique se trouve en réalité à l'index... 49 (décimal) ou, en hexadécimal 0x31.

Si j'insiste là dessus, c'est parce qu'il est important que tu comprennes la différence.

Pour faire un parallèle, '1' sera ce que tu lira en ouvrant un fichier texte avec notepad (ou autre) alors que 1 sera ce que tu obtiens en lisant un fichier binaire.

Ici, ce que tu obtiens, c'est un flux équivalent à un fichier binaire, tu ne peux donc pas le gérer comme si c'était du texte!

---

J'ai bien compris la difference entre 0x01 et 0d01
Comment faire pour faire un recv() sur un socket qui transforme la reception (donc quelque chose du style 01 01 01 01) en vector<unsigned char> directement, sans passer par mes fonctions ?
Car VS ne veux meme pas que j'utilises de std::string... (D'ou mon choix de ne pas utiliser ca ..)

nico

Citation:

---

Envoyé par DakM

J'ai bien compris la difference entre 0x01 et 0d01
Comment faire pour faire un recv() sur un socket qui transforme la reception (donc quelque chose du style 01 01 01 01) en vector<unsigned char> directement, sans passer par mes fonctions ?
Car VS ne veux meme pas que j'utilises de std::string... (D'ou mon choix de ne pas utiliser ca ..)

nico

---

Exactement comme je le dis depuis le début:

• utilise un tableau de unsigned char ( donc avec un unsigned char* buffer, si tu veux ;))
• manipule ce tableau non pas comme s'il s'agissait d'une chaine de caractères mais bien comme ce que c'est réellement à savoir : un tableau de valeurs numérique, ce qui revient à... ne pas utiliser la moindre fonction strXXX

Si tu suit ces deux instructions toutes bêtes, tu auras beaucoup moins de problèmes ;)

Mais, j'y penses... le serveur, est-ce toi qui t'en occupe :question:

Montre nous un peu la partie qui construit la réponse et qui l'envoie, STP...

De cette manière, nous pourrons avoir la certitude de la manière dont les données sont émise ;)

Je n'ai meme pas fais le client encore, dabord je fais mon serveur (qu'il puisse traiter les packet et que les threads marchent bien, ensuite je ferais le client ! =)

Le truc que je ne comprend pas, cmment utiliser recv() avec un unsigned char *buffer pour pouvoir le traiter avec des valeurs numériques...
La fonction c'est recv(SOCKET, char*, int, int);

Citation:

---

Envoyé par DakM

Je n'ai meme pas fais le client encore, dabord je fais mon serveur (qu'il puisse traiter les packet et que les threads marchent bien, ensuite je ferais le client ! =)

Le truc que je ne comprend pas, cmment utiliser recv() avec un unsigned char *buffer pour pouvoir le traiter avec des valeurs numériques...
La fonction c'est recv(SOCKET, char*, int, int);

---

Peut etre as tu alors une spec du protocole à utiliser :question:

Laisses moi deviner, vas tu me poser la question de "c'est quoi une specs?" maintenant :question:

Quoi qu'il en soit, je ne peux que répéter ma recommandation d'utiliser boost asion pour l'ensemble de ton projet.

Les tutoriaux aidant, tu arriveras beaucoup plus facilement et rapidement à un résultat correct et qui plus est portable que si tu dois apprendre à gérer tes sockets et tes threads avec les bibliothèques C et sous window!

Le propre d'un bon programmeur, lorsqu'il est confronté à un problème qu'il n'a jamais rencontré n'est pas forcément de s'arracher les cheveux à tout faire par lui-même, mais de commencer par voir si, par hasard, il n'existerait pas une bibliothèque éprouvée qui puisse lui servir de base!

Nous avons d'ailleurs eu, il y a quelques mois, un débat fort intéressant sur l'utilisation de bibliothèques et le syndrome NIH (Not Implemented Here)

Quand on pense que boost sera ton "compagnon de voyage au pays de C++" exactement au même titre que la STL, on se dit qu'il serait vraiment aberrant, meme pour un débutant, meme si c'est pour n'utiliser qu'une des bibliothèque qu'elle offre, alors que tu ne vois meme pas comment convertir un char* en unsigned char *, ce qui semble indiquer qu'il te manque peut etre quelque notions de base en C++ !

Plus vite tu prendras l'habitude d'écrire du C++ et non du C et plus vite tu te mettras à l'utilisation de la STL et de parties de boost qui t'apparaitront utiles pour les besoins que tu rencontre de manière journalière, plus vite tu pourras évoluer et devenir efficace en C++!!!