Pourquoi ce code ne m'affiche pas la bonne valeur ?

Pourquoi ce petit bout de code ne m'affiche pas la valeur attendue ? :(

Code:

---

1 2	unsigned char * tab = new unsigned char['h','e','l','l','o','\0']; cout << *tab << endl; //il m'affiche 'î' au lieu de 'h', pareil pour *(tab+1) il m'affiche 'à' au lieu de 'e'

---

Comment je dois faire pour afficher les valeurs correctes de mon taleau de char ?
Merci :roll:

Ne devrais-tu pas faire une boucle pour afficher chaque valeur de ton tableau ?

tab est un poiteur sur la chaîne de caractères. C'est lui qu'ils faut injecter dans le flux. Pas besoin de le déréférencer.

Code:

---

cout << tab << endl;

---

Salut,
Essaye plutôt

Code:

---

1 2 3 4

// Ce code suggère une compilation ANSI char *tab = new char[10]; sprintf(tab, "hello"); std::cout << tab << std::endl;

---

Je ne crois pas que tu puisses utiliser des chaînes de caractères avec des unsigned char*, je crois que tu es obligé de les signer.

Citation:

---

Envoyé par Mr. X

Pourquoi ce petit bout de code ne m'affiche pas la valeur attendue ? :(

Code:

---

unsigned char * tab = new unsigned char['h','e','l','l','o','\0'];

---

---

Ca ne marche pas comme ça! Entre les crochets [] tu dois mettre la taille de ton tableau. Tu ne peux pas mettre des valeurs d'initialisation.

Code:

---

1 2 3

unsigned char tab[]="hello"; std::cout<<tab<<std::endl;

---

ou

Code:

---

1 2 3 4 5 6 7 8 9

unsigned char * tab = new unsigned char[6]; tab[0]='h'; tab[1]='e'; tab[2]='l'; tab[3]='l'; tab[4]='o'; tab[5]='\0'; std::cout << tab << std::endl;

---

Citation:

---

Envoyé par Mr. X

cout << *tab << endl; //il m'affiche 'î' au lieu de 'h', pareil pour *(tab+1) il m'affiche 'à' au lieu de 'e'

---

Normal, tu n'avais alloué qu'un buffer de taille 0. Et encore, tu as de la chance, car si ta zone mémoire n'avait pas contenu un 0, tu aurais pu aller jusqu'à une zone interdite et là :aie: !

Citation:

---

Envoyé par Mr. X

Comment je dois faire pour afficher les valeurs correctes de mon taleau de char ?
Merci :roll:

---

Réviser les notions de tableau et de new[].

Citation:

---

Envoyé par Spout

Je ne crois pas que tu puisses utiliser des chaînes de caractères avec des unsigned char*, je crois que tu es obligé de les signer.

---

Sous visual et sous gcc/windows, std::cout<<unsigned char* fonctionne et ça ne me choque pas sinon cela voudrait dire une conversion implicite vers du short/int/long.

En c++ il est préfèrable d'utiliser les std::string

Citation:

---

Envoyé par Mongaulois

En c++ il est préfèrable d'utiliser les std::string

---

Sur le principe, tout à fait d'accord. std::string est faite pour ça. Mais dans son exemple, le problème ne venait pas de là.

Citation:

---

Envoyé par 3DArchi

Mais dans son exemple, le problème ne venait pas de là.

---

oui oui mais vous lui avez déjà expliqué.
C'est juste une précision car faire des char * c'est faire du C.


D'ailleurs, à votre avis que fait cette ligne? ou pourquoi les , compile?
(j'adore ce bug , c'est le gros bug de mon premier projet que j'ai dû corrigé 8-))

Code:

---

unsigned char * tab = new unsigned char['h','e','l','l','o','\0'];

---

Citation:

---

Envoyé par Mongaulois

oui oui mais vous lui avez déjà expliqué.

---

Oublie le vous.:mouarf:

Citation:

---

Envoyé par Mongaulois

D'ailleurs, à votre avis que fait cette ligne? ou pourquoi les , compile?

Code:

---

unsigned char * tab = new unsigned char['h','e','l','l','o','\0'];

---

---

Super bonne question. A la lecture, j'avais pensé qu'il faisait: évalutation de 'h' puis évaluation de 'e', etc... puis évaluation de '\0' , empilement de cette dernière évaluation et appel de new. Pour moi ',' agit comme opérateur binaire (c'est pour ça que ça compile): d'abord l'expression de gauche est évaluée, puis l'expression de droite est évaluée, et la valeur retour est l'expression de droite (http://msdn.microsoft.com/en-us/library/t29e07xd.aspx). Donc, je me serais attendu à ce qu'il fasse un new ['0'=48]. Or, Visual me génère un new[0], mais là je suis pas capable d'expliquer...
Voilà le code assembleur généré jusqu'à l'appel du new:

Code:

---

1 2 3 4 5 6 7 8 9

xor ecx,ecx mov eax,1 xor edx,edx mul eax,edx seto cl neg ecx or ecx,eax push ecx call operator new[]

---

le dernier char est '\0' pas '0', donc la valeur ascii 0, donc 0

Citation:

---

Envoyé par 3DArchi

Normal, tu n'avais alloué qu'un buffer de taille 0. Et encore, tu as de la chance, car si ta zone mémoire n'avait pas contenu un 0, tu aurais pu aller jusqu'à une zone interdite et là :aie: !

---

un bon vieux crash est bien meilleur pour montrer que ca ne marche pas qu'un comportement qui "marche presque", et quand ca "marche presque" j'appelle jamais ca de la chance, plutot de la malchance. dans un monde parfait, des que tu ferais une erreur, tu meurs :)
(a ne pas appliquer en politique)

C'est bien cela :king:
la virgule sert de Séquenceur dans les opération.
donc on fait bien un new unsigned char['\0']; => new unsigned char[0];

Bon la c'est moins drôle.
Mais je l'ai eu dans un simulateur qu'il fallait corriger où un nombre était écrit avec une virgule et non un point
http://www.developpez.net/forums/d71...float-virgule/

J'adore ce bug. Tellement subtile qu'il passe inaperçu.

[edit]
par contre je ne sait plus si résultat de 'h','e','l','l','o','\0' est définie.L'ordre est respecté(il me semble) mais que le retour utilisé non...

Citation:

---

Envoyé par 3DArchi

Sous visual et sous gcc/windows, std::cout<<unsigned char* fonctionne et ça ne me choque pas sinon cela voudrait dire une conversion implicite vers du short/int/long.

---

OK, mais je parlais de toutes les fonctions CRT pour les chaînes de caractères (sprintf, strlen, ...)

Merci pour vos réponses ! ;)

Je dois aussi remplir mon tableau de unsigned char à partir d'un unsigned entré en paramètre mais je n'arrive pas à convertir un unsigned en unsigned char :?

Code:

---

1 2 3 4 5 6 7 8 9 10 11

fonction(unsigned n) { unsigned i=0, chiffre; taille = SMB::getNbDigits(n); //cette fonction donne bien le nombre de chiffres du nombre passé en paramètre tab = new unsigned char[taille]; while(n > 0) { chiffre = n%10; n = n/10; tab[i] = (unsigned char)chiffre; //c'est ici qu'il faut que je convertisse chiffre en char mais apparement ma méthode n'est pas bonne alors comment faire ??? i++; } }

---

Code:

---

unsigned i=0

---

le type me semble incomplet: char | int | ...

signed, unsigned, long et short, impliquent par défaut "int".

Donc unsigned == unsigned int, signed long == signed long int,...

Citation:

---

Envoyé par Mr. X

Je dois aussi remplir mon tableau de unsigned char à partir d'un unsigned entré en paramètre mais je n'arrive pas à convertir un unsigned en unsigned char :?

---

Tu veux dire 0 en '0', etc..., j'imagine?

Citation:

---

Envoyé par Mr. X

Code:

---

1 2 3 4 5 6 7 8 9 10 11

fonction(unsigned n) { unsigned i=0, chiffre; taille = SMB::getNbDigits(n); //cette fonction donne bien le nombre de chiffres du nombre passé en paramètre tab = new unsigned char[taille]; while(n > 0) { chiffre = n%10; n = n/10; tab[i] = '0' +(unsigned char)chiffre; //c'est ici qu'il faut que je convertisse chiffre en char mais apparement ma méthode n'est pas bonne alors comment faire ??? i++; } }

---

---

Pour en revenir à la source du problème

Code:

---

1 2	unsigned char * tab = new unsigned char['h','e','l','l','o','\0'];

---

Visual ne dit rien sur cette ligne, mais gcc met la puce à l'oreille. Car il génère quelques warning du type:

Citation:

---

Envoyé par gcc

warning: left-hand operand of comma has no effect

---

Le code ASM généré par gcc est on ne peut plus simple:

Code:

---

1 2 3

movl $0x0,(%esp,1) call 0x402270 <operator new[](unsigned)>

---

OK, c'est ce qu'on en attend en décortiquant l'opérateur ',' (en en ne se trompant pas entre '\0' et '0':marteau: )
Par contre, visual génère :

Code:

---

1 2 3 4 5 6 7 8 9 10

xor ecx,ecx mov eax,1 xor edx,edx mul eax,edx seto cl neg ecx or ecx,eax push ecx call operator new[]

---

Et là, j'avoue que je suis perdu. Qu'est ce que cela signifie, et donc comment Visual a-t-il compris le code?
Je comprend: on met le registre ecx à 0, on met 1 dans eax, on met edx à 0, on multiplie eax par edx, donc on se retrouve avec eax à 0! Met dans cl l'overflow flag (0), complément à 2 de ecx (donc, 0), or ecx(0) avec eax(0), donc 0, puis push de 0 et appel de new. Je ne vois pas du tout le rapport avec l'instruction de départ: 'h','e','l','l','o','\0'. Si quelqu'un peut m'expliquer, ce serait avec plaisir.