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