probleme dans le code

Version imprimable

Voir 40 message(s) de cette discussion en une page

Voila je viens d'aprendre le c++ et je j'essaye de mettre en pratique mais ce n'est pas facile.Je viens de faire un programme mais le compilateur(devc++) affiche une erreur sur int choix.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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
// Alcootest
//
//
#include <cstdio>
#include <cstdlib>
#include <iostream>
using namespace std; 
 
int main() 
 
    //Declaration des variables
    int choix;
    float vin;
    float biere;
    float whisky;
    float pastis;
    float champagne;
    float alcoolémie;
    float factor = 0.7;
 
do
 {
 
    cout << "Ver(s) de vin : 1 " << endl;
    cout << "Ver(s) de biere : 2 " << endl;
    cout << "Ver(s) de whisky : 3 " << endl;
    cout << "Ver(s) de pastis : 4 " << endl;
    cout << "Ver(s) de champagne : 5 " << endl;
    cin >> choix;
 
      //Declaration du premier argument 
      int poids;
      cout << "Entrez votre poids : ";
      cin >> poids; 
 
    if (choix == 1) { //Si l'utilisateur a choisi 1 
      cout << "Entrez le nombre(s) de ver(s) de vin bu(s) : " << endl; 
      cin >> vin; //Enregistrer dans la variable vin
      alcolémie = vin / poids * factor;
      cout << "Votre taux d'alcoolémie est de : " << taux alcoolémie << endl;
 
    }
 
    if (choix == 2) { //Si l'utilisateur a choisi 2
      cout << "Entrez le nombre(s) de ver(s) de biere bu(s)
      cin >> vin; //Enregistrer dans la variable biere
      taux alcoolémie = biere / poids * factor;
      cout << "Votre taux d'alcoolémie est de : " << alcoolémie << endl;
 
    }
 
    if (choix == 3) { //Si l'utilisateur a choisi 3
      cout << "Entrez le nombre(s) de ver(s) de biere bu(s)
      cin >> biere; //Enregistrer dans la variable whisky
      alcoolémie = whisky / poids * factor;
      cout << "Votre taux d'alcoolémie est de : " << alcoolémie << endl;
 
    }
 
    if (choix == 4) { //Si l'utilisateur a choisi 4
      cout << "Entrez le nombre(s) de ver(s) de pastis bu(s)
      cin >> pastis; //Enregistrer dans la variable pastis
      alcoolémie = pastis / poids * factor;
      cout << "Votre taux d'alcoolémie est de : " << alcoolémie << endl;
 
    }
 
    if (choix == 5) { //Si l'utilisateur a choisi 5
      cout << "Entrez le nombre(s) de ver(s) de champagne bu(s)
      cin >> pastis; //Enregistrer dans la variable champagne
      alcoolémie = champagne / poids * factor;
      cout << "Votre taux d'alcoolémie est de : " << alcoolémie << endl;
      }
        cout << "Pour revenir au menu pricipal tapez 1" << endl; 
  cout << "Pour quitter tapez 2" << endl;  
  cin >> choix; 
} 
while(choix == 1); 
 
  return EXIT_SUCCESS; 
}

merci d'avance pour votre aide.

30/10/2005, 22h26
Médinoc

accolade ouvrante manquante entre le int main() et le int choix...
30/10/2005, 22h53
vivu

Merci il manquait bien une accolade mais maintenant il m'affiche une erreur a float alcoolémie;
30/10/2005, 23h28
Médinoc

les accents ne sont pas des caractères valides.

tu as visiblement encore beaucoup à apprendre, parmi les bases...
31/10/2005, 00h47
vivu

Merci pour ta reponse sa marche.J'ai lu encore une fois et j'ai detecté des fautes et sa marche.Je pense que je vais encore l'amelioré car il n'est pas parfait et mois je suis encore qu'un debutant.C'est vrai le coups des accents j'avais completement oublier pourtant c'etait sou mon nez.J'espere m'améliorer avec le temps.

Merci encore pour l'aide
31/10/2005, 10h13
diogene

Vivu,
tu as déja fait un code très analogue dans sa structure concernant des conversions de monnaies. Utilise les conseils qui t'ont été donnés à cette occasion pour améliorer le code (parmi ceux-ci, utilise un switch ), ne te contente pas de reproduire toujours le même code sinon tu ne feras pas de progrès
Amicalement :)
31/10/2005, 10h51
vivu

ok.Je vais essayer.Qui ne tente rien n'a rien.
31/10/2005, 21h47
vivu

Diogene tu veut dire un peut comme le programme poids ideal c'est ça?
31/10/2005, 23h21
diogene

Citation:

Diogene tu veut dire un peut comme le programme poids ideal c'est ça?

???????? Connais pas 8O
02/11/2005, 11h54
vivu

Petite question sa sert a quoi les printf et les else avec le switch?

Et mes variables je les declares de la meme façon car j'ai vu sur un autre programme avec switch qu'elles etaient declarées d'une autre manieres

Bonjour,

dans ton code, tu teste toujours ansi:

Code:

1
2
3
4
5
6
7
 
    if (choix == 1)
    //{traitements}
    ...
    if (choix == 4)
    //{traitements}
    ...

Donc le test porte systématiquement sur la même variable avec des valeurs statiques différentes.

Dans ce cas, la structure la plus approprié pour des raisons de lisibilité (entre autre) c'est le branchement multiple, ou switch.

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
 
switch ( choix ) {
    case 1:  //Si l'utilisateur a choisi 1 
      cout << "Entrez le nombre(s) de ver(s) de vin bu(s) : " << endl; 
      cin >> vin; //Enregistrer dans la variable vin 
      alcolémie = vin / poids * factor; 
      cout << "Votre taux d'alcoolémie est de : " << taux alcoolémie << endl; 
      break;      
 
    case 2:  //Si l'utilisateur a choisi 2 
      cout << "Entrez le nombre(s) de ver(s) de biere bu(s) 
      cin >> vin; //Enregistrer dans la variable biere 
      taux alcoolémie = biere / poids * factor; 
      cout << "Votre taux d'alcoolémie est de : " << alcoolémie << endl; 
     break;    
 
   case 3:  //Si l'utilisateur a choisi 3 
      cout << "Entrez le nombre(s) de ver(s) de biere bu(s) 
      cin >> biere; //Enregistrer dans la variable whisky 
      alcoolémie = whisky / poids * factor; 
      cout << "Votre taux d'alcoolémie est de : " << alcoolémie << endl; 
      break;    
    case 4 : //Si l'utilisateur a choisi 4 
      cout << "Entrez le nombre(s) de ver(s) de pastis bu(s) 
      cin >> pastis; //Enregistrer dans la variable pastis 
      alcoolémie = pastis / poids * factor; 
      cout << "Votre taux d'alcoolémie est de : " << alcoolémie << endl; 
    break;    
 
    case 5 : //Si l'utilisateur a choisi 5 
      cout << "Entrez le nombre(s) de ver(s) de champagne bu(s) 
      cin >> pastis; //Enregistrer dans la variable champagne 
      alcoolémie = champagne / poids * factor; 
      cout << "Votre taux d'alcoolémie est de : " << alcoolémie << endl; 
      break;
   default:
     cout << "Erreur de paramètre" << endl;
};

Tu peux de plus largement amélioré ton code:

Code:

1
2
3
4
5
6
 
    cout << "Ver(s) de vin : 1 " << endl; 
    cout << "Ver(s) de biere : 2 " << endl; 
    cout << "Ver(s) de whisky : 3 " << endl; 
    cout << "Ver(s) de pastis : 4 " << endl; 
    cout << "Ver(s) de champagne : 5 " << endl;

Peut s'écrire tout simplement:

Code:

1
2
 
cout << "Ver(s) de vin : 1 " << endl << "Ver(s) de biere : 2 " << endl << "Ver(s) de whisky : 3 " << endl << "Ver(s) de pastis : 4 " << endl << "Ver(s) de champagne : 5 " << endl;

Tu n'as pas besoin de répéter cout systématiquement.

Pourquoi redéclaré dans ta boule le poids !

C'est avec les autres déclarations que doit ce trouvé ce qui suit.

Code:

1
2
3
 
      //Declaration du premier argument 
      int poids;

De plus tes traitement dans les branchements sont très similaire:

Code:

1
2
3
4
5
6
7
8
 
void      Afficher_Calcul_Alcolemie (string Boisson, float& NombreVerres, float& Alcolemie, float poids, float factor) {
 
      cout << "Entrez le nombre(s) de ver(s) de"<< Boisson <<" bu(s) : " << endl; 
      cin >> NombreVerres; //Enregistrer dans la variable vin 
      Alcolemie = NombreVerres / poids * factor; 
      cout << "Votre taux d'alcoolémie est de : " << Alcolemie << endl; 
}

Maintenant regarde ce que ça donne :

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
 
switch ( choix ) {
    case 1:  //Si l'utilisateur a choisi 1 
      Afficher_Calcul_Alcolemie("vin",vin,alcolemie,poids,factor);       
      break;      
 
    case 2:  //Si l'utilisateur a choisi 2 
      Afficher_Calcul_Alcolemie("biere",biere,alcolemie,poids,factor);      
     break;    
 
   case 3:  //Si l'utilisateur a choisi 3 
      Afficher_Calcul_Alcolemie("whisky",whisky,alcolemie,poids,factor);
      break;    
    case 4 : //Si l'utilisateur a choisi 4 
     // A toi de remplir    
break;    
 
    case 5 : //Si l'utilisateur a choisi 5 
     // A toi de remplir    
 
      break;
   default:
     cout << "Erreur de paramètre" << endl;
};

Plus que deux corrections (je te laisses trouver) dans ton code original, plus ce que je t'ai indiqué, et ça marche :)

02/11/2005, 14h35
vivu

Merci pour tes explications

Dnas le dernier bout de codes que tu m'as donné il n'y a pas de calcule de fait donc comment le programme peut deviner quels calcul il faut faire?

J'ai compris le debut mais je me perd un peut sur la fin.Désoler je sais que sa doit pas etre facile d'expliquer a un débutant.
03/11/2005, 12h30
Caine

Tu veux parler de quel cas?

Les cas 4 et 5? Là je te laisse transformer ton code en utilisant la fonction que je t'ai donnée, pour vérifier que tu as compris comment cela fonctionne.

La clause default ? Cette clause est une sécurité du switch. En effet, la variable qui conditionne le switch doit prendre une des valeurs des "case". Et pour les autres valeurs? Cette clause n'est exécutée que si choix ne vaut pas 1,2,3,4 ou 5.
Elle ne correspond donc à aucun calcul. Dans ce cas, il s'agirait même d'une erreur de saisie de l'utilisateur.
03/11/2005, 20h14
vivu

En faite ce sont dans les cases 1 et 2.Dois-je remettre les meme calcules que mon code original?
03/11/2005, 20h18
Bethoring

S'il tappe un char ca devrait foire et pas passer dans le default?
si?
04/11/2005, 12h35
Caine

Un char est un type entier, s'il tape une lettre au lieu d'un chiffre, la valeur de cette lettre sera endehors des cas du switch.

Essai pour voir :)
04/11/2005, 12h37
Caine

Citation:

Envoyé par vivu

En faite ce sont dans les cases 1 et 2.Dois-je remettre les meme calcules que mon code original?

Visiblement tu n'as pas compris la fonction que j'ai ajouté.

Remplace dans chaque case la fonction par son coprs :)

J'ai essayer de refaire le code mais je ne suis vraiment pas sur de mon code alors le voila.

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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
// Alcootest
//
//
#include <cstdio>
#include <cstdlib>
#include <iostream>
using namespace std; 
 
int main() 
{
 
    //Declaration des variables
    int choix;
    float vin =10;
    float biere = 10;
    float whisky =10 ;
    float pastis =10;
    float champagne =10;
    float alcoolemie =10;
    float factor = 0.7;
 
do
 {   
 
    cout << "Ver(s) de vin : 1 " << endl << "Ver(s) de biere : 2 " <<
    endl << "Ver(s) de whisky : 3 " <<
    endl << "Ver(s) de pastis : 4 " << endl << "Ver(s) de champagne : 5 " <<
    endl; 
 
    //Declaration du premier argument 
      int poids;
      cout << "Entrez votre poids : ";
      cin >> poids; 
 
      switch ( choix ) { 
    case 1:  //Si l'utilisateur a choisi 1 
        cout << "Entrez le nombre(s) de ver(s) de vin bu(s) : " << endl; 
      cin >> vin; //Enregistrer dans la variable vin
      alcoolemie = vin / poids * factor;
      cout << "Votre taux d'alcoolemie est de : " << alcoolemie << endl;        
      break;   
 
      }   
 
    case 2:  //Si l'utilisateur a choisi 2 
       cout << "Entrez le nombre(s) de ver(s) de biere bu(s) : " << endl;
      cin >> vin; //Enregistrer dans la variable biere
      alcoolemie = biere / poids * factor;
      cout << "Votre taux d'alcoolemie est de : " << alcoolemie << endl;     
     break;  
 
     }  
 
   case 3:  //Si l'utilisateur a choisi 3 
      cout << "Entrez le nombre(s) de ver(s) de biere bu(s) : " << endl;
      cin >> biere; //Enregistrer dans la variable whisky
      alcoolemie = whisky / poids * factor;
      cout << "Votre taux d'alcoolemie est de : " << alcoolemie << endl;
     break; 
 
     }
 
    case 4 : //Si l'utilisateur a choisi 4 
      cout << "Entrez le nombre(s) de ver(s) de pastis bu(s) : " << endl;
      cin >> pastis; //Enregistrer dans la variable pastis
      alcoolemie = pastis / poids * factor;
      cout << "Votre taux d'alcoolemie est de : " << alcoolemie << endl; 
     break;    
 
    case 5 : //Si l'utilisateur a choisi 5 
     cout << "Entrez le nombre(s) de ver(s) de champagne bu(s) : " << endl;
      cin >> pastis; //Enregistrer dans la variable champagne
      alcoolemie = champagne / poids * factor;
      cout << "Votre taux d'alcoolemie est de : " << alcoolemie << endl;
      break;
 
 
   default: 
     cout << "Erreur de paramètre" << endl; 
 
      system ("PAUSE");
 
 
}

tu as des problémes avec les } :
Code:

1 2 3 4 switch ( choix ) { case 1: .... break; }
cette dernière } ferme le switch. Alors ensuite le case 2: n'est pas placé correctement. Tu dois avoir
Code:

1 2 3 4 5 6 7 switch(choix) { case 1 :.... break; case 2 : ... break; .... default : ... }
Le tout est enfermé dans un do...while. Il y a le do mais pas le while!. La structure du programme doit être :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 //Déclarations ... do { // questions & réponses pour le choix .... switch(choix) { // voir ci-dessus } } while(choix>5 || choix <=0); system ("PAUSE"); }//fin de main;

04/11/2005, 19h55
vivu

Salut.

Je viens de corriger les fautes et le programme ce compile bien mais quand je lance le programme il passe directement a l'entré du poids on ne peut pas choisir dans le menu pourtant le menu est bien affiché mais il ne donne pas le choix.Bizard

Voir 40 message(s) de cette discussion en une page