JS:在具有时间循环的区间内获取x个随机数

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;
}

我已经有一段时间没有使用“been循环”这个东西了，所以我对它感到不太舒服。

我的问题是：

1. 这是一个好的做法，还是有更好的解决办法？
2. 是否有任何性能或其他实际原因，这个解决方案是坏的？
3. 我是不是想得太多了？

Answer 1

1. 在我看来，我认为不需要函数getRandomNumber()。尽管如此，这仍取决于人们的喜好。 我会说： 导出函数getRandomSuits(nrOfPlayers) { const rangeMin = 1；const rangeMax = 6；设num= []；while (nums.length < nrOfPlayers) { let ( nr = Math.floor((Math.random() * rangeMax) + rangeMin)；var numberExists =_.indexOf(num，nr)；if(numberExists < 0) { nums.push(nr)；}}返回num；}
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;
}