[C] [réglé] Récupérer le contenu d'un fichier texte ?

Récupérer le contenu d'un fichier texte ? [C] [réglé] - C - Programmation

Marsh Posté le 16-06-2005 à 13:29:36    

Bonjour, je viens vous demander un peu d'aide car là je sèche :D
Petit détail de ce que je veux faire :
J'ai un fichier nommé "test" (juste pour l'instant hein ;)) qui contient des infos que je dois analyser (une en tout cas), et des trous à remplir par l'application !
 
Le fichier se présente comme celà au début (balises de changement de ligne type unix) :

Code :
  1. 1
  2. 0.0
  6. sylver
  13. 37650432


 
J'ai choisi (mauvais choix :??:) de récupérer les informations non connues par mon appli et de reconstruire le fichier de zéro, ca m'a semblé le plus simple vu que l'appli met à jour ce fichier assez souvent !
 
j'ai fait un fichier de test pour vérifier mon analyse du fichier :

Code :
  1. #include <stdio.h>
  4. FILE *stat_tf;
  5. char *stat_file= (char*) 0;
  7. void main (void)
  8. {
  9.   int stat_state = 0;
  10.   char stat_user[80];
  11.   char stat_size[80];
  13.   stat_file = "test";
  14.   stat_tf = fopen(stat_file, "r+" );
  15.   fscanf(stat_tf, "%d%*s%*s%*s%*s%s%*s%*s%*s%*s%*s%*s%s",&stat_state, stat_user, stat_size);
  17.   printf("%d %s %d", stat_state, stat_user, stat_size);
  18. }


et voilà, mon problème c'est que quand le fichier est dans son état initial, la ligne pour lire les infos qui m'interessent est :

Code :
  1. fscanf(stat_tf, "%d%*s%s%s",&stat_state, stat_user, stat_size);


mais par la suite mon application modifie le fichier pour compléter les parties vides, et quand le fichier est completement rempli, la ligne qui lit correctement le fichier est alors :

Code :
  1. fscanf(stat_tf, "%d%*s%*s%*s%*s%s%*s%*s%*s%*s%*s%*s%s",&stat_state, stat_user, stat_size);


exemple de fichier "test" rempli :

Code :
  1. 1
  2. 0.0
  3. 1
  4. 2
  5. 33
  6. sylver
  7. 11
  8. 55
  9. 77
  10. 99
  11. 55
  12. 22
  13. 37650432


 
Le problème (ou j'ai cherché un moment) vient du fait que fscanf considère les retours chariots comme du vent et passe aux prochaines infos utiles => pleins de retours chariot sont équivalent à un espace par exemple ! Ca n'arrange pas du tout mes affaires, donc si vous aviez un piste pour que je m'en sorte, ça serait cool (par exemple un balise type %[] pour que fscanf considère les \n comme de l'info ou éventuellement un approche différente basée sur autre chose que du fscanf :sweat:)
 
Merci d'avance :jap:
 
Sylver


Message édité par _Sylver_ le 16-06-2005 à 16:48:35
Reply

Marsh Posté le 16-06-2005 à 13:29:36   

Reply

Marsh Posté le 16-06-2005 à 13:39:39    

c'est du C

Reply

Marsh Posté le 16-06-2005 à 13:41:24    

lol +1 pour taz

Reply

Marsh Posté le 16-06-2005 à 13:45:39    

Oui c'est du C pour le test, mais mon appli finale est en C++, je ne suis donc pas limité aux primitives C ...
 
Des idées ?


Message édité par _Sylver_ le 16-06-2005 à 13:46:37
Reply

Marsh Posté le 16-06-2005 à 13:48:11    

ben fais le en C++ alors avec des std::string et VERIFIE que les fonctions d'E/S n'échoue pas (si tu sors une seule fois un eof, je me casse)

Reply

Marsh Posté le 16-06-2005 à 13:53:45    

Ca a l'air tellement simple :D (je ne manipule pas souvent les chaines de caractères, c'est pour ça que je ne connais que le scanf :D), je vais chercher sur google :D, en tout cas tu me confirmes que j'aurais toutes les opérateurs pour faire ce que je souhaite ?
 
Merci en tout cas !
 
PS : pourquoi les fonctions d'IO échoueraient ?

Reply

Marsh Posté le 16-06-2005 à 15:16:05    

Je déplace dans [C] car en C++ c'est un peu lourd je trouve !
Edit : bon j'ai trouvé une méthode qui marche, à base de gros fseek des familles !
 

Code :
  1. #include <stdio.h>
  4. FILE *stat_tf;
  5. char *stat_file= (char*) 0;
  7. void main (void)
  8. {
  9.   int stat_state = 0;
  10.   char stat_user[80];
  11.   char stat_size[80];
  12.   long int offset;
  13.   int  compteur;
  15.   stat_file = "test";
  16.   stat_tf = fopen(stat_file, "r+" );
  18.   /* Get state */
  19.   fscanf(stat_tf, "%d", &stat_state);
  21.   /* Jump to owner line */
  22.   for (compteur=0;compteur<6;compteur++){
  23. while (fgetc(stat_tf) != '\n'){
  24.  offset++;
  25.  fseek (stat_tf,offset,SEEK_SET);
  26. }
  28.     offset++;
  29.     fseek (stat_tf,offset,SEEK_SET);
  30.   }
  32.   fscanf(stat_tf, "%s", stat_user);
  33.  
  34.   /* Jump to size line */
  35.   for (compteur=0;compteur<8;compteur++){
  36. while (fgetc(stat_tf) != '\n'){
  37.  offset++;
  38.  fseek (stat_tf,offset,SEEK_SET);
  39. }
  41.     offset++;
  42.     fseek (stat_tf,offset,SEEK_SET);
  43.   }
  45.   fscanf(stat_tf, "%s", stat_size);
  46.  
  47.   printf("%d %s %d", stat_state, stat_user, stat_size);
  48. }


 
C'est moche mais ça marche ! En attendant de trouver mieux, au moins je peux avancer ;)


Message édité par _Sylver_ le 16-06-2005 à 15:44:39
Reply

Marsh Posté le 16-06-2005 à 18:11:50    

et bien pris pour que ton fichier soit bien formé ... parce que là c'est n'importe quoi

Reply

Marsh Posté le 16-06-2005 à 19:22:44    

[:hide]  
 
C'est ni du C++, ni du C. C'est portnawak.
 
Je me permets de commenter :
 

Code :
  1. #include <stdio.h>
  3. /* ERK ! Pourquoi des globales ?! */
  4. FILE *stat_tf;
  6. /* re-ERK ! tu as donc *stat_file = NULL, je parie que tu  
  7.    ne lui alloueras rien en memoire, invoquant un superbe
  8.    comportement indefini. Voyons la suite */
  9. char *stat_file= (char*) 0;
  11. /* Non : main() est cense retourner un int.
  12.    donc : int main(void) */
  13. void main (void)
  14. {
  15.   int stat_state = 0;
  16.   char stat_user[80];
  17.   char stat_size[80];
  18.   long int offset;
  19.   int  compteur;
  21.   /* bingo, le beau comportement indefini que voila. Tu n'as  
  22.      absolument aucun espace memoire pour stocker ta chaine  
  23.      dans stat_file. Ca passe ou ca casse (coredump sous *nix,
  24.      violation machin error sous windows), c'est aleatoire.
  25.      Voir malloc() pour corriger ca, au plus vite */
  26.   stat_file = "test";
  28.   /* admettons que fopen() echoue, genre le fichier existe pas,
  29.      tu n'as pas les droits ou autre, stat_tf == NULL.
  30.      Il faut le tester pour s'assurer que tout fonctionne.
  31.      Et je ne vois pas l'utilité du "+" dans "r+" */
  32.   stat_tf = fopen(stat_file, "r+" );
  35.   /* Pourquoi pas, mais ce serait bien... voire necessaire, de
  36.      verifier que fscanf() a reussi a recuperer un entier. */
  37.   fscanf(stat_tf, "%d", &stat_state);
  39.   /* Ton programme n'a decidement aucune chance
  40.      face a la moindre malformation de ton fichier. */
  41.   for (compteur=0;compteur<6;compteur++){
  42.         /* Oh, le beau while (fgetc() ...)
  43.            Pourquoi ne pas plutot utiliser fgets() ? */
  44. while (fgetc(stat_tf) != '\n'){
  45.                 /* offset++ ?! Mais il n'est jamais  
  46.                    initialisé, il y a potentiellement
  47.                    n'importe quoi dedans ! */
  48.  offset++;
  49.                 /* WTF ?! fseek() ? */
  50.  fseek (stat_tf,offset,SEEK_SET);
  51. }
  53.     /* o_O */
  54.     offset++;
  55.     fseek (stat_tf,offset,SEEK_SET);
  56.   }
  58.   /* Ouh la la. Mais comment ca arrive a fonctionner tout ca  
  59.      Usine a gaz. */
  60.   fscanf(stat_tf, "%s", stat_user);
  61.  
  62.   /* Ok. En fait c'est pas une usine a gaz, c'est Tchernobyl */
  63.   for (compteur=0;compteur<8;compteur++){
  64. while (fgetc(stat_tf) != '\n'){
  65.  offset++;
  66.  fseek (stat_tf,offset,SEEK_SET);
  67. }
  69.     offset++;
  70.     fseek (stat_tf,offset,SEEK_SET);
  71.   }
  73.   fscanf(stat_tf, "%s", stat_size);
  74.  
  75.   printf("%d %s %d", stat_state, stat_user, stat_size);
  76. }


 

Citation :


C'est moche mais ça marche ! En attendant de trouver mieux, au moins je peux avancer ;)


 
C'est pire que ça.
Avancer sur une telle base, c'est courir à la catastrophe. Enfin "courir à la catastrophe", disons plutôt que tu viens de t'asperger d'essence, habillé par un pagne en paille, en train de tourner tes saucisses sur le barbecue, et que non content de la situation, tu décides d'allumer une clope en prime.
 
Sérieusement, commence par une base solide et bien plus saine que cela, parce que là c'est ignoble.
Vu ton fichier, je lirais tout bêtement le fichier ligne à ligne, en stockant au passage toutes les lignes non vides, pourquoi pas dans une liste chaînée.
Je me débrouillerais ensuite pour tester l'intégrité des données lues.
Une fois ça fait, le plus dur est passé. Il suffit de compléter la liste chaînée, et de la stocker dans un nouveau fichier.


Message édité par Elmoricq le 16-06-2005 à 19:30:15
Reply

Marsh Posté le 16-06-2005 à 19:55:44    

je ne suis pas d'accord pour
stat_file = "test";
 
c'est moche oui, mais le comportement n'est pas indefinis, tu affecte au pointeur l'adresse d'un tableau constant

Reply

Marsh Posté le 16-06-2005 à 19:55:44   

Reply

Marsh Posté le 17-06-2005 à 00:14:52    

- Oui oui pour le stat_file = "test", pas de soucis de mémoire, on mets juste à jour le pointeur vers une zone prédéfinie par le compilo !
Mais bon c'est pas fait comme ça dans le vrai soft (passage du nom par parametre, là c'est un soft de test !
- J'avais bien oublié le offset = 0 au début (pas vu car sous windows c'était mis à 0 sans init) !
- le r+, c'est parceque la suite de mon soft réécrit les infos, mais bon c'est pas mis dans la partie de test en effet :)
- pour fgets(), je ne connaissais pas, je vais regarder :jap:
- Et en effet, fichier mal formaté = cata :D
 
Merci pour ton analyse :jap:
 
J'ai fait quelques simplifications/protections :D

Code :
  1. #include <stdio.h>
  4. FILE *stat_tf;
  5. char *stat_file= (char*) 0;
  7. void main (void)
  8. {
  9.   int stat_state = 0;
  10.   char stat_user[80];
  11.   char stat_size[80];
  12.   char read_char;
  13.   int  i;
  15.   stat_file = "test";
  16.   stat_tf = fopen(stat_file, "r+" );
  18.       /* Get state */
  19.       fscanf(stat_tf, "%d", &stat_state);
  21.       /* Jump to owner line */
  22.   for (i=0;i<5;i++){
  23.    while ((((read_char=fgetc(stat_tf))) != '\n')&&(read_char != EOF));
  24.   }
  26.   fscanf(stat_tf, "%s", stat_user);
  28.   /* Jump to size line */
  29.    for (i=0;i<7;i++){
  30.    while ((((read_char=fgetc(stat_tf))) != '\n')&&(read_char != EOF));
  31.   }
  33.   fscanf(stat_tf, "%s", stat_size);
  35.   printf("%d %s %s", stat_state, stat_user, stat_size);
  36. }


Message édité par _Sylver_ le 17-06-2005 à 10:01:01
Reply

Marsh Posté le 18-06-2005 à 09:13:55    

Je rappelle que Taz a écrit

Citation :

(si tu sors une seule fois un eof, je me casse)


 :whistle:  
 
==> fgets...
 
Tout cela est bien moche, en effet.


Message édité par el muchacho le 18-06-2005 à 09:19:25
Reply

Marsh Posté le 18-06-2005 à 13:54:08    

C'est ça, le code qui va se retrouver dans ton appli ? Et tu dis dans le titre du topic que ton pb est réglé ?? :heink:

Reply

Marsh Posté le 20-06-2005 à 09:42:23    

En effet le code est pourri, mais pour l'instant, comme il fonctionne et qu'il est protégé, je me concentre sur d'autres problèmes !
Je pense que la solution que m'a donné Elmoricq (merci beaucoup Elmoricq au fait ;)) est la plus propre (lire toutes les lignes et les stocker soit dans une liste chainée, soit dans une structure), je réécrirai le code de cette façon quand le reste fonctionnera !

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed