code douteux ou pas

**t_2_m** · 30/04/2006, 10h42

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** · 30/04/2006, 10h53

Envoyé par 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)

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** · 30/04/2006, 10h56

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

**Trap D** · 30/04/2006, 10h57

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** · 30/04/2006, 11h08

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** · 30/04/2006, 12h10

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

**crocodilex** · 30/04/2006, 12h17

Envoyé par CSoldier

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** · 30/04/2006, 12h22

Envoyé par crocodilex

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** · 30/04/2006, 12h24

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** · 30/04/2006, 12h37

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.

**Emmanuel Delahaye** · 30/04/2006, 13h33

Envoyé par t_2_m

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.

**Emmanuel Delahaye** · 30/04/2006, 13h35

Envoyé par 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.

Uniquement sous Borland C...

**Trap D** · 30/04/2006, 13h59

Mon info est tirée du MSDN ...

**Trap D** · 30/04/2006, 14h05

Envoyé par seriousme

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** · 30/04/2006, 15h36

Envoyé par Trap D

Mon info est tirée du MSDN ...

Il y a eu pas mal d'échange de "bon procédés" (non portables, bien sûr) entre Microsoft et Borland. Pendant un moment ils se vouaient du "respect mutuel"...

**Médinoc** · 01/05/2006, 16h56

Ah, c'est donc pour cela que getch() et compagnie, que l'on clame souvent l'apanage de Borland, sont aussi compris par les compilateurs Microsoft?

Et donc, le "rt" n'est pas standard, c'est pas malin... Surtout que pour moi, c'est le comportement logique "normal" que de croire qu'un fichier est ouvert non-trafiqué (comprendre : binaire) par défaut...

code douteux ou pas

C

Discussions similaires

Partager

Partager