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

Python Python Collections (Retired) Dictionaries Teacher Stats

Create a function named num_teachers that takes a dictionary of teachers and returns the number of teachers.

I have tested my code for num_teachers and I am getting the correct answers. What is wrong with my code? Thanks in advance.

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.
def most_classes(teachers):
  max_count = 0
  max_teach = ''
  for teacher in teachers:
    if len(teachers[teacher]) > max_count:
      max_count = len(teachers[teacher])
      max_teach = teacher
  return max_teach

def num_teachers(teachers):
  max_teach = most_classes(teachers)
  return len(teachers[max_teach])

I forgot to mention, it is returning 2 more than it should when run here. When I use test data and run it in from Python 3.4 on my computer it gives me the correct number.

4 Answers

Hanley Chan
Hanley Chan
27,771 Points

Hi,

I think your function is returning the number of classes of the teacher with the most classes. The question is asking to return the number of teachers in the dictionary.

I'd also say it might help to name your variables so they are easier to track

teachers[teacher]

is harder to understand then

teachers[name]

Thank you. smh. what a dork I am.

C H
C H
6,587 Points

You may be overthinking it. Since the '.keys()' method lists the keys out, and all of the keys are teacher names, let's just take the length of that.

def num_teachers(teachers):
  return len(teachers.keys())