bonjour à tous,
j'ai un problème qui me prend la tête depuis hier :
j'ai écrit une petite fonction qui ajoute un '0' à un entier passé en argument s'il est inférieur à 10 et renvoie un pointeur sur une chaîne de char :
 
char *AjouteZero (int entier)
{
   char chaine [3],tmp [3],*p;
 
   if (entier<=9)
   {
      strcpy (chaine,"0" );
      strcat (chaine,itoa (entier,tmp,10));
   }
   else
   {
      strcpy (chaine,itoa (entier,tmp,10));
   }
   p=chaine;
   return (p);
}
 
le problème est que quand j'essaie de récupérer la valeur renvoyée ça veut pas ...
j'essaie comme ça : strcpy (tmp,AjouteZero (8)); où tmp est : char tmp [10] => j'obtient des hiéroglyphes en faisant un printf de tmp
ou bien en faisant directement un printf ("\n%s\n",AjouteZero (8));
 
ce qui m'épate c'est que j'ai une autre fonction que j'utilise depuis des années qui est pareille au niveau des types de variables et ça marche sans problème.
 
merci d'avance pour toute aide.

L'espace alloué à "chaine" n'existe qu'à l'intérieur de ta fonction.
Utilise malloc() pour pouvoir propager ta variable.
 
Et utilise plutôt snprintf() que itoa(), qui n'est pas standard.

ça c'est de la réponse rapide !
c'est ce que je suspectais, mais ce que je comprends pas c'est que ça marche dans un autre cas :
 
char *litchamp (int position,char lecligne[])
{
 int i,j;
 char lec[1000],*point;
 
 i=1;
 j=0;
 while (i<position)
 {
  while (lecligne[j]!='|' && lecligne[j]!='\0' && lecligne[j]!='\n')
  {
   j++ ;
  }
  j++;
  i++;
 }
 i=j;
 while (lecligne[j]!='|' && lecligne[j]!='\0' && lecligne[j]!='\n')
 {
  j++;
 }
 strncpy (lec,&lecligne[i],j-i);
 lec[j-i]='\0';
   point = lec;
 return point;
}
 
la seule différence c'est que LecLigne est globale, mais elle est recopiée dans lec qui elle est locale à la fonction, donc ça ne devrait pas marcher non plus ...
 
enfin le principal c'est d'avoir la solution, merci Elmoricq

3 n'est pas un nombre suffisant pour stocker un entier non-signée dans une chaine de caractères

 
c'est pour des nombres de 2 chiffres au max (les jours et les mois en fait) donc avec le '\0' ça fait 3 ...

Juste une question : tu fais ça pour apprendre à manipuler les chaines de caractères (donc dans ce cas programmer toi-même cette fonction est un but en soi), ou bien tu en as juste besoin pour une appli de plus grande envergure ?
 
Dans le deuxième cas, je te conseille d'utiliser tout simplement un snprinft("%02d", ... ) qui fera le boulot très bien sans que tu aies à t'occuper de ce qui se passe derrière.

qui t'empeche de faire du code propre?

 
Sauf erreur, c'est juste un coup de bol. Comportement aléatoire.

ça marche à tous les coups, j'ai plusieurs programmes qui tournent toutes les nuits avec cette fonction depuis plusieurs années et y aucun soucis.
 
maintenant c'est vrai que c'est pas logique ...
 
va vraiment falloir que je reprenne tous mes vieux prg, doit y avoir des belles merdes dedans ...
 
merci encore

Quelle est l utilité de retourner p ???
p = chaine ne sert a rien ... tu devrais mieux retourner directectement "chaine", et tu pourrais te passer de "p"

ben on peut pas retourner une chaîne en C ... c'est soit une variable simple (int, char, float, ...) soit un pointeur (sauf erreur ou omission ...)

 

 
En plus, ton char *p = lec, ça permet juste de masquer le warning suivant :

 
L'affectation à p n'enlève absolument rien au fait que tu retournes l'adresse d'une variable locale. C'est un pur coup de bol si rien n'écrase cet endroit de la mémoire au moment où tu l'utilises.
 
Il aurait fallu écrire (j'en profite au passage pour effectuer quelques modifications cosmétiques) :

Le type chaine n existe pas en C. La preuve avec le code suivant j ai remis au propre ton code tout simplement -> ca fait la meme chose en trois lignes et sans faire appel a des fonctions de lib externes (a part malloc).
 

Regarde bien j ai bien un pointeur au depart (char *str)
j alloue 3 octets pour stocker la chaine de carateres
ensuite quelques manipulations simples basees sur la table ASCII et un cast
si tu fais printf("%s", str); tu as bien ce que tu voulais.
tu peux donc en deduire que la chaine de carateres n est autres qu un pointeur en C.
Si je n ai pas ete clair dit le moi

effectivement ç'est mieux ... par contre les modifications cosmétiques, comme tu dis, j'ai pas trop l'habitude ... mais va falloir que je m'y mette. à ma décharge, je n'ai eu qu'une semaine de formation sur le c ansi, on n'a même pas vu les pointeurs ni l'allocation dynamique ...
du coup je programme à la bourrin depuis des années (que avec des tableaux et les pointeurs quand vraiment je peux pas faire autrement, et le malloc que quand y a pas assez de place en mémoire).
 
mais bon la j'arrête, je veux faire bien.
 
sur tes conseils j'ai modifié ma fonction comme ça :
 
char *AjouteZero (int entier)
{
   char *chaine,tmp [3];
 
   chaine=(char *) malloc (3*sizeof (char));
   if (entier<=9)
   {
      strcpy (chaine,"0" );
      strcat (chaine,itoa (entier,tmp,10));
   }
   else
   {
      strcpy (chaine,itoa (entier,tmp,10));
   }
   return (chaine);
}
 
et ça marche nickel. on verra après pour snprintf et autres standardisation et amélioration, faut que j'avance un peu dans la suite du prg.

Au fait je vois pas mal d exemple de code dans lesquels il n y a pas de test de valeur de retour lors d utilisation de fonctions provenant de librairies externes ... je ne l ai pas fait juste avant mais il faut toujours tester la valeur de retour d une allocation dynamique de memoire car d une part ca permet d eviter les SEGFAULT et au moins quand on a un probleme on sait d ou ca vient

 
ok, c'est ce que j'ai fait aussi, pour le pointeur. ce que je voulais dire c'est qu'on ne pouvait pas retourner un tableau de char (ce que j'appelle une chaîne de caractères).
 
par contre je suis pas trop habitué aux trucs style : (i / 10 != 0) ? (char)(i / 10 + 48) : '0'; j'aime bien qu'il y ait des if, strcpy, etc ...
question d'habitude.
merci à toi

 
exact, ça par contre j'en mets un max de tests et tout tracé dans une log, là je voulais juste faire fonctionner le bazarre, je vais blinder après

surtout eviter les strcpy, c est une fonction qui n est pas fiable et qui peut permettre des buffer overflow ... m enfin apres c est au niveau de la securite du programme, je ne pense pas que ca te gene enormement.
 

attention tout de meme a l utilisation du if (pour les test), c est l une des instructions les plus gourmandes en ressources, il faut eviter d en abuser et s en passer permet souvent de gagner pas mal de cycle d execution. Mais ca n est la encore qu une question d optimisation.
 

il ne s agit que d une facon d ecrire une condition if / then / else. De plus j utilise des cast (char) ce qui est beaucoup moins gourmand que itoa dans ton cas car tu ne traite que des petits nombres.
Autre reproche, n utilise pas strcat dans ta condition else, en effet itoa alloue deja de la memoire et strcat le fait aussi c est donc inutile est tu pourrais ecrire par consequent chaine = itoa(entier, tmp, 10);, de plus avec strcat tu concatene une chaine vide (chaine) avec la valeur de retour de itoa, ce qui n est pas tres judicieux.
 
Fais attention avec les allocations memoires, car c est la encore une operation tres couteuse en ressource processeur (je sais de quoi je parle ... j ai recode malloc() free() et realloc()).
 
Mais ce ne sont que des conseils, libre a toi de faire ce qu il te plait  

la sécurité je m'en fous pas, c'est des prg qui tournent en exploitation réellement que je fais. j'ai quasiment aucun soucis avec malgré la façon dont c'est codé ... quand y a des problèmes c'est que j'ai oublié de tester des trucs, mais c'est tout.
 
tu utilises quoi à la place de strcpy (sur des tableaux de char) ? le risque c'est de dépasser la taille de la variable dans laquelle on copie, c'est ça ?
 
de même, quoi mettre à la place d'un if, si on veut dire "si c'est comme ça, tu fais ça" ... ?
 
l'écriture j'avais saisi, le truc c'est que c'est à peu près aussi parlant que du russe traduit en chinois par un ougandais quand tu le lis ...
 
en tout cas, merci pour toutes ces explications messieurs.

J utilise mes propres fonctions (c est pour ca que j ai recodé malloc, printf et compagnie). strcpy (et scanf d ailleurs) contrairement a strncpy, ne controle pas la taille de la chaine a mettre en tampon, on peut donc ecrire librement dans la memoire jusqu a l adresse de retour de la fonction en cours d execution, on peut ainsi injecter du code qui sera executer en lieu et place de la suite de ton programme. Pour un programme s executant sans droits particuliers ce n est pas problematique mais pour un programme s executant en root ou en wheel ca peut vite aboutir a la catastrophe.
 
Pour le if il y a les automates mais c est tres complexe et ca ne vaut pas le coup pour ce genre de programme. On utilisera les automates pour faire de l IA et du pathfinding. Dans ton cas n en abuse pas essaye d'utiliser les conditions le moins souvent possible (tout en verifiant toujours les valeurs de retour des fonctions telles que malloc et compagnie).

 
j'avais pas vu ton post franceso (je t'ai pas snobé ...)
je connaissais pas snprintf et comme Elmoricq en a parlé pour remplacer itoa, je pensais que ça ne faisait que ça. mais là je viens de regarder, ça fait ce que je veux en fait.
 
c'est dans le cadre d'un programme d'interface entre applis, et effectivement, moins c'est long à écrire + ça m'arrange (enfin surtout le boss ...)
 
je pense que deuxsous (qui a carrément écrit ses propre fonctions à la place de celles des librairies !) va dire que snprintf c'est pas bien, vu que ç'est comme printf et fprintf ...
à ce propos deuxsous (si tu as le temps hein, c'est pas de la plus haute importance, c'est juste pour comprendre ...), j'ai pas très bien compris le coup de strcpy qui risque de saturer un buffer et d'écrire un peu n'importe où en mémoire :
ça veut dire que les octets à copier sont mis dans un buffer de taille limitée avant de copier, et que strcpy ne contrôle pas s'il à atteint la limite et peut donc dépasser cette limite ? si c'est ça c'est complètement c.., je pensais qu'il copiait les caractères un par un.

c est un peu plus complexe que ca ... mais si ca t interesse vraiment je ne peux que te conseiller ce document vraiment tres complet et tres abordable pour tout personne s etant deja essayée au C.
www.zone-h.fr/download/file=14/
je n aurais pas pu faire mieux en termes de details et clareté

C'est faux. Une chaine de caractères est un tableau de char initialisé et terminé par un 0.
 
Par contre, un pointeur a le droit d'avoir pour valeur l'adresse du premier élément d'une chaine...
 
http://mapage.noos.fr/emdel/notes.htm#char_star

 
Magnifique, mais encore trop compréhensible.
En plus les magic numbers, c'est Entreprisey©, donc c'est vraiment du code de qualitaÿ.
 

 
Il y a pas mal de cas où tu connais, de manière certaine, la taille de tes chaînes source et destination, et dans ce cas-là, tu vas pas t'embêter avec un strncpy() hein.
 

 
En dehors du fait que itoa() n'est pas une fonction standard, on se moque totalement de la "gourmandise" et des micro-optimisation de daube, hein. A moins de cas très spécifiques, et qui ne concernent certainement pas un débutant, tu t'en moques éperdument de la "gourmandise" !  
C'est inutile, ça rend le code illisible, quand tu ne maîtrises pas pleinement ton sujet ça mène n'importe où sauf au résultat escompté, et surtout, ça ne sert à rien, parce que les performances se mesurent essentiellement à l'aune de l'algorithme utilisé.
 

 
Si c'est pour dire des âneries pareilles, tu ferais mieux de t'abstenir d'écrire.
http://www.linux-france.org/articl [...] cat-3.html
 
 

non mais rassurez moi là : y'a que moi que ça choque ce code ?

Bien sûr qu'il n'y a que toi qu'il choque, ce code.
 
(relis le topic en commençant par la toute première réponse )

ok je préfere

C'est ptet surtout le meilleur moyen d'avoir un programme bancale
 
Et tu utilise quoi à la place ???
 
 

Ben, rien. Cf. son magnifique snippet, destiné à un débutant, montrant un malloc() sans aucun test pour valider que la mémoire a été allouée.