sets.py

Question

This is doing what I want in workspace not sure why it's not passing...

sets.py

COURSES = {
    "Python Basics": {"Python", "functions", "variables",
                      "booleans", "integers", "floats",
                      "arrays", "strings", "exceptions",
                      "conditions", "input", "loops"},
    "Java Basics": {"Java", "strings", "variables",
                    "input", "exceptions", "integers",
                    "booleans", "loops"},
    "PHP Basics": {"PHP", "variables", "conditions",
                   "integers", "floats", "strings",
                   "booleans", "HTML"},
    "Ruby Basics": {"Ruby", "strings", "floats",
                    "integers", "conditions",
                    "functions", "input"}
}

def covers(value):
    for objects in COURSES.items():
        if len(value & objects[1]) >= 1:
            return [objects[0]]

Answer 1 · 2018-02-07T12:04:45Z

February 7, 2018 12:04pm

I agree that your code is perfect for the single answer case they supplied!

I think they want you to interpret that the solution to be that the function will find multiple courses which contain the same topic.

Example:

courses({'strings'}) should return ['Python Basics', 'Java Basics', 'PHP Basics', 'Ruby Basics']

currently, your code returns the first ['Python Basics'] which is correct, but misses the others.

Your code is well written, so all you need to do is to add a "memory" list of courses you detected.

Keep up the good work! Python is an amazing language that has nearly infinite possibilities!

def covers(value):
    courses = [] # will allow us to remember each course object that contained the topic
    for objects in COURSES.items():
        if len(value & objects[1]) >= 1:
           courses.append(objects[0]) # append (save) the course name to the course list
    return courses

Welcome to the Treehouse Community

Looking to learn something new?

Christian Lamoureux

Christian Lamoureux

sets.py

1 Answer

Jeff Muday

Jeff Muday