FizzBuzz using JavaScript Switch

Question

Can't seem to point out something obvious. This solution doesn't work. Please help?

FizzBuzz question: Print numbers from 1 to 100. Print Fizz instead of numbers divisible by 3. Print Buzz instead of numbers divisible by 5. Print FizzBuzz instead of numbers divisible by both 3 and 5.

for(var i = 1; i <= 100; i++) {
 
 switch (i) {

      case (i%3 === 0 && i%5 === 0):
        console.log('FizzBuzz'); 
        break;

      case (i%3 === 0):
        console.log('Fizz');
        break;

      case (i%5 === 0):
        console.log('Buzz');
        break;

      default:
        console.log(i);

 }
    }

Define "doesn't work". Errors? Wrong output? Something else? — takendarkk, May 01 '18 at 11:19
Why are you using switch? Is that something that task requests or is it just your choice? — Mladen Ilić, May 01 '18 at 11:19
The closest you can get to your code whilst still using a `switch` block is [this](http://jsbin.com/yexowopiku/1/edit?js,console). — Phylogenesis, May 01 '18 at 11:21

score 3 · Accepted Answer · answered May 01 '18 at 11:21

The problem is, you are switching on the value of variable i. Then you are comparing that with expressions like i % 3 === 0, so the comparison becomes i === (i % 3 === 0).

Instead, you can switch on true, so any expression evaluating to true will be switched into.

for (var i = 1; i <= 100; i++) {
  switch (true) {
    case (i % 3 === 0 && i % 5 === 0):
      console.log('FizzBuzz');
      break;

    case (i % 3 === 0):
      console.log('Fizz');
      break;

    case (i % 5 === 0):
      console.log('Buzz');
      break;

    default:
      console.log(i);
  }
}

Jonas Wilms · Answer 2 · 2018-05-01T11:31:42.313

1

A switch enters the branch if the case matches the value you are switching for, therefore:

  switch (i) {
    case (i%3 === 0 && i%5 === 0):

Will enter the first case if:

 i === (i%3 === 0 && i%5 === 0)

E.g. for 15 it will be:

 15 === true // -> false

so it wont enter the branch. Therefore instead of switching for i you have to switch for true:

  switch(true) {

edited May 01 '18 at 11:31

answered May 01 '18 at 11:22

Jonas Wilms

132,000
20
149
151

score -3 · Answer 3 · answered May 01 '18 at 11:19

-3

Did you try not to break the switch, because when it matches it won't print the other stuff it is supposed to? Maybe?

answered May 01 '18 at 11:19

Petar Yonkov

1

FizzBuzz using JavaScript Switch

3 Answers3