FizzBuzz

Question

func fizzBuzz(n: Int) -> String {
    if n % 3 == 0 {
        return "Fizz"
    } else if n % 5 == 0 {
        return "Buzz"
    } else if n % 3 == 0 && n % 5 == 0 {
        return "FizzBuzz"
    } else {
      return  "(n)"
    }
}
fizzBuzz.swift
func fizzBuzz(n: Int) -> String {
    if n % 3 == 0 {
        return "Fizz"
    } else if n % 5 == 0 {
        return "Buzz"
    } else if n % 3 == 0 && n % 5 == 0 {
        return "FizzBuzz"
    } else {
      return  "\(n)"
    }
}

james south · Accepted Answer

it says it will take care of the default so you can delete the last else. then you want to start your conditional block with the most restrictive case first, then get more general, so the multiples of 3 and 5 need to be checked first, then the remaining numbers can be checked for being multiples of only 3 or only 5. right now you are sending returns for multiples of 15 as 3s, then 5s, and then 15s, which i think is causing it to not pass.

arnoldmukasa · Answer

Much Appreciated!! :)

Welcome to the Treehouse Community

Looking to learn something new?

arnoldmukasa

arnoldmukasa

FizzBuzz

2 Answers

james south

james south

arnoldmukasa

arnoldmukasa