Discussion :

[Guizmo95]

Bonjour,
Voici mon problème :
Je créer un tableau de char : char buf[1000];
je créer un tableau de pointeurs : char* table [ nb_idents ];
Je demande la saisie dans le buffer : scanf("%s",buf);
je met la valeur du buffer dans mon tableau :
table[tailleTab]=(char*) malloc(sizeof(char*)*strlen(buf));
table[tailleTab]=buf;
j'incremente la taille de mon tableau : TailleTab+=1;
et je boucle pour revenir sur la demande de saisie.

Mon problème est que quand je rentre une deuxième valeur dans mon buffer, mon tableau Table prend imédiatment la valeur de mon buffer, donc j'en conclue que je met l'adresse de mo buffer dans le tableau...
Ma question est la suivante : comment mettre simplement la valeur de mon buffer dans mon tableau?

Merci pour les réponses que vous pourrez m'apporter

[Trap D]

Tout simplement en faisant une recopie des données saisies par le scanf.
En fait, ici, tu perds de la mémoire, car le malloc demande de la mémoire au système, qui te retourne une adresse stockée dans Table[tailleTab] à cause du Table[TailleTab] =(char*) malloc(sizeof(char*)*strlen(buf));
mais comme ensuite tu fais table[tailleTab]=buf; l'adresse précédemment mémorisée est perdue (c'est ce qu'on appelle une fuite mémoire).
Deuxième erreur, tu stocke toujours la même adresse celle de buf, donc à la fin de la saisie, les données mémorisée par l'intermédiaire du tableau sont toutes les mêmes.
Troisième erreur , il ne faut pas oublier que les chaînes de caractères valides se terminent par un \0, il faut le compter dans l'allocation mémoire.
Quatrième "erreur" : sizeof char vaut 1 par définition, tu peux te contenter d'une malloc(strlen(buf) + 1);
Cinquième erreur : on ne caste pas en C !
Tu as deux possibilités :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 table[tailleTab]= malloc(strlen(buf)+ 1);
strcpy(table[tailleTab], buf);

ou tout simplement :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 table[tailleTab]= strdup(buf);

Ne pas oublier de libérer la mémoire (car strdup en fait fait la première possibilité) à la fin du programme.

[ge-lt]

effectivement, tu fais pointer l'élément de ton tableau sur la même zone mémoire que ton buffer. Pour solutionner ça je ferais une boucle sur ton buffer et copierai caractère par caractères dans ton tableau.
Solution pas terrible je sais mais ça fait tellement longtemps que je n'ai pas fais de C.

A ce que je vois un expert à répondu plus vite que moi hi hi hi

[Guizmo95]

Merci bien pour vos réponse je vais régler mes erreur

[Emmanuel Delahaye]

Voici mon problème :
Je créer un tableau de char : char buf[1000];
je créer un tableau de pointeurs : char* table [ nb_idents ];
Je demande la saisie dans le buffer : scanf("%s",buf);
je met la valeur du buffer dans mon tableau :
table[tailleTab]=(char*) malloc(sizeof(char*)*strlen(buf));
table[tailleTab]=buf;

Bug. C'est pas comme ça qu'on affecte le contenu d'un tableau. Comme c'est une chaine de caractères, utilise strcpy().

Mais le plus simple est encore d'utiliser directement strdup() :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

   table[tailleTab] = strdup (buf);

Cette fonction n'est pas standard C, mais elle est POSIX.1, donc très portable. La libération se fait par free().