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

Posted by Nils Garland
Nils Garland
18,416 Points

FizzBuzz Problems :P

So I can't find the problem, does anybody see it? Thanks a lot :D

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) {
            return "FizzBuzz"
        } else if(n % 3 == 0) {
            return "Fizz"
        } else if(n % 5 == 0) {
            return "Buzz"
        } else {
            return "\(n)"
        }
    }
    return "\(n)"
}

2 Answers

Jennifer Nordell
seal-mask
STAFF
.a{fill-rule:evenodd;}techdegree
Jennifer Nordell
Treehouse Teacher
Jennifer Nordell
.a{fill-rule:evenodd;}techdegree
Jennifer Nordell
Treehouse Teacher

Hi again! If you're referring to this challenge, then I'd point out that you missed a note in the instructions.

The challenge also does not need you to loop over a range of values (using for or while). I'll take care of that.

If I copy/paste your code into the challenge and then remove the surrounding for loop (including the closed curly brace for that loop), it passes! Your logic is spot on. Hope this helps! :sparkles:

Nils Garland
Nils Garland
18,416 Points

Thanks again :D

Leandro Pezán
PLUS
Leandro Pezán
Courses Plus Student 2,049 Points
Leandro Pezán
Leandro Pezán
Courses Plus Student 2,049 Points

There's no need for a double return (see the two last ones). Since this is the default case, it should go inside the else curly braces. The code should look something 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"
    } else {
        return "\(n)"
    }
}
Nils Garland
Nils Garland
18,416 Points

Thank you for answering :D