Discussion :

[BlackFridge]

Bonjour !

Voici mon problème :

J'ai créé un programme simple de conversion de devise afin d'apprendre le C# (pour mon stage).

Avec : -deux RadioButtons, un pour de l'€ vers le Francs et inversement.
-deux TextBoxs, la somme à convertir et le résultat de la conversion.
-Un Button pour exécuter la conversion.

Lors de l'exécution, le button pour convertir applique bien le code et sort le bon résultat quelque soit le sens de conversion choisi.
Cependant, je ne peux plus re-convertir après une première exécution.

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
 private void m_convertir_Click(object sender, EventArgs e)
        {
            try
            {
                Program p = new Program(Double.Parse(TB_entree.Text, 0)
                    , Double.Parse(TB_sortie.Text, 0));
 
                //Checking Radio Buttons
                if (RB_toEuro.Checked){
                    //Conversion
                    p.convertirFunctionFtoE();
                    //Envoi du résultat de la conversion
                    TB_sortie.Text = p.sortieToString();
                }
 
                if (RB_toFranc.Checked){
                    //Conversion
                    p.convertirFunctionEtoF();
                    //Envoi du résultat de la conversion
                    TB_sortie.Text = p.sortieToString();
                }
            }
            catch (FormatException f) {
                TextWriter errorWriter = Console.Error;
                errorWriter.WriteLine(f.Message);
            }
        }

Je ne vois pas ce que je peux faire car logiquement, la fonction s'applique à chaque clic.

Merci d'avance de votre aide =)

[Jabbal'H]

Tu passe dans le code ? T'as essaye en debug pas à pas.
Sinon une petite remarque, je pense que tu ne peux pas avoir les deux radio bouton de selectionné, donc un else suffit plutot que de re tester le cas

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
if (RB_toEuro.Checked){
                    //Conversion
                    p.convertirFunctionFtoE();
                    //Envoi du résultat de la conversion
                    TB_sortie.Text = p.sortieToString();
                }
 
                else{
                    //Conversion
                    p.convertirFunctionEtoF();
                    //Envoi du résultat de la conversion
                    TB_sortie.Text = p.sortieToString();
                }

[BlackFridge]

Merci pour le conseil du débogage (même si c'est censé être une broutille).
C'était un problème dans l'initialisation de ma classe Program. Une mauvaise utilisation de la fonction "Double.Parse".

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
 private void m_convertir_Click(object sender, EventArgs e)
        {
            try
            {
 
                this.p = new Program(Double.Parse(TB_entree.Text, System.Globalization.NumberStyles.AllowDecimalPoint)
                    , Double.Parse(TB_sortie.Text, System.Globalization.NumberStyles.AllowDecimalPoint));
 
                //Checking Radio Buttons
                if (RB_toEuro.Checked){
                    //Conversion
                    p.convertirFunctionFtoE();
                    //Envoi du résultat de la conversion
                    TB_sortie.Text = p.sortieToString();
                }
 
                else if(RB_toFranc.Checked){
                    //Conversion
                    p.convertirFunctionEtoF();
                    //Envoi du résultat de la conversion
                    TB_sortie.Text = p.sortieToString();
                }
            }
            catch (FormatException f) {
                MessageBox.Show(f.Message);
            }
        }