ÉCRITURE DE ITOA

Commentaire de acx01b le 14/06/2007 12:21:36

salut

à mon avis tu pourrais renvoyer un int : le nombre de caractère écrits au lieu de renvoyer un char *

et tu pourrais aussi écrire directement à l'envers dans chaine comme ça tu pourrais copier chaine dans resultat avec un memcpy

ton const char num devrait être déclaré static ?

pour mettre le signe moins tu fais deux tests un seul suffirait

tu utilises des indices sur tes pointeurs, alors que tu pourrais directement incrémenter et décrémenter les pointeurs, ça simplifirait le travail d'optimisation du compilateur

la macro MAX je la mettrais à 64 au moins ça ne coûte rien

et enfin il y a un bug quand on fait itoa de 0

a+

Commentaire de acx01b le 14/06/2007 13:07:13

salut

static est très simple: une variable déclarée static c'est une variable "globale" acessible que dans le fichier, la fonction, ou le bloc où tu l'as déclarée

c'est à dire qu'elle ne sera pas localisée dans la pile, et donc elle ne sera pas crée au début de la fonction et détruite à la fin de la fonction

par exemple:

int func() {
   static int k = 0;
   return k++;
}

printf(" %d ",func()); // 0 : premier appel
printf(" %d ",func()); // 1
printf(" %d ",func()); // 2
printf(" %d ",func()); // 3
...

Commentaire de thomwebster le 14/06/2007 13:38:36

Je ne vois pas l'utilité de renvoyer le nombre de caractères écrits... Une chaine, ça se parcours jusqu'à trouver '\0' donc il est inutil de renvoyer le nombre de caractères écrits... De plus c'est une écriture de la fonction itoa, j'ai donc respecté le typage de cette fonction...

Commentaire de thomwebster le 14/06/2007 14:09:44

Donc si j'ai bien compris, ce qu'il faut que je retourne, c'est le pointeur qui pointe sur le '\0' de ma chaine?

Commentaire de BruNews le 14/06/2007 14:19:33 administrateur CS

ben oui, si j'écris (ou REécris) une fonction, c'est pour obtenir le mieux possible. Si sprintf me suffit, aucun besoin de fonction perso.

A quoi sert la chaine[MAX] dans ta fonction ???
void ReverseStr(char *psz)
{
  char *c = psz, a;
  while(*c) c++;
  while(psz < --c) {a = *psz; *psz++ = *c; *c = a;}
}
quan on voit que le renversement se limite à "while(psz < --c)" autant écrire direct sur le buffer passé en param, sans compter qu'on s'épargne le count inutile en ce cas.