在 JS 中推送到数组时遇到问题

Question

下面只是我的代码的一部分，但我知道这是有问题的，因为我无法让它返回除“未定义”之外的任何值。我已经经历了几个小时，无法弄清楚。

我希望能够输入一个数字并将其因子推送到一个数组中。我已经通过警告数组中的第一项来测试它，但我什么也没得到。我敢肯定这很容易，但我就是想不通。代码如下：

    var numberInQuestion = prompt("Of what number are you wanting to find the largest        prime factor?");

    //determine factors and push to array for later use
    var factorsArray = [];
    function factors(numberInQuestion){
        for(var i = 2; i < numberInQuestion-1; i++){
            if(numberInQuestion % i === 0){
                return factorsArray.push[i];
            } else {
                continue;
            }
        }
    };
    factors(numberInQuestion);
    alert(factorsArray[0]);

感谢您的帮助！

Answer 1

• 你只能返回一个值
• 您必须使用()，而不是[] 来调用push
• factorsArray 应该是 factors 的本地（将定义放在函数中）
• else { continue; } 没用

这是完全更正的代码：

var numberInQuestion = prompt("Of what number are you wanting to find the factors of?");

//determine factors
function factors(numberInQuestion){
    var factorsArray = []; // make it local
    for (var i = 2; i < numberInQuestion-1; i++){
        if(numberInQuestion % i === 0){
            factorsArray.push(i); // use (), and don't return here
        } // no need for else { continue; } because it's a loop anyway
    }
    return factorsArray; // return at the end
};
var result = factors(numberInQuestion); // assign the result to a variable
alert(result);

Here's a JSFiddle.

Answer 2

您的推送语法有误。推送的正确语法是 -

factorsArray.push(i);

在找到第一个除数后立即从函数返回不会给你完整的列表。找出所有除数后，您可能想返回。

考虑到以上所有，你应该重写你的函数如下 -

function factors(numberInQuestion){

    for(var i = 2; i < numberInQuestion - 1; i++){
        if(numberInQuestion % i === 0) {
            factorsArray.push(i);
        }
    }
}

你会没事的。

Answer 3

您已经对此进行了编码，因此当您找到第一个因素时，您的函数会立即返回。只需删除该语句中的 return 关键字即可。 （“返回”在 JavaScript 和其他类似语言中的意思是立即退出函数并从调用函数的位置恢复。）

哦，还有，你用括号而不是方括号来调用函数（如.push()）。

Answer 4

推入数组时函数不应返回。执行循环后返回数组。 else 子句也是不必要的。

var numberInQuestion = prompt("Of what number are you wanting to find the largest        prime factor?");

function factors(numberInQuestion){
    var factorsArray = [];
    for(var i = 2; i < numberInQuestion-1; i++){
        if(numberInQuestion % i === 0 && isPrime(i)){
            factorsArray.push(i);
        }
    }
    return factorsArray; 
};
var factors = factors(numberInQuestion);
alert(factors[factors.length-1]);

//From: http://stackoverflow.com/questions/11966520/how-to-find-prime-numbers
function isPrime (n)
{
    if (n < 2) return false;
    var q = Math.sqrt (n);

    for (var i = 2; i <= q; i++)
    {
        if (n % i == 0)
        {
            return false;
        }
    }

    return true;
}

鉴于示例的目的，必须考虑两项

代码无法确定数字是否实际上是素数。代码将返回可能的最小因子，因为循环从 2 开始并递增，然后返回数组中的第一个元素。最大的因素实际上是数组中的最后一个元素。我已经更正了这个例子以找到最大的素数。你可以通过这个小提琴来测试它：http://jsfiddle.net/whKGB/1/