optymalizacja zagniezdzonych ifow c#

Question

Witam. Nie wiem co jest w tej petli blednie zrobione. Wg kompilatora jest niesprecyzowany wyjatek. Jakies pomysly? Kod wykomentowany to oryginal a ten w forze jest proba optymalizacji. Wykomentowany dziala. Comboboxes[] to tablica w ktorej sa wszystkie comboboxy

//if (comboBox1.Text == "Cheese" || comboBox1.Text == "Vegetable" || comboBox1.Text == "Meat")
            //{
            //    if (comboBox2.Text == "Cheese" || comboBox2.Text == "Vegetable" || comboBox2.Text == "Meat")
            //    {
            //        if (comboBox3.Text == "Cheese" || comboBox3.Text == "Vegetable" || comboBox3.Text == "Meat")
            //        {
            //            return true;
            //        }
            //        else { comboBox3.Text = "Wrong input"; return false; }
            //    }
            //    else { comboBox2.Text = "Wrong input"; return false; }
            //}
            //else { comboBox1.Text = "Wrong input"; return false; }
            int isvalid = 0;
            for (int i = 0; i < 3; i++)
            {
                if (ComboBoxes[i].Text == "Cheese" || ComboBoxes[i].Text == "Vegetable" || ComboBoxes[i].Text == "Meat")
                {
                    isvalid++;
                }
                else { ComboBoxes[i].Text = "Wrong input"; }
            }
            if (isvalid == 3)
            { return true; }
            else { return false; }

 

Comments

Zastosuj try i catch i dowiesz się co to za błąd. Prawdopodobnie wychodzisz poza zakres tablicy/listy lub odwołujesz się do null.

---

yyy a cos wiecej? xD

---

try{
int isvalid = 0;
            for (int i = 0; i < 3; i++)
            {
                if (ComboBoxes[i].Text == "Cheese" || ComboBoxes[i].Text == "Vegetable" || ComboBoxes[i].Text == "Meat")
                {
                    isvalid++;
                }
                else { ComboBoxes[i].Text = "Wrong input"; }
            }
            if (isvalid == 3)
            { return true; }
            else { return false; }
}
catch(Exception e){
Console.WriteLine(e);
Console.ReadKey();
}

Jeżeli nie korzystasz z konsoli to wyświetl sobie w postaci messagebox'a

---

MessageBox.Show(e) nie moze przekonwertowac tego na string.

---

e.ToString(); podstawy c#... 

Ale lepiej w MessageBox e.Message. 

Answer 1

Po prostu użyj debuggera do sprawdzenia błędu. Załóż break point-y w kilku miejscach i uruchom program w trybie debug i zobacz w którym miejscu wyrzuca aplikację.

Równie dobrze te porównywania stringów mógłbyś wrzucić do tablicy stringów i sprawdzać czy tekst z comboBoxów jest w kolekcji zamiast rozbudowywać tak cały warunek. Np coś takiego:

 int isvalid = 0;
string [] values= {"Cheese","Vegetable","Meat"};

            for (int i = 0; i < ComboBoxes.Length; i++)
            {
                if (values.Contains(ComboBoxes[i].Text))
                {
                    isvalid++;
                }
                else { ComboBoxes[i].Text = "Wrong input"; }
            }

            if (isvalid == 3)
            { return true; }
            else { return false; }

Pisze to z głowy więc nie gwarantuje że wszystko będzie poprawnie :)

Zamiast pętli for mógłbyś również użyć pętli foreach wtedy od razu masz zapewnione sprawdzanie wielkości kolekcji.

Comments

Dobry pomysl z ta tablica stringow