aide petit prog c

aide petit prog c - C - Programmation

Marsh Posté le 23-06-2006 à 11:02:38    

Bonjour,
 
Débutant en C, je souhaiterai savoir comment changer ce programme initiale (liste chainée) qui demande à l'utilisateur de choisir les options selon l'affichage :
1 : ajout dans la liste
2 : suppression dans liste
3 : trie de la liste
4 : affiche la liste
0 : quit programme
 
Je voudrai que celui-ci lance le programme suivi de l'option puis les paramêtres (selon option choisi).
 
Par explemple, ./monprog -a elementAajouter (qui correspond à l'option 1)
  ./monprog -b elementAsupp (qui correspond à l'option 2)
  ./monprog -c (option 3)
  ./monprog -d (option 4)
  ./monprog -q (option 0)
 
Avez vous des conseils, des exemples, des idées ?
 
Merci
 
Voici le code :
 

Code :
  1. #include<stdio.h>
  2. #include<malloc.h>
  4. #define Faux 0
  5. #define Vrai 1
  7. typedef struct Element{
  8.                int val;
  9.                struct Element *next;
  10.         }Element;
  13. int estVide(Element* L){
  14.          if(L == NULL){
  15.                  return Vrai;
  16.               }
  17.          else{
  18.                  return Faux;
  19.               }           
  20.      }         
  21.  
  22. void ajouter(Element **L, int v){
  23.        Element *tmp = *L;
  24.        Element *nouveau = (Element*)malloc(sizeof(Element));
  25.        if(nouveau == NULL){
  26.                   exit(1);
  27.              }
  28.        nouveau->val = v;
  29.        nouveau->next = NULL;
  30.        if(estVide(*L)){
  31.             *L = nouveau;         
  32.        }
  33.        else{
  34.             tmp = *L;
  35.             while(tmp->next != NULL){
  36.                  tmp = tmp->next; 
  37.             }
  38.             tmp->next = nouveau;
  39.        }
  40.      }
  42. void Trier(Element **L){
  43.        Element *tmp1, *tmp2;
  44.        Element *Min;
  45.        int t;
  46.        tmp1 = *L;
  47.        while(tmp1 != NULL){
  48.              Min = tmp1;
  49.              tmp2 = tmp1;
  50.              while(tmp2 != NULL){
  51.                  if(Min->val > tmp2->val) Min = tmp2;
  52.                  tmp2 = tmp2->next;     
  53.              }
  54.              t = tmp1->val;
  55.              tmp1->val = Min->val;
  56.              Min->val = t;
  57.            
  58.              tmp1=tmp1->next;   
  59.        }
  60.   }   
  61. void Supprimer(Element **L, int v){
  62.       Element *tmp;
  63.       int d;
  64.       if(!estVide(*L)){
  65.           tmp = *L;
  66.           d = 0;
  67.           if(tmp->val == v){
  68.              *L = (*L)->next;       
  69.           }
  70.           else{
  71.              while( d == 0 && tmp->next != NULL){
  72.                    if(tmp->next->val == v) d = 1;
  73.                    else  tmp = tmp->next;               
  74.              }
  75.              if(d == 1)  tmp->next = tmp->next->next;
  76.              else printf("!cette valeur n'est pas trouvable!\n" );     
  77.           }                 
  78.       }
  79.       else{
  80.            printf("!La liste est vide. Impossible de faire la suppression!\n" );
  81.       }
  82. } 
  84. int Longueur(Element *L){
  85.      int l=0;
  86.      Element *tmp = L;
  87.      while(tmp != NULL){
  88.          l++;
  89.          tmp = tmp->next;   
  90.      }
  91.      return l;
  92. }
  93. void afficher(Element *L){
  94.          Element *tmp;
  95.          if(estVide(L)){
  96.              printf("!la liste est vide!\n" );         
  97.          }
  98.          else{
  99.            tmp = L;
  100.            while(tmp != NULL){
  101.                 printf("%d ",tmp->val);
  102.                 tmp = tmp->next;   
  103.            }
  104.            printf("   - longueur : %d\n",Longueur(L));   
  105.          }
  106.       }
  107.    
  108.          
  109. int main(){
  111.   Element *L = NULL;
  112.   int choix;
  113.   char *fichier;
  114.   int v;
  116.   do{
  117.       printf("1-Ajouter Element.\n" );
  118.       printf("2-Supprimer Element.\n" );
  119.       printf("3-Trier la Liste.\n" );
  120.       printf("4-Afficher.\n" );
  121.       printf("0-Quitter.\n" );
  122.      
  123.       printf("   -->choix(0-4):" );scanf("%d",&choix);
  124.      
  125.       switch(choix){
  126.          case   1:printf(" - Entrer une valeur : " ); scanf("%d",&v);
  127.                   ajouter(&L, v);
  128.                 break;
  129.          case   2:printf(" - Entrer une valeur : " ); scanf("%d",&v);
  130.                   Supprimer(&L,v);
  131.                 break;
  132.          case   3:Trier(&L);
  133.                 break;
  134.          case   4:afficher(L);
  135.                 break;
  136.       }
  138.   }while(choix != 0);
  140. return 0;
  141. }

Message cité 1 fois
Message édité par mayapour le 23-06-2006 à 13:11:54
Reply

Marsh Posté le 23-06-2006 à 11:02:38   

Reply

Marsh Posté le 23-06-2006 à 11:04:04    

lire le k&r, rechercher sur google, utiliser les argc et argv.

Reply

Marsh Posté le 23-06-2006 à 11:58:29    

ok, j'ai fait un début d'ajout des options en paramètres mais ça ne fontionne pas (j'ai une erreur de déclaration)
 

Code :
  1. #include<stdio.h>
  2. #include<malloc.h>
  4. #define Faux 0
  5. #define Vrai 1
  7. typedef struct Element{
  8.                int val;
  9.                struct Element *next;
  10.         }Element;
  12. int estVide(Element* L){
  13.          if(L == NULL){
  14.                  return Vrai;
  15.               }
  16.          else{
  17.                  return Faux;
  18.               }           
  19.      }
  20.    
  21. int woptions(int ac,char ***av)
  22. {
  23. int option2return;
  24. option2return=getopt(ac, *av,"abcd" );
  25. if(getopt(ac,*av,"abcd" ) != -1)
  26.        {
  27.               printf("listeChaine: usage -[abcd]\n" );
  28.        exit(EXIT_FAILURE);
  29.         }
  30. else
  31.  return option2return;
  32. }
  33.    
  34. void ajouter(Element **L, int v){
  35.        Element *tmp = *L;
  36.        Element *nouveau = (Element*)malloc(sizeof(Element));
  37.        if(nouveau == NULL){
  38.                   exit(1);
  39.              }
  40.        nouveau->val = v;
  41.        nouveau->next = NULL;
  42.        if(estVide(*L)){
  43.             *L = nouveau;         
  44.        }
  45.        else{
  46.             tmp = *L;
  47.             while(tmp->next != NULL){
  48.                  tmp = tmp->next; 
  49.             }
  50.             tmp->next = nouveau;
  51.        }
  52.      }
  54. void Trier(Element **L){
  55.        Element *tmp1, *tmp2;
  56.        Element *Min;
  57.        int t;
  58.        tmp1 = *L;
  59.        while(tmp1 != NULL){
  60.              Min = tmp1;
  61.              tmp2 = tmp1;
  62.              while(tmp2 != NULL){
  63.                  if(Min->val > tmp2->val) Min = tmp2;
  64.                  tmp2 = tmp2->next;     
  65.              }
  66.              t = tmp1->val;
  67.              tmp1->val = Min->val;
  68.              Min->val = t;
  69.            
  70.              tmp1=tmp1->next;   
  71.        }
  72.   }   
  73. void Supprimer(Element **L, int v){
  74.       Element *tmp;
  75.       int d;
  76.       if(!estVide(*L)){
  77.           tmp = *L;
  78.           d = 0;
  79.           if(tmp->val == v){
  80.              *L = (*L)->next;       
  81.           }
  82.           else{
  83.              while( d == 0 && tmp->next != NULL){
  84.                    if(tmp->next->val == v) d = 1;
  85.                    else  tmp = tmp->next;               
  86.              }
  87.              if(d == 1)  tmp->next = tmp->next->next;
  88.              else printf("!cette valeur n'est pas trouvable!\n" );     
  89.           }                 
  90.       }
  91.       else{
  92.            printf("!La liste est vide. Impossible de faire la suppression!\n" );
  93.       }
  94. } 
  96. int Longueur(Element *L){
  97.      int l=0;
  98.      Element *tmp = L;
  99.      while(tmp != NULL){
  100.          l++;
  101.          tmp = tmp->next;   
  102.      }
  103.      return l;
  104. }
  106. int nbrconf(char ac)
  107. {
  108. if(ac!=4)
  109. {
  110.  printf("listeChaine: usage -[abcd]\n" );
  111.  exit(EXIT_FAILURE);
  112. }
  113. else
  114.   return 1;
  115. }
  117. void afficher(Element *L){
  118.          Element *tmp;
  119.          if(estVide(L)){
  120.              printf("!la liste est vide!\n" );         
  121.          }
  122.          else{
  123.            tmp = L;
  124.            while(tmp != NULL){
  125.                 printf("%d ",tmp->val);
  126.                 tmp = tmp->next;   
  127.            }
  128.            printf("   - longueur : %d\n",Longueur(L));   
  129.          }
  130.       }
  132. void priseoption(char choixoption, char *v)
  133. {
  134.   do{
  135.   switch (choixoption)
  136.          {
  137.              case 'a':scanf("%d",&v);
  138.                     ajouter(&L, v);
  139.                  break;
  140.              case 'b':scanf("%d",&v);
  141.                     Supprimer(&L,v);
  142.           break;
  143.          case 'c':
  144.                 Trier(&L);
  145.           break;
  146.           case 'd':
  147.                     afficher(L);
  148.           break;
  149.          }
  150.      }while(choixoption != 0);
  151.    return 0;
  152. }
  153.          
  154. int main(char ac, char **av)
  155. {
  156.   Element *L = NULL;
  157.   char choixoption;
  158.   char *v;
  159. }


Message édité par mayapour le 23-06-2006 à 13:12:55
Reply

Marsh Posté le 23-06-2006 à 11:59:35    

- met des balises [ code ], ce sera plus lisible.
- copie/colle le msg d'erreur stp.

Reply

Marsh Posté le 23-06-2006 à 12:05:00    

Erreurs (avec compilateur gcc):
 
In function `priseoption':
138: error: `L' undeclared (first use in this function)
138: error: (Each undeclared identifier is reported only once
138: error: for each function it appears in.)
138: warning: passing arg 2 of `ajouter' makes integer from pointer without a cast
141: warning: passing arg 2 of `Supprimer' makes integer from pointer without a cast
151: warning: `return' with a value, in function returning void
 
Merci

Message cité 1 fois
Reply

Marsh Posté le 23-06-2006 à 12:09:13    

mayapour a écrit :

138: error: `L' undeclared (first use in this function)


 
T'as des problèmes de vue ?
Mets des balises [ code ] stp

Reply

Marsh Posté le 23-06-2006 à 13:11:53    

mayapour a écrit :

Bonjour,
...
0 : quit programme
Par exemple, ...
  ./monprog -q (option 0)


C'est quoi l'interet?  :pt1cable:  
 

Reply

Marsh Posté le 23-06-2006 à 13:16:17    

J'ai ajouté les [CODE] (un peu tard dsl)
l'intéret du -q c'est de passer au programme des fichiers tests a stocker dans des logs et de pouvoir y spécifier -q tout comme l'utilisateur qui taperait 0 pour quitter le programme.

Reply

Marsh Posté le 23-06-2006 à 13:29:31    

ok, à froce de mofifs :
Je ne sais pas quel pointeur utiliser pour récuper la bonne info pour la prise des options :
 

Code :
  1. void priseoption(char choixoption, char *L, char *v)
  2. {
  3.   do{
  4.    switch (choixoption)
  5.          {
  6.              case 'a':scanf("%d",&v);
  7.                     ajouter(&L, v);
  8.                  break;
  9.              case 'b':scanf("%d",&v);
  10.                     Supprimer(&L,v);
  11.            break;
  12.           case 'c':
  13.                  Trier(&L);
  14.            break;
  15.            case 'd':
  16.                     afficher(L);
  17.            break;
  18.          }
  19.      }while(choixoption != 0);
  20.    return 0;
  21. }
  22.          
  23. int main(int ac, char **av)
  24. {
  25.   Element *L = NULL;
  26.   char choixoption;
  27.   char *v;
  28. }


 
du coup j'ai des types de pointeurs imcompatibles ?? (le problème doit venir de L)
non ?

Reply

Marsh Posté le 23-06-2006 à 13:52:02    

Je crois que tu n'as malheureusement pas compris grand chose.
 
Dans ton main, où est l'appel à ta fonction priseoption ?
Nulle part. Résultat ton programme ne fait rien du tout, et c'est normal.
 
D'autre part, tu n'as pas de cas default dans ton case, c'est toujours bien d'en avoir un.?
 
Encore une fois, renseigne les messages d'erreurs ou soit plus clair dans la description de ton probleme.

Reply

Marsh Posté le 23-06-2006 à 13:52:02   

Reply

Marsh Posté le 23-06-2006 à 14:11:27    

Exact ... MERCI
J'ai modifier mon main comme ceci : c'est mieux ?
 

Code :
  1. int main(int ac, char **av)
  2. {
  3.   Element *L = NULL;
  4.   char choixoption;
  5.   char *v;
  6.   nbrconf(ac);
  7.   choixoption=woptions(ac,&av);
  8.   priseoption(choixoption,av[2]);
  9. }


 
Cependant j'ai 161: error: too few arguments to function `priseoption' dans le main
Il attend quel argument ?

Reply

Marsh Posté le 23-06-2006 à 14:16:09    

t'as déjà programmé avant ? [:pingouino]
ou tu fais ton tp d'année à l'arrache avant de le rendre ?
 
T'as écrit une fonction à 3 paramètres et t'en renseigne que 2, y'aurait pas comme un oubli ?  [:pingouino]

Reply

Marsh Posté le 23-06-2006 à 16:12:46    

En effet, je suis novice en C ... (c'est pourquoi je recherche de l'aide de personnes qui savent programmer)
 
j'ai ajouté un paramètre à "priseoption".

Code :
  1. int main(int ac, char **av)
  2. {
  3.   Element *L = NULL;
  4.   char choixoption;
  5.   char *v;
  6.   nbrconf(ac);
  7.   choixoption=woptions(ac,&av);
  8.   priseoption(choixoption,av[2],v);
  9. }


 
Mais actuellement, tous mes appelles de fonction dans priseoption génèrent l'erreur : makes integer from pointer without a cast pour ajouter
et from incompatible pointer type pour Supprimer, Tier, afficher.
 
Je ne trouve pas bcp d'info sur le net sur ce type d'erreur.
Merci de m'éclairer...

Message cité 1 fois
Message édité par mayapour le 23-06-2006 à 16:13:18
Reply

Marsh Posté le 23-06-2006 à 16:15:16    

Reprend de 0, en ayant quelques connaissances, sinon tu vas jamais t'en sortir.
 
Trouve un livre , lis les bibliolinks, jette un oeil là ( http://mapage.noos.fr/emdel/ ), commence par des tutoriaux, parce que là tu n'as pas les bases.

Reply

Marsh Posté le 23-06-2006 à 16:22:08    

ok, merci pour le lien

Reply

Marsh Posté le 23-06-2006 à 16:39:58    

Code :
  1. int main(int ac, char **av)
  2. { 
  3.   Element *L = NULL; 
  4.   char choixoption; 
  5.   char *v; 
  6.   nbrconf(ac); 
  7.   choixoption=woptions(ac,&av); 
  8.   priseoption(choixoption,av[2],v);
  9. }


 
un ptit

Code :
  1. return 0;

à la fin de ton main ne serait pas superflu


Message édité par Tamahome le 23-06-2006 à 16:40:46
Reply

Marsh Posté le 23-06-2006 à 16:41:37    

Code :
  1. void priseoption(char choixoption, char *v)
  2. {
  3. ...
  4. return 0;
  5. }


 
un void qui renvoit un int, c'est conceptuel [:roane]

Reply

Marsh Posté le 23-06-2006 à 16:56:59    

Oui, mauvais placement du return (MERCI)
 
c'est mieux comme cela :
 

Code :
  1. void priseoption(char choixoption, char *L, char *v)
  2. {
  3.   do{
  4.   switch (choixoption)
  5.          {
  6.              case 'a':scanf("%d",&v);
  7.                     ajouter(&L, v);
  8.                  break;
  9.              case 'b':scanf("%d",&v);
  10.                     Supprimer(&L,v);
  11.           break;
  12.          case 'c':
  13.                 Trier(&L);
  14.           break;
  15.           case 'd':
  16.                     afficher(L);
  17.           break;
  18.          }
  19.      }while(choixoption != 0);
  20. }
  21.          
  22. int main(int ac, char **av)
  23. {
  24.   Element *L = NULL;
  25.   char choixoption;
  26.   char *v;
  27.   nbrconf(ac);
  28.   choixoption=woptions(ac,&av);
  29.   priseoption(choixoption,av[2],v);
  30.   return 0;
  31. }


 
Il me reste néanmoins toujours les erreurs :
In function `priseoption':
138: warning: passing arg 1 of `ajouter' from incompatible pointer type
138: warning: passing arg 2 of `ajouter' makes integer from pointer without a cast
141: warning: passing arg 1 of `Supprimer' from incompatible pointer type
141: warning: passing arg 2 of `Supprimer' makes integer from pointer without a cast
144: warning: passing arg 1 of `Trier' from incompatible pointer type
147: warning: passing arg 1 of `afficher' from incompatible pointer type


Message édité par mayapour le 23-06-2006 à 16:59:21
Reply

Marsh Posté le 23-06-2006 à 23:19:20    

utiliser scanf mondieu mondieu, as tu lu le lien qu'on t'a filé plus haut ? fgets il pue ?
 

Code :
  1. Element *nouveau = (Element*)malloc(sizeof(Element));


 
on ne cast pas un malloc.


Message édité par Tamahome le 26-06-2006 à 11:43:08

---------------
Hobby eien ／人◕ ‿‿ ◕人＼
Reply

Marsh Posté le 24-06-2006 à 08:55:07    

On ne peut pas résoudre ses problèmes erreur par erreur, il y a vraiment un souci de fond là [:spamafote], il doit vraiment lire ses cours cette fois ;)


---------------
Töp of the plöp
Reply

Marsh Posté le 24-06-2006 à 20:00:31    

mayapour a écrit :

En effet, je suis novice en C ...


http://mapage.noos.fr/emdel/images/c_warn.png


Message édité par Emmanuel Delahaye le 24-06-2006 à 20:01:00

---------------
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 26-06-2006 à 11:42:43    

voila, et encore "maitriser" est un bien grand mot. J'aurais plutot dit "bricoler".

Reply

Marsh Posté le 26-06-2006 à 12:11:58    

quand meme pas, le langage seul est assez simple, mais bien programmer en c requiert beaucoup de methodologie

Reply

Marsh Posté le    

Reply

Sujets relatifs:

Leave a Replay

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