Conversion décimale => binaire

Bonsoir tlm,
j'ai un pb un une ptite fonction pas très compliquée à mon avis :

char * itob( int n )
{
char * chaine;
int i = 0;
while ( n > 0 )
{
*(chaine+i) = ((n%2) + '0');
n = n / 2;
i++;
}
return chaine;
}

ca marche pas et jcomprend pas pk .. Si qq1 pouvait me corriger ca serait sympa svp :( (merci !)

Neo [email] [site web]

tu fait :
*(chaine+i)

mais chaine ne pointe sur rien, fait une allocation dynamique sur chaine avant !
du genre : char *chaine = new char[16];

et si tu modifie la chaine avec un pointeur, n'oublie pas de placer un zéro binaire là où il faut

1) un int c'est sur 32 bits (en tous cas pour les processeurs 32bits). donc c'est pas char[16] qu'il faut mais char[32]. Et d'ailleurs ce n'est pas la peine de faire un new char[32]...qui soit dit en passant est du C++. Il faut juste faire char chaine[32];

2) tu ne peux pas retourner un char* en esperant que ce soit une chaine de caracteres. En fait quand tu fais char * str ="coucou"; str est un pointeur sur le premier caractère. Donc ceque renvoie ta fonction c'est juste un pointeur sur le premier caractère de la valeur binaire. En fait tu dois faire qqc comme le fais scanf par exemple. C'est-à-dire prendre en paramètre un pointeur sur la variable qui va stocker le résultat.

Par exemple
void itob(int n, char* chaine)
{
...
}



On ne force pas une curiosité, on l'éveille.
.................................................Daniel Pennac

si tu fait un char chaine[32]; elle sera détruite a la fin de la fonction nan ? tandis qu'une allocation dynamique devrai rester (g des gros doute mais bon, ca me coute rien de demander).

Et je ne comprend pas pourquoi tu tiens a mettre une deuxieme param, un return chaine; marche très bien et renvoi l'adresse du premier caractère, oui

donc s'il fait :
char *chaine = itob(434);
c bon, puisque la chaine existe encore...

Enfin, je demande confirmation, on sait jamais

Mon 1) c'étatit juste pour info. Mais effectivement il ne faut pas s'en servir si on utilise le 2).

En fait si tu n'utilises pas le 2ème paramètre voila ce qui va se passe:
chaine[32] va exister tant que la fonction n'est pas terminée, et tu vas retourner l'adresse (enfin un pointeur) du premier caractère. Cependant lorsque ceci sera renvoyé la variable chaine n'existera plus. Ou du moins l'espace qu'elle occupe sera considéré comme libre. Donc si tu as de la cahnce et que rien n'y a été mis ça marchera mais cet espace a été occupé par une nouvelle allocation, tant pis pour toi.


On ne force pas une curiosité, on l'éveille.
.................................................Daniel Pennac