Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

iOS Swift Collections and Control Flow Control Flow With Conditional Statements FizzBuzz Challenge

Ingunn Augdal Fløvig
Ingunn Augdal Fløvig
1,507 Points
Posted by Ingunn Augdal Fløvig
Ingunn Augdal Fløvig
1,507 Points

FizzBuzz challenge error compiling code

I first got the error code "Double check your logic for Buzz values and make sure you're returning the correct string!" and found an answered question that said that I do not need the ´´´ for n in 1...100 ´´´ part of the code, so I removed that part and now it doesn't compile. Not sure what went wrong

fizzBuzz.swift 
func fizzBuzz(n: Int) -> String {

  for n {
    if (n % 3 == 0) && (n % 5 == 0) {
    return "FizzBuzz"
    } else if (n % 3 == 0) {
    return "Fizz"
    } else if (n % 5 == 0) {
    return "Buzz"
    }

  }

  return "\(n)"
}

2 Answers

andren
andren
28,521 Points
andren
andren
28,521 Points

You need to remove your loop entirely. You have just removed the in 100 part of the condition for the loop, which is not enough and results in your code being invalid.

Like this:

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