Discussion :

[mandok]

Bonjour tout le monde,

Je tente depuis deux jours de réaliser une énumération et ensuite l'utilisé dans un menu. Cependant, malgré toutes mes bidouilles je n'obtient que des erreurs de compilations.

Voilà le code que j'utilise pour réaliser mon menu et cela de manière simpliste sans utilisé l'énumération que je définis au début. Ce code compile.

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
typedef enum Figure Figure;
	enum Figure
		{
			POINT, CERCLE, TRIANGLE
		};
 
 
int saisirFigure()
	{
		int choixFigure;
 
		while (choixFigure < 0 || choixFigure > 2)
		{
			printf("Figures disponibles :\n");
			printf("1 : Point\n");
			printf("2 : Cercle\n");
			printf("3 : Triangle\n");
			printf("Votre choix ?");
			scanf("%d", &choixFigure);
		}
 
		return choixFigure;
	}
 
 
 
 
 
 		saisirFigure();
 
 
 				switch (saisirFigure())
		{
			case 1:
				printf("Vous avez choisi la figure Point\n");
					break;
			case 2:
				printf("Vous avez choisi la figure Cercle\n");
					break;
			case 3:
				printf("Vous avez choisi la figure Triangle \n");
					break;
			default: 
            	printf ("Figure inconnue\n"); 
            		break; 
		}
 
  		return 0;
  	}

Je ne vois vraiment pas comment faire, tous les exemples dont j'ai essayé de m'inspirer n'ont pas vraiment aider.

Est ce que quelqu'un pourrait m'expliquer la marche à suivre?

Merci d'avance.

[LinuxUser]

Dans ta fonction int saisirFigure() tu utilises dans le while la variable choixFigure sans lui avoir attribut de valeur (par saisi de l'utilisateur).
De plus dans ton choix de figure, on ne peux jamais choisir de triangle d'après ta condition:

Code :

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

choixFigure > 2

Par contre on peut saisir un 0 qui correspond à aucune figure:

Code :

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

choixFigure < 0

[mandok]

Dans ta fonction int saisirFigure() tu utilises dans le while la variable choixFigure sans lui avoir attribut de valeur (par saisi de l'utilisateur).

Pourtant il y a un scanf("%d", &choixFigure) à la fin du while pour y stocker le choix de l'utilisateur.

Je ne comprend pas

[LinuxUser]

Oui mais au moment du while, choixFigure n'a pas de valuer.
Cela aurait pu marcher avec un do{} while(); du genre:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
  int choixFigure;
 
 do
{
  printf("Figures disponibles :\n");
  printf("1 : Point\n");
  printf("2 : Cercle\n");
  printf("3 : Triangle\n");
  printf("Votre choix ?");
  scanf("%d", &choixFigure);
}while (choixFigure <= 0 || choixFigure > 3);

[mandok]

Ok, je ce que tu veux dire.

C'est juste que déjà à la base je ne vois pas comment intégrer l'énumération à mon menu et surtout quelle sera la différence fondamentale avec le fait de laisser mon menu comme je l'ai fait avant.

Est ce que tu pourrais, si tu as le temps, me l'expliquer avec des mots? Je n'arrive pas à assimiler ce concept...

Merci d'avance.

[edgarjacobs]

Hello,

Pour ton problème, et pour ne rien changer à ton while, le plus simple est de faire

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
 
 
int saisirFigure()
{
	int choixFigure.
 
        choixFigure=POINT-1;
	while (choixFigure < POINT || choixFigure > TRIANGLE) {
		printf("Figures disponibles :\n");
		printf("1 : Point\n");
		printf("2 : Cercle\n");
		printf("3 : Triangle\n");
		printf("Votre choix ?");
		scanf("%d", &choixFigure);
		}
	return choixFigure;
}

Mais voilà, ça va poser un petit problème, puisque POINT vaut 0, et TRIANGLE vaut 2.

Donc, ou tu changes to input (0: point, 1: cercle, 2: triangle), ou tu changes ton enum:

Code :

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

1
2
3
4
enum Figure
{
	POINT=1, CERCLE, TRIANGLE
};

[mandok]

Merci, ça marche plutôt bien en utilisant cette technique.

J'ai un seul soucis c'est que le printf du default n'apparait pas lorsque je rentre autre chose que les options disponible, le programme renvoie automatiquement le menu sans afficher le message "figure indisponible" entre temps.

Je souhaiterais aussi savoir, s'il vous plaît, comment empêcher mon programme d'aller dans une boucle infinie lorsque je rentre une valeur autre que integer pour le choix de la figure.

Merci d'avance.

[edgarjacobs]

Une soution est la suivante (mais je n'ai pas l'habitude des entrées console)

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
int saisirFigure()
{
	int choixFigure;
	char tmp[5];
 
        choixFigure=POINT-1;
	while (choixFigure < POINT || choixFigure > TRIANGLE) {
		printf("Figures disponibles :\n");
		printf("1 : Point\n");
		printf("2 : Cercle\n");
		printf("3 : Triangle\n");
		printf("Votre choix ?");
		fgets(tmp,sizeof(tmp),stdin);
		choixFigure=atoi(tmp);
		}
	return choixFigure;
}

[Roland Chastain]

Bonjour !

Il y a encore une dernière erreur, c'est que la fonction saisirFigure est appelée deux fois.

Code :

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

1
2
3
4
5
6
saisirFigure();
 
switch (saisirFigure()){
  case 1:
  printf("Vous avez choisi la figure Point\n");
  break;

[edgarjacobs]

Il y a encore une dernière erreur, c'est que la fonction saisirFigure est appelée deux fois.

Oui, j'avais vu, je comptais y venir par la suite

[moins1]

Je souhaiterais aussi savoir, s'il vous plaît, comment empêcher mon programme d'aller dans une boucle infinie lorsque je rentre une valeur autre que integer pour le choix de la figure.
Merci d'avance.

Tu vas devoir vider le buffer clavier. Ça pourrait être une fonction dédié, disont: ViderBuffer(). Et pour être sûr que l'entrée est un entier, tu peux vérifier le retour de ton scanf(). La fonction retourne le nombre d'argument correctement lu.

Genre:

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
#include <stdio.h>
 
void ViderBuffer(void);
int LireEntier(void);
 
int main(void)
{
    int e = LireEntier();
    printf("Entier lu: %d", e);
    return 0;
}
 
void ViderBuffer(void)
{
    int c;
    while((c = getchar()) != '\n' && c != EOF)
        ;
}
 
int LireEntier(void)
{
    int r, e;
    do
    {
        printf("Entrer un entier: ");
        r = scanf("%d", &e);
        ViderBuffer();
    }while(!r); //En ajoutant tes choix d'entier à la condition
    return e;
}

[Bktero]

Attention quand même à n'appeler ViderBuffer() que si le buffer stdin n'est pas vide !

Le problème vient de scanf() : cette fonction n'est pas fait pour des entrées utilisateurs. Mieux vaut utiliser fgets() et en extraire le contenu. Si on y arrive pas, on affiche un message d'erreur. On n'a besoin de vider le buffer clavier que si le buffer de fgets() était trop petit.

[moins1]

Et tu vérifies comment pour savoir si le buffer est vide? Parce que bon même si le nombre d'argument lu est bon il peut rester des trucs dans le buffer. Par exemple quand scanf() attend un entier et que l'on entre disont: 12abcd la fonction retourne 1.

[Bktero]

A ma connaissance, tu ne peux pas le savoir sans lire dedans justement. Tu montres bien là un problème de scanf() : elle n'est pas adaptée aux saisies utilisateur. Ou alors, il faut faire des choses bien plus compliquées que juste scanf("%d", &nb) : http://xrenault.developpez.com/tutoriels/c/scanf/

[Médinoc]

Aucun problème, le buffer d'entrée ne sera jamais vide après un simple scanf("%d") : Il y aura toujours au moins le retour à la ligne qui sera toujours présent dans le buffer.