Czy ten kod jest poprawny ?

Question

Witam

Pytanie tak jak w temacie, mianowicie chodzi o to czy to naciśnięcie przycisku poprawnie wyświetla losowe liczby z zakresu jaki został podany jest w dwóch textBoxach ? Z góry dziękuje za pomoc.

     private void button3_Click(object sender, EventArgs e)
        {
            int z1 = int.Parse(textBox1.Text);
            int z2 = int.Parse(textBox2.Text);
            Random r = new Random();                                                                 
             wynik = r.Next(z1,z2);
              z1 = int.Parse(textBox1.Text);
              z2 = int.Parse(textBox2.Text);
             z1.ToString(textBox1.Text);
             z2.ToString(textBox2.Text);
             listBox1.Items.Add(wynik);    
            listBox1.Text = wynik.ToString();           
        }

 

Answer 1

Nie wiem po co parsujesz ponownie z1 i z2 po wylosowaniu już liczby, w dodatku nie robisz z nimi potem nic sensowego (z1.ToString(x) zwraca string, który ginie, bo nic z nim nie robisz, podobnie następna instrukcja). Wywaliłbym linijki 7-10. Zastanawiam się co chciałeś zrobić w linijce nr 12. Jak dla mnie ta linijka również jest niepotrzebna  Ogólnie zamiast parsowania użyłbym albo Convert.ToInt32, albo metody int.TryParse. Poczytaj o różnicach pomiędzy nimi na msdnie.

Answer 2

Dzięki za pomoc zastosowałem się do twoich wskazówek a linijka 12 faktycznie jest nie potrzebna  ,kombinowałem jak dodać itemy do listboxa niestety nie zauważyłem linijki wyżej :)