Prosta funkcja odnośnie losowania liczby zwraca wartość 'undefined'

Question

Witam radośnie wszelkich forumowiczów,

otóż piszę skrypt szyfrujący, w którym potrzebuję wylosować cyfrę. Funkcja posiada dwa typy, odwołanie do pierwszego ( drawNumber(1) ) sprawia, iż funkcja powinna zwrócić losową liczbę parzystą, natomiast gdy przekażemy do funkcji parametr o wartości 2, to zwracana wartość powinna być liczbą nieparzystą. Jednakże zwraca mi ta funkcja wartość 'undefined', czy mógłby ktoś pomóc mi znaleźć wyjście z labiryntu nieprawidłowości?

 

function drawNumber(type){
    var result;

    if(type == 1){
        result =  Math.floor(Math.random() * 22);
        if(result%2 == 0){
            return result;
        } else drawNumber(1);
    } else if(type == 2){
        result =  Math.floor(Math.random() * 22);
        if(result%2 == 1){
            return result;
        } else drawNumber(2);
    }
}

 

Comments

Dodaje, że zwracana wartość np. poprzez console.log(drawNumber(2)) nie zawsze zwraca 'undefined', czasami zwraca poprawnie.

Answer 1

Przy

else drawNumber(1);

oraz

else drawNumber(2);

brakuje return, przez co wynik zwrócony z rekurencyjnego wywołania jest "ignorowany" i całość zwraca undefined.

Comments

Kocham Cię stary! W życiu bym na to nie wpadł, z rekurencjami bardzo rzadko miałem okazję pracować, dzięki wielkie! Miłego!

Answer 2

Zamiast var daj let.

Aha i możesz dopisać corner case, kiedy rekurencja jakimś cudem wykona się, np. ponad 100 razy. if x over 100 then result 0

Comments

Tylko, że to dalej nie naprawia błędu (mowa o zmianie z var na let). Zastanawia mnie, dlaczego mówisz "jakimś cudem"? Może w niej jest gdzieś błąd? Może dlatego niekiedy działa, bo wtedy nie potrzebuje rekurencji, a jeśli chce uzyskać parzystą, a randomowa wychodzi nieparzysta i właśnie w tym momencie korzysta rekurencji żeby jeszcze raz spróbować, może wtedy właśnie coś się psuje i wyskakuje 'undefined'..? Już dorzucam tego corner case'a, chociaż nie wiem do końca po co, bo chcę, żeby ona się wykonywała tak długo, aż uzyska poprawny wynik...

---

W sumie tak, funkcja random nie powinna milion razy losować nieparzystych. Wspomniałem dla czujności.