Problème de boucle do..while

Version imprimable

Je sors de ma boucle alors que les conditions ne sont pas remplies, quelqu'un peut m'aider : voici mon code :

Code:

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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 
int ArretProgramme()
{
    /* Fonction renvoyant 0 ou 1
    0 traduit o et 1 traduit n
    0=On quitte le programme
    1=On reste dans le programe
    */
    int i;
    int car;   
    printf("\nConfirmez-vous vraiment l'arr%ct du programme : (O)OUI et (N)NON :",136);
    car=getche(); /*Recupération de la valeur saisie sur écran */
    if ((car=='o') || (car=='O')) /* Si on a tapé o ou O donc on quitte */
    {
        i=1;
    }
    else /*Sinon n ou autre valeur on ne quitte pas et on renvoie une erreur si ce n'etait n */
    {
        if ((car=='N') || (car=='n'))
        {
            i=0;
        }
        else
        {
            printf("Retour au menu principal, car mauvais choix.\n");
            i=0;
        }
    }
    /*printf("Nombre retourné :%d\n",i);*/
    return i; /*Renvoi de la valeur saisie */
}
 
 
 
 
int main(int argc, char *argv[])
{
    int iOption;
    int iArret;
    /*Appel de l'affichage du menu principal*/
    do
    {
        iArret=1;
        AffichageMenu();
        /*iOption=ChoixOption();*/
        printf("Veuillez-S%cl%ctionner une option 1,2,3,4 : ",130,130);
        iOption=getche();
        /*printf("Valeur de l'option : %c\n",iOption);*/
        switch (iOption)
        {
            case 49:
                printf("\nModule de saisie\n");
                /*iOption=*/getche();
               continue;
            case 50:printf("\nModule de tri\n");getche();break;
            case 51:printf("\nModule de suppression\n");getche();break;
            case 52 :iArret=ArretProgramme();break;
            default : iOption=getche();printf("\nValeur par defaut\n");continue; 
        }
 
    }while(iArret != 1 && iOption != 53); /* tant que iArret(flag de la fonction Arret programme ne vaut pas 1 et iOption choix menu
    différent de 4 on affiche le menu principal */  
    printf("\nValeur de iOption : %d\n",iOption);
	return 0;
}

02/08/2006, 11h43
fearyourself

Bien le bonjour,

- Quelle condition ?
- Quelle est l'exécution que tu fais qui pose problème...

Jc
02/08/2006, 11h48
chental

condition

salut,

la voici mes conditions

while(iArret != 1 && iOption != 53);
02/08/2006, 11h52
floflo_2006

tu n'aurait pas fait une erreur ??

Code:

case 52 :iArret=ArretProgramme();break;

et plus loin

Code:

}while(iArret != 1 && iOption != 53);

ca vient peut etre de là

Premièrement :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
cond.c: In function 'ArretProgramme':
cond.c:10: warning: implicit declaration of function 'printf'
cond.c:10: warning: incompatible implicit declaration of built-in function 'printf'
cond.c:11: warning: implicit declaration of function 'getche'
cond.c: In function 'main':
cond.c:43: warning: implicit declaration of function 'AffichageMenu'
cond.c:45: warning: incompatible implicit declaration of built-in function 'printf'
cond.c:62: warning: incompatible implicit declaration of built-in function 'printf'
cond.c:35: warning: unused parameter 'argc'
cond.c:35: warning: unused parameter 'argv'
/tmp/ccCsRWfB.o: In function `ArretProgramme':cond.c:(.text+0x1b): undefined reference to `getche'
/tmp/ccCsRWfB.o: In function `main':cond.c:(.text+0x5c): undefined reference to `AffichageMenu'
:cond.c:(.text+0x7d): undefined reference to `getche'
:cond.c:(.text+0x97): undefined reference to `getche'
:cond.c:(.text+0xf2): undefined reference to `getche'
:cond.c:(.text+0x11f): undefined reference to `getche'
:cond.c:(.text+0x14c): undefined reference to `getche'

Voilà ce que me rend le compilateur...

Ensuite :

Code:

1
2
3
4
5
6
7
8
9
10
11
        switch (iOption)
        {
            case 49:
                printf("\nModule de saisie\n");
                /*iOption=*/
               continue;
            case 50:printf("\nModule de tri\n");break;
            case 51:printf("\nModule de suppression\n");break;
            case 52 :iArret=ArretProgramme();break;
            default : iOption=getchar();printf("\nValeur par defaut\n");continue; 
        }

Utiliser les codes ascii en numérique rend le code non portable... Il faut utiliser '1', '2', '3' ...

Enfin, je ne sais pas ce que tu fais pour voir le problème... Donne un exemple d'exécution à problème...

Jc

02/08/2006, 11h59
chental

condition

Oui, peut être à ce niveau, mais je viens de corriger et je continue toujoirs à sortir du boucle.
En tapant 1(49), je devrais me retrouver toujours à l'écran avec module de saisie mais au contraire je sors.
02/08/2006, 12h02
fearyourself

Citation:

Envoyé par chental

Oui, peut être à ce niveau, mais je viens de corriger et je continue toujoirs à sortir du boucle.
En tapant 1(49), je devrais me retrouver toujours à l'écran avec module de saisie mais au contraire je sors.

Non, en tapant 1, iArret demeure à 1 donc la boucle se termine...

Jc
02/08/2006, 12h05
chental

Conditions

Ci-joint la fonction main modifiée mais le problème persiste.
Il faut que les deux conditions suivantes soient remplies pour sortir de la boucle :
Pour sortir de la boucle, il faut que iArret valle 1 et iOption valle 5

int main(int argc, char *argv[])
{
int iOption=1;
int iArret=0;
/*Appel de l'affichage du menu principal*/
do
{
iArret=1;
AffichageMenu();
/*iOption=ChoixOption();*/
printf("Veuillez-S%cl%ctionner une option 1,2,3,4 : ",130,130);
iOption=getche();
/*printf("Valeur de l'option : %c\n",iOption);*/
switch (iOption)
{
case '1':
printf("\nModule de saisie\n");
/*iOption=*/getche();
continue;
case '2':printf("\nModule de tri\n");getche();break;
case '3':printf("\nModule de suppression\n");getche();break;
case '4':printf("\nModule de sauvegarde\n");getche();break;
case '5' :iArret=ArretProgramme();break;
default : iOption=getche();printf("\nValeur par defaut\n");break;
}

}while(iArret != 1 && iOption != '5'); /* tant que iArret(flag de la fonction Arret programme ne vaut pas 1 et iOption choix menu
différent de 4 on affiche le menu principal */
printf("\nValeur de iOption : %d\n",iOption);
return 0;
}
02/08/2006, 12h08
floflo_2006

il faut que tu initailise

Code:

iArret = 0;

et dans la condition de ta boucle tu poura enlever

Code:

&& iOption != '5'

car ca ne servira plus je pense
02/08/2006, 12h09
fearyourself

Je crois que t'as un problème de compréhension sur le fonctionnement de la condition...

Si la condition est fausse -> On sort

Or : Si iArret == 1 alors (!iArret && n'importe quoi) -> FAUX donc la boucle se termine...

Ce que tu voudrais serait plutôt :

Code:

!((iArret==1)&&(iOption=='5'))

ou

Code:

((!iArret)||(iOption!='5'))

(Loi de Morgan)
Jc
02/08/2006, 12h20
Ti-R
Pourquoi pas juste

Code:

while(iArret != 1)

Vu que la fonction "ArretProgramme()" indique vraiment si on doit sortir ou non

et bien placer iArret à 0 et pas à 1
Code:

1 2 3 4 5 int iArret=0; /*Appel de l'affichage du menu principal*/ do { iArret=1; // à supprimer !
02/08/2006, 12h27
chental

Conditions

J'ai enlevé la condition iOption et laissé le iArret à (iarret!=1) et cela fonctionne, merci les gars de votre participation, en effet j' en conclus un problème de compréhension car en ce moment stressé par ma non rapidité dans l'écriture de ce futuire dll. encore une fois merci
02/08/2006, 19h26
Ti-R

Mais de rien Chantal, nous sommes des chevaliers servant, sans peur et sans reproche, qui volons à la rescousse des plus démunis dans ce monde impitoyable qu'es le C :P ;)