Czy mój sposób na losowanie liczb jest dobry?

Question

Witam, mam następujący kod:

var luck = 5; //nadanie 5% szans na trafienie
var lucky_number = Math.floor(Math.random()*100)+1; //losowanie numeru
if(lucky_number<=luck)
{
	//losowanie się udało, lucky_number trafiło w luck
}
else
{
	//losowanie się nie udało, nie trafiono w luck
}

Czy jest to dobry sposób na tworzenie losowości czy jest coś lepszego? Np. jakaś wbudowana funkcja czy coś takiego.

Mam również inne pytanie: czy używając

var lucky_number = Math.floor(Math.random()*100)+1;

może się zdarzyć tak, że lucky_number po zaokrągleniu wyniesie 0? Ponieważ wtedy szansa jest większa i trzeba usunąć to +1.

Answer 1

Chciałem dodać do tego co napisali @Us i @Borisoveck, że często stosowany jest np. taki zapis  

function getRandomNumber(min, max) {
  return Math.floor(Math.random() * (max - min + 1)) + min;
}

const lucky_number = getRandomNumber(1, 100);

 

Answer 2

Cześć,

To dobry sposób na tworzenie losowości i właśnie użyłeś wbudowanej funkcji. Możesz napisać własną funkcję, tylko po co?

Co do drugiego pytania, w Twoim przypadku nie wyniesie 0, bo po to przecież dałeś "+1" po nawiasie, aby zaczynało się od 1.

Answer 3

Hmmm... Javascriptem się nie zajmuje ale patrząc na ten kod twierdze że powinien on wygenerować liczby od 1 do 100. Gdybyś nie dał +1 to byłoby od 0 do 99. Wsadziłem ten kod do interpretera i powiem tyle: Ani razu nawet zamieniając 100 na 5 nie wylosowało 0. Myślę że moja odpowiedź cię usatysfakcjonowała, życzę miłego dnia. 

 

Comments

A no tak, zapomniałem, że to +1 odnosi się do wylosowanej liczby, a nie do zakresu. Dziękuję i również życzę miłego dnia.