Discussion on: FizzBuzz JavaScript

View post

Replies for: I think you made a small typo in for loop, for( i = 1;i<=n ; i++). I was asked this same question and I gave the exact answer. The answer has on...

Mind you that with that approach every n is evaluated 4 times before a result sting is printed , so it's less efficient than Tadea's solution.

Akhil • May 3 '20 • Edited

Nope, it would be the same since here I have modified the question for 4 conditions, ie 3,5,7,11. If we follow the original question ie checking for 3 & 5, then code will be :

let str = "";
for( i = 1;i<=n ; i++){
    if(n%3 == 0) str += "fizz";
    if(n%5 == 0) str += "buzz";
    else str += n;
}
return str;
}

JP Antunes • May 3 '20 • Edited

Interesting. What do you think happens after each if statement?
Or in another way, what is the difference between two or more if's and a chain of if, else if, else?

edit: a picture of my cat doing a code review :-)

Akhil • May 3 '20

oh yeah, that else part is a really bad bug from my side
it should be

if(str ="") str += n;

what this video : youtube.com/watch?v=QPZ0pIK_wsc