Problème de boucle do..while

**chental** · 02/08/2006, 11h41

Je sors de ma boucle alors que les conditions ne sont pas remplies, quelqu'un peut m'aider : voici mon 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
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;
}

**fearyourself** · 02/08/2006, 11h43

Bien le bonjour,

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

Jc

**chental** · 02/08/2006, 11h48

salut,

la voici mes conditions

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

**floflo_2006** · 02/08/2006, 11h52

tu n'aurait pas fait une erreur ??

Code :

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

case 52 :iArret=ArretProgramme();break;

et plus loin

Code :

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

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

ca vient peut etre de là

**fearyourself** · 02/08/2006, 11h55

Premièrement :

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
 
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 :

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

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

**chental** · 02/08/2006, 11h59

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.

**fearyourself** · 02/08/2006, 12h02

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

**chental** · 02/08/2006, 12h05

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;
}

**floflo_2006** · 02/08/2006, 12h08

il faut que tu initailise

Code :

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

iArret = 0;

et dans la condition de ta boucle tu poura enlever

Code :

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

&& iOption != '5'

car ca ne servira plus je pense

**fearyourself** · 02/08/2006, 12h09

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 :

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

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

ou

Code :

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

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

(Loi de Morgan)
Jc

**Ti-R** · 02/08/2006, 12h20

Pourquoi pas juste

Code :

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

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 :

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

1
2
3
4
5
int iArret=0;
/*Appel de l'affichage du menu principal*/
do
{
iArret=1; // à supprimer !

**chental** · 02/08/2006, 12h27

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

**Ti-R** · 02/08/2006, 19h26

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

Problème de boucle do..while

C

Discussions similaires

Partager

Partager