Probleme sur la lecture d'un fichier

Version imprimable

Bonjour, j'ai réalisé un petit fichier de test afin de réaliser une fonction qui permette de lire un fichier. Ainsi, je lance une première lecture de mon fichier, ça marche, mais le system("pause") final n'apparait pas, et si je demande une nouvelle lecture, le programme plante.
Apres l'affichage de la dernière ligne de mon fichier, il est marqué "nom de commande ou de fichier incorrect".
J'ai mis des #include, et les system("pause") que j'avais placés avant de faire cette fonction marchent parfaitement.

Un coup d'oeil extérieur permettra peut etre de voir l'erreur que je n'arrive pas à trouver
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 int action_fichiers(char* fic_entree) { char* ligne; FILE* fic; ligne=malloc(sizeof(char)*40); fic=fopen(fic_entree,"r"); fscanf(fic,"%s\n",ligne); while( !feof(fic) ) { fscanf(fic,"%s\n",ligne); printf("%s\n",ligne); } fclose(fic); free(ligne); system("pause"); return 1; }
le fichier est composé de petites lignes de textes finies par un retour à la ligne :
Code:

1 2 3 test1 test2 ...
en vous remerciant
Isk

07/05/2007, 18h45
odsen.s
Bonjour,

:tagcode:

Quelques remarques sur le code de la fonction :
- Pourquoi utiliser malloc pour une taille fixe ? De plus, sizeof(char) vaut 1 par définition
- Il faut impérativement tester l'ouverture du fichier
- feof ne fait pas ce que tu crois, la forme canonique pour lire un fichier est
  Code:
  
  1 2 3 4 while(fgets(ligne, sizeof ligne, fichier) != NULL) { /* instructions */ }
- system("pause") n'est pas portable. Ceci est portable pour faire une pause et attendre l'appui de la touche <entree> :
  Code:
  
  1 2 3 4 5 int c = getchar(); if(c != '\n' && c != EOF) { while((c = getchar()) != '\n' && c != EOF); }
- La fonction retourne 1 quoi qu'il se passe.
Il faut donc corriger ces points avant d'aller plus loin.

ATTENTION......

feof ne fait pas ce que tu penses qu'il fait...

Lit les manuels.....

Et fscanf n'est pas conseillé, mais fgets l'est...

Pour lire un fichier :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
int action_fichiers(char* fic_entree) {

    char* ligne;
    FILE* fic;
    ligne=malloc(sizeof(char)*40);
    if ( ligne == NULL )  /* Tester si l'allocation a réussi */
       return 0 ;
    
    fic=fopen(fic_entree,"r");
    if ( fic == NULL )  /* Tester si l'ouverture a réussie */
       return 0 ;

    while( fgets(ligne, 40, fic) != NULL )  {
        printf("%s\n",ligne);
    }
        
    fclose(fic);
    free(ligne);
 /* code d'attente tel que cité */
    return 1;
}

08/05/2007, 15h00
Iskander81
Oui, mais par la suite, je vais etre amené à avoir des fichiers de la forme
Code:

1 2 3 4 var1,var2,var3 var1,var2,var3 ...
le fscanf me paraissait judicieux pour récupérer rapidement les variables.

L'existence du fichier est testée dans une autre fonction.
Mon codage est destiné pour une interface msdos. La portabilité n'est pas nécéssaire.
08/05/2007, 15h04
odsen.s

Si tu tiens vraiment à utiliser fscanf, il faut que tu écrives une fonction qui vérifie le format du fichier avant.

Toutefois, la meilleure solution est d'utiliser fgets(), puis de traiter la ligne saisie. Tu pourras tout à fait récupérer des données séparées par des virgules.

Citation:
Envoyé par Iskander81
Oui, mais par la suite, je vais etre amené à avoir des fichiers de la forme
Code:

1 2 3 4 var1,var2,var3 var1,var2,var3 ...
le fscanf me paraissait judicieux pour récupérer rapidement les variables.

L'existence du fichier est testée dans une autre fonction.
Mon codage est destiné pour une interface msdos. La portabilité n'est pas nécéssaire.
le fscanf est à déconseiller....

Et récupérer "rapidement" les variables avec fscanf n'est pas garanti...

D'autre part, je répète, feof ne fait pas ce que tu crois qu'il fait..

Enfin, un exemple de récupération des paramètres séparés par des virgules :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 char *p = NULL ; char *p1 = NULL ; int i = 0 ; while( fgets(ligne, 40, fic) != NULL ) { p1 = ligne ; while ( (p = strtok ( p1, ",\n")) != NULL ) { i = i + 1 ; fprintf ( stderr, "valeur %d : %s",i,p); p1 = NULL ; } }

09/05/2007, 01h05
Skyounet
Citation:
Envoyé par souviron34

Code:

1 2 p1 = &ligne ;
C'est pas plutôt

Code:

p1 = ligne

??
09/05/2007, 10h55
souviron34

Citation:

Envoyé par Skyrunner

C'est pas plutôt

Code:

p1 = ligne

??

:oops: oooppsss tu as raison... Désolé j'ai été trop vite... Je corrige..
:oops:
09/05/2007, 11h51
Iskander81

Je vous remercie pour toutes vos réponses, je vais faire avec un fgets pour ensuite retraiter la ligne.
Merci pour la rapidité, la précision et les conseils.
Isk
10/05/2007, 14h48
roryprogc

Zut...

Salut
Je planche depuis quleques semaines sur le fameux fscanf et je me demandais si lorsque l'on appelle un fichier de données dans lequel figure des lignes de commentaires et des lignes de données chiffrées(intéressantes pour moi) on peut faire en sorte que le fscanf "zappe" les lignes de commentaires?
De plus je suis allée dans le man scanf et je ne comprends pas très bien si l'option %s peut me permettre de lire toute la ligne d'un coup au lieu d'avoir à la traiter caractères par caractères...
En gros je suis désespérée...
Merci d'avance à tous ceux qui pourront m'aider et à tous ceux qui ont pris le temps de me lire...
Rory
10/05/2007, 14h51
Skyounet

Citation:

Envoyé par roryprogc

Salut
Je planche depuis quleques semaines sur le fameux fscanf et je me demandais si lorsque l'on appelle un fichier de données dans lequel figure des lignes de commentaires et des lignes de données chiffrées(intéressantes pour moi) on peut faire en sorte que le fscanf "zappe" les lignes de commentaires?
De plus je suis allée dans le man scanf et je ne comprends pas très bien si l'option %s peut me permettre de lire toute la ligne d'un coup au lieu d'avoir à la traiter caractères par caractères...
En gros je suis désespérée...
Merci d'avance à tous ceux qui pourront m'aider et à tous ceux qui ont pris le temps de me lire...
Rory

Lis ce petit tuto sur scanf et ces soeurs
http://xrenault.developpez.com/tutoriels/c/scanf/