Discussion :

[salseropom]

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

Mon fichier input.txt est le suivant

TEMP 350
PRES 1.5
EGR 40

et mon code source est ici :

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

[Emmanuel Delahaye]

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 :

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

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

[salseropom]

Salut, voici le code revu et corrigé

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

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

option de compilation

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)

[Thierry Chappuis]

Voici ma version:

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

[salseropom]

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

[Emmanuel Delahaye]

option de compilation

Mes paramètres ne sont pas 'en plus'. Il sont 'à la place'. Tu dois donc retirer les premiers paramètres.

[Emmanuel Delahaye]

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