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!

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

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,554 Points
andren
andren
28,554 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)"
}