Gdzieś jest błąd przy switch case ?

Question

Witam! Na "ifach" mi to działa ale chciałbym dla testu zrobić na switch case i już nie działa. Dwie funkcje jedna wywołuje drugą (tak potrzebuję). Mam trzy radioButtony i gdy wybiorę któryś z nich i kliknę button to w textBox nie pokazuje mi który został włączony. Gdzieś zrobiłem błąd...

Działający na "ifach":

namespace TwoFunction
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            liczenie();
        }

        public void liczenie() //cały kod 
        {
            string[] zmienna = new string[] { "rB1", "rB2", "rB3" };

            if (radioButton1.Checked)
            {
                textBox1.Text = zmienna[0];
            }

            else if (radioButton2.Checked)
            {
                textBox1.Text = zmienna[1];
            }

            else if (radioButton3.Checked)
            {
                textBox1.Text = zmienna[2];
            }
        }
    }
}

Nie działający switch case:

namespace TwoFunction
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            liczenie();
        }

        public void liczenie() //cały kod 
        {

            string[] zmienna = new string[] { "rB1" , "rB2", "rB3" };

            RadioButton radioBtn = new RadioButton();
            if (radioBtn.Enabled == true)
            {
                switch (radioBtn.Name)
                {
                    case "radioButton1":
                        textBox1.Text = zmienna[0];
                        break;

                    case "radioButton2":
                        textBox1.Text = zmienna[1];
                        break;

                    case "radioButton3":
                        textBox1.Text = zmienna[2];
                        break;
                }
            }
        }
    }
}

 

Answer 1

Czemu?

if (radioBtn.Enabled == true)

a nie

if (radioBtn.Checked == true)

 

spróbuj np. iterując po radioButton-ach

        public void liczenie()
        {
            string[] zmienna = new string[] { "rB1", "rB2", "rB3" };

            foreach (RadioButton radioButton in new RadioButton[] { radioButton1, radioButton2, radioButton3 })
            {
                if (radioButton.Checked == true)
                {
                    switch (radioButton.Name)
                    {
                        case "radioButton1":
                            textBox1.Text = zmienna[0];
                            break;

                        case "radioButton2":
                            textBox1.Text = zmienna[1];
                            break;

                        case "radioButton3":
                            textBox1.Text = zmienna[2];
                            break;
                    }
                }
            }
        }

lub np.

        public void liczenie()
        {
            string[] zmienna = new string[] { "rB1", "rB2", "rB3" };
 
            foreach (RadioButton radioButton in new RadioButton[] { radioButton1, radioButton2, radioButton3 })
            {
                if (radioButton.Checked == true)
                {
                    int index = -1;
                    switch (radioButton.Name)
                    {
                        case "radioButton1": index = 0; break;
                        case "radioButton2": index = 1; break;
                        case "radioButton3": index = 2; break;
                    }

                    if (index > -1) textBox1.Text = zmienna[index];
                }
            }
        }

 

możesz też np.

        public void liczenie()
        {
            string[] zmienna = new string[] { "rB1", "rB2", "rB3" };
            RadioButton[] radioButtons = { radioButton1, radioButton2, radioButton3 };

            for (int i=0; i<radioButtons.Length; i++)
            {
                if (radioButtons[i].Checked == true)
                {
                    textBox1.Text = zmienna[i];
                }
            }
        }

 

możesz też spróbować wykorzystać radioButton.TabIndex

        public void liczenie()
        {
            string[] zmienna = new string[] { "rB1", "rB2", "rB3" };

            foreach (RadioButton radioButton in new RadioButton[] { radioButton1, radioButton2, radioButton3 })
            {
                if (radioButton.Checked == true)
                {
                    textBox1.Text = zmienna[radioButton.TabIndex];
                }
            }
        }

 

możesz też tak jak już Tobie @Velta; sugerował użyć Linq, np.

        public void liczenie()
        {
            RadioButton radioButton = Controls.OfType<RadioButton>()
             .Where(control => control.Checked)
             .FirstOrDefault();

            if (radioButton != null)
            {
                string[] zmienna = new string[] { "rB1", "rB2", "rB3" };
                textBox1.Text = zmienna[radioButton.TabIndex];
            }
        }

 

możesz też spróbować wykorzystać radioButton.Tag

        public Form1()
        {
            InitializeComponent();

            string[] zmienna = new string[] { "rB1", "rB2", "rB3" };
            RadioButton[] radioButtons = { radioButton1, radioButton2, radioButton3 };

            for (int i=0; i<radioButtons.Length; i++)
                radioButtons[i].Tag = zmienna[i];
        }

        private void liczenie()
        {
            RadioButton radioButton = Controls.OfType<RadioButton>()
             .Where(control => control.Checked)
             .FirstOrDefault();

            if (radioButton != null)
                textBox1.Text = (string)radioButton.Tag;
        }

 

Answer 2

W kodzie z if-ami używasz zmiennych radioButton1, radioButton2, radioButton3 istniejącymi gdzieś na formularzu.
W kodzie ze switchem tworzysz nową kontrolkę, nie ustawiasz w niej pola Name i oczekujesz, że magicznie coś się wykona.