Discussion :

[Greggg1]

Salut à tous ,

Je suis un grand débutant en C++ et j'ai un petit probleme avec mon exo :

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
 
#include <cstdlib>
#include <iostream>
using namespace std;
 
int main(int argc, char *argv[])
{
    cout<<"bonjour\n";
 
    char S ; //sexe de la personne
    int A ; // age de la personne 
    bool droit ; //initialiser a vrai si la personne a droit a la carte
 
    cout<<"Saisir l'age et le sexe de la personne : " "\n" ;
 
    cin>>S>>A ;
 
    if (S=='F') 
    {   
        if (A>=60) 
         {
             droit=true ;
         }
         else
         {
             droit=false ;
         }          
    } 
    else 
    {
        if (S=='H') 
        {   
        if (A>=65) 
         {
             droit=true ;
         }
         else 
         {
             droit=false ;
         }          
    }  
    if (droit=false) 
    {
        cout<<"Vous n'avez pas le droit a la carte vermeil" "\n" ;
    }
    if (droit=true)
    { 
        cout<<"Vous avez le droit a la carte vermeil" "\n" ;
    }                     
    system("PAUSE");
    return EXIT_SUCCESS;
}
}

Si vous ne comprenez pas le probleme je vous explique en esperant que vous puissiez m'aider .

Greggg1.

[bretus]

Bonsoir,

if (droit=false)
if (droit=true)

1) == revient à comparer, = revient à affecter...


3) astuce anti-étourderie : if ( <constante_litterale> = <variable> ) provoque une erreur de compilation.

Ecrire :

Code :

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

1
2
 
if ( 23 == a ){}

Plutôt que

Code :

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

1
2
 
if ( a == 23 ){}

Si jamais on oublie un =, le compilo hurle dans le premier cas... (impossible d'affecter a à 23)

[cob59]

Essaie peut-être de structurer un peu tes données et de les nommer plus explicitement dans ton programme, au lieu de balancer des 's' 'a' 'droit' en vrac dans ton main(). Pense aussi à gérer les cas où l'utilisateur rentre autre chose que 'F' ou 'M'.

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
#include <iostream>
#include <cstdlib>
 
using namespace std;
 
struct Personne {
	unsigned int age;
	char sexe;
	bool sexeValide() {
		return sexe=='F' || sexe=='M';
	}
	bool droitCarteVermeil() {
		switch (sexe) {
		case 'F': return (age >= 60);
		case 'M': return (age >= 65);
		}
		return false; 
	}
};
 
int main(void)
{
	Personne p;
	cout << "Age ?" << endl;
	cin >> p.age;
	cout << "Sexe ?" << endl;
	cin >> p.sexe;
	if (!p.sexeValide()) {
		cerr << "[Erreur] 'M' et 'F' uniquement" << endl;
		exit (EXIT_FAILURE);
	}
 
	if (p.droitCarteVermeil()) {
		cout << "Vous avez le droit a la carte vermeil" << endl;
	} else {
		cout << "Vous n'avez pas le droit a la carte vermeil" << endl;
	}
 
	return EXIT_SUCCESS;
}

[Greggg1]

2) "unBooleen == true" est d'un intérêt limité...

D'abord merci pour ta reponse mais en fait en cours le prof nous l'a fait marquer en algo et en tp il a voulu qu'on le passe en C++ donc on doit garder la structure qu'il nous a filer et ne rien changer ...

Ce que je comprends c'est que, que je mette n'importe quel sexe et n'importe quel âge, j'ai toujours "vous n'avez pas le droit a la carte vermeil ".

en esperant que tu puisse m'aider .

Greggg1.

[bretus]

D'abord merci pour ta reponse mais en fait en cours le prof nous l'a fait marquer en algo et en tp il a voulu qu'on le passe en C++ donc on doit garder la structure qu'il nous a filer et ne rien changer ...

C'est marrant, le miens nous avait clairement expliqué que si on comparait un booléen à "true", on était bon pour la bulle...

C'est plus clair. Et ça a l'intérêt de rappeller que c'est un booléen, le C++ n'appliquant pas un typage fort dans les tests booléens.

Le résultat est pourtant sous vos yeux, ça apporte beaucoup d'injecter des instructions inutiles...

[koala01]

Salut,

Il faut savoir que, si tu travailles avec un booléen et qu'il n'est pas vrai, il sera d'office... faux

Ce qui tombe bien, parce que le pendant à un test if(quelque chose a tester) qui entrera dans la partie qui suit, mais que tu peux aussi prévoir le cas où le résultat du test donne faux en utilisant le mot clé else

Plutôt que d'écrire tes tests sous la forme (corrigée) de

Code :

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

1
2
3
4
5
6
7
8
    if (droit==false) 
    {
        cout<<"Vous n'avez pas le droit a la carte vermeil" "\n" ;
    }
    if (droit==true)
    { 
        cout<<"Vous avez le droit a la carte vermeil" "\n" ;
    }

tu peux donc l'écrire sous la forme de

Code :

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

1
2
3
4
5
6
7
8
9
 
    if (droit==true) 
    {
        cout<<"Vous avez le droit a la carte vermeil" "\n" ;
    }
    else
    { 
        cout<<"Vous n'avez pas le droit a la carte vermeil" "\n" ;
    }

[Greggg1]

D'abord merci à tous pour vos réponses.

Salut,

Il faut savoir que, si tu travailles avec un booléen et qu'il n'est pas vrai, il sera d'office... faux

Ce qui tombe bien, parce que le pendant à un test if(quelque chose a tester) qui entrera dans la partie qui suit, mais que tu peux aussi prévoir le cas où le résultat du test donne faux en utilisant le mot clé else

Plutôt que d'écrire tes tests sous la forme (corrigée) de

Code :

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

1
2
3
4
5
6
7
8
    if (droit==false) 
    {
        cout<<"Vous n'avez pas le droit a la carte vermeil" "\n" ;
    }
    if (droit==true)
    { 
        cout<<"Vous avez le droit a la carte vermeil" "\n" ;
    }

tu peux donc l'écrire sous la forme de

Code :

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

1
2
3
4
5
6
7
8
9
 
    if (droit==true) 
    {
        cout<<"Vous avez le droit a la carte vermeil" "\n" ;
    }
    else
    { 
        cout<<"Vous n'avez pas le droit a la carte vermeil" "\n" ;
    }

Sache koala01 que je suis tout a fait d'accord avec toi mais c'est le prof qui nous l'a fait ecrire en algo de cette façon et je dois garder la structure initial...

Greggg1.

[koala01]

Sache koala01 que je suis tout a fait d'accord avec toi mais c'est le prof qui nous l'a fait ecrire en algo de cette façon et je dois garder la structure initial...

Greggg1.

Un prof qui force à utiliser deux tests au lieu de la branche else en algorithmie, on peut sérieusement douter de sa compétance!...

Laisse moi deviner, il en est encore au FlowChart