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 ?