Lecture d'un fichier et stocker le contenu dans des variables

Version imprimable

Bonjour

J'essaye de lire un fichier qui contient des donnees, et mon programme devra lire le contenu et stocker le contenu lu dans des variables de type char. Mon code maintenant ne commence a stocker qu'a partir de la troisieme ligne. Je ne vois pas ou est le probleme.
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 int parser (void) { int n = 0; char buffer [ 81 ]; char arg[50][9]; // tableau pour les donnees /* Verifier le succes de l'ouverture */ if ( ( sf = fopen ( fname, "r" ) ) == NULL ) { fprintf ( stderr, "MAKELIST: Unable to open %s!\n", fname ); exit ( 1 ); } /* Lecture du fichier */ fgets ( buffer, 81, sf ); while ( ! feof ( sf ) ) { fputs ( buffer, stdout ); /* si un des mots contenus dans le fichier est "add", chercher si un deuxieme mot dans la meme ligne existe, puis le stocker lui aussi */ if (strcmp(arg[n],"add")){ fscanf(sf, "%s %s\n", arg[n], arg[n+1]); } // Stocker tout les mots tant que la while loop s'execute fscanf(sf, "%s\n", arg[n]); // imprimer ce qui a ete stocker pour tester printf("%s\n", arg[n]); fgets ( buffer, 81, sf ); n++; // incrementer le compteur } fclose ( sf ); //fermer le fichier return 0; }
Un exemple de fichier sf
Code:

1 2 3 4 5 un deux add qqchose trois quat
L'output de la ligne printf("%s\n", arg[n]);
Code:

1 2 add qqchose
Normalement, l'output devrait etre le meme que le fichier sf.

Comment faire pour ne pas avoir le retour de la ligne quand add est lue.

Pourquoi un et deux n'ont pas ete lue et imprimee ?

PS: la variable fname a ete declare en tant que static (pour une utilisation ulterieur dans d'autres fonctions).

Merci de votre aide.

02/11/2007, 19h01
diogene

C'est lié au mélange fgets qui lit une ligne et les fscanf qui lisent des chaînes sur le même fichier. On devrait avoir une lecture par ligne avec fgets et une analyse de chaque ligne par sscanf

Merci, j'ai utilise sscanf, et ca semble marcher, mais seulement un mot par ligne :roll:

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
 
int parser (void) {
  int n = 0;
  char buffer [ 81 ], arg[50][9];
 
  if ( ( sf = fopen ( fname, "r" ) ) == NULL ) {
    fprintf ( stderr, "MAKELIST: Unable to open %s!\n", fname );
    exit ( 1 );
    }
 
  fgets ( buffer, 81, sf );
  while ( ! feof ( sf ) ) {
    if (strcmp(arg[n],"add")){
        sscanf(buffer, "%s %s\n", arg[n], arg[n]+strlen(arg[n]));
        }
    sscanf(buffer, "%s\n", arg[n]);
    char *p;
    p = arg[n]+strlen(arg[n]);
    if (arg[n] != NULL && !p){
        printf("%s\n", arg[n]);
        }
    if (arg[n] != NULL && p){
        printf("%s %s\n", arg[n], p);
        }
    fgets ( buffer, 81, sf );
    n++;
    }
    fclose ( sf );
  return 0;
}

quand le fichier contient par exemple dans n'importe quel ligne: add qchose Le programme ne stocke que "add" dans arg[n] et ne fait rien pour le mot qui suit add. J'ai essaye d'utiliser strlen... sans succes.
Des idees sur comment y remedier ?

02/11/2007, 21h48
nicolas.sitbon

pourrais tu mettre le code complet s'il te plaît.
Cordialement.

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
 
# include <stdio.h>
# include <string.h>
# include <stdlib.h>
# include <ctype.h>
 
void createfile(char []);
 
static FILE *sf;
static char fname[30];
static char com[81];
 
int parser (void) {
  int n = 0;
  char buffer [ 81 ], arg[50][9];
 
  if ( ( sf = fopen ( fname, "r" ) ) == NULL ) {
    fprintf ( stderr, "MAKELIST: Unable to open %s!\n", fname );
    exit ( 1 );
    }
 
  fgets ( buffer, 81, sf );
  while ( ! feof ( sf ) ) {
    //fputs ( buffer, stdout );
    if (strcmp(arg[n],"add")){
        sscanf(buffer, "%s %s\n", arg[n], arg[n]+strlen(arg[n]));
        }
    sscanf(buffer, "%s\n", arg[n]);
    char *p;
    p = arg[n]+strlen(arg[n]);
    if (arg[n] != NULL && !p){
        printf("%s\n", arg[n]);
        }
    if (arg[n] != NULL && p){
        printf("%s %s\n", arg[n], arg[n]+strlen(arg[n]));
        }
    fgets ( buffer, 81, sf );
    n++;
    }
    fclose ( sf );
  return 0;
}
 
int newfile ( )
{
  printf ("\n\tPlease, specify a filename: " );
  gets(fname);
  createfile(fname);
  return 0;
}
 
void createfile(char file[]){
   sf = fopen(file, "w");
   if ( sf != NULL ) {
      printf("\n\tFile \"%s\" created.\n", file);
      printf("\n\tEnter commands : \n");
      fclose (sf);
      }
}
 
void append(){
   sf = fopen(fname,"a");
   if ( sf != NULL ) {
       fprintf(sf, "%s\n", com);
       fclose(sf);
       }
}
 
int main (void) {
 
 printf("Enter commands : \n");
 
 while ( strcmp(com, "quit") != 0 ) {
   printf("\t >  ");
   gets(com);
   if (sf == NULL) {
      printf("\n\tError, no script file detected.\n ");
      newfile();
      }
     append();
   }
  parser();
return 0;
}

Citation:

Envoyé par virtuadrack

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
# include <stdio.h>
# include <string.h>
# include <stdlib.h>
# include <ctype.h>

void createfile(char []);

static FILE *sf; // Ces globales ne sont pas justifiées . A éviter absolument!
static char fname[30];
static char com[81];

int parser (void) {
  int n = 0;
  char buffer [ 81 ], arg[50][9];

  if ( ( sf = fopen ( fname, "r" ) ) == NULL ) {
    fprintf ( stderr, "MAKELIST: Unable to open %s!\n", fname );
    exit ( 1 );
    }

  fgets ( buffer, 81, sf );
  while ( ! feof ( sf ) ) {
    //fputs ( buffer, stdout );
    if (strcmp(arg[n],"add")){ // la chaîne arg[n] n'est pas initialisée pour n =0
        sscanf(buffer, "%s %s\n", arg[n], arg[n]+strlen(arg[n]));
         // strlen(arg[n]) n'est pas signifiant à l'entrée de la fonction puique arg[n] n'est rempli qu'à la sortie de la fonction!
// De plus ca n'a aucun sens d'aller écrire à cette adresse
        }
    sscanf(buffer, "%s\n", arg[n]);
    char *p;
    p = arg[n]+strlen(arg[n]); // idem, cette adresse n'a pas de sens
    if (arg[n] != NULL && !p){ // arg[n] et p sont != NULL, c'est sur
        printf("%s\n", arg[n]);
        }
    if (arg[n] != NULL && p){
        printf("%s %s\n", arg[n], arg[n]+strlen(arg[n])); // idem
        }
    fgets ( buffer, 81, sf );
    n++;
    }
    fclose ( sf );
  return 0;
}

int newfile ( )
{
  printf ("\n\tPlease, specify a filename: " );
  gets(fname); // ne pas utilier gets mais fgets sur le flux stdin  createfile(fname);
  return 0;
}

void createfile(char file[]){
   sf = fopen(file, "w");
   if ( sf != NULL ) {
      printf("\n\tFile \"%s\" created.\n", file);
      printf("\n\tEnter commands : \n");
      fclose (sf);
      }
}

void append(){
   sf = fopen(fname,"a");
   if ( sf != NULL ) {
       fprintf(sf, "%s\n", com);
       fclose(sf);
       }
}

int main (void) {

 printf("Enter commands : \n");

 while ( strcmp(com, "quit") != 0 ) {
   printf("\t >  ");
   gets(com);
   if (sf == NULL) {
      printf("\n\tError, no script file detected.\n ");
      newfile();
      }
     append();
   }
  parser();
return 0;
}

Quelques remarques en commentaires dans le code.
Pourtant, sur le principe, ce n'est pas si compliqué :
Jusqu'à la fin du fichier :
- lire une ligne
- Extraire la première chaîne de la ligne
- Si cette chaîne est "add" ( je ne sais pas ce que tu veux faire du "add") extraire la seconde chaîne
- recommencer

03/11/2007, 19h02
virtuadrack
Citation:
Envoyé par diogene

Code:

1 2 static FILE *sf; // Ces globales ne sont pas justifiées . A éviter absolument!
Que dois-je utiliser a la place ? Mes globales sont censee servir pour d'autres fonctions si tu regardes le code entierement.
Citation:
Code:

1 2 gets(fname); // ne pas utilier gets mais fgets sur le flux stdin
avec fgets(fname, 81, stdin) un fichier se cree mais avec un retour a la ligne (^\0)

output avec fgets
Code:

1 2 File "fichier1 " created.
avec gets:

Code:

File "fichier1" created.

Quel est le probleme ici ?
03/11/2007, 19h15
diogene

Citation:

Mes globales sont censee servir pour d'autres fonctions si tu regardes le code entierement.

Alors on les passera en arguments de ces fonctions.

Citation:

Quel est le probleme ici ?

gets ne garantit aucunement contre un dépassement de longueur du tableau où on stocke la ligne. C'est extrèmement dangereux pour une entrée clavier où l'utilisateur peut taper n'importe quoi.
fgets conserve le '\n' c'est vrai. Si il te gêne, il suffit de le rechercher (strchr) et de le remplacer par '\0'. C'est embêtant mais l'inconvénient de gets justifie cet embarras.