Problème de boucle

**Neocid** · 05/05/2008, 10h15

Bonjour, j'ai écrit une fonction qui me permet de choisir une option. J'aimerai blinder le programme en permettant à l'utilisateur de faire une erreur de choix et de recommencer. Seulement la boucle que j'ai écrite une fois lancée (utilisateur entre un caractère invalide != (1 || 2 || 3) :p) elle se relance indéfiniement, l'instruction de mon scanf n'est plus prit en compte.

Une idée de mon erreur ? merci d'avance !
ps: Code édité après correction. la boucle fonctionne mais pas la vérification de fonction

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
 
void introduction(char bla[5])
{
	int i;
	do
	{
		i=0;
		printf("/|===============================================|\\\n");
		printf("||    Bonjour et bienvenue dans notre programme  ||\n");
		printf("||	        				 ||\n");
		printf("||    Veuillez choisir l'option de votre choix	 ||\n");
		printf("||          [1]- Lancer une recherche            ||\n");
		printf("||	    [2]- Afficher les crdits		 ||\n");
		printf("||	    [3]- Quitter			 ||\n");
		printf("\\|===============================================|/\n");
		scanf("%d",&i);
		getchar();
		if (isdigit(i))
		{
 
			if(i==1)
			{
				saisie_repertoire(bla);
				lecture_repertoire(bla,NULL);
			}
			if(i==2)
			{
				credit();
			}
			if(i==3)
			{
				break;
			}
			else
			{
				printf("L'option choisie n'est pas valide, veuillez recommencer");
			}
		}		
		else
		{
			printf("L'option choisie n'est pas valide, veuillez recommencer");
		}
	}while(!((i==1)||(i==2)||(i==3)));
}

**emprex** · 05/05/2008, 10h28

Après ton scanf, il me semble qu'il faut manger le retour chariot.
tu peux ecrire juste apres ton scanf
getchar();

**Neocid** · 05/05/2008, 10h45

Super merci

ça marche.

**mailaka** · 05/05/2008, 10h49

D'apres un tuto, mieux vaut ne pas melanger scanf et getchar a priori. Et privilegier fgets pour avoir des entrees fiable si on ne veut pas rentrer dans les arcanes du fonctionnement de scanf.

**emprex** · 05/05/2008, 10h50

Lorsqu'on saisit un caractere ou une donnée avec un scanf ou un getchar on doit appuyer sur enter qui se retrouve alors dans le flot en entrée du programme, il sera lu par le scanf ou le getchar ce qui provoque un bug.
On l'élimine donc par un getchar();.

**snake264** · 05/05/2008, 13h09

Je pense que ceci résoudra ton problème si j'ai bien compris ce que tu cherche à 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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
void introduction(char bla[5])
{
	int i;
	do
	{
		i=0;
		printf("/|===============================================|\\\n");
		printf("||    Bonjour et bienvenue dans notre programme  ||\n");
		printf("||	        				 ||\n");
		printf("||    Veuillez choisir l'option de votre choix	 ||\n");
		printf("||          [1]- Lancer une recherche            ||\n");
		printf("||	    [2]- Afficher les crdits		 ||\n");
		printf("||	    [3]- Quitter			 ||\n");
		printf("\\|===============================================|/\n");
		scanf("%d",&i);
			if (i == 1)
			{
				saisie_repertoire(bla);
				lecture_repertoire(bla,NULL);
			}
			else if (i == 2)
			{
				credit();
			}
			else if (i == 3)
			{
				break;
			}
			else
			{
				printf("L'option choisie n'est pas valide, veuillez recommencer\n");
			}
	}while(!((i==1)||(i==2)||(i==3)));
}

**Neocid** · 05/05/2008, 13h43

Merci pour cette aide.

j'ai finalement opté pour cette solution, c'est ptete pas très "jolie" mais "c'est moi qui l'ai fait" :p

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
 
int introduction(char bla[5])
{
	int i;
	do
	{
		i=0;
		printf("/|===============================================|\\\n");
		printf("||    Bonjour et bienvenue dans notre programme  ||\n");
		printf("||	        				 ||\n");
		printf("||    Veuillez choisir l'option de votre choix	 ||\n");
		printf("||          [1]- Lancer une recherche            ||\n");
		printf("||	    [2]- Afficher les crdits		 ||\n");
		printf("||	    [3]- Quitter			 ||\n");
		printf("\\|===============================================|/\n");
		scanf("%d",&i);
		getchar();
 
	}while(! ((i==1)||(i==2)||(i==3)) || (isdigit(i)));
 
		if(i==1)
		{
			saisie_repertoire(bla);
			lecture_repertoire(bla,NULL);
		}
		else if(i==2)
		{
			credit();
		}
		else if(i==3)
		{
			return 0;
		}
 
 
}

**snake264** · 05/05/2008, 13h54

Sa marche aussi et c'est toujours mieux quand on arrive à trouver soit même

bravo

**dapounet** · 05/05/2008, 13h58

Pour éviter les appels inutiles à printf() :

Code :

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

1
2
3
4
5
6
7
8
printf("/|===============================================|\\\n"
	"||    Bonjour et bienvenue dans notre programme  ||\n"
	"||	        				 ||\n"
	"||    Veuillez choisir l'option de votre choix	 ||\n"
	"||          [1]- Lancer une recherche            ||\n"
	"||	    [2]- Afficher les crdits		 ||\n"
	"||	    [3]- Quitter			 ||\n"
	"\\|===============================================|/\n");

gl · 05/05/2008, 14h42

Envoyé par snake264

Je pense que ceci résoudra ton problème si j'ai bien compris ce que tu cherche à 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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
void introduction(char bla[5])
{
	int i;
	do
	{
		i=0;
		printf("/|===============================================|\\\n");
		printf("||    Bonjour et bienvenue dans notre programme  ||\n");
		printf("||	        				 ||\n");
		printf("||    Veuillez choisir l'option de votre choix	 ||\n");
		printf("||          [1]- Lancer une recherche            ||\n");
		printf("||	    [2]- Afficher les crdits		 ||\n");
		printf("||	    [3]- Quitter			 ||\n");
		printf("\\|===============================================|/\n");
		scanf("%d",&i);
			if (i == 1)
			{
				saisie_repertoire(bla);
				lecture_repertoire(bla,NULL);
			}
			else if (i == 2)
			{
				credit();
			}
			else if (i == 3)
			{
				break;
			}
			else
			{
				printf("L'option choisie n'est pas valide, veuillez recommencer\n");
			}
	}while(!((i==1)||(i==2)||(i==3)));
}

Ça ne résout le problème qu'en partie, la fonction introduction() se comportera à priori comme attendue mais s'il y a une autre saisie utilisateur ailleurs dans le programme, celle-ci peut mal se passer puisqu'il reste des caractères dans le flux stdin (au minimum le \n, potentiellement n'importe quoi si l'utilisateur n'entre pas 1, 2 ou 3)

**miss_marrakchia** · 06/05/2008, 14h55

Envoyé par Neocid

Merci pour cette aide.

j'ai finalement opté pour cette solution, c'est ptete pas très "jolie" mais "c'est moi qui l'ai fait" :p

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
 
int introduction(char bla[5])
{
	int i;
	do
	{
		i=0;
		printf("/|===============================================|\\\n");
		printf("||    Bonjour et bienvenue dans notre programme  ||\n");
		printf("||	        				 ||\n");
		printf("||    Veuillez choisir l'option de votre choix	 ||\n");
		printf("||          [1]- Lancer une recherche            ||\n");
		printf("||	    [2]- Afficher les crdits		 ||\n");
		printf("||	    [3]- Quitter			 ||\n");
		printf("\\|===============================================|/\n");
		scanf("%d",&i);
		getchar();
 
	}while(! ((i==1)||(i==2)||(i==3)) || (isdigit(i)));
 
		if(i==1)
		{
			saisie_repertoire(bla);
			lecture_repertoire(bla,NULL);
		}
		else if(i==2)
		{
			credit();
		}
		else if(i==3)
		{
			return 0;
		}
 
 
}

Pourquoi autant de If Else , un switch case aurait été mieux adapté à ce genre de traitement !!!

**Thierry Chappuis** · 06/05/2008, 19h12

Envoyé par miss_marrakchia

Pourquoi autant de If Else , un switch case aurait été mieux adapté à ce genre de traitement !!!

Utiliser switch plutôt que if ... else if ... else if ... ici, c'est plus une question de style qu'autre chose. Il y a peut-être un infinitésimal gain de performance en utilisant une structure de contôle switch, mais je doute que ce soit significatif.

Thierry

**Neocid** · 07/05/2008, 18h46

Hmm j'ai un nouveau soucis lors du choix numéro 3 ( quitter le programme ).

Sous Linux tout va bien, mais sous Windows la boucle se contente de recommencer. Il n'aimerait pas le "return 0" ?

Une idée ?

**droggo** · 07/05/2008, 18h57

Fao,

Envoyé par Neocid

Hmm j'ai un nouveau soucis lors du choix numéro 3 ( quitter le programme ).

Sous Linux tout va bien, mais sous Windows la boucle se contente de recommencer. Il n'aimerait pas le "return 0" ?

Une idée ?

Poste ton code en l'état actuel.

Problème de boucle

C

Discussions similaires

Partager

Partager