函数和 if else 语句不起作用

Question

我只是在学习如何编码，我现在正在使用 CodeWars 练习基础知识。由于 CodeWars 允许您查看解决方案，因此我查看了一些解决方案以寻求指导，这很有帮助。我已使用此站点作为指导，但无法弄清楚为什么我的功能不起作用。它是用 javascript 编写的。它只输出 []。这是问题、代码和输出（按以下顺序）：

问题

编写一个方法，它将获取一个整数数组作为参数，并处理该数组中的每个数字。 返回一个新数组，处理输入数组的每个数字，如下所示： 如果数字有整数平方根，则取这个，否则取平方。 [4,3,9,7,2,1] -> [2,9,3,49,4,1]

代码

function squareOrSquareRoot(array) {

    var newValues = []; // new array for new values

    for (i = 0; i < array.length; i++){ // for loop to look through the values

        var initial = array[i]; // extracting one value from the array to evaluate
        var sqrt = Math.sqrt(initial); // finding the square root of initial
        if (Number.isInteger(sqrt) == 'true'){ // determining if sqrt is an integer 
                                            // and if so .....
            newValues.push[sqrt];
        } // .... adding sqrt to the newValues array
        else if (Number.isInteger(sqrt) == 'false') { // determining if sqrt is not 
                                                      // an integer
            newValues.push[initial*initial];  // then multiplying initial by itself 
                                           //and adding to newValues
        }
    }
    return newArray; // returning newValues onto the screen
}

输出

Expected: '[2, 9, 3, 49, 4, 1]', instead got: '[]'

Expected: '[10, 10201, 25, 25, 1, 1]', instead got: '[]'

Expected: '[1, 4, 9, 2, 25, 36]', instead got: '[]'

Answer 1

你的情况有问题。改变

Number.isInteger(sqrt) == 'true'

到

Number.isInteger(sqrt) == true

Number.isInteger 返回布尔值而不是字符串。第二个 else if 也是多余的，如果 isInteger 返回 false 则只需执行 else 部分而不是再次检查。 最后你需要返回newValues 而不是newArray。希望对您有所帮助。

Answer 2

你做错了很多事情。

1. 您没有返回正确的数组。您应该返回 newValues 而不是 newArray。
2. 您不应在true 或false 中使用引号
3. 不需要使用 if/else if 来判断 true/false。改用 if/else

这是可行的解决方案：

function squareOrSquareRoot(array) {

    var newValues = []; 
    for(var i = 0 ; i<array.length ;i++) {
        Number.isInteger(Math.sqrt(array[i]))?newValues.push(Math.sqrt(array[i])):newValues.push(array[i]*array[i]);
    }
    return newValues;
}

var a = [3,4,5,9,7,16,36,11];
console.log('Input : ' + a);
console.log('Output: ' + squareOrSquareRoot(a));

输入：[3,4,5,9,7,16,36,11]

输出：[9,2,25,3,49,4,6,121]

如果这个三元表达式让你感到困惑，这里是 if/else 语句的示例：

function squareOrSquareRoot(array) {

    var newValues = []; 
    for(var i = 0 ; i<array.length ;i++) {
        var initial = array[i];
        var sqrt = Math.sqrt(initial);
        if(Number.isInteger(sqrt)) 
            newValues.push(sqrt);
        else
            newValues.push(array[i]*array[i]);
    }
    return newValues;
}

var a = [3,4,5,9,7,16,36,11];
console.log('Input : ' + a);
console.log('Output: ' + squareOrSquareRoot(a));

Answer 3

最好不要使用 for 循环，而是完全适用 array.map。

  function squareOrSquareRoot(array) {
    return array.map(function(int) {
      var sqrt = Math.sqrt(int);
      return Number.isInteger(sqrt) ? sqrt : int * int;
    })
  }

Array.map 将每个项目映射到给定的函数，并在最后返回一个新数组。