Administrace komentářů

Před rekurzivním voláním random(min, max) chybí return. Od toho se celý problém odvíjí. Hodnota vrácená rekurzivním voláním se tím pádem zahodí a nikam dál se nevrací.

Vůbec nepíšeš, co přesně má ta funkce dělat, jaké vstupy má podporovat a jaké výstupy má generovat. Mají to být všechno integery? Libovolná čísla? Něco jiného? Hint: Co když bude min mít (zhruba) hodnotu pí? Na takový dotaz se pak odpovídá těžko.

Budu tedy dál předpokládat, že argumenty mají být integery. (To by se ale mělo v prakticky použitelném kódu ověřit, případně specifikovat, co se má stát, pokud to nebudou integery.) Takže první iterace:

function random(min, max) {
  var num = Math.floor(Math.random() * (max - min + 1)) + min;
  return num == 0 ? random(min, max) : num;
}

To^^^ je ale pořád ještě fujtajbl, protože dobu běhu té funkce a počet úrovní rekurze je potřeba deterministicky omezit. Sice je krajně nepravděpodobné, že by hloubka rekurze překročila povolenou hloubku zásobníku, ale spoléhat se tímto způsobem na náhodu je ošklivé. Vznikají tak chyby, kvůli kterým pak software jednou za rok za podivných okolností zařve. Takové chyby jsou těžko reprodukovatelné a těžko odhalitelné.

Takový jednoduchý nápad: Co takhle zjistit, jestli zadaný rozsah čísel obsahuje nulu, a pokud ano, jednoduše se jí při generování náhodného čísla vyhnout?

function random(min, max) {
  if (min > max) [min, max] = [max, min];
  var num = function(range) {
    return Math.floor(Math.random() * range) + min;
  };
  if (min > 0 || max < 0) {
    return num(max - min + 1);
  } else {
    var result = num(max - min);
    return result >= 0 ? result + 1 : result;
  }
}