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

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

Kjetil Korsveien
Kjetil Korsveien
1,713 Points
Posted by Kjetil Korsveien
Kjetil Korsveien
1,713 Points

Does´t get the right answer correct

In the FizzBuzz solution I get a wrong answer even though I copied it from the video? What is wrong here?

I also made a solution using switch, but that wasn't approved as correct either, worked in Playground?

fizzBuzz.swift 
func fizzBuzz(n: Int) -> String {
  // Enter your code between the two comment markers
      for n in 1...100 {

        if (n % 3 == 0) && (n % 5 == 0) {
            print("FizzBuzz")
        }
        else if (n % 3 == 0) {
            print("Fizz")
        }
        else if (n % 5 == 0) {
            print("Buzz")
        }
        else {
            print(n)
        }
    }
  // End code
  return "\(n)"
}
Paul Ryan
Paul Ryan
4,584 Points
Paul Ryan
Paul Ryan
4,584 Points

I think you need return statements rather than print statements. I have never coded in Swift but thats what I am guessing

2 Answers

Ben Bastow
Ben Bastow
24,657 Points

Hey!

Your answer wasn't right because you have your code surrounded in a For In loop so for your number you put in you run the code 100 times and from numbers from 1 - 100 when actually what you want is for the code to run once with the user inputted number. Also, you print fizzbuzz fizz and buzz but you actually need to return these values instead of returning them

If you took away the For In Loop and changed your print statements to return values then your code would be correct for the challenge.

This would be the correct code:

func fizzBuzz(n: Int) -> String {
    // Enter your code between the two comment markers
    if (n % 3 == 0) && (n % 5 == 0) {
        return ("FizzBuzz")
    }

    else if (n % 3 == 0) {
        return ("Fizz")
    }

    else if (n % 5 == 0) {
        return ("Buzz")
    }
    // End code
    return "\(n)"
}

I hope this helps!

Kjetil Korsveien
Kjetil Korsveien
1,713 Points
Kjetil Korsveien
Kjetil Korsveien
1,713 Points

I think that was the only combination I didn't try. Tnx mate :)