ComboBox pour appli calculatrice

Version imprimable

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:

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 :)

08/01/2012, 19h30
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.

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

Code:

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

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

Code:

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

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:

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

Illisible

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

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

09/01/2012, 09h53
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 :lol:
09/01/2012, 19h13
Olivier782

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