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 Functions in Swift Adding Power to Functions Function Scope

Michael Dean
Michael Dean
821 Points
Posted by Michael Dean
Michael Dean
821 Points

Double Double is producing errors in playgrounds and not accepted in challenge

I'm not sure why this doesn't work. I'm getting two warnings: Line one I get "All paths through this function will call itself" Line two I get "Variable 'locationCoordinates' was written to, but never read"

func coordinates(for location: String) -> (Double, Double){

    var locationCoordinates: (Double, Double)

    switch location {

    case "Eiffel Tower": locationCoordinates = (48.8582, 2.2945)

    case "Great Pyramid": locationCoordinates = (29.9792, 31.1344)

    case "Sydney Opera House": locationCoordinates = (33.8587, 151.2140)

    default: locationCoordinates = (0, 0)

    }
    return coordinates(for: "Eiffel Tower")

}

2 Answers

Jhoan Arango
Jhoan Arango
14,575 Points
Jhoan Arango
Jhoan Arango
14,575 Points

Hello,

You are using the function within its own body. The return should be the variable that you are assigning values to in the switch cases. 

func coordinates(for location: String) -> (Double, Double){

    var locationCoordinates: (Double, Double)

    switch location {

    case "Eiffel Tower": locationCoordinates = (48.8582, 2.2945)

    case "Great Pyramid": locationCoordinates = (29.9792, 31.1344)

    case "Sydney Opera House": locationCoordinates = (33.8587, 151.2140)

    default: locationCoordinates = (0, 0)

    }
    return locationCoordinates

}

Hope this helps.

Michael Dean
Michael Dean
821 Points
Michael Dean
Michael Dean
821 Points

Thank you Jhoan ... yes that worked. Now I understand what I need to do with the return.