Discussion :

[Elragail]

Bonjour à tous,

Le but du programme est de calculer la moyenne de notes inscrites par l'utilisateur.

J'ai débuté l'apprentissage du C++ il y a peu, et me voila bloquée sur un programme qui s'exécute normalement mais qui, après avoir entré la première note, demande à l'utilisateur s'il souhaite continuer ou pas. Et c'est là que je bloque. Je pense avoir fait une erreur sur les boucles, mais j'ai beau lire et relire mon code, je ne trouve pas la solution.

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
#include <iostream>
 
using namespace std;
 
int main()
{
    //declaration
    int note, cumul, nbre;
    string rep;
 
    //initialisation
    cumul = 0;
    nbre = 0;
 
    do{
        cout << "Entrez une note" << endl;
        do {
            cin >> note;
        } while (note<0 || note>20);
 
        //calculs
        cumul = cumul + note;
        nbre++;
 
        do{
        cout << "Voulez vous continuer ? (Oui/Non)" << endl;
        cin >> rep;
        } while ((rep == "Non") || (rep == "Oui"));
 
    } while (rep == "Non");
 
    //affichage de la moyenne
    cout << "La moyenne de ces" << nbre << " notes est de " << (cumul/nbre) << endl;
 
}

Je viens donc quémander votre aide

[Neckara]

Bonjour,

while(condition) indique "tant que 'condition' est vrai faire des tours de boucles".
Ainsi, il ne faut pas faire while(rep == "non") mais while(rep != "non")

[Elragail]

Merci Neckara pour votre aide. Effectivement, il y avait bien une erreur à ce niveau là.

A présent, le programme bloque et réclame toujours une réponse de l'utilisateur "Voulez vous continuer ? (Oui/Non). Il y a aussi un bug au niveau de la boucle qui vérifie la note (entre 0 et 20).

Les boucles me donnent beaucoup de peine décidément.

Heureusement, je me suis achetée 3 bouquins qui sont cités dans la rubrique livres C++ de ce site. Peut-être qu'alors je réussirais à comprendre et modifier mon code.

[Neckara]

Pour la boucle du "oui"/"non", relisez ce que je vous ai dit au post précédent.

Pour les notes, il faudrait nous dire quel est le bug pour que nous puissions vous aider.

A votre niveau, ce n'est pas un problème de connaissance mais de méthode et de rigueur.

[Elragail]

J'approuve le fait que je n'ai pas la méthodologie nécessaire, mais il faut bien un début à tout. Je suis assez réticente à poster dans les forums pour demander de l'aide car ce n'est pas toujours bien vu par les pros de la programmation. Or, il ne s'agit pas de fainéantise mais d'une réel problème que je n'arrive pas à solutionner.

Donc, je vous remercie de m'aider.

[Obsidian]

Hello,

Je suis assez réticente à poster dans les forums pour demander de l'aide car ce n'est pas toujours bien vu par les pros de la programmation. Or, il ne s'agit pas de fainéantise mais d'une réel problème que je n'arrive pas à solutionner.

C'est pour cela qu'il y a ici un sous-forum « débuter » qui sert précisément à cela (celui dans lequel tu lis ce message).

Ne t'inquiète pas. Ce qui n'est pas apprécié ici, ce sont les gens qui déposent leur énoncé tel quel sans s'y intéresser, ou les gens qui se contentent de dire « ça marche pas » sans donner plus de précision. Mais si tu fais l'effort de choisir le bon forum, de rédiger proprement tes messages et d'expliquer clairement ce sur quoi tu bloques, tu trouveras toujours quelqu'un pour t'aider.

[Elragail]

Merci Obsidian, vous me rassurez.

Bon, au niveau du code ça s'est un peu arrangé mais pas totalement puisque lors de l'affichage en mode console, je n'arrive pas à sortir de la boucle "Voulez-vous continuer (Oui/Non).

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
#include <iostream>
 
using namespace std;
 
int main()
{
    //declaration
    int note, cumul, nbre;
    string rep;
 
    //initialisation
    cumul = 0;
    nbre = 0;
 
    do{
        cout << "Entrez une note entre 0 et 20" << endl;
        do {
            cin >> note;
        } while (note<0 || note>20);
 
        //calculs
        cumul = cumul + note;
        nbre++;
 
        do {
        cout << "Voulez vous continuer ? (Oui/Non)" << endl;
        cin >> rep;
        } while (rep != "Oui");
 
    } while (rep != "Non");
 
    //affichage de la moyenne
    cout << "La moyenne de ces" << nbre << " notes est de " << (cumul/nbre) << endl;
 
}

Voilà le soucis, c'est que la moyenne ne s'affiche pas car on reste dans la boucle si on répond Non. Et je ne comprends pas pourquoi.

[Neckara]

Essayez de parcourir votre code et de réfléchir à ce qu'il fait en le parcourant ligne à ligne.

Code :

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

while (rep != "Oui");

Dans quel cas la condition est-elle fausse, ie dans quels cas sort-on de la boucle ?

[Elragail]

On sort de la boucle quand la condition du while est vraie donc que la réponse soit différente que "Oui". La condition sera fausse si l'utilisateur répond "Oui". Est-ce cela ?

[Obsidian]

Indice : attention aux doubles négations !

On sort de la boucle quand la condition du while est vraie donc que la réponse soit différente que "Oui". La condition sera fausse si l'utilisateur répond "Oui". Est-ce cela ?

« while » signifie « tant que ». Ta boucle peut donc être traduite en français par :

« Effectue { <contenu du bloc> } Tant Que ( rep Est Différent De "Oui" ) ».

[Elragail]

J'ai beau triturer mon code, il n'y a rien à faire, je suis bloquée. Je m'en veux de ne pas y arriver.

[imperio]

La bidouille c'est bien, mais dans ton code il n'y en a pas besoin. Si tu n'as pas tout compris sur les boucles ou d'autres bases, tu peux toujours regarder des tutos sur le net. Il y a de bons tutoriels pour apprendre C++ sur ce site pour debuter. Sinon montre-nous ton code pour voir ou t'en es.

[JolyLoic]

Au début, tu avais :

Code :

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

1
2
3
4
5
6
        do{
        cout << "Voulez vous continuer ? (Oui/Non)" << endl;
        cin >> rep;
        } while ((rep == "Non") || (rep == "Oui"));
 
    } while (rep == "Non");

Ce qui est bien, c'est que tu sépare la gestion dela validité de la saisie utilisateur (le premier while) avec ce que tu dois en faire par la suite (le second). Le problème, c'est que tu t'es trompée sur le sens des conditions.

Par la suite, tu as écrit :

Code :

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

1
2
3
4
5
6
        do {
        cout << "Voulez vous continuer ? (Oui/Non)" << endl;
        cin >> rep;
        } while (rep != "Oui");
 
    } while (rep != "Non");

Cette fois-ci, les conditions sont dans le bon sens, mais ton premier while n'autorise que la saisie de "Oui". Si l'utilisateur saisi quoi que ce soit d'autre, y compris "Non", on continue à lui demander s'il veut boucler.

Si on parle en français, on veut que la première boucle continue tant qu'il n'a pas saisi "Oui" ou "Non" (mais attention à l'utilisation de "ou" dans la langue courante). C'est à dire (tu l'écris comme tu veux, c'est équivalent, mais certains y arrivent mieux d'une manière que de l'autre ):

Code :

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

1
2
while (!((rep == "Non") || (rep == "Oui"))); // Tant qu'on n'a pas soit oui, soit non
while (rep != "Non" && rep != "Oui"); // Tant que la réponse est différente de oui et qu'elle est différente de non

[Elragail]

Merci JolyLoic, j'étais loin de pouvoir corriger mon erreur toute seule. Mais j'ai compris pourquoi ça ne fonctionnait pas.

[SkyZoThreaD]

Tu peux donc passer le thread en "résolu"