Programme C Fermé

Question

Salut!! je veux s'il vous plait un programme qui lit des chaînes et les met dans un pointeur , ça veux dire chaque chaîne dans un pointeur. mais ça veut pas quand je depasse 8 lettre je vois un probleme c'est mon travail : #include #include #include int main(){char a;char ***t;int i=1,b,tt;int j=1;int n=sizeof(char);t = malloc(sizeof(char**));*t=malloc(sizeof(char*));//**t=malloc(sizeof(double));do{gets(*t+i);*t=realloc(*t,(i+1)*sizeof(char**));i=i+1;} while(i<4);for(j=1;j<4;j++){ printf("%s ",(*t)+j);}//printf("%d",n);//printf("%s ",t+j);}

fiddy · Answer

Bonjour,

gets(*t+i); 
gets() est à proscrire, utilise plutôt fgets(). Pour info, tu peux utiliser t[i] au lieu de *t+i, c'est la même chose en plus clair.

Ce que tu souhaites nécessite donc 2 dimensions (1 pour stocker une chaîne, une autre pour stocker les chaînes). Donc char **t; suffit.

Tu as oublié d'allouer **t : **t=malloc(...*sizeof **t);

for(j=1;j<4;j++){
Les tableaux (au sens général) commencent à l'indice 0. Donc j=0.

Tu as oublié de faire un return 0 à la ifn du main().

PS : quand tu postes ton code, utilise la balise "code" et sélectionne "C" pour qu'il soit correctement indenté.

Cdlt,

xanonyme47 · Answer

Merci Mr :) Merci beaucoup , mais y a encore un problem just aprés l execution : #include #include #include int main(){ char a; char **t; int i=0,j=0; int n=sizeof(char); t = malloc(sizeof(char**)); *t=malloc(sizeof(char*)); **t=malloc(sizeof(char)); do{ fgets(stdin,t[i],100); *t=realloc(*t,(i+1)*sizeof(char**)); i=i+1; } while(i<3); for(j=0;j<3;j++){ printf("%s ",(*t)+j); } return(0); }

fiddy · Answer

je pence que j'ai besoin de 3 dimensions parceque je veux chaque chaîne dans **t est chaque phrase dans *t.
Donc, ça fait 2 dimensions (la première : les phrases, la seconde : le contenant).
Il ne fallait pas mettre des 100 partout . Juste pour la dimension recevant les phrases.

t = malloc(100*sizeof(char***));
Non. t est de type char***, et il contient des char**. Donc sizeof(char**).

Du coup, tout est décalé:
***t=malloc(100*sizeof(char));
***t est de type char. Donc rien à allouer.

Donc cela donnerait :

char **t;
t = malloc(sizeof(char*)); /*création d'un conteneur d'une seule chaîne)*/
*t=malloc(100*sizeof(char)); /*la première chaîne pourra contenir 99 caractères*/

do{
 fgets(t[i],100,stdin); /*tu as inversé t[i] et stdin*/
 t[i]=realloc(t[i],(strlen(t[i])+1)*sizeof(char)); /*on réajuste la taille de la chaîne (initialement c'était à 100)*/
 i=i+1;
        t = realloc(t, (i+1)*sizeof(char*)); /*on agrandit le conteneur d'une case*/
        t[i]= malloc(100*sizeof(char)); /*on alloue la place qui contiendra la chaîne*/
} while(i<3);
/*Ne pas oublier les free()*/


En axe d'amélioration, tu pourrais économiser les premiers mallocs (un realloc sur une chaîne NULL vaut un malloc). Ensuite, au lieu de mettre "3" en dur dans le code, tu pourrais stocker toutes les chaînes de l'utilisateur jusqu'à ce qu'il rentre par exemple "FIN" (strcmp()).
Je te conseille aussi de faire des fonctions pour rendre le code plus propre (ici une fonction d'affichage, 1 fonction contenant les malloc, etc.). Il ne faut pas oublier non plus les free().
Sinon, les realloc, c'est assez lourd en mémoire. Pour ce genre de programme, on utilise plutôt les listes chaînées (enfin tu verras cette notion par toi-même si tu poursuis le C).

Cdlt,

Google is your friend

Programme C

3 réponses

Discussions similaires