[C] Plantage incompréhensible...

Plantage incompréhensible... [C] - C - Programmation

Marsh Posté le 18-04-2006 à 19:49:41    

Pour mes études, je dois réaliser un projet qui permet, à partir de fiches de stage en .txt remplies par les étudiants selon le modèle ci-dessous, d'extraire les informations intéressantes (après les ":" ou la case cochée) et d'écrire seulement toutes ces informations dans un seul fichier texte qui devra à terme contenir des instruction SQL qui serviront à remplir une base de donnée mais là n'est pas le sujet.  
 
--------------------------------------------------------------------------------
DESCRIPTIF DU STAGE
 
 
Titre du rapport :
Raison sociale de l'entreprise :
Code postal :
Ville :
 
 
année :
cd :
répertoire :
 
 
Domaine(s) d'activités abordés durant le stage :
[  ]  Type informatique
[  ]  Type réseaux
[  ]  Type internet
[  ]  Type téléphonie
[  ]  Type commercial
[  ]  Type maintenance gros systèmes
[  ]  Type opérateur radiocom
[  ]  Type maintenance et gestion de matériel réseau ou téléphonie de grande capacité
[  ]  Type prestation de services
 
 
Résumé :  
 
 
--------------------------------------------------------------------------------
 
 
J'ai fait le programme suivant :
 

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <string.h>
  4. #include <dir.h>
  5. #include<dirent.h>
  7. void chercher (char* nomfichier, FILE *f2)
  8. {
  9. FILE *f; // Pointeur vers fichier f  
  10. char information[3000]; // Chaine de 3000 caractères au max  
  11. char *pos; // Chaine de caractère représentant le pointeur  
  12. char donnee [3000]; // Chaine de 3000 caractères au max  
  13. char info[3000]; //Chaine de 3000 caractères au max  
  14. f=fopen(nomfichier, "r" ); //OUverture du fichier texte comme celui ci-dessus en lecture  
  15. while (!feof(f)) //Tant qu'on est pas à la fin du fichier faire  
  16. {
  17.   fgets(information, 256, f); /* Récupération de la ligne et stockage dans information*/
  18.   pos=strstr(information, ": " ); /*strstr renvoie la valeur du pointeur se trouve  la première ocurence des ":_" et stockage dans pos */
  19.   printf("%s",pos) ;
  20.   if (pos != NULL) //Si la valeur du pointeur n'est pas nulle...
  21. {
  22.      pos++; // Incrémentation de pos  
  23.      pos++ ; // On enlève le blanc suivant les ":" aussi  
  24.      fprintf(f2, "%s", pos); // Ecriture de la chaine dans le fichier texte crée  
  25. }
  26. }
  27.   fclose(f); //On ferme le fichier f
  28. }
  30. int main()
  31. {
  32. FILE *f; //Pointeur sur fichier f
  33. FILE *f2; //Pointeur sur fichier f2
  34. int l;
  35. char nomfichier[256];
  36. printf("Bienvenue dans le programme de tri des fiches de stage\n" );
  37. system("dir *.txt /b  > res.txt" ); // Liste toutes les fiches de stage dans res.txt
  38. f=fopen("res.txt","r" ); ///OUvre ce fichier
  39. system(" echo. > fiche.txt" ); //Création d'un fichier texte vide  
  40. f2=fopen("fiche.txt", "w" ); //Ouverture de ce fichier texte en écriture  
  41. while (!feof(f)) //Tant qu'on est pas à la fin de ce fichier
  42. {
  43.   printf(".\n" ); 
  44.   fgets( nomfichier, 256, f ) ;  //Récupération d'un nom de fiche de stage
  45.   l=strlen(nomfichier); //Affectation de la longueur de ce nom à l
  46.   nomfichier[l-1]='\0'; //On remplace le /n par /0
  47.   if(nomfichier!=NULL)//Si il n'y a pas d'erreur lors de l'ouverture du fichier...
  48.   {
  49.    chercher(nomfichier, f2); //Appel à la fonction chercher
  50.   }
  51.   else
  52.   {
  53. printf("une erreur est survenue pendant l'execution\n" );
  54.   }
  56. }
  57.   fclose(f); //Fermeture du fichier f
  58.   return 0; //Fin du programme
  59. }


 
Cependant lorsque je compile ce programme sous dev C++ ou sous turbo C, il ne me met aucune erreur ni warning mais lorsque je lance l'éxécutable, le programme plante ou sous turbo C quand je fais run, ça me donne une sorte de boulce infinie avec que des <NULL> à l'écran...
Si quelqu'un pouvait me dire où j'ai fait l'erreur, ça serait sympa...
 :)

Message cité 1 fois
Reply

Marsh Posté le 18-04-2006 à 19:49:41   

Reply

Marsh Posté le 18-04-2006 à 20:40:24    

Code :
  1. system(" echo. > fiche.txt" );


 
pas necessaire, regardes la doc de fopen
 

Code :
  1. while (!feof(f))


 
regardes la doc de feof
 

Code :
  1. nomfichier[l-1]='\0'; //On remplace le /n par /0


 
pas forcement, regardes la doc de fgets
 

Code :
  1. if(nomfichier!=NULL)


 
faute de frappe ? ce test est toujours vrai
 

Code :
  1. fclose(f); //Fermeture du fichier f


 
faudrais aussi fermer f2 et faire plus de controle sur les e/s
 

Code :
  1. printf("%s",pos) ;
  2. if (pos != NULL)


 
le printf devrait etre dans le if sinon ca plante si pos est nul

Reply

Marsh Posté le 18-04-2006 à 21:32:32    

Joker28 a écrit :


Cependant lorsque je compile ce programme sous dev C++ ou sous turbo C, il ne me met aucune erreur ni warning mais lorsque je lance l'éxécutable, le programme plante ou sous turbo C quand je fais run, ça me donne une sorte de boulce infinie avec que des <NULL> à l'écran...
Si quelqu'un pouvait me dire où j'ai fait l'erreur, ça serait sympa...
 :)


 

  • fopen() peut échouer
  • feof() ne fait pas ce que tu crois. Il faut tester le retour des fonctions de lecture.
  • Un fichier ouvert en "w" est automatiquement créé sii il n'existe pas.
  • f2 n'est pas fermé.
  • La trace 'printf("%s", pos)' doit être mise après le test de pos...


A la vache, grillaid à 100% !
 

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <string.h>
  5. #define DBG 0
  7. static void clean (char *s, FILE *fp)
  8. {
  9.    /* search ... */
  10.    char *p = strchr (s, '\n');
  11.    if (p != NULL)
  12.    {
  13.       /* ... and kill */
  14.       *p = 0;
  15.    }
  16.    else
  17.    {
  18.       /* purge */
  19.       int c;
  20.       while ((c = fgetc(fp)) != '\n' && c != EOF)
  21.       {
  22.       }
  23.    }
  24. }
  26. static void chercher (char* nomfichier, FILE *f2)
  27. {
  28.    FILE *f = fopen(nomfichier, "r" );
  29.    if (f != NULL)
  30.    {
  31.    char information[256];
  32.    while (fgets(information, sizeof information, f) != NULL)
  33.    {
  34.       /* strstr() renvoie la valeur du pointeur qui se trouve
  35.          a la première ocurence des ":_" et stockage dans pos */
  36.       char *pos = strstr(information, ": " );
  37.       if (pos != NULL) //Si la valeur du pointeur n'est pas nulle...
  38.       {
  39. #if DBG
  40.       printf("%s", pos) ;
  41. #endif
  42.          /* +2 : On saute ": " */
  43.          /* Ecriture de la chaine dans le fichier texte cree */
  44.          fprintf(f2, "%s", pos + 2);
  45.       }
  46.    }
  47.       fclose (f), f = NULL;
  48.    }
  49.    else
  50.    {
  51.       perror(nomfichier);
  52.    }
  53. }
  55. int main(void)
  56. {
  57.    printf("Bienvenue dans le programme de tri des fiches de stage\n" );
  59.    /* Liste toutes les fiches de stage dans res.txt */
  60.    system("dir *.txt /b  > res.txt" );
  62.    {
  63.       /* OUvre ce fichier */
  64.       char const fres[] = "res.txt";
  65. #define  FICHE "fiche.txt"
  67.       FILE *f = fopen(fres, "r" );
  69.       if (f != NULL)
  70.       {
  71.          FILE *fw = fopen(FICHE, "w" );
  72.          if (fw != NULL)
  73.          {
  74.             char nomfichier[256];
  75.             while (fgets( nomfichier, sizeof nomfichier, f ) != NULL)
  76.             {
  77.                clean(nomfichier, f);
  78.                chercher(nomfichier, fw);
  79.             }
  80.             fclose(fw), fw = NULL;
  81. #if 1
  82.             system ("type "FICHE);
  83. #endif
  84.          }
  85.          else
  86.          {
  87.             perror(FICHE);
  88.          }
  89.          fclose(f), f = NULL;
  90.       }
  91.       else
  92.       {
  93.          perror("res.txt" );
  94.       }
  95.    }
  97.    return 0;
  98. }


Message édité par Emmanuel Delahaye le 19-04-2006 à 11:19:43

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 19-04-2006 à 08:41:15    

Merci beaucoup les gars, j'vais étudié tout ça de plus près!

Reply

Sujets relatifs:

Leave a Replay

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