我的 FizzBuzz 尝试[关闭]答案

【问题标题】：My FizzBuzz attempt [closed]我的 FizzBuzz 尝试[关闭]
【发布时间】：2015-02-13 07:27:29
【问题描述】：

这是我的 FizzBuzz 尝试

for (i=1;i<=20;i++) {
    if (i % 3 == 0 && i % 5 !== 0) {
        console.log ("Fizz");
    }
    else if (i % 3 ==0 && i % 5 == 0) {
        console.log ("FizzBuzz");
    }
    else if (i % 5 ==0 && i % 3 !== 0){
        console.log ("Buzz");
    }
    else {
        console.log (i);
    }
    
};

Codecademy 认为它是正确的，但我想确保它确实如此。非常感谢您提前。附言我真诚地希望这次我的问题不是太模糊或抽象或离题:)

【问题讨论】：

条件else if (i % 5 ==0 && i % 3 !== 0)是多余的，else if (i % 5 == 0)就够了
如果您的代码正在运行，那么它就是题外话。我认为对于codereview.SE来说也太模糊了
如果问题要求关于工作代码的非特定 cmets，它可能是题外话，更适合 codereview.stackexchange.com

标签： javascript fizzbuzz

【解决方案1】：

这是正确的，但很冗长。

让我们稍微分解一下任务。你有两个因素：它是 3 的倍数，还是 5 的倍数？当然，它可以是两种情况，也可以两者都不是，总共有四种情况。像这样：

       | Not x5 |    x5
-------+--------+---------
Not x3 | number |   Buzz
  x3   |  Fizz  | FizzBuzz

您可以非常简单地在代码中表示：

function getFizzBuzz(n) {
    if(n%5) { // not x5
        if(n%3) // not x3 either
            return n;
        else // x3
            return 'Fizz';
    }
    else {
        if(n%3)
            return 'Buzz';
        else
            return 'FizzBuzz';
    }
}

然后做你的循环：

for(i=1; i<=20; i++) console.log(getFizzBuzz(i));

如果你真的想要，你可以让你的代码超级紧凑。当然，可读性要差得多！

function getFizzBuzz(n) {
    return n%5?(n%3?n:'Fizz'):(n%3?'Buzz':'FizzBuzz');
}

有很多方法可以完成这项任务。不同之处在于其他程序员以后阅读您的代码的难易程度。

【讨论】：

我认为使用 cmets 表明这不是一个简单的解决方案。如果它简单易懂，它们就没有必要了。它简约而巧妙，但并不那么简单。使用一些 && 和仅四个 if/else 语句而不是嵌套的 if/else 可能有助于清晰。

【解决方案2】：

我认为它可以有多种解决方案，正如 Niet 指出的那样，问题是让它更具可读性。

这是我的版本，Working Fiddle

代码：

for (var a = 1; a <= 20; a++) {
    var log = (a % 3 == 0 && a % 5 == 0) ? 'fizzbuzz' : null;
    if (!log) log = a % 3 == 0 ? 'fizz' : null;
    if (!log) log = a % 5 == 0 ? 'buzz' : null;
    if (log) console.log(log);
    else console.log(a);
}

【讨论】：

我个人觉得这不是特别可读...

【解决方案3】：

我认为您的解决方案很好。它的可读性也很好 - 这是编程中的一个大交易。

我建议的唯一方法是从 3 和 5 的倍数检查开始（因此您可以删除非多重检查），使用严格的相等比较运算符 (===)，并稍微更改您的代码样式 -同样，可读性巨大。（我建议的一些风格改变可能不被推荐，这取决于你最终为谁工作或与谁一起工作，所以对它们持保留态度。）

以下是代码形式的建议：

for (i = 1; i <= 20; i++) {
    if (i % 3 === 0 && i % 5 === 0)
        console.log("FizzBuzz")
    else if (i % 3 === 0)
        console.log("Fizz")
    else if (i % 5 === 0)
        console.log("Buzz")
    else
        console.log(i)
};

【讨论】：

【解决方案4】：

有很多方法可以做到这一点，但重要的是遵循编码指南并使其更简单易读。

class fizzbuzz {
	
	constructor(fizz,buzz,length){
		this.fizz = fizz;
		this.buzz = buzz;
		this.length = length;
	}

	output() {
		for(let i = 0; i < this.length; i++){
			console.log(this.getFizzBuzz(i));
		}
	}

	getFizzBuzz(i) {
		if(i % this.fizz == 0 && i % this.buzz == 0 )
			return 'fizzbuzz'
		else 
			return i % this.fizz == 0 ? 'fizz' : 'buzz';
	}
}

new fizzbuzz(3,5,100).output();

【讨论】：

【解决方案5】：

你会说这太迂回了吗？我有点喜欢这件事的开始，但可能有更好的方法来做到这一点。

编辑：尽管我的其他解决方案肯定比其他解决方案更冗长 XD。

//declare the three arrays to store variables
var threeArray = [];
var fiveArray = [];
var threeFiveArray = [];

//prep your fizzes 
for (i = 3; i < 100; i+=3){
    threeArray.push(i); 
}   

//prep your buzzes 
for (i = 5; i < 100; i+=5){
    fiveArray.push(i);
}

//iterate through 0-99 to see if any of the numbers are in fizz, buzz, or both.
for (i = 0; i < 100; i++){
  if(threeArray.includes(i) && fiveArray.includes(i)){
    threeFiveArray.push(i + ' fizzbuzz');
  } else if(threeArray.includes(i)){
    threeFiveArray.push(i + ' fizz');
  } else if(fiveArray.includes(i)){
    threeFiveArray.push(i + ' buzz');
  } 
}

//return your results
console.log(threeFiveArray);

【讨论】：