Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.
Start your free trial
arnoldmukasa
3,924 PointsFizzBuzz
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)" } }
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)"
}
}
2 Answers
james south
Front End Web Development Techdegree Graduate 33,271 Pointsit 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
3,924 PointsMuch Appreciated!! :)