Discussion :

[t_2_m]

Bonjour,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include <stdio.h>
#include <stdlib.h>
int main(void) {
            FILE *f = fopen("mama.txt", "rt");
            float sum = 0;
            while(!feof(f)) {
                      char ligne[32];
                      if (fgets(ligne, 32, f) != NULL)
                                sum = sum + atof(ligne);
           }
           printf("%f\n", sum);
           return 0;
}

Avez vous des remarques concernant ce code ?


(source http://www-gtr.iutv.univ-paris13.fr/Cours/Mat/LangageC)

[piff62]

Bonjour,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include <stdio.h>
#include <stdlib.h>
int main(void) {
            FILE *f = fopen("mama.txt", "rt");
            float sum = 0;
            while(!feof(f)) {
                      char ligne[32];
                      if (fgets(ligne, 32, f) != NULL)
                                sum = sum + atof(ligne);
           }
           printf("%f\n", sum);
           return 0;
}

Avez vous des remarques concernant ce code ?


(source http://www-gtr.iutv.univ-paris13.fr/Cours/Mat/LangageC)

Peut etre faudrait il verifier que la fonction fopen arrive bien a ouvrir le fichier .. et donc en meme temps que le fichier mama.txt existe bien ?

Code :

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

1
2
3
4
5
 
if(f == NULL)
printf("mama.txt inexistant\n");
else
//ton traitement

[Franck.H]

Je ne crois pas déjà avoir vu un 't' dans le mode d'ouverture de fichier:

Code :

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

FILE *f = fopen("mama.txt", "rt");

[crocodilex]

Je ne crois pas déjà avoir vu un 't' dans le mode d'ouverture de fichier:

Code :

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

FILE *f = fopen("mama.txt", "rt");

J'ai effectivement déjà vu le mode "t" (pour texte) et "b" (pour binaire) sur certains systèmes (vxWorks ou pSOS, je ne me souviens plus).
Sur des systèmes plus "standard", ces deux modes sont simplement ignorés par la fonction "fopen".

[Franck.H]

J'ai effectivement déjà vu le mode "t" (pour texte) et "b" (pour binaire) sur certains systèmes (vxWorks ou pSOS, je ne me souviens plus).
Sur des systèmes plus "stantard", ces deux modes sont simplement ignorés par la fonction "fopen".

Ok merci pour l'info !

[t_2_m]

Pour info ce code fait partie d'un controle donne en janvier 2005 (voir 1er post pour l'url du site et obtenir le controle en entier)

En tout cas merci pour vos remarques.

[Trap D]

Pour l'histoire du fopen(... "rb") ou "rt" il faut en fait consulter la variable globale _fmode qui indique le mode par défaut, généralement c'est t.

[Trap D]

fgets ne se manipule pas comme celà :

Code :

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

1
2
3
             while(fgets(ligne, sizeof(ligne), f) != NULL) {
                     sum = sum + atof(ligne);
           }

feof indique simplement si, en cas de fin de lecture (retour de fgets = NULL), celà est du à une erreur de lecture ou à une fin de fichier.
atof est une fonction dépréciée, celà veut dire qu'il ne faut plus l'utiliser, à la place prend strtod(ligne, NULL).
Très juste CSoldier, c'est fopen(... "r").

Bref, code douteux

[Franck.H]

D'ailleurs tu avais oublié de fermer le fichier !
Voici un code un peu corrigé, je ne sais toujours pas d'où sort le 't' dans l'ouverture du fichie, d'ailleurs je ne crois même pas l'avoir vu dans mon book un jour, en tous s'il existe je veut bien savoir à quoi il sert !

Code :

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

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
36
37
38
39
40
41
42
#include <stdio.h>
#include <stdlib.h>
 
 
#define BUF_LEN 32
 
 
int main (void)
{
   FILE * f = NULL;
   char ligne [BUF_LEN];
   float sum = 0;
 
 
   f = fopen ("mama.txt", "r");
 
   if (f)
   {
      while (fgets (ligne, BUF_LEN, f))
      {
         sum = sum + atof (ligne);
      }
 
 
      if (! feof (f))
      {
         fprintf (stderr, "Erreur de lecture\n");
      }
      else
      {
         printf ("%f\n", sum);
      }
      fclose (f);
   }
   else
   {
      fprintf (stderr, "Erreur d'ouverture du fichier\n");
   }
 
 
   return EXIT_SUCCESS;
}

[Pragmateek]

De plus il faut préférer les adresses absolues aux relatives si le programme ou le fichier change de place.

[Trap D]

De plus il faut préférer les adresses absolues aux relatives si le programme ou le fichier change de place.

Oui, mais attention, si tu veux faire ce genre de chose, la méthode est de créer un fichier .ini où tu indiqueras la racine de l'arborescence des fichiers à exploiter.

[Emmanuel Delahaye]

Avez vous des remarques concernant ce code ?

(source http://www-gtr.iutv.univ-paris13.fr/Cours/Mat/LangageC)

Douteux est un mot faible. Je dirais catastrophique et dangereux...

• Pas de contrôle de l'ouverture du fichier (Comportement indéfini)
• "rt" n'existe pas (Comportement indéfini)
• evaluation de feof() avant la lecture (Comportement indéfini). Pas de fin de lecture si fgets() retourne NULL. En cas d'erreur : boucle infinie...
• usage non justifié de float (double est le type retourné par atof())
• Contrairement à strtod(), atof() ne permet pas le contrôle de la conversion.
• Pas de fermeture du fichier.

à part ça, tout va bien... L'enseignement du C en milieu universitaire a encore des progrès à faire...

Un retour vers l'auteur (Emmanuel Viennet emmanuel.viennet@lipn.univ-paris13.fr) est envisageable.