pb liste chainée et lecture de fichier C

pb liste chainée et lecture de fichier C - C - Programmation

Marsh Posté le 28-02-2012 à 16:53:47    

Bonjour à tous,
 
j'ai un projet à remettre pour l'école, c'est un programme qui gère le stock d'un magasin composé de structure liste chainée et lecture de fichier principalement.
 
Le soucis est qu'il ne m'affiche pas correctement ce qu'il faut, c'est peut etre un problème au niveau des fgets ou alors de l'allocation en mémoire mais je vois pas où.
 
Si quelqu'un pourrait m'éclairer de sa lumière ce serait sympa...
 
Voici le projet ( j'utilise codeblocks) sur rapidshare, jmet pas le code ici, il est en plusieur fichier (le tout est dans un fichier .rar avec dedans une image de la liste chainée et un screenshot de ce qu'il m'affiche et ce qu'il devrait m'afficher) :
https://rapidshare.com/#!download|9 [...] 4fa3175%29  
 
 
 
Merci.

Reply

Marsh Posté le 28-02-2012 à 16:53:47   

Reply

Marsh Posté le 28-02-2012 à 20:48:15    

Bon déja en modifiant data.c avec:
 
 

Code :
  1. void printItem(ITEM *item) {
  2.     printf(" - [%d] %s (%s): %.2lf \n", item->id, item->name, item->category, item->price);
  3. }
  5. void printStockItem(STOCKITEM *stockItem) {
  6.     printf(" - [%d] %d unite expirant le %s \n",stockItem->item->id, stockItem->quantity, stockItem->expirationDate);
  7. }


 
et stock.c avec:
 

Code :
  1. void printStock(STOCK *stock) {
  2.     int i;
  3.     if (stock->nbItems) {
  4. printf("%d items\n", stock->nbItems);
  5. for(i = 0; i < stock->nbItems; ++i) {
  6.     printItem(&stock->items[i]);
  7. }
  9. if(stock->size == 0) {
  10.     printf("\n [] \n" );
  11. }
  12. else {
  13.     struct STOCKNODE *current = stock->first;
  14.     printf("%d stockitems\n", stock->size);
  15.     for(i = 0; i < stock->size; ++i) {                      // voir dans data.c
  16.  printStockItem(current->item);                    // voir dans data.c
  17.  current=current->next;
  18.     }
  19. }
  20.     }
  21.     else {
  22. printf("Stock vide\n" );
  23.     }
  24. }
  26. STOCK *loadStock(char *path) {
  27.     STOCK *liste = newStock();                                            //initialisation de la liste
  28.     FILE *file;
  29.     char buff[30];                                                       // ouverture du fichier afin de prendre les éléments pour construire les structures
  30.     int i, id = 0, quantity;
  31.     double price[30];
  32.     char *category[20];
  33.     char *name[30];
  34.     char *expiDate[20];
  35.     char *c;
  37.     if (liste == NULL) {
  38.         return NULL;
  39.     }
  41.     file = fopen(path,"r" );
  42.     if(file == NULL) {
  43.         printf("\n impossible d'ouvrir le fichier %s \n\n",path);
  44.         return NULL;
  45.     }
  47.     fgets(buff,30,file);                                          // lis la première ligne du fichier stock.txt qui correspond au nombre 3 et qui est le nombre d'article
  48.     int nbItems = atoi(buff);
  49.     liste->nbItems = nbItems;
  50.     liste->items = malloc(nbItems * sizeof(ITEM));                  // déclaration du tableau de structures ITEM
  52.     for (i = 0; i < nbItems; ++i) {
  53.         category[i] = malloc(sizeof(buff));
  54.         fgets(buff, 30, file);                                   // lis la ligne suivante et la place dans la variable char category
  55.         if ((c = strrchr(buff, '\n')) != NULL) {
  56.             *c = '\0';          // purge du '\n' terminant chaque ligne d'un fichier texte
  57.         }
  58.         strcpy(category[i], buff);
  60.         name[i] = malloc(sizeof(buff));
  61.         fgets(buff, 30, file);
  62.         if ((c = strrchr(buff, '\n')) != NULL) {
  63.             *c = '\0';   // purge du '\n' terminant chaque ligne d'un fichier texte
  64.         }
  65.         *name[i] = '\0';
  66.         strcpy(name[i], buff);
  68.         fgets(buff, 30, file);
  69.         price[i] = atof(buff);
  71.         liste->items[i] = *buildItem(id, name[i], category[i], price[i]);     // place dans la case i du tableau de structures, la structure construite voir data.c
  72.         id++;
  73.     }
  75.        fgets(buff,30,file);                                            // ligne vide
  77.        for (i = 0; i < nbItems; ++i) {
  78.         fgets(buff, 30, file);
  79.         id = atoi(buff);
  81.         fgets(buff, 30, file);
  82.         quantity = atoi(buff);
  84.         expiDate[i] = malloc(sizeof(buff));
  85.         fgets(buff, 30, file);
  86.         if ((c = strrchr(buff, '\n')) != NULL) {
  87.             *c='\0';   // purge du '\n' terminant chaque ligne d'un fichier texte
  88.         }
  89.         strcpy(expiDate[i], buff);
  91.         STOCKITEM *receive_stock = malloc(sizeof(STOCKITEM));
  92.         receive_stock = buildStockItem(&liste->items[i], quantity, expiDate[i]);             //créaction d'une structure STOCKITEM au fur et a mesure qu'on parcourt la liste
  93.         // liste->items[id] = *receive_stock->item;
  94.         addToStock(liste, receive_stock);                                               // on ajoute ce qu'on a construit
  95. }
  96.    
  97.     fclose(file);
  99.     return liste;
  101. }


 
et en virant tous les blancs inutiles de stock.txt
 

3
DVD
Les Simpson - Saison 14
12.99
Livre
Langage C
7.99
Charcuterie
Jambon Cuit - 4 tranches
3.99
 
0
10
-
2
10
25/12/2011
1
1
-
2
5
27/12/2011


On obtient un truc qui marche presque:

3 items
 - [0] Les Simpson - Saison 14 (DVD): 12.99
 - [1] Langage C (Livre): 7.99
 - [2] Jambon Cuit - 4 tranches (Charcuterie): 3.99
3 stockitems
 - [0] 10 unite expirant le -
 - [1] 10 unite expirant le 25/12/2011
 - [2] 1 unite expirant le -


 
Mais la ou on tombe sur ton erreur de conception, c'est ici:

Citation :


 
       fgets(buff,30,file);                                            // ligne vide
 
      for (i = 0; i < nbItems; ++i) {
        fgets(buff, 30, file);
        id = atoi(buff);
 
        fgets(buff, 30, file);
        quantity = atoi(buff);
 

Il y a aucune raison que le nb de stockitems soit égal au nb d'items, dans ton fichier d'ailleurs, 3 items et 4 stockitems, et ton code fait qu'on n'en lit que 3.
 
Commences par penser à un format plus utilisable (s'il ne t'es pas imposé) comme fichier de sauvegarde des stocks pour t'en sortir
- soit en notant le nb de stockitems
- soit en ayant une marque de suite d'items et une de suite de stockitems en tête de ce type d'enregistrement (avec une fin par exemple quand on rencontre une nouvelle marque)
ou autre (lecture de tous les stockitems jusqu'a la fin du fichier, etc), en faisant gaffe de choisir une solution qui puisse évoluer si ton projet le fait (si tu dois pouvoir ajouter des items et des stockitems à l'avenir par exemple)
 
A+,


---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
Reply

Sujets relatifs:

Leave a Replay

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