Unity - skrypt na losową kolejność. Dlaczego działa?

Question

Witam,

zrobiłem funkcje wywoływaną na starcie, która przypisuje do ośmiu zmiennych losową liczbę od 1 do 8, ale ta liczba nie może się powtórzyć. W miejscu gdzie teraz są dziewiątki na początku było 8. Po odpaleniu gry w Unity się zawieszało. Myślałem że to przez to że ta ostatnia liczba nie chce się wylosować jako odpowiednia czy coś. Zmieniłem 8 na 9 aby sprawdzić czy to w czymś pomoże. I teraz działa tak jakby tam cały czas było 8. Wszystko działa tak jak miało na początku i dziewiątka nie wylosowuje się ani razu, mimo tego że powinna być tak samo często jak wszystkie inne liczby.  Ktoś jest w stanie mi wytłumaczyć dlaczego tak się dzieje?

    void Kolejnosc()
    {

        Losowanko = Random.Range(1, 9);
        JanuchaOrder = Losowanko;

        Losowanko = Random.Range(1, 9);

        for (int i = 0; i == 0;)
        {
            if (JanuchaOrder != Losowanko)
            {
                KlawiszOrder = Losowanko;
                i = 1;
                Losowanko = Random.Range(1, 9);
            }
            else
            {
                Losowanko = Random.Range(1, 9);
            }
        }

        for (int i = 0; i == 0;)
        {
            if (JanuchaOrder != Losowanko && KlawiszOrder != Losowanko)
            {
                KakOrder = Losowanko;
                i = 1;
                Losowanko = Random.Range(1, 9);
            }
            else
            {
                Losowanko = Random.Range(1, 9);
            }
        }

        for (int i = 0; i == 0;)
        {
            if (JanuchaOrder != Losowanko && KlawiszOrder != Losowanko && KakOrder != Losowanko)
            {
                KiedroOrder = Losowanko;
                i = 1;
                Losowanko = Random.Range(1, 9);
            }
            else
            {
                Losowanko = Random.Range(1, 9);
            }
        }

        for (int i = 0; i == 0;)
        {
            if (JanuchaOrder != Losowanko && KlawiszOrder != Losowanko && KakOrder != Losowanko && KiedroOrder != Losowanko)
            {
                Opponent1Order = Losowanko;
                i = 1;
                Losowanko = Random.Range(1, 9);
            }
            else
            {
                Losowanko = Random.Range(1, 9);
            }
        }

        for (int i = 0; i == 0;)
        {
            if (JanuchaOrder != Losowanko && KlawiszOrder != Losowanko && KakOrder != Losowanko && KiedroOrder != Losowanko && Opponent1Order != Losowanko)
            {
                Opponent2Order = Losowanko;
                i = 1;
                Losowanko = Random.Range(1, 9);
            }
            else
            {
                Losowanko = Random.Range(1, 9);
            }
        }

        for (int i = 0; i == 0;)
        {
            if (JanuchaOrder != Losowanko && KlawiszOrder != Losowanko && KakOrder != Losowanko && KiedroOrder != Losowanko && Opponent1Order != Losowanko && Opponent2Order != Losowanko)
            {
                Opponent3Order = Losowanko;
                i = 1;
                Losowanko = Random.Range(1, 9);
            }
            else
            {
                Losowanko = Random.Range(1, 9);
            }
        }

        for (int i = 0; i == 0;)
        {
            if (JanuchaOrder != Losowanko && KlawiszOrder != Losowanko && KakOrder != Losowanko && KiedroOrder != Losowanko && Opponent1Order != Losowanko && Opponent2Order != Losowanko && Opponent3Order != Losowanko)
            {
                Opponent4Order = Losowanko;
                i = 1;
                Losowanko = Random.Range(1, 9);
            }
            else
            {
                Losowanko = Random.Range(1, 9);
            }
        }
    }

 

Answer 1

Wszystko działa tak jak miało na początku i dziewiątka nie wylosowuje się ani razu, mimo tego że powinna być tak samo często jak wszystkie inne liczby.

Z dokumentacji:

Return a random integer number between min [inclusive] and max [exclusive]

Boczne uwagi:

for (int i = 0; i == 0;)

Co to wszystko jest...?

I jeśli chcesz mieć 8 losowych liczb bez powtórzeń z zakresu 1..8, to... nie prościej będzie zrobić zbiór [1, 2, 3, 4, 5, 6, 7, 8] i go przemieszać?

Comments

Dobra, teraz wszytko jasne.

Co do tego zbioru to nie wiem w jaki sposób mam go przemieszać. Jestem początkującym programistą.

---

Co do tego zbioru to nie wiem w jaki sposób mam go przemieszać. Jestem początkującym programistą.

Jasne. Niestety, nie jest to wbudowane :/ Osobiście bym na przykład użył: https://stackoverflow.com/questions/273313/randomize-a-listt

Answer 2

dziewiątka nie wylosowuje się ani razu, mimo tego że powinna być tak samo często jak wszystkie inne liczby

 No własnie nie powinno jej być w ogóle :P 

Random.Range losuje liczbę od x do y-1 to znaczy w twoim przypadku Random.Range losuje od 1 do 9-1

czyli górna granica losowania musi być o 1 większa, dlatego 9 nigdy się nie wylosuje.

Jeżeli np. losował być liczby typu float (Random.Range(1.0,9.0) to 9.0 mogło by się pojawić, natomiast dla liczb typu int największa wartość się nie losuje.