Problème d'encodage de liste chainée

Problème d'encodage de liste chainée - C - Programmation

Marsh Posté le 06-05-2017 à 15:28:18    

Voilà mon problème : je dois créer une liste chainée pour contenir des articles (sur une facture). Je pense avoir fait les bonnes fonctions mais mon encodage rencontre des problèmes : lorsque j'encode (dans la fonction EncoderArticle) c'est bien encodé correctement (le printf me renvoie le bon string) mais quand je l'affiche juste après ma fonction le printf renvoie quelque chose comme ça : (Gx (et x change régulièrement). Si quelqu'un a une idée pour régler leproblème, je vous en serais reconnaissante.
 

Code :
  1. void EncoderArticle(article Produit)
  2. {
  3.   char nom[51];
  5.   printf ("\nVeuillez entrer le nom de l'article : " );
  6.   fflush(stdin);
  7.   scanf("%s", nom);
  8.   printf("%s", nom);
  9.   system("pause" );
  10.   Produit.Designation = (char*) malloc (sizeof(nom)+1);
  11.   strcpy(&(*Produit.Designation),nom);
  13.   printf ("\nVeuillez entrer le prix hors TVA de l'article : " );
  14.   fflush(stdin);
  15.   scanf ("%f", &(Produit.PrixHTVA));
  16.   printf ("\nVeuillez entrer la quantite desiree : " );
  17.   fflush(stdin);
  18.   scanf ("%d", &(Produit.Quantite));
  19.   printf("encodage : %s\n", Produit.Designation);
  20.   system("pause" );
  21. }
  23. void InsererElementListe(liste **ListeProduits)
  24. {
  25.   int trouve = 1;
  26.   int i = 1;
  27.   liste *parcours = NULL;
  28.   liste *inserer = NULL;
  29.   liste *precedent = NULL;
  30.   parcours = *ListeProduits;
  31.   inserer = (liste*) malloc(sizeof(liste));
  32.   EncoderArticle(inserer->Donnees);
  33.   printf("parcours %s\n", (parcours->Donnees).Designation);
  34.   system("pause" );
  35.   printf("Encoder OK\n" );
  36.   printf("inserer : %s\n", (inserer->Donnees).Designation);
  37.   system("pause" );
  39.   do
  40.   {
  41.       printf("condition boucle OK\n" );
  42.       printf("%s, %s, %d\n", ((inserer->Donnees).Designation), ((parcours->Donnees).Designation), i++);
  43.       system("pause" );
  45.     if (ComparerProduit(((inserer->Donnees).Designation), ((parcours->Donnees).Designation)) < 0)
  46.         {printf("Comparer OK" );
  47.         system("pause" );
  48.       trouve = 0;}
  49.     else
  50.     {
  51.         printf("else" );
  52.       precedent = parcours;
  53.       parcours = parcours->Suite;
  54.     }
  55.   }
  56.   while(parcours != NULL && !trouve);
  57.   inserer->Suite = parcours;
  58.   printf("inserer OK" );
  59.   system("pause" );
  60.   if (precedent == NULL)
  61.     *ListeProduits = inserer;
  62.   else
  63.     precedent->Suite = inserer;
  64. }


 
P.S.: les printf me servent pour le debug...

Reply

Marsh Posté le 06-05-2017 à 15:28:18   

Reply

Marsh Posté le 06-05-2017 à 16:17:33    

Un rapide coup d’œil me dit qu'il faut passer ton paramètre par adresse dans ta fonction EncoderArticle

Reply

Marsh Posté le 06-05-2017 à 16:30:18    

+1 pour Totoche17.
 
Pour la prochaine fois: Il est préférable de donner un code compilable, c'est à dire avec la définition/déclaration (je confonds toujours :o ) de "liste", les #include qui vont bien (même si dans ce cas présent c'est pas bien difficile de deviner ce qu'il faut) et le code pour ComparerProduit() et une main() réduit au max.
 
Surtout pour ce genre de bricolages avec des pointeurs il est très utile d'activer les warnings sur son compilateur (ce qui de toute façon devrait toujours être le cas!!), tu l'as fait? (GCC: rajouter -Wall comme paramètre en ligne de commande et -Werror si souhaité)
 
Quelque remarques:
-Tu ne vérifies pas ce que retourne malloc() (!=NULL).
-strcpy() est une fonction non sécurisée contre les buffer overflow, utiliser strncpy()!
-fflush(stdin); a un comportement indéfini selon le standard C (mais beaucoup de personnes l'utilisent et souvent ça fonctionne...).
-Il n'y a pas besoin de cast pour la valeur retournée par malloc(), sauf si par erreur tu compiles ton code avec un compilateur C++.

Reply

Sujets relatifs:

Leave a Replay

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