Discussion :

[ziz2005]

Bonjour,
je suis entrain de faire un petit programme en C sous linux qui présente un menu de choix qui me fait passer à un autre sous menu. mon problème c'est que j'arrive à afficher le premier menu et faire un choix mais ce dernier ne me mène pas au sous choix correspondant et le programme exit. je ne sais pas où est le probleme
Merci de m'aider.

voici une partie du code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
#include <stdio.h>
#include <types.h>
 
int main()
{
 
FILE * FicEntree;
char rep;
 
printf("\n*******************Menu Test ********************\n");
 
printf ("   1   GetParameterNames   \n");
printf ("   2   AddObject   \n");
printf ("   3   SetParameterValues    \n");
printf ("   4   DeleteObject          \n");
rep=getchar();
switch (rep)
{
       case '1': 
{
      printf ("  avec 1 param   \n");
      printf ("   avec 2 param \n");
      rep2= getchar();
             switch (rep2)
              {
                         case '1' : printf ("  sous menu   \n");
                              break;
              }
break;
}
 
}
return 0;
}

[frizou11]

De plus il te manque le case default dans ton switch pour tester s'il ne rentre pas dans les case définis.
Enfin rep2 n'est pas déclaré.

Je pense que l'erreur vient du getchar()

Et à quoi sert le

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

#include <types.h>

?

[crocodilex]

Ton code ne compile pas :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
Project   : Console application
Compiler  : GNU GCC Compiler (called directly)
Directory : D:\CodeBlocks_Project\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:2:19: types.h: No such file or directory
main.c:5: warning: function declaration isn't a prototype
main.c: In function `main':
main.c:23: error: `rep2' undeclared (first use in this function)
main.c:23: error: (Each undeclared identifier is reported only once
main.c:23: error: for each function it appears in.)
main.c:7: warning: unused variable `FicEntree'
Process terminated with status 1 (0 minutes, 0 seconds)
4 errors, 2 warnings

La solution se trouve dans la FAQ : Comment vider le buffer clavier

[ziz2005]

bonjour, merci à vous tous mais j'ai mentionné que c'est une partie du code

voici un code plus complet et qui compile mais qui n'entre pas dans le second menu.
Merci.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, int**argv)
{
 
FILE * FicEntree;
char rep,rep2;
printf("\n*******************Menu Test *************************\n");
printf ("   1   GetParameterNames   \n");
printf ("   2   AddObject   \n");
printf ("   3   SetParameterValues    \n");
printf ("   4   DeleteObject          \n");
printf ("   5   GetParameterValues    \n");
printf ("   6   SetParameterAttributes \n");
printf ("   7   GetParameterAttributes \n");
printf ("   8   Test RPC methods   \n");
printf ("   9   Test RPC reboot    \n");
printf(" \n Entrez votre choix : ");
rep=getchar();
 
switch (rep)
{
  case '1': 
            {
 
            printf("GetParameterNames with level 0 \n");
            printf("GetParameterNames with level 1 \n");
            printf("GetParameterNames with wrong level  \n");
            printf("GetParameterNames with wrong name level 1 \n");
            printf ("Entrez votre choix");
            rep2=getchar();
 
             switch(rep2)
                {
              case '1':
              printf ("second menu"); 
            } break;
 
               }
 
  case  '2':
            printf("AddObject with valid object Name\n");
            printf("AddObject with wrong object Name \n");
            printf("AddObject not an object \n");
            printf("AddObject for an instanciated object and its number of instances is maximum  \n");
            break;
default: printf ("gggggggggggggggggggg");
               break;
}

[PRomu@ld]

Désolé, mais ça ne compile pas chez moi.

test.c:5: warning: second argument of 'main' should be 'char **'
test.c: In function 'main':
test.c:50: error: expected declaration or statement at end of input
test.c:7: warning: unused variable 'FicEntree'
test.c:50: warning: control reaches end of non-void function
test.c: At top level:
test.c:4: warning: unused parameter 'argc'
test.c:4: warning: unused parameter 'argv'

[Melem]

Un exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
int main()
{
    int choix;
 
    do
    {
        choix = AfficheMenuPrincipal();
        MenuPrincipal(choix);
    }
    while (choix != QUITTER);
 
    return 0;
}

La fonction AfficheMenuPrincipal() affiche le menu principal puis attend un choix de l'utilisateur. Cette fonction ne doit retourner que lorsque le choix de l'utilisateur est bien valide.

La fonction MenuPrincipal() est sans doute un gros switch-case. Selon choix, soit on fait un truc utile, soit on affiche un sou menu. On crée autant de fonctions d'affichage de sous menus et de traitement du choix que l'on en a besoin.

[PRomu@ld]

Melem, tu n'as pas compris le problème de ziz2005 : son code ne fonctionne pas et il aimerait savoir pourquoi.

ziz2005, affiches (via un printf par ex) la valeur de rep, juste pour voir.

[frizou11]

bonjour, merci à vous tous mais j'ai mentionné que c'est une partie du code

voici un code plus complet et qui compile mais qui n'entre pas dans le second menu.
Merci.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, int**argv)
{
 
FILE * FicEntree;
char rep,rep2;
printf("\n*******************Menu Test *************************\n");
printf ("   1   GetParameterNames   \n");
printf ("   2   AddObject   \n");
printf ("   3   SetParameterValues    \n");
printf ("   4   DeleteObject          \n");
printf ("   5   GetParameterValues    \n");
printf ("   6   SetParameterAttributes \n");
printf ("   7   GetParameterAttributes \n");
printf ("   8   Test RPC methods   \n");
printf ("   9   Test RPC reboot    \n");
printf(" \n Entrez votre choix : ");
rep=getchar();
 
switch (rep)
{
  case '1': 
            {
 
            printf("GetParameterNames with level 0 \n");
            printf("GetParameterNames with level 1 \n");
            printf("GetParameterNames with wrong level  \n");
            printf("GetParameterNames with wrong name level 1 \n");
            printf ("Entrez votre choix");
            rep2=getchar();
 
             switch(rep2)
                {
              case '1':
              printf ("second menu"); 
            } break;
 
               }
 
  case  '2':
            printf("AddObject with valid object Name\n");
            printf("AddObject with wrong object Name \n");
            printf("AddObject not an object \n");
            printf("AddObject for an instanciated object and its number of instances is maximum  \n");
            break;
default: printf ("gggggggggggggggggggg");
               break;
}

Bonjour,
- ton code ne compile pas : il manque une accolade fermante (sûrement celle du main)
- ton main est déclaré comme retournant un int, or tu ne retournes rien
Ceci résolu ça compile chez moi et le premier menu s'affiche bien.
Si j'entre '1' alors le second menu s'affiche très bien mais le programme quitte. Ceci est dû au second getchar() qui doit prendre le retour-chariot de la première réponse.

[frizou11]

[EDIT: oups, désolé double post...]


[EDIT : il faut vider le buffer clavier]
ton code devient alors :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
case '1': 
        {
 
            printf("GetParameterNames with level 0 \n");
            printf("GetParameterNames with level 1 \n");
            printf("GetParameterNames with wrong level  \n");
            printf("GetParameterNames with wrong name level 1 \n");
            printf ("Entrez votre choix");
            /* vider le buffer clavier */
            while ((c = getchar()) != '\n' && c != EOF); /* mange le tampon */
            rep2=getchar();
 
            switch(rep2)
            {
                case '1':
                    printf ("second menu"); 
            } break;
        }

avec c déclaré comme int plus haut avec tes déclarations

[odsen.s]

Ceci est dû au second getchar() qui doit prendre le retour-chariot de la première réponse.

En effet, getchar() n'est pas adpaté à la saisie d'un caractère.
Il faut vider stdin après usage :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
int lire_car (void)
{
   int c = getchar ();
   if (c != '\n' && c != EOF)
   {
      int c;
      while((c = getchar ()) != '\n' && c != EOF);
   }
 
   return c;
}

[Melem]

Melem, tu n'as pas compris le problème de ziz2005 : son code ne fonctionne pas et il aimerait savoir pourquoi.

Justement : On a affaire à un programme qui va afficher des menus et des sous menus et on met tout dans le main()? En découpant le programme en modules, on pourra plus facilement détecter les erreurs ...

[ziz2005]

Bonjour,
merci pour votre aide, j'ai bien trouvé mon erreur et tout marche bien maintenent.
pour ceux qui sont intéressé par ce genre de menu voici un petit exemple que j'ai trouvé sur internet et qui est parfait bonne journée à tous !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
 
 
#include <stdio.h>
 
int main(void)
{
    int choix = 0;
    printf("Menu du programme.\n\n");
    printf("1. Faire une adition\n");
    printf("2. Faire une soustraction\n");
    printf("3. Faire une multiplication\n");
    printf("4. Faire une division\n");
    printf("5. Autre operation\n");
    /* Demande du choix */
    scanf("%d", &choix);
    switch (choix)
    {
        case 1: 
            printf("|   Addition   |\n");
            /* reste du code */
            break;
        case 2:
            printf("|   Soustraction   |\n");
            /* reste du code */
            break;
        case 3:
            printf("|   Multiplication   |\n");
            /* reste du code */
            break;
        case 4:
            printf("|   Division   |\n");
            /* reste du code */
            break;
        case 5:
            printf("Autre operation\n");
            /* reste du code */
            break;
        default:
            printf("Non gere\n");
    }
    getchar();
    return 0;
}

[odsen.s]

Salut,

pour ceux qui sont intéressé par ce genre de menu voici un petit exemple que j'ai trouvé sur internet et qui est parfait

Il n'est pas parfait car il utilise scanf ().
Je rappelle que scanf () n'est pas adaptée pour effectuer des saisies.
Si on effectue ton programme en boucle, ça serait une catastrophe.
Il vaut mieux utiliser une fonction ressemblant à celle citée plus haut, c'est tout à fait adapté pour les choix dans un menu :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
int lire_car (void)
{
   int c = getchar ();
   if (c != '\n' && c != EOF)
   {
      int c;
      while((c = getchar ()) != '\n' && c != EOF);
   }
 
   return c;
}

[Médinoc]

Il n'est pas parfait car il utilise mal scanf ().
Je rappelle que scanf () n'est pas adaptée pour les débutants.

Contrairement à gets(), scanf() n'est pas un bug, et peut être correctement utilisée.
Toutefois, il est très facile de mal l'utiliser. Les deux pires utilisations possibles de scanf() sont :

• scanf("%s", ...);
• Enseigner scanf() à des débutants.

Pour apprendre à utiliser scanf() correctement :
http://xrenault.developpez.com/tutoriels/c/scanf/

[BainE]

Bonjour,

de plus pour revenir sur la perfection,

come cité un peu plus haut, il faudrait encapsuler un "menu" dans un do - while pour boucler dessus... (en cas d'erreur de frappe notement)
et truc bete, le jour ou ton menu a 10 choix possible, ca va pas etre facile avec le getchar()...

Mais bon...

[Emmanuel Delahaye]

voici une partie du code

C'est bien le problème. Ce code est incomplet et il ne compile pas. Il ne peut donc pas être mis au point sans faire d'hypothèses.