Discussion :

[karimDevelopper]

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

Code :

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

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);
 
                }
}

[karimDevelopper]

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

Code :

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

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);
                }
        }
}

[diogene]

Deux erreurs.
La première :

Code :

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

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

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

Code :

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

char mot[....];

La deuxième :

Code :

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

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 :

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

printf("%s",mot);

[ram-0000]

La grosse erreur est ici

Code :

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

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 :

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

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 ), lui préférer l'instruction fgets

[Edit] Grillé par diogene

[karimDevelopper]

tres bien
merci

[Médinoc]

La grosse erreur est ici

Code :

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

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.

[ram-0000]

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