Co jest złego w instrukcji switch? Błąd dotyczy nieprzypisanej wartości liczbaPierwiastków.

Question

namespace x0_z_użycem_switch
{
    class Program
    {
        static void Main(string[] args)
        {
            double a, b, c, delta, x1, x2;
            byte liczbaPierwiastków;
            Console.WriteLine("Program obliczy miejsca zerowe funkcji ax^2+bx+c");
            Console.Write("Podaj wartrtość a: ");
            a = double.Parse(Console.ReadLine());
            if (a == 0)
                Console.WriteLine("Współczynnik a nie może być równy 0! Naciśnij ENTER, aby zakończyć działanime programu.");
            else
            {
                Console.Write("Podaj wartość b: ");
                b = double.Parse(Console.ReadLine());
                Console.Write("Podaj wartość c: ");
                c = double.Parse(Console.ReadLine());

                delta = b * b - 4 * a * c;
                if (delta < 0) liczbaPierwiastków = 0;
                if (delta == 0) liczbaPierwiastków = 1;
                if (delta > 0) liczbaPierwiastków = 2;

                switch (liczbaPierwiastków)
                {
                    case 0 :
                        {

                        }
                    
                        break;
                }
  
            }
        }
    }
}

Answer 1

A nie lepszym pomysłem jest typ int zamiast typ byte? Warto dodać że zwykle nie są potrzebne kamerki po case.

Comments

Błąd nadal jest aktywny.

Wartość zmiennej i tak jest z zakresu 0 - 2, więc myślę, ze nie ma to większego znaczenie chociaż zminiłem na int, spróbowałem usunąć klamerki. Błąd nadal się pojawia.

---

Napisz co to za błąd.

Answer 2

            byte liczbaPierwiastków;

zamień na:

            byte liczbaPierwiastków = 0;

Tak zrobiłem i program się uruchomił. Switch jest OK, wystarczy dodać dwa pozosrtałe warunki.

Comments

A rzeczywiście, teraz działa. Dziękuję :)

A mogę wiedzieć dlaczego przy nieprzypisanej wartości wyskakiwał błąd?

---

Tego nie wiem, widocznie tak już jest. Zmieniłem tym zmiennej na int, ale też musi być przypisany :) Widocznie na etapie kompilacji kompilator musi mieć pewność, że switch się nie posypie.

---

Nie dawało mi to spokoju więc musiałem sprawdzić mimo późnej pory:)

Możesz też zamiast przypisywać wartość na początku do 

byte liczbaPierwiastków;

 

dokonać innej zmiany. Zamiast:

if (delta < 0) liczbaPierwiastków = 0;
if (delta == 0) liczbaPierwiastków = 1;
if (delta > 0) liczbaPierwiastków = 2;

napisać:

if (delta < 0) liczbaPierwiastków = 0;
else if (delta == 0) liczbaPierwiastków = 1;
else liczbaPierwiastków = 2;

i też się skompiluje.

---

Tak, rzeczywiście działa. Dzięki! :)