Problème avec mon fichier d'entrées

Version imprimable

Bonjour, j'ai un pb avec mon fichier d'entrée.

Mon fichier input.txt est le suivant

Citation:

TEMP 350
PRES 1.5
EGR 40

et mon code source est ici :

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
 
#include<stdlib.h>
#include<stdio.h>
#include<string.h>
 
int main(void)
{
  char line[1000],keyword[50];
 
  enum Keyword {TEMP,PRES,EGR} kwd;
 
  int nbkeywords=3; /* nombre de mot cle == "longueur" de Keyword */
 
  char * listkwd={"TEMP","PRES","EGR"}; /* liste des mots-cle */
 
  int keywordOK[3]={0}; /* keywordOK[i]==1 : le mot-cle a ete saisi
                           keywordOK[i]==0 : le mot-cle n'a pas ete saisi
			 */
 
  double temp,pres,egr;
  int err;
  char * p=NULL;
 
  FILE * file=fopen("input.txt","r");
  if(file==NULL)
  {
    fprintf(stderr,"erreur ouverture\n");
    exit(1);
  }
 
  while(fgets (line, sizeof line, file) != NULL)
  {
    p=strchr(line,'\n');
    if(p!=NULL) *p=NULL;
 
    if(strstr(line,"TEMP ")!=NULL)
    {
      sscanf(line,"%s%lf",keyword,&temp);
      kwd=TEMP;
      keywordOK[kwd]=1;
    }
    else if(strstr(line,"PRES ")!=NULL)
    {
      sscanf(line,"%s%lf",keyword,&pres);
      kwd=PRES;
      keywordOK[kwd]=1;
    }
    else
    {
      sscanf(line,"%s%lf",keyword,&egr);
      kwd=EGR;
      keywordOK[kwd]=1;
    }
  } /* fin while(fgets (line, sizeof line, file) != NULL) */
 
  if(!feof(file))
  {
    fprintf(stderr,"%s %d : Error : a misreading occurred in file input.txt\n",__FILE__,__LINE__); 
    return EXIT_FAILURE;
  }
 
  if(ferror(file))
  {
    fprintf(stderr,"%s %d : a misreading occurred in file input.txt\n",__FILE__,__LINE__);
    return EXIT_FAILURE;
  }
 
  fclose(file); file=NULL;
 
  unsigned i;
  err=0;
 
  for(i=0;i<nbkeywords;++i)
  {
    if(keywordOK[i]==0)
    {
      fprintf(stderr,"keyword %s must be specified\n",listkwd[i]);
      err=1;
    }
  }
 
  if(err==1) exit(1);
 
  printf("T = %f P = %f EGR = %f\n",temp,pres,egr);
 
  return 0;
}

Je veux vérifier que l'utilisateur a bien saisi les 3 mots-clé (TEMP, PRES, EGR), d'où mes variables kwd, listkwd, keywordOK.

Y aurait-il une manière un peu plus judicieuse que celle que j'ai faite ? Je trouve que enum Keyword et listkeyword sont redondants et en plus il faut que la liste soit saisie dans le même ordre ...

Merci encore.

Citation:

Envoyé par salseropom

et mon code source est ici :

Depuis le temps, tu devrais savoir configurer ton compilateur pour qu'il te dise ce qui passe dans ton code ...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `main':
main.c:13: warning: initialization discards qualifiers from pointer target type
main.c:13: warning: excess elements in scalar initializer
main.c:13: warning: (near initialization for `listkwd')
main.c:13: warning: excess elements in scalar initializer
main.c:13: warning: (near initialization for `listkwd')
main.c:33: warning: assignment makes integer from pointer without a cast
main.c:72: warning: comparison between signed and unsigned
main.c:76: warning: format argument is not a pointer (arg 3)
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 8 warnings

Ma dernière proposition

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
 
#include<stdlib.h>
#include<stdio.h>
#include<string.h>
 
#define FNAME "data.txt"
 
enum Keyword
{ TEMP, PRES, EGR, NB_KWD };
 
static char const *listkwd[] = { "TEMP", "PRES", "EGR" }; /* liste des mots-cle */
 
int get_value (char const *line, enum Keyword e_kwd, int keywordOK[],
               double val[])
{
   int ok = 0;
 
   if (e_kwd < NB_KWD)
   {
      char const *s_kwd = listkwd[e_kwd];
 
      char *p;
      if ((p = strstr (line, s_kwd)) != NULL)
      {
         val[e_kwd] = strtod (p + strlen (s_kwd), NULL);
         keywordOK[e_kwd] = 1;
         ok = 1;
      }
   }
   return ok;
}
 
int main (void)
{
   int ret;
 
   FILE *file = fopen (FNAME, "r");
   if (file != NULL)
   {
      double val[3] = { 0 };
      int keywordOK[3] = { 0 }; /* keywordOK[i]==1 : le mot-cle a ete saisi
                                   keywordOK[i]==0 : le mot-cle n'a pas ete saisi
                                 */
      char line[1000];
 
      while (fgets (line, sizeof line, file) != NULL)
      {
         {
            char *p = strchr (line, '\n');
            if (p != NULL)
            {
               *p = 0;
            }
         }
         {
            enum Keyword e_kwd;
            for (e_kwd = 0;
                 e_kwd < NB_KWD && !get_value (line, e_kwd, keywordOK, val);
                 e_kwd++)
            {
            }
         }
      }                         /* fin while */
 
      fclose (file), file = NULL;
 
      {
         unsigned i;
         int err = 0;
 
         for (i = 0; i < NB_KWD; ++i)
         {
            if (keywordOK[i] == 0)
            {
               fprintf (stderr, "keyword %s must be specified\n", listkwd[i]);
               err = 1;
            }
         }
 
         if (err == 1)
         {
            fprintf (stderr, "erreur ouverture\n");
            ret = EXIT_FAILURE;
         }
         else
         {
            for (i = 0; i < NB_KWD; ++i)
            {
               printf ("%s = %g ", listkwd[i], val[i]);
            }
            printf ("");
            ret = EXIT_SUCCESS;
         }
      }
   }
   else
   {
      perror (FNAME);
      ret = EXIT_FAILURE;
   }
   return ret;
}

Pose des questions si tu ne comprends pas.

Salut, voici le code revu et corrigé
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 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 #include<stdlib.h> #include<stdio.h> #include<string.h> int main(void) { char line[1000],keyword[50]; enum Keyword {TEMP,PRES,EGR} kwd; int nbkeywords=3; /* nombre de mot cle == "longueur" de Keyword */ char listkwd[3][50]={"TEMP","PRES","EGR"}; /* liste des mots-cle */ int keywordOK[3]={0}; /* keywordOK[i]==1 : le mot-cle a ete saisi keywordOK[i]==0 : le mot-cle n'a pas ete saisi */ double temp,pres,egr; int err; char * p=NULL; FILE * file=fopen("input.txt","r"); if(file==NULL) { fprintf(stderr,"erreur ouverture\n"); exit(1); } while(fgets (line, sizeof line, file) != NULL) { p=strchr(line,'\n'); if(p!=NULL) *p=0; if(strstr(line,"TEMP ")!=NULL) { sscanf(line,"%s%lf",keyword,&temp); kwd=TEMP; keywordOK[kwd]=1; } else if(strstr(line,"PRES ")!=NULL) { sscanf(line,"%s%lf",keyword,&pres); kwd=PRES; keywordOK[kwd]=1; } else { sscanf(line,"%s%lf",keyword,&egr); kwd=EGR; keywordOK[kwd]=1; } } /* fin while(fgets (line, sizeof line, file) != NULL) */ if(!feof(file)) { fprintf(stderr,"%s %d : Error : a misreading occurred in file input.txt\n",__FILE__,__LINE__); return EXIT_FAILURE; } if(ferror(file)) { fprintf(stderr,"%s %d : a misreading occurred in file input.txt\n",__FILE__,__LINE__); return EXIT_FAILURE; } fclose(file); file=NULL; int i; err=0; for(i=0;i<nbkeywords;++i) { if(keywordOK[i]==0) { fprintf(stderr,"keyword %s must be specified\n",listkwd[i]); err=1; } } if(err==1) exit(1); printf("T = %f P = %f EGR = %f\n",temp,pres,egr); return 0; }
mais ma question, bien qu'il y a des warnings (pb avec le fgets), porte plus sur la méthode pour vérifier que l'utilisateur a bien entré tous les mots-clé que sur ma qualité de programmation (qui bien sûr est à la base de tout...)

Citation:

main.c:30: attention : passage de l'argument n°2 de « fgets » a une largeur différente en raison du prototype

option de compilation

Citation:

ED_PARANOIA=-Wchar-subscripts -Wcomment -Wformat=2 -Wimplicit-int -Werror-implicit-function-declaration -Wmain -Wparentheses -Wsequence-point -Wreturn-type -Wswitch -Wtrigraphs -Wunused -Wuninitialized -Wunknown-pragmas -Wfloat-equal -Wundef -Wshadow -Wpointer-arith -Wbad-function-cast -Wconversion -Wsign-compare -Waggregate-return -Wmissing-prototypes -Wmissing-declarations -Wmissing-noreturn -Wformat -Wmissing-format-attribute -Wno-deprecated-declarations -Wpacked -Wredundant-decls -Wnested-externs -Winline -Wlong-long -Wunreachable-code

CFLAGS=-Wall -W -O2 -Wwrite-strings -Wstrict-prototypes $(ED_PARANOIA)

Voici ma version:

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
enum Keyword {
    KEY_TEMP,
    KEY_PRES,
    KEY_EGR,
    KEY_NB
};
 
void fclean(char *tampon, FILE *fp);
 
int main(void)
{
    int err = EXIT_SUCCESS;
    FILE * file = NULL;
 
    file = fopen("input.txt", "r");
    if (file != NULL)
    {
        char line[1000] = "";
        double keywordVal[KEY_NB] = {0.0};
        int keywordOK[KEY_NB] = {0};
        char const * keywords[] = {"TEMP", "PRES", "EGR"};
        int i;
 
        while(fgets (line, sizeof line, file) != NULL)
        {
            fclean(line, file);
 
            if(strstr(line, "TEMP ") != NULL)
            {
                sscanf(line, "TEMP %lf", keywordVal + KEY_TEMP);
                keywordOK[KEY_TEMP] = 1;
            }
            else if(strstr(line,"PRES ") != NULL)
            {
                sscanf(line, "PRES %lf", keywordVal + KEY_PRES);
                keywordOK[KEY_PRES] = 1;
            }
            else
            {
                sscanf(line, "EGR %lf", keywordVal + KEY_EGR);
                keywordOK[KEY_EGR] = 1;
            }
        }
 
        if (feof(file) == 1)
        {
            for (i = 0; i < KEY_NB; i++)
            {
                if (keywordOK[i] == 0)
                {
                    fprintf(stderr,"keyword %s must be specified\n", keywords[i]);
                    err = EXIT_FAILURE;
                }
            }
        }
        else
        {
            fprintf(stderr,"%s %d : Error : a misreading occurred in file"
                    "input.txt\n",__FILE__,__LINE__);
            err = EXIT_FAILURE;
        }
 
        fclose(file), file = NULL;
    }
    else
    {
        fprintf(stderr, "Impossible d'ouvrir le fichier!\n");
        err = EXIT_FAILURE;
    }
 
    return 0;
}
 
void fclean(char *tampon, FILE *fp)
{
    if (tampon != NULL && fp != NULL)
    {
        char *pc = strchr(tampon, '\n');
 
        if (pc != NULL)
        {
            *pc = 0;
        }
        else
        {
            int c;
 
            while ((c = fgetc(fp)) != '\n' && c != EOF)
            {}
        }
    }
}

Thierry

11/10/2007, 18h05
salseropom

oups, je n'avais pas vu que tu avais corrigé mon code...
11/10/2007, 18h40
Emmanuel Delahaye

Citation:

Envoyé par salseropom

option de compilation

Mes paramètres ne sont pas 'en plus'. Il sont 'à la place'. Tu dois donc retirer les premiers paramètres.
11/10/2007, 19h08
Emmanuel Delahaye

Citation:

Envoyé par salseropom

oups, je n'avais pas vu que tu avais corrigé mon code...

Et j'ai posté une nouvelle proposition de solution. Je pense qu'il faudrait regrouper tout ça dans une structure...