阶乘/组合在不应该的情况下产生 NaN

Question

我正在求解 y 选择 p 组合。我是 JavaScript 初学者。

我给出了阶乘函数的定义： 它的目的是执行 x * (x-1) * (x-2) ... * 1。

function factorial(z)
{
    var product = z;
    if (z == 0)
    {
        return 1;
    }
    else
    {
        for (var m = 1; m < z; m++)
        {
            product = m * product;
            return product;
        }
    }
}

然后我有了这个函数，它使用阶乘函数。

function placeBoxes()
{
    var ids = 0;
    for (var y = 0; y < (numOfRows-1); y++)
    {
        for (var p = 0; p < (y+1); p++, ids++)
        {
            x = document.createElement('div');
            x.className = "box";
            x.id = ids;
            x.innerHTML = (factorial(y))/((factorial(y-p))*(factorial(p)));
            document.getElementById('holdsBoxes').appendChild(x);
        }
    }
}

它应该选择组合。出于实验目的，numOfRows 实际上在 5 到 30 之间。0C0 1C0 1C1 2C0 2C1 2C2 等等... 这相当于 1, 1, 1, 1, 2, 1 等等...

有谁知道我做错了什么？我得到 NaN 而不是第二个、第三个、第五个、第八个、第九个和许多其他值的值。

编辑：谢谢大家！问题解决了。阶乘函数被搞乱了。

Answer 1

更改

    for (var m = 1; m < z; m++)
    {
        product = m * product;
        return product;
    }

为

    for (var m = 1; m < z; m++)
    {
        product = m * product;
    }
    return product;

您永远不会完成循环

Answer 2

for (var m = 1; m < z; m++)
{
    product = m * product;
    return product;
}

您在循环内返回的product。你确定吗？您不想在循环之外返回 product 吗？

这也是您的 NaN 的原因。如果 z 是 1 呢？那么m < z 是 m 1 这总是错误的。所以这个循环体永远不会被调用。并且 return 语句永远不会被调用。

因此该函数返回未定义。并且 undefined * 1 === NaN

Answer 3

我认为阶乘函数是不正确的。尝试将其更改为这样：

function factorial(z)
{
    var product = z;
    if (z == 0)
    {
        return 1;
    }
    else
    {
        for (var m = 1; m < z; m++)
        {
            product = product*(product - m);
        }
        return product;
    }
}