Discussion :

[ouinih]

Bonjour,

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

Code :

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

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!!!

[Médinoc]

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

[ouinih]

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

[diogene]

ouinih :

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

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

Code :

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

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

[corentin59]

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

Code :

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

char mot[100];

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

[ouinih]

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

[Melem]

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 :

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

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 :

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

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.

[ouinih]

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

Code :

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

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

Merci

[Melem]

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

Code :

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

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

Merci

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