JS: получить x количество случайных чисел внутри интервала с циклом while

Question

Я работаю над карточной игрой с шестью доступными костюмами, но может быть 2-6 игроков, и мне нужно только, чтобы количество костюмов соответствовало количеству игроки.

let players = 4  
const suits = [1, 2, 3, 4, 5, 6] 

Конечно, я хочу, чтобы они вышли случайным образом.

я придумал следующее решение:

export function getRandomSuits(nrOfPlayers) { 
    const rangeMin = 1; 
    const rangeMax = 6; 
    const amount = nrOfPlayers; 
    let nums = []; 

    let getRandomNumber = function() { 
     return Math.floor((Math.random() * rangeMax) + rangeMin); 
    } 

    while (nums.length < amount) { 
     let nr = getRandomNumber(); 
     var numberExists = _.indexOf(nums, nr); 
     console.log(numberExists); 
     if(numberExists < 0) { 
      nums.push(nr); 
     } 
    } 

    return nums; 
} 

Это было некоторое время, так как я использовал «в то время как петля» штуковина, так что я не чувствую себя очень комфортно с ним.

Мои вопросы:

1. Это хорошая практика, или есть лучшее решение?
2. Есть ли какие-либо результаты или другие практические причины, почему это решение плохое?
3. Я переусердствую?

Answer 1

1. На мой взгляд, я не вижу никакой необходимости в функции getRandomNumber(). Тем не менее, это зависит от предпочтений.
   я бы с:

   export function getRandomSuits(nrOfPlayers) { 
       const rangeMin = 1; 
       const rangeMax = 6; 
       let nums = []; 
   
       while (nums.length < nrOfPlayers) { 
        let nr = Math.floor((Math.random() * rangeMax) + rangeMin); 
        var numberExists = _.indexOf(nums, nr); 
        if(numberExists < 0) { 
         nums.push(nr); 
        } 
       } 
   
       return nums; 
   } 
   

2. не обязательно. Просто вопрос чистоты и предпочтения.

3. Может быть? :-)

Второй метод (Немного лучше) с временным массивом:

export function getRandomSuits(nrOfPlayers) { 

    const rangeMin = 1; 
    const rangeMax = 6; 
    var tempNum = []; 
    for(i = 0; i <= rangeMax - rangeMin; i++){ 
     tempNum[i] = rangeMin + i; 
    } 
    let nums = []; 
    while (nums.length < nrOfPlayers) { 
     let index = Math.floor(Math.random() * tempNum.length); 
     var numberExists = _.indexOf(nums, tempNum[index]); 
     if(numberExists < 0) { 
      nums.push(tempNum[index]); 
      tempNum.splice(tempNum, index)); 
     } 
    } 

    return nums; 
}