0

我有一个任务,我必须用 Javascript 做 FizzBu​​zz 游戏。问题是我的循环在第一次迭代后停止并且只返回第一个值(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));

它返回:答案 = 未定义

4

3 回答 3

4

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

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

  • 你想让这个fizzBuzz函数做什么?它应该在某处打印文本,还是应该返回一个值?
  • 如果fizzBuzz应该返回一个值,您希望它返回什么?一行文字?一次多行文本?
于 2015-03-11T21:54:21.013 回答
1

无论在你的 for 循环中采用什么分支,总会有return一个以返回值退出函数的分支。所以,当你从 4 开始时,程序进入 else 分支并返回 4。

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

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

希望我能帮上忙。;)

于 2015-03-11T22:03:38.977 回答
0

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));
于 2019-09-18T05:34:48.320 回答