Question

我正在尝试执行一项功能，在我的console.log中打印1-27之间的数字。

当一个数字除以3时，它应该用＆＃34; Fizz＆＃34;
替换数字如果一个数字可以除以5，请将其替换为＆＃34; Buzz＆＃34; 如果数字可以除以3和5，请将其替换为＆＃34; Fizzbuzz＆＃34;

参考：http://en.wikipedia.org/wiki/Fizz_buzz）

这是我的代码：

 var fizzbuzz = function(start,stop) {
    for (var x=1;x <= stop; x++)
        var string =',';
    if (x%3 == 0) {
            string += 'Fizz';
    }
    if (x%5 ==  0){
        string += 'Buzz';
    }
    if (x%5 && x%3){
        string += 'Fizzbuzz';
    }
    return string;
};

Console.log正在给我＆＃34;，＆＃34;而且我不确定自己做错了什么。

只是澄清一下。我希望我的答案可以打印1,2,Fizz,4,Buzz,Fizz,7,8,Fizz,Buzz,11,Fizz,13,14,Fizz Buzz,16,17,Fizz,19,Buzz,Fizz,22,23,Fizz,Buzz,26,Fizz等等，具体取决于＆＃39;停止＆＃39;在If语句中。

Answer 1

Valentins评论是正确的，你需要在你的循环周围添加括号。但是，您还要在循环的每次迭代中重新定义字符串var。

最后一个if也会使输出有点不对，例如15会打到所有3个语句并打印FizzBuzzFizzBuzz

所以选择像

这样的东西

var fizzbuzz = function(start,stop) {
  var string = '';
  var addComma = false;
  for (var x=1;x <= stop; x++){
    addComma = false;
    if (x%3 == 0) {
        string += 'Fizz';
        addComma = true;
    }
    if (x%5 ==  0){
        string += 'Buzz';
        addComma = true;
    }
    if(addComma && x!== stop){
        string+=','
    }
  }
  return string;
};

这不是跟踪添加逗号的位置的最佳方式，但它可以完成任务。

Answer 2

for(let x = 0; x <=30; x++) {
if (x % 15 === 0) {
    console.log('Fizzbuzz')
} else if (x % 5 === 0) {
    console.log('Buzz')
} else if (x % 3 === 0) {
    console.log('Fizz')
} else {
    console.log(x)
}

}

我是这样处理的

Answer 3

您需要更正for循环

你有

for (var x=1;x <= stop; x++)
    var string =',';

执行到x <= stop。

如果你想执行一行代码，Javascript允许你避免使用括号。

if (a===true)
    alert(a); // This is executed when a === true

alert(b); // This is always executed no matter what a is

这里的缩进是为了说明一点，但if语句会执行所有内容直到第一个分号。

另一方面，如果您想要a === true执行多行代码，您会选择使用大括号

// Alert a and alert b are only executed if a is true
if (a===true) {
    alert(a);
    alert(b);
}

if语句将执行大括号中的所有内容。

重要的是要注意return停止执行并退出函数。一旦到达return语句，循环就会退出。这就是你应该在for循环之后返回整个字符串的原因。

这是一个更好的实施，但你一定要自己尝试实施

var fizzbuzz = function(start,stop) {
    var string = '';

    for (var x=1;x <= stop; x++) {
        var status = x.toString(); //Each time the loop executes a new variable `status`is created and set to the value `x` for that loop.

        // x is checked as to whether it is divisible by 3 or 5 or both, if it is divisible its status is set to a that value
        if (x%3 === 0) {
            status = 'Fizz';
        }
        if (x%5 ===  0){
            status = 'Buzz';
        }
        if (x%5 === 0 && x%3 === 0){
            status = 'Fizzbuzz';
        }

        string += status; // Append status to the end

        if (x !== stop){ // If x is not equal to the value of stop add a comma
            string += ',';
        }
    }

    return string; //This returns the string value which has had statuses and commas appended to it.
};

Answer 4

这有多个问题：（1）构造

for (var x=1;x <= 10; x++)
    statement;
otherstatement;

在执行其他语句之前执行语句10次。没有大括号，Javascript假定下一个语句是for循环的内容;

（2）在每个循环中重新定义字符串变量，除去前面的版本，因此return语句只打印出字符串的最后一个值。

（3）fizzBuzz if语句的逻辑错误。如果对于除以15的语句执行此操作，则会执行所有三个语句。因此，第三个iff语句完全是多余的。

解决方案如下：

var fizzBuzz = function(x){
    if(x%15==0){
         return "Fizzbuzz";
    }
    if(x%3==0){
          return "Fizz";
    }
    if(x%5==0){
          return "Buzz";
    }
    return x;
    };

var mainFunction = function(start,stop){
    var str="";
    for(var i=start; i < stop; i++){
          str += fizzBuzz(i) + ", ";
    }
       return str;
    };

请注意，仅当此版本要求您打印Fizzbuzz而不是FizzBuzz时，才需要％15的第三个if语句。

Fizzbuzz游戏与for循环

4 个答案: