pb. lecture d'un pointeur "segmentation error"

Version imprimable

Bonjour,
voila un ptit code, j'ai une erreur de segmentation,,,je ne vois pas dutout l'erreur§§§ merci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
int main () {
        FILE *stream;
        char *file_name="file";
        stream= fopen (file_name,"r");
        char *mot=NULL;
        if (stream != NULL) {
                (fscanf(stream,"%s",mot));
                        printf("%s",*mot);
 
                }
}

j'ai modifé un peu le code mais c'est toujours la meme erreur

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
int main () {
        FILE *stream;
        char *file_name="new_geno_tot.ped";
        stream= fopen (file_name,"r");
        char *mot;
        if (stream != NULL) {
                while ((fscanf(stream,"%s",mot)) != EOF) {
                        printf("%s",*mot);
                }
        }
}

23/06/2009, 15h54
diogene

Deux erreurs.
La première :

Code:

fscanf(stream,"%s",mot);

mot doit être un tableau (suffisamment grand), pas un pointeur.

Code:

char mot[....];

La deuxième :

Code:

printf("%s",*mot);

*mot est un char. Le format %s est approprié pour les chaines de caractères (tableau de caractères terminé par '\0'), pas pour les caractères.

Code:

printf("%s",mot);
23/06/2009, 15h55
ram-0000
La grosse erreur est ici
Code:

1 2 char *mot; fscanf(stream,"%s",mot)
Tu lis un mot et du dis de le stocker dans un pointeur qui vaut NULL, cela part donc en live

Correction :
Code:

1 2 char mot[256]; // par exemple 256, je ne connait pas la taille maximale d'un mot fscanf(stream,"%s",mot)
Autre probleme, je ne suis pas sûr que scanf retourne EOF, a confirmer.

Dant tous les cas, scanf() est toujours délicat à manipuler. C'est l'instruction que l'on apprend aux débutants à l'école mais ce n'est pas précisemment une instruction pour débutant. A moins de contraintes particulières (c'est obligatoire parce que c'est comme cela :P), lui préférer l'instruction fgets

[Edit] Grillé par diogene
23/06/2009, 15h56
karimDevelopper

tres bien
merci
23/06/2009, 16h47
Médinoc
Citation:
Envoyé par ram-0000

La grosse erreur est ici

Code:

1 2 char *mot; fscanf(stream,"%s",mot)

Tu lis un mot et du dis de le stocker dans un pointeur qui vaut NULL, cela part donc en live
Pire, le pointeur n'est pas initialisé: Il peut valoir n'importe quoi.
23/06/2009, 16h56
ram-0000

Citation:

Envoyé par Médinoc

Pire, le pointeur n'est pas initialisé: Il peut valoir n'importe quoi.

Si si, il était initialisé à NULL mais j'ai oublié de reporter cette initialisation dans ma recopie du code