Question

我正在完成一项任务，我必须在Javascript中使用FizzBuzz游戏。问题是我的循环在第一次迭代后停止并且只返回第一个值（4）。我可能对我的代码视而不见但我无法找到错误的位置。如果你能把我推向正确的方向，我会很高兴。提前致谢。此致，托马斯。

function fizzBuzz(start, stop) {

    for(var i = start; i <= stop; i++) {
        if (i % 3 == 0) {
            return "fizz";
        }else if( i % 5 == 0) {
            return "buzz";
        }else if(i % 15 == 0) {
            return "fizz buzz";
        }else {
            return i;
        }
    }
}

ANSWER = (fizzBuzz(4, 22));

新代码：

function fizzBuzz(start, stop) {

for(var i = start; i <= stop; i++) {
    if (i % 3 == 0 && i % 5 == 0) {
        document.write ("Fizz Buzz");
    }else if(i % 3 == 0) {
        document.write ("Fizz");
    }else if(i % 5 == 0) {
        document.write ("Buzz");
    }else {
        document.write(i);
    }
}
}

ANSWER = (fizzBuzz(4, 22));

返回：Answer = undefined

Answer 1

当您的代码遇到return语句时，将从整个函数返回给定的值。这会阻止for循环进一步迭代。

这些是您需要问自己的问题：

您希望fizzBuzz函数做什么？它应该在某处打印文本，还是应该返回值？
如果fizzBuzz应该返回一个值，您会期望它返回什么？一行文字？一次多行文本？

Answer 2

无论你的for循环中采用了什么分支，总会有一个return以退货值退出功能。因此，当您从4开始时，程序进入else分支并返回4.

您想要的是打印值而不是从函数返回。

另外，我可以在代码中看到逻辑错误。假设我15岁，可以被3和5整除。你的程序将进入i % 3分支并返回“fizz”而不是“fizz buzz”。您可能希望更改if语句和/或使用字符串连接。

希望我能提供帮助。 ;）

Answer 3

JavaScript中针对Fizzbuzz问题的最佳解决方案之一

const fizzbuzz = (start, stop) => {
  const arr = [];
  let str;
  for(let i=start; i<=stop; i++) {
    str = "";
    if(i%3===0) {
      str = "fizz";
    }
    if(i%5===0) {
      str += "buzz";
    }
    arr.push(str||i);
  }
  return arr;
}

console.log(fizzbuzz(0, 105));

Fizz Buzz返回值javascript

3 个答案: