What is the fault in this code

Hello :

Your code looks good, the only thing is that you have to reverse it a bit. Make sure that the FizzBuzz is the first one to be evaluated. And then also consider using "else if". Here I made 2 solutions to this challenge.

var numbers = 1...100

// Switch Statement
for number in numbers {
    switch number {
    case number where number % 3 == 0 && number % 5 == 0 :
        print("FizzBuzz")
    case number where number % 3 == 0:
        print("Fizz")
    case number where number % 5 == 0:
        print("Buzz")
    default:
        print(number)
    }
}

// If statement
for n in numbers {
    if n % 3 == 0 && num % 5 == 0{
        print("FizzBuzz")
    } else if n % 3 == 0 {
        print("Fizz")
    } else if n % 5 == 0 {
        print("Buzz")
    } else {
        print(n)
    }
}

Although I know, Pasan did not go over the where clause in the switch statement, I will post it on here, so that you can see the power of the switch statement as well.

Good luck.

Ravirayappan Chinnappan and Jhoan Arango,

The issue here is that your code includes a case for when the the number does not match either FizzBuzz, Fizz or Buzz (which is the correct way to do it). The code in the challenge already takes care of that for you. If you notice at the bottom of the code provided there is a line

return "\(n)" 

I'm using string interpolation to return the number as a string.

I've updated the directions to make it more clear that you don't have to worry about the default case.

And yes: Cheating with the where clause

For what it's worth Jhoan, you can optimize that code by ignoring the number variable in your cases like this:

case _ where number % 3 == 0 && number % 5 == 0 :

Since you don't actually need the number in the case statement and you're performing the logic in the where clause, you can just discard the number. This allows the compiler to optimize memory by not maintaining a copy of that variable.