Please can someone review my Fizz Buzz code?

Carrie,

Aside from the "counter = counter + 1" adding 101 to your count, your code is perfect! If you change the increment to "counter++", it works correctly and stops printing at 100. I'm not entirely sure why this is happening, but that seems to be the culprit. Perhaps someone else will jump on and provide further explanation for that cause, but in any case, if you switch that little part out, you're good to go.

Good work!

Erik

Hi Erik!

Thanks for coming back to me so quickly. When I run the code it says:

... 97 98 Fizz (99) Buzz (100)

Is this not correct?

Also, how do I post the code so it has the black background? :(

Thanks again :)

Carrie,

If it is stopping at that last "Buzz" for you, rather than also then putting out "101", then you're good to go! When I copied and pasted your code into the console, it printed "101" after the final "Buzz", but if you're not seeing that, then no worries, I suppose =p

As far as posting code into the forums, this and this are handy posts to take a look at for help. But, basically, to post code blocks with that black background that you're seeing, you simply type three backticks (the [ ` ] character found usually in the top-left of your keyboard, perhaps directly to the left of your 1 key. It usually also has the tilde [ ~ ] symbol on its key), then type the name of the language you're going to be coding in (HTML, CSS, JavaScript, PHP, etc.), then hit enter and type another three backticks, then in between those two lines, you type your code.

Erik

Hi Erik

Not sure why you were getting to 101, very strange! Thanks for the tips about posting the code properly, looks to have worked :)

Wow really?? Thanks Erik! :D

Your code is solid, I would suggest you work on your code formatting to increase your readability.

For instance this is more readable so compare and contrast the whitespace in the 2 programs:

var counter = 1;
while (counter <= 100) {
    if (counter % 3 == 0 && counter % 5 == 0) {
        console.log("FizzBuzz");
    } else if (counter % 3 == 0) {
        console.log("Fizz");
    } else if (counter % 5 == 0) {
        console.log("Buzz");
    } else console.log(counter);
    counter = counter + 1;
}

2 small nitpicks:

• You can use counter++ rather than counter = counter + 1 which I think makes your code look cleaner but that's really a matter of style.

• Also you probably should use === to compare rather than == I don't think the difference has yet been where you are in the course.

Hello, I am new to to programming and wondering some stuffs according this assignment. But first, I will post my own code here for this assignment. The structure looks kinda similar

for (var counter = 100; counter; counter = counter - 1) {
    if(counter % 15 == 0) {
    console.log("fizzbuzz");
    } else if (counter % 3 == 0) {
        console.log("Fizz");
    } else if (counter % 5 == 0) {
        console.log ("Buzz");
    } else console.log(counter);
    }

console.log(x);

1. I wonder if it is a bad idea to use the "for-loop". Are there cases where you prefer to use for loops instead of while loops and vice-versa?
2. Are there any benefits to use incrementing instead of decrementing?
3. Do you prefer that i use counter % 3 == 0 && counter % 5 == 0 instead of counter % 15 == 0? Since it means the same thing, but the code maybe looks better if you type "counter % 3 == 0 && counter % 5 == 0"? :)

Sorry for my (probably) silly questions but i am new to this and really wants to learn it :)

Thanks.

Thank you james, very straight forward and good answers :D