Created
January 11, 2012 01:39
-
-
Save jaysonrowe/1592432 to your computer and use it in GitHub Desktop.
FizzBuzz JavaScript solution
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
for (var i=1; i <= 20; i++) | |
{ | |
if (i % 15 == 0) | |
console.log("FizzBuzz"); | |
else if (i % 3 == 0) | |
console.log("Fizz"); | |
else if (i % 5 == 0) | |
console.log("Buzz"); | |
else | |
console.log(i); | |
} |
persianturtle
commented
Nov 7, 2023
const fizzBuzz = (num, rules) => {
const result = [];
for (let i = 1; i <= num; i++) {
result.push(applyRules(i, rules));
}
return result;
};
const applyRules = (num, rules) => {
const output = rules.reduce((result, rule) => {
if (rule.condition(num)) {
result += rule.output;
}
return result;
}, '');
return output || num;
};
// Example usage:
const isMultipleOf = (n) => (num) => num % n === 0;
const rules = [
{ condition: isMultipleOf(3), output: 'Fizz' },
{ condition: isMultipleOf(5), output: 'Buzz' },
];
const resultArray = fizzBuzz(15, rules);
console.log(resultArray);
//para imprimir en la página web const n= window.prompt("Ingrese un número: "); for(i=1; i<=n; i++) if(i%3==0 && i%5==0){ console.log("FizzBuzz\n"); } else if(i%3==0 && i%5!=0){ console.log("Fizz\n"); } else if(i%3!=0 && i%5==0){ console.log("Buzz\n"); } else{ console.log(i\n); }
Which way would be "better regarded" as a best answer, by using for loops or by using .forEach() and .map()? Or it really does not matter as long as the program executes what is expected of it? Cheers
@iuliaGrig The answer is: It depends.
In this example, it's so simple that it doesn't matter for most intents and purposes. For more complex examples a rule of thumb could be:
.forEach
and.map
are generally more readable (even though plenty of people already might disagree)..forEach
should be used if there are side-effects to the function (likeconsole.log
),.map
must be used if you want to create a new list (.forEach
doesn't return anything)for
-loops are generally more performant, even though the difference only really matters for very large lists
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment