aide pour un exo

Version imprimable

aide pour un exo

Salut à tous ,

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

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
 
#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.

21/11/2011, 23h14
bretus
Bonsoir,

Citation:

Envoyé par Greggg1

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:

1 2 if ( 23 == a ){}
Plutôt que
Code:

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

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:

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

22/11/2011, 18h40
Greggg1

Citation:

Envoyé par bretus

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.
22/11/2011, 20h34
bretus

Citation:

Envoyé par Greggg1

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

Citation:

Envoyé par oodini

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

Salut,

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

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:

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:

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

D'abord merci à tous pour vos réponses.
Citation:
Envoyé par 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 :D

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:

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:

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.

23/11/2011, 12h02
koala01

Citation:

Envoyé par Greggg1

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