Return number of teachers. Expected 5 got 2. There are only two teachers int the dictionary

Question

The prior step didn't like me using my own dictionary of teachers I used the example dictionary. My code works but the answers says it expects five when there are only 2 teachers? Should I just add more teachers to the dictionary? and is this just me misunderstanding the wording or is it a bug?

teachers.py

# The dictionary will be something like:
# {'Jason Seifer': ['Ruby Foundations', 'Ruby on Rails Forms', 'Technology Foundations'],
#  'Kenneth Love': ['Python Basics', 'Python Collections']}
#
# Often, it's a good idea to hold onto a max_count variable.
# Update it when you find a teacher with more classes than
# the current count. Better hold onto the teacher name somewhere
# too!
#
# Your code goes below here.

master_class_list =  {'Jason Seifer': ['Ruby Foundations', 'Ruby on Rails Forms', 'Technology Foundations'], 'Kenneth Love': ['Python Basics', 'Python Collections']}


def most_classes(master_class_list): 
    max_count = 0
    most_busy_teacher = ''
    for teacher in master_class_list:
        classes = master_class_list[teacher]
        if len(classes) > max_count:
            max_count = len(classes)
            most_busy_teacher = teacher
    print(most_busy_teacher)
    print(max_count)
    return most_busy_teacher

def num_teachers(dict): 
    count = 0
    for teacher in master_class_list:
        count +=1 
    print(count)
    return count


most_classes(master_class_list)
num_teachers(master_class_list)

Answer 1 · 2016-01-08T09:11:06Z

January 8, 2016 9:11am

It is not necessary to use your own data. It is supplied by the challenge grader. The code error is in not using the passed argument dict. Instead, you are using your own data master_class_list.

def most_classes(master_class_list): 
    max_count = 0
    most_busy_teacher = ''
    for teacher in master_class_list:
        classes = master_class_list[teacher]
        if len(classes) > max_count:
            max_count = len(classes)
            most_busy_teacher = teacher
    print(most_busy_teacher)
    print(max_count)
    return most_busy_teacher

def num_teachers(dict): 
    count = 0
    for teacher in dict:  # <-- changed to dict
        count +=1 
    print(count)
    return count

Welcome to the Treehouse Community

Looking to learn something new?

Brigette Eckert

Brigette Eckert

Return number of teachers. Expected 5 got 2. There are only two teachers int the dictionary

1 Answer

Chris Freeman

Chris Freeman