Discussion :

[romcollard]

Bonsoir

J'ai une combobox qui contient +,-,* et /
L'orsque l'ulisateur saisi les deux valeurs dans TxtPremierNbr.Text etTxtDeuxiemeNbr.Text, le résultat s'afiche dans un Label.

J'ai utilisé ce 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
switch (CbxOp.Text)
             {
                 (CbxOpération.Text) "+":
                    LblResultat = Convert.ToString(Convert.ToDouble(TxtPrmNbr.Text) + Convert.ToDouble(TxtDxNbr.Text));break;
 
                (CbxOpération.Text) "-":
                    LblResultat = Convert.ToString(Convert.ToDouble(TxtPrmNbr.Text) - Convert.ToDouble(TxtDxNbr.Text));break;
 
                (CbxOpération.Text) "/":
                    LblResultat = Convert.ToString(Convert.ToDouble(TxtPrmNbr.Text) / Convert.ToDouble(TxtDxNbr.Text));break;
 
                (CbxOpération.Text) "*":
                    LblResultat =Convert.ToString (Convert.ToDouble(TxtPrmNbr.Text) * Convert.ToDouble(TxtDxNbr.Text));break;
 
                default: MessageBox.Show("Opérateur inconnu");}

Sans succès. Merci d'avance à ceux qui pourront m'aider

[Olivier782]

Bonsoir,

Quel est ton problème exactement? Problème à la compilation? à l'exécution? Une exception? un résultat incorrect?

Est-ce vraiment ton code ou est-ce un pseudo-ode ce que tu as mis? Car je ne vois pas les "case" dans le switch.

[romcollard]

Voici le code que nous a fourni notre prof sur lequel on doit se baser :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
switch (operateur)
{
case "+":
resultat = nb1 + nb2 ; break;
case "-":
resultat = nb1 - nb2 ; break;
case "x":
resultat = nb1 * nb2 ; break;
case "/":
resultat = nb1 / nb2 ; break;
default: MessageBox.Show("Opérateur inconnu");
break;
}

[Olivier782]

Ton code ne devrait-il pas ressembler plus à ça?

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
 
 
string operateur = CbxOp.Text;
 
switch (operateur )
{
case "+":
LblResultat = Convert.ToString(Convert.ToDouble(TxtPrmNbr.Text) + Convert.ToDouble(TxtDxNbr.Text));break;
 
case "-":
LblResultat = Convert.ToString(Convert.ToDouble(TxtPrmNbr.Text) - Convert.ToDouble(TxtDxNbr.Text));break;
 
case "/":
LblResultat = Convert.ToString(Convert.ToDouble(TxtPrmNbr.Text) / Convert.ToDouble(TxtDxNbr.Text));break;
 
case "*":
LblResultat =Convert.ToString (Convert.ToDouble(TxtPrmNbr.Text) * Convert.ToDouble(TxtDxNbr.Text));break;
 
default: MessageBox.Show("Opérateur inconnu");}

[Aeronia]

A quoi correspond exactement la variable LblResultat ?

Est ce un string, ton tabel ?

Car si tu souhaites mettre le résultat de ton opération dans le le label nommé LblResultat , tu dois le faire via la propriété Text de ton label.

Comme suit : LblResultat.Text = ...

Si je me bases sur le code fourni, cela ressemblerait plus à ceci :

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
 
 
switch (CbxOp.Text)
{
case "+":
LblResultat.Text = Convert.ToString(Convert.ToDouble(TxtPrmNbr.Text) + Convert.ToDouble(TxtDxNbr.Text));break;
 
case "-":
LblResultat.Text = Convert.ToString(Convert.ToDouble(TxtPrmNbr.Text) - Convert.ToDouble(TxtDxNbr.Text));break;
 
case  "/":
LblResultat.Text = Convert.ToString(Convert.ToDouble(TxtPrmNbr.Text) / Convert.ToDouble(TxtDxNbr.Text));break;
 
case  "*":
LblResultat.Text =Convert.ToString (Convert.ToDouble(TxtPrmNbr.Text) * Convert.ToDouble(TxtDxNbr.Text));break;
 
default: MessageBox.Show("Opérateur inconnu");}

A noter que dans ce code, aucune vérification est faites sur le fait que l'utilisateur ai bien encodé des nombres valides

[StringBuilder]

Personne sur ce forum n'a l'idée de faire :

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
 
try
{
   double resultat;
   double operateur;
   double operande;
 
   if (!double.TryParse(TxtPrmNbr.Text, ref operateur))
   {
       throw new Exception("L'opérateur n'est pas un nombre valide")
   }
 
   if (!double.TryParse(TxtDxNbr.Text, ref operande))
   {
       throw new Exception("L'opérande n'est pas un nombre valide")
   }
 
   string operateur = CbxOp.Text;
 
   switch (operateur)
   {
      case "+":
         resultat = operateur + operande;
         break;
      case "-":
         resultat = operateur - operande;
         break;
      case "x":
         resultat = operateur * operande;
         break;
      case "/":
         resultat = operateur / operande;
         break;
      default:
         throw new Exception("L'opération n'est pas valide");
         break;
   }
 
   LblResultat.Text = resultat.ToString();
}
catch (Exception e)
{
   LblResultat.Text = e.Message;
}

C'est pas un peu plus lisible ?

[Aeronia]

On est tout à fait d'accord sur le fait que ton code est bien plus propre.

Maintenant je dois avouer que je me suis contenté de corriger le code fourni dans le post, même si je le suis ai signalé que le code n'était pas parfait

[Olivier782]

Pareil. Je me contente d'aider à corriger des codes ou donner des conseils, pas à donner la solution toute faite...