Discussion :

[CompuTux]

Bonjour,

Dans une application console, je dois implémenter une sorte de menu càd choisir entre 4 carburants en tapant 1 pour le carburant 1, 2 pour le 2ème, etc.

Je considère l'utilisateur non intelligent càd qu'il peut taper 5 ou 56,45745745 pour choisir le carburant. J'aimerais donc que mon programme revienne au menu de choix du carburant lorsque l'on tape n'importe quoi.

Mon code est :

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
// définition de la fonction choix du prix du carburant
 
        public static double prix_carburant(int choix_carburant)
        {
            double prix; 
 
            if (choix_carburant == 1)
            {
                prix = 1.309;
                return prix;
            }
            else if (choix_carburant == 2)
            {
                prix = 1.287;
                return prix;
            }
            else if (choix_carburant == 3)
            {
                prix = 1.049;
                return prix;
            }
            else 
            {
                prix = 0.57;
                return prix;
            }
 
        }

Pour l'instant le prix du carburant est 0,57 losque l'on tape autre chose que 1 ou 2 ou 3.

J'ai essayé avec un Environnement.Exit() mais ça ne marche pas.

Quelqu'un peut-il m'aider svp ?

[teddyalbina]

Pour pour commencer un peu de ménage dans ton 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
 
 
// définition de la fonction choix du prix du carburant
 
        public static double prix_carburant(int choix_carburant)
        {
            double prix;
            switch(choix_carburant)
            {
                  case 1:
                    prix = 1.309;
                    break;
                  case 2:
                    prix = 1.287;
                    break;
                  case 3:
                    prix = 1.049;
                    break;
                  default:           
                    prix = -1; // erreur de frapppe         
                    break;
            }
            return prix;       
        }

Pour alors ici utilise plutot un switch. Pour le reste tu test chacun des cas prévu, si l'entrée choix_carburant ne correspond pas le switch saute à default: ici j'ai mis -1 qui te permettra de savoir qu'une erreur de frappe a eu lieu

[CompuTux]

Le prof veut qu'on utilise uniquement les conditionnels if else. Dans ce cas comment faire ?

[teddyalbina]

Ton prof est relou

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
 
 
// définition de la fonction choix du prix du carburant
 
        public static double prix_carburant(int choix_carburant)
        {
            double prix;
 
            if (choix_carburant == 1)
                prix = 1.309;
            else if (choix_carburant == 2)
                prix = 1.287;
            else if (choix_carburant == 3)
                prix = 1.049;
            else 
                prix = -1; // problème
 
            return prix;
        }

Evite autant que possible de mettre des return dans tout les sens, parce que ça donne mal au crâne

[CompuTux]

Merci teddyalbina !

Est il possible de mettre une phrase comme "Votre choix est incorrect ! Veuillez rechoisir :" et faire recommencer la fonction prix_carburant au début ?

Genre avec un goto... ça existe ?

[teddyalbina]

OUlala non pas de GOTO oublie de suite.

Ce qu'il te faut c'est boucler tant que la condition prix != -1 n'est pas remplie

Code :

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

1
2
3
4
5
TANT QUE prix == -1
   AFFICHER MESSAGE
   RECUPERER SAISIE UTILISATEUR
      prix = prix_carburant(SAISIE);
FIN TANT QUE

[olibara]

Ton prof est relou

Si c'est pour l'experience academique du else if je peux peut-etre comprendre....

Mais pour ma part, j'ai toujours trouvé que les cascade de else if etaient un piege pour la compréhension logique car souvent au bout du 3eme else if on a du mal a savoir d'ou on vient et le chemin eventuel qui reste a parcourir

Dans ton cas effectivement le switch case serait plus approprié et plus lisible

Ou meme ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
if (condition a)
{
  // traitement 
  return;
}
if (condition b)
{
  // traitement 
  return;
}

[realpasso]

Si c'est pour l'experience academique du else if je peux peut-etre comprendre....

Mais pour ma part, j'ai toujours trouvé que les cascade de else if etaient un piege pour la compréhension logique car souvent au bout du 3eme else if on a du mal a savoir d'ou on vient et le chemin eventuel qui reste a parcourir

Dans ton cas effectivement le switch case serait plus approprié et plus lisible

Ou meme ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
if (condition a)
{
  // traitement 
  return;
}
if (condition b)
{
  // traitement 
  return;
}

Je partage cet avis.

Le param de fonction ne devrait pas être int mais une énumeration de Carburant. Et aucun prix n'existe sans devise.

Si input est invalide il faut "lancer" une exception (ArgumentException ?).

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
 
// Generalement on travaille en unité indivisible de la devise
// car _amount reste entier et on a pas d'arrondis et plus
class Price{ int _amout; Devise _devise; ...
 
Price PrixCarburant(Carburant carburant)
{
 
if( Carburant.SansPlomb95 == carburant)
return( new Price(1.095, Devise.Euro) );
 
...
// connait pas ce carburant
throw new ArgumentException("carburant non pris en charge", "carburant");
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
void Main(...
{
try
{
...
}
catch(ArgumentException ex)
{
 ... //Saisie incorrecte
}
}

et une classe carburant ?

Pascal