lecture tableau de char

Bonjour,

J'ai fait un prog qui lit des valeurs à partir d'un fichier.
Voila le code

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35

#include <stdio.h> #include <stdlib.h>   int main () { int i, l; FILE *src = NULL; FILE *f = NULL; int size[12]; int taille; char* mot; char* tab[12];   src = fopen ("huffman_DC.txt", "r"); f = fopen("test.txt", "w");   if((src == NULL) || (f == NULL)) { printf("pb ouverture fichier\n"); }   for (i=0;i<12;i++) { fscanf(src,"%d %s", &taille, mot); fprintf(f, "%s", mot); size[i] = taille; tab[i] = mot; printf("voila le mot de code %d %s \n", i, tab[i]); }   printf("voila tab mot %s \n", tab[4]);   fclose(src); fclose(f);

---

Pour le premier printf, les valeurs affichées sont correctes (correspond à ce que j'ai dans le fichier) Par Contre pour le 2eme printf (affichage de tab[4]), il m'affiche la dernière valeur insérée dans le tableau. Je ne comprend pas d'où ca vient!!!

Le pointeur mot n'est pas initialisé: Il peut pointer sur n'importe quoi.
Comportement indéfini.

mot est un pointeur qui me permet de lire le fichier. tu me suggère de l'initialiser à NULL.
Sinon je ne vois pas le rapport avec mon pb!!
Merci

ouinih :

Citation:

---

tu me suggère de l'initialiser à NULL.

---

Ce n'est pas du tout ce que te suggères Médinoc

Code:

---

1 2 3

char* mot; .... fscanf(src,"%d %s", &taille, mot);

---

Pour stocker une chaîne de caractères, il faut un tableau de char, pas un char*. C'est la définition de mot qui est incorrecte, il FAUT que mot soit un tableau (suffisamment grand) de char

il faut que tu alloues de la mémoire pour mot : soit tu le déclares comme cela

Code:

---

char mot[100];

---

auquel cas tu auras un tableau de 100 caractères, soit tu utilises malloc() (sans oublier free())

j'ai fait ce que vous me demander, mais j'ai tjrs le meme pb: l'execution du prog donne la meme chose

tab non plus n'est pas initialisé. Plutôt que d'utiliser des pointeurs utilise plutôt des tableaux comme ça t'auras moins de problèmes.

Code:

---

1 2 3 4 5 6 7 8

int main () { ... char mot[100]; char tab[12][100]; ... return 0; }

---

A part ça (rien à voir avec ton problème) tu gères mal les erreurs d'ouverture de fichiers. En cas d'erreur on termine le programme, alors que toi tu affiches un message mais continue quand même l'exécution ...

EDIT : J'ai répondu trop vite. En fait, j'ai lu le programme et je constate que tu copies mot dans tab[i] à chaque itération. Pour copier une chaîne il faut utiliser strcpy. Copier une chaîne et pointer sur une chaîne (comme tu le fais) ne sont pas la même chose. Par exemple :

Code:

---

1 2 3 4 5 6 7 8

char s[100]; char * p;   strcpy(s, "bonjour"); /* on met "bonjour" dans s */ p = s; /* p pointe sur s */ strcpy(s, "au revoir"); /* maintenant s contient "au revoir" */   printf("%s\n", p); /* affiche "au revoir" ! */

---

En effet, p n'est pas une chaîne mais simplement un pointeur. Dans l'exemple il pointe sur s.

En résumé, utilise des tableaux.

en faisant comme tu dis le reste du code ne change pas.
La j'ai une eurreur de compilation

Code:

---

1 2	binaire_jeudi.c:42: erreur: incompatible types in assignment

---

Merci:cry:

Citation:

---

Envoyé par ouinih

en faisant comme tu dis le reste du code ne change pas.
La j'ai une eurreur de compilation

Code:

---

1 2	binaire_jeudi.c:42: erreur: incompatible types in assignment

---

Merci:cry:

---

Remplace tab[i] = mot par strcpy(tab[i], mot). On ne peut pas faire <tableau> = <quoi que ce soit>.