4th challenge

Hi there,

You're on the most_courses method, right?

I created two variables at the top of the method - one to hold the max number of courses currently seen which I initialized to zero. The second was a string to hold the teacher's name, initialised to a blank string.

I then for looped through the dictionary using the .items() method. At each iteration I can access the keys and values; I called them teacher and courses, I asked if the length of courses was greater than the current max_value. Since this was initialized to zero, the first iteration would always meet this condition.

If the length of the courses (the values aspect of the .items() from the dictionary), was greater, then assign that value into the variable storing the max number of courses and store this teacher's name in the string.

At the end of the loop, return that name.

I hope that helps; I can be more specific if you want but I thought I'd talk you through my thought process to see if that helps you reach a solution.

Steve.

Hi Steve,

When first completing this I did the method you described above but I am still not able to get the correct result from my code. Any recommendations on the changes I should make/consider

def most_courses(dictionary):
    most_courses = None
    teacher = None
    for keys in dictionary.keys():
        if len(dictionary[key]) > most_courses:
            most_courses = len(dictionary[key])
            teacher = key
        else:
            pass
        return teacher

Thanks

Hi again!

Three things - unindent your return as it is inside the loop.

Second, you called the loop variable keys and you then use key inside the loop. They need to be called the same.

Lastly, create empty local variables of the correct type; you can't compare an integer to a NoneType:

    most_courses = 0
    teacher = ""

That should now pass.

Steve.

Thanks Steve!

I need to keep double checking on what I name my variables

Wow, that one way waaaay harder.

def most_courses(dictionary_list):
    most_courses = 0
    teacher = ""
    for keys in dictionary_list.keys():
        if len(dictionary_list[keys]) > most_courses:
            most_courses = len(dictionary_list[keys])
            teacher = keys
        else:
            pass
    return teacher

print most_courses(dictionary_list)