Last active
April 8, 2016 10:36
-
-
Save bsthomsen/8b199fd0dd5b768ee4f5a841bf54073d to your computer and use it in GitHub Desktop.
This file contains hidden or 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
// Write a program that prints the numbers from 1 to 100. | |
// But for multiples of three print "Fizz" instead of the number and for the multiples of five print "Buzz". | |
// For numbers which are multiples of both three and five print "FizzBuzz". | |
for (var i = 1; i <= 100; i++) { | |
var output = ""; | |
if (i % 3 == 0) { | |
output += "Fizz"; | |
} | |
if (i % 5 == 0) { | |
output += "Buzz"; | |
} | |
if(output == "") { | |
output = i; | |
} | |
console.log(output); | |
} |
schonert
commented
Apr 8, 2016
@schonert ser nice ud men ville nok ikke ha skrevet det så minimalistisk. Nemmere at kode videre på senere samt at lade en minifier optimere det senere. Den bliver nemlig ikke særlig pæn hvis først du sku ha en 3. condition på i%9 el.l.
Og udover det får du aldrig skrevet FizzBuzz(v. 15, 30, 45, 60, 75, 90) hvor begge conditions er opfyldt.
For at være lidt sjov, kan man faktisk lave det til en oneliner ... umuligt at læse, men det virker
for( var i = 1; i <= 100; i++ ) console.log( ( i%3 == 0 && i%5 == 0 ) ? "FizzBuzz" : ( i%3 == 0 ) ? "Fizz" : ( i%5 == 0 ) ? "Buzz" : i );
@Vaff
det bare dårlig kode praksis 💃
En let læselig udgave
for (var i = 1; i < 101; i++) {
var buzz = (i % 5 == 0) ? 'Buzz' : i;
var fizz = (i % 3 == 0) ? 'Fizz' : buzz;
var fizzBuzz = (i % 3 == 0 && i % 5 == 0) ? 'FizzBuzz' : fizz;
console.log("fizzBuzz: ", fizzBuzz);
}
@sp90 - Jeg sagde heller ikke noget om, at det var best praksis :P
@bsthomsen rimelig væsentlig del jeg overså med (15, 30, 45, 60, 75, 90) - håbløst 😅
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment