Discussion :

[aboubakar25]

Salut à tous,

Je souhaite récupérer une suite de valeurs séparées par un espace dans un fichier avec fscanf(). Voici mon morceau de code mais le programme plante.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
char *data[500];
int i, retour;
FILE *fichier = NULL;
 
fichier = fopen("C:\\fichier.txt", r);
 
if(fichier != NULL) {
    for(i = 0; i < 500; i++) {
        if( retour = fscanf(fichier,"%s", data[i]) != EOF &&retour != 0)
        {
         printf("%s" data[i]);
        }
}

N.B: Quand j'annule la boucle for(), j'ai une seule valeur qui s'affiche.

Merci.

[transgohan]

data est un tableau de 500 caractères.
Puis tu as une boucle qui fait 500 fscanf de taille chaîne de caractères (%s).
Pour rappel une chaîne de caractères c'est plusieurs caractères.

Donc si tu as des chaînes de caractères de taille x dans ton fichier alors tu stockes dans ton tableau data : x * 500 caractères.

Edit : bien vu VDD !
Pour ma défense l'étoile était trop petite...

[edgarjacobs]

Hello,

data est un tableau de 500 caractères....

Pas vraiment data est un tableau de 500 pointeurs sur des données de type caractère. Il faudrait, pour chaque data[i], faire un malloc( /* d'une taille suffisante */ ) pour que cela fonctionne.

[Sve@r]

Code :

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

if( retour = fscanf(fichier,"%s", data[i]) != EOF && ...)

Bonjour

En dehors de la magnifique remarque d'edgarjacobs qui a mis le doigt pile poil sur le noeud principal du problème, ton instruction de test n'est pas bonne.
Les comparaisons se font de la droite vers la gauche. Donc au lieu de mettre le résultat de fsanf(...) dans la variable "retour", tu y mets le résultat de la comparaison fsanf(...) != EOF.

N.B: Quand j'annule la boucle for(), j'ai une seule valeur qui s'affiche.

Oui, c'est normal. Boucle sur une instruction = plein d'appels à cette instruction ; pas de boucle sur une instruction = instruction appelée une seule fois.

[aboubakar25]

Merci a tous voila mon nouveau code qui marche bien:

Code :

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

1
2
3
4
5
6
7
8
9
char *data[500];
int i;
FILE *fichier = NULL;
 
if(fichier != NULL) {
       for(size_t i = 0; i < 500 && (1 == fscanf(fichier, "%d", &data[i])); i++){
       printf("Lecture des donnees en cours...");
       }
}

[edgarjacobs]

Merci a tous voila mon nouveau code qui marche bien:

Code :

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

1
2
3
4
5
6
7
8
9
char *data[500];
int i;
FILE *fichier = NULL;
 
if(fichier != NULL) {
       for(size_t i = 0; i < 500 && (1 == fscanf(fichier, "%d", &data[i])); i++){
       printf("Lecture des donnees en cours...");
       }
}

C'est du grand n'importe quoi. Voir ton autre discussion sur oc.

[Sve@r]

Il faudrait, pour chaque data[i], faire un malloc( /* d'une taille suffisante */ ) pour que cela fonctionne.

voila mon nouveau code qui marche bien:

Mouais. Ca "marchera bien" pendant 15 jours et le 16°...

[aboubakar25]

Code :

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

1
2
3
4
5
6
7
8
int data[500], i;
FILE *fichier = NULL;
 
if(fichier != NULL) {
       for( i = 0; i < 500 && (1 == fscanf(fichier, "%d", &data[i])); i++){
       printf("Lecture des donnees en cours...");
       }
}

C'est plutot ce code qui marche bien.

[Sve@r]

Code :

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

int data[500], i;

C'est plutot ce code qui marche bien.

Là on est d'accord.

[aboubakar25]

cool