Pb de lecture dans un fichier avec la fonction fgets

Version imprimable

18/09/2005, 23h03
Vince57

Pb de lecture dans un fichier avec la fonction fgets
Bonjou,

J'ai un pb très blocant de lecture d'une donnée dans un fichier:

mon main est:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 int main() { char x[10]; char in[10]; FILE *fin; scanf("%s",in); fin=fopen(in,"r"); fgets(x,10,fin); fclose(fin); printf("x vaut %s",x); }
=> le fichier en entrée contient un seul nombre (1.12) et la fonction me renvoit
x vaut {\rtf1\ma

alors que je m'attends à x vaut 1.12 !!

où est mon erreur?

Marci

Edit : merci de penser a la balise code
18/09/2005, 23h10
Bob

Tu es sur que la lecture dans le fichier a bien lieu ?
Deja, alloue plus que 10 octets pour ton tableau in
Ensuite verifie les retour de fopen() et de fgets() pour etre sur que la lecture fonctionne.

Re: Pb de lecture dans un fichier avec la fonction fgets

:tagcode:
Code incomplet. Merci de poster du code compilable.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
main.c: In function `main_':
main.c:12: error: `FILE' undeclared (first use in this function)
 
main.c:12: error: (Each undeclared identifier is reported only once
main.c:12: error: for each function it appears in.)
main.c:12: error: `fin' undeclared (first use in this function)
main.c:14: error: implicit declaration of function `scanf'
main.c:14: warning: nested extern declaration of `scanf'
<internal>:0: warning: redundant redeclaration of 'scanf'
main.c:15: error: implicit declaration of function `fopen'
main.c:15: warning: nested extern declaration of `fopen'
main.c:17: error: implicit declaration of function `fgets'
main.c:17: warning: nested extern declaration of `fgets'
main.c:19: error: implicit declaration of function `fclose'
main.c:19: warning: nested extern declaration of `fclose'
 
main.c:21: error: implicit declaration of function `printf'
main.c:21: warning: nested extern declaration of `printf'
<internal>:0: warning: redundant redeclaration of 'printf'

Citation:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
int main()
{
char x[10];
char in[10];
FILE *fin;
 
scanf("%s",in);
fin=fopen(in,"r");
 
fgets(x,10,fin);
 
fclose(fin);
 
printf("x vaut %s",x);
 
}

=> le fichier en entrée contient un seul nombre (1.12) et la fonction me renvoit

Code:

x vaut {\rtf1\ma

alors que je m'attends à x vaut 1.12 !!

où est mon erreur?

Manque <stdio.h>
fopen() peut échouer (NULL).
main() retournant un int, il faut un return 0,; par exemple (mais facultatif en C99)

Avec le fichier texte 'xx':

Code:

1.12

et ce code

Code:

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>
 
int main (void)
{
   int err = 1;
   char in[10];
   char x[10];
   int n = scanf ("%s", in);
 
   if (n == 1)
   {
      FILE *fin = fopen (in, "r");
 
      if (fin != NULL)
      {
         if (fgets (x, sizeof x, fin) != NULL)
         {
            err = 0;
         }
 
         fclose (fin);
      }
      else
      {
         perror (in);
      }
   }
 
   if (!err)
   {
      printf ("x vaut %s\n", x);
   }
   return 0;
}

Ca fonctionne :

Code:

1
2
3
 
xx
x vaut 1.12

Je ne recommande pas scanf() pour les saisies (ici, l'usage que tu en fais est très dangereux).

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

19/09/2005, 09h33
Médinoc

Ton fichier n'est pas un fichier texte, mais un .rtf (qui contient plein de balises de mise en forme)
19/09/2005, 12h29
Trap D

Citation:

le fichier en entrée contient un seul nombre (1.12) et la fonction me renvoie

sous quelle forme ? binaire ou ASCII ?
C'est-à-dire si tu fais un type du fichier qu'obtiens-tu à l'écran ?
24/09/2005, 16h02
artatum
pardonnez-moi ma curiosité, mais:
quel est l'intérêt du [scanf("%s",in);] au début du code ? (1er post).

je l'aurais écrit:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 #include <stdio.h> int main(void) { char x[10]; FILE *fin; fin=fopen(in,"r"); scanf("%s",x); fclose(fin); printf("x vaut %s",x);
c'est plus imple et plus clair, non ?
24/09/2005, 16h03
artatum

pardon, j'ai oublié
. un return 0, puisque la fonction main est un int
. le } terminal
24/09/2005, 17h46
Trap D

Citation:

Envoyé par artatum

pardonnez-moi ma curiosité, mais:
quel est l'intérêt du [scanf("%s",in);] au début du code ? (1er post).

Il faut bien connaître le nom du fichier à ouvrir, non ? C'est à ça que sert le scanf !
24/09/2005, 19h13
Emmanuel Delahaye
Citation:
Envoyé par artatum

quel est l'intérêt du [scanf("%s",in);] au début du code ? (1er post). je l'aurais écrit:

Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 #include <stdio.h> int main(void) { char x[10]; FILE *fin; fin=fopen(in,"r"); scanf("%s",x); fclose(fin); printf("x vaut %s",x);

c'est plus simple et plus clair, non ?
Et 'in' est défini comment ?
24/09/2005, 19h31
artatum

c'est bon, pas besoin de m'agresser, je pose des questions pour apprendre, j'ai bien dit que j'étais débutant.

au lieu de balancer ce genre de choses vous pourriez développer, comme ça tout le monde en profiterait :P
24/09/2005, 21h31
Emmanuel Delahaye

Tu parles à qui ? (merci d'utiliser le bouton [citer])

Citation:

Envoyé par artatum

c'est bon, pas besoin de m'agresser,

Gni ? Personne ne t'a agressé. Tu proposes une solution bourrée d'erreurs, et on se contente de signaler tes erreurs. Tu préfères quoi ? Qu'on ne dise rien ?

Citation:

je pose des questions pour apprendre,

Ok.

Citation:

j'ai bien dit que j'étais débutant.

J'ai bien relu le fil, je n'ai rien vu de tel. Citation ?

Citation:

au lieu de balancer ce genre de choses vous pourriez développer, comme ça tout le monde en profiterait

Quelle genre de choses ? Tu n'as pas cité le contexte.
24/09/2005, 22h23
artatum
d'accord, désolé je m'emporte un peu facilement quand je me sens agressé. :lol:

donc pour en revenir à ce code, ne pourrait-on pas écrire

Code:

fin=fopen("nom du fichier","r");

?

et le sens de ma question
Code:

1 2 pardonnez-moi ma curiosité, mais: quel est l'intérêt du [scanf("%s",in);] au début du code ? (1er post).
c'est que pour moi cette fonction signifie "doit lire une chaîne de caractère (s) dans l'entrée standard (scanf), et la mettre dans le tableau de caractère in[] (in remplaçant *in[0] ).

donc je ne comprenais pas sa présence, d'autant plus vu sa place ds le code AVANT l'ouverture en lecture.

éclaire-moi :D
24/09/2005, 22h38
artatum

mais comme je connais mal la fonction fgets (je l'apprends d'ailleurs en ce moment ds le K&R en lisant ce forum), je commence à comprendre,
mea culpa.

donc il ouvre le fichier "in", y lit x[10], (au passage referme fin et non in).

alors pourquoi cela ne renvoie pas la bonne chaîne ? est-ce parce qu'il aurait dû écrire fopen(*in, "r") ? pourtant in est l'équivalent de *in[0], donc ça ne devrait pas poser pb.

je ne vois pas, j'aimerais bien que Vince nous donne le retour de fopen, pour voir. 8O
24/09/2005, 22h59
Emmanuel Delahaye
Citation:

Envoyé par artatum

donc pour en revenir à ce code, ne pourrait-on pas écrire

Code:

fin=fopen("nom du fichier","r");

On peut techniquement faire beaucoup de chose. Le problème est de respecter les specs. Je suppose que si le posteur original s'est cassé les c...ll.s à saisir le nom du fichier, c'est qu'on lui a demandé de le faire... Un programmeur qui commence à changer les specs ne fera pas long feu dans ce métier...
Citation:
et le sens de ma question

Code:

1 2 pardonnez-moi ma curiosité, mais: quel est l'intérêt du [scanf("%s",in);] au début du code ? (1er post).
De demander à l'utilisateur de rentrer le nom du fichier à traiter. On suppose que ça correspond à une demande du client.

Citation:

c'est que pour moi cette fonction signifie "doit lire une chaîne de caractère (s) dans l'entrée standard (scanf), et la mettre dans le tableau de caractère in[] (in remplaçant *in[0] ).

donc je ne comprenais pas sa présence, d'autant plus vu sa place ds le code AVANT l'ouverture en lecture.

Comment ouvrir un fichier dont on ne connait pas le nom ? Il me parait logique de commencer par demander le nom, et ensuite d'ouvrir le fichier, non ?

Je te dirais un truc dans le genre 'achète toi un cerveau', mais tu risques de le prendre mal, alors je ne le dis pas! (humour).
24/09/2005, 23h05
Emmanuel Delahaye

Citation:

Envoyé par artatum

donc il ouvre le fichier "in",

Non. Il ouvre le fichier dont le nom est dans la chaine in.

Citation:

y lit x[10], (au passage referme fin et non in).

Il faut apprendre le sens des fonction fopen() et fclose(). Ici, in est la chaine qui contient le nom du fichier, alors que fin est le nom du pointeur sur l'objet FILE crée par fopen(), et s'utilise ensuite dans les fonctions de fichier, y compris fclose().

Citation:

alors pourquoi cela ne renvoie pas la bonne chaîne ? est-ce parce qu'il aurait dû écrire fopen(*in, "r") ? pourtant in est l'équivalent de *in[0], donc ça ne devrait pas poser pb.

je ne vois pas, j'aimerais bien que Vince nous donne le retour de fopen, pour voir.

STP. Relis ton K&R au chapitre sur les I/O (chap. 7), fait les exercices et revient avec des questions qui montrent que tu as lu le bouquin... On est pas là pour t'enseigner les bases qui se trouvent dans les bouquins.
24/09/2005, 23h32
artatum

il n'y a rien de plus horrible que de relire ce qu'on a écrit & de voir ses erreurs.... bon d'accord.

et je ne connais pas le K&R par coeur, d'abord parce que cela ne fait pas longtemps que je me suis mis au C, ensuite parce que je n'ai pas que ça à faire.

enfin bon, une fois que mes innombrables erreurs & ignorances sont corrigées et que l'on a rendu à César ce qui appartient à César, répondons à la question première qui était:

mais enfin, pourquoi est-ce que ça ne marche pas ?
24/09/2005, 23h40
Emmanuel Delahaye

Citation:

Envoyé par artatum

mais enfin, pourquoi est-ce que ça ne marche pas ?

Quels sont les mots que tu ne comprends pas dans

Citation:

Tu parles à qui ? (merci d'utiliser le bouton [citer])

et j'ajoute "tu parles de quoi ?"
25/09/2005, 01h43
artatum

bon, laissons tomber ça devient stérile.
25/09/2005, 09h18
Emmanuel Delahaye

Citation:

Envoyé par artatum

bon, laissons tomber ça devient stérile.

Si tu veux des réponses interessantes, fait au moins l'effort de lire les regles du forum et de les appliquer.
25/09/2005, 12h49
artatum

et ? je ne vois pas en quoi je contreviens aux règles du forum, que j'ai lues bien évidemment.

Tu trouves mes questions débiles, c'est ton droit, maintenant j'attendais une autre attitude. dans mon travail la connaissance théorique est indispensable, c'est la base de tout savoir, mais rien ne remplace ce que peut t'apporter un Maître, car lui seul a la possibilité d'imprimer de façon indélébile ses connaissances au fond de ton esprit.

C'est avec des explications pédagogiques et de la patience que l'on devient un Maître, pas de cette façon. taper du code, tout le monde peut l'apprendre, et j'espère bien y arriver un jour de façon satisfaisante.

Enfin pour terminer, quand on pose une question c'est qu'on cherche une réponse, et quand je soumets un post avec des erreurs, ce qui doit souvent arriver pour l'instant dans ce langage, j'attends autre chose que "retourne lire et achète-toi un cerveau".

ça me fait un peu penser aux gamins SMS d'un autre forum, qui s'envoient en permanence des "boulet tu sors".

sur ce, je ne lirai plus toute réponse postée à la suite sur ce topic.
j'aimerais bien continuer à apprendre, à poser des questions, des questions idiotes même, et qu'on me corrige. mais j'aimerais que les réponses viennent de quelqu'un qui veuille corriger, pas envoyer bouler.

merci.
25/09/2005, 15h07
Emmanuel Delahaye

Citation:

Envoyé par artatum

et ? je ne vois pas en quoi je contreviens aux règles du forum, que j'ai lues bien évidemment.

Et pourtant, tu t'obstines à ne pas citer à qui tu réponds, ni ce à quoi tu réponds. Ca rend tes propos illisibles, mais après tout, c'est ton problème.

Mais il me parait simple et évident qu'une question bien formulée a de meilleures chances de recevoir une réponse utile. Maintenant, c'est toi qui vois...

Citation:

Tu trouves mes questions débiles,

Non, elle sont mal posées, et on ne peut y répondre. Quand tu dis:

Citation:

mais enfin, pourquoi est-ce que ça ne marche pas ?

sans aucune référence, avoue qu'il n'est pas facile de répondre. Et si on te fait remarquer que la question est mal posée, tu t'offusques et tu conclus par un

Citation:

bon, laissons tomber ça devient stérile.

Avoue que tu n'es pas un client facile, et ce malgré des trésors de patience. Crois moi, d'autres t'auraient envoyé bouler (ou pire ignoré) depuis longtemps.

Citation:

c'est ton droit, maintenant j'attendais une autre attitude. dans mon travail la connaissance théorique est indispensable,

C'est pour ça que je te demande parfois de relire ton livre de C, ce que je ne peux pas faire à ta place. Je ne peux pas t'enseigner les bases. C'est à toi de le faire. Nous tout ce qu'on peut faire, c'est répondre à des questions précises (dont la réponse n'est pas dans les livres), expliquer des passges difficiels trouvésdans certains livres, ou commenter (voire corriger) le code posté.

Citation:

c'est la base de tout savoir, mais rien ne remplace ce que peut t'apporter un Maître, car lui seul a la possibilité d'imprimer de façon indélébile ses connaissances au fond de ton esprit.

On va peut être laisser la philosophie indienne et le rôle du 'Maître' (quatre-vingt comme disait Pierre Dac) de coté pour aujourd'hui, et je dirais surtout que rien ne remplace l'expérience personnelle. C'est à toi de lire les livres, et c'est à toi de faire les exercices.

Citation:

C'est avec des explications pédagogiques et de la patience que l'on devient un Maître, pas de cette façon. taper du code, tout le monde peut l'apprendre, et j'espère bien y arriver un jour de façon satisfaisante.

Avant de vouloir être un Maître, commence déjà par être un bon apprenti.