code douteux ou pas

Version imprimable

code douteux ou pas

Bonjour,

Code:

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)

Citation:

Envoyé par t_2_m

Bonjour,

Code:

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:

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

30/04/2006, 10h56
Franck.H

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

Code:

FILE *f = fopen("mama.txt", "rt");
30/04/2006, 10h57
Trap D
fgets ne se manipule pas comme celà :
Code:

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 :mouarf:

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:

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;
}

30/04/2006, 12h10
Pragmateek

De plus il faut préférer les adresses absolues aux relatives si le programme ou le fichier change de place.
30/04/2006, 12h17
crocodilex

Citation:

Envoyé par CSoldier

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

Code:

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".
30/04/2006, 12h22
Franck.H

Citation:

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 !
30/04/2006, 12h24
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.
30/04/2006, 12h37
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.
30/04/2006, 13h33
Emmanuel Delahaye
Citation:

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.
30/04/2006, 13h35
Emmanuel Delahaye

Citation:

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...
30/04/2006, 13h59
Trap D

Mon info est tirée du MSDN ... :mouarf:
30/04/2006, 14h05
Trap D

Citation:

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.
30/04/2006, 15h36
Emmanuel Delahaye

Citation:

Envoyé par Trap D

Mon info est tirée du MSDN ... :mouarf:

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"...
01/05/2006, 16h56
Médinoc

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...