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 Basics (2015) Shopping List App Refactor

Posted by dima Dima
dima Dima
901 Points

What should I do here

I can not understand where is my mistake

shopping_list.py 
def show_help():
    # print out instructions on how to use the app
    print("What should we pick up at the store?")
    print("""
Enter 'DONE' to stop adding items.
Enter 'HELP' for this help.
Enter 'SHOW' to see your current list.
""")

def show_list(shopping_list):
    # print out the list
    print("Here's your list:")

    for item in shopping_list:
        print(item)

def add_to_list(shopping_list, new_item):
    # add new items to our list
    shopping_list.append(new_item)
    print("Added {}. List now has {} items.".format(new_item, len(shopping_list)))
    return shopping_list

def main():
    return

main:    
    show_help()

# make a list to hold onto our items
    shopping_list = []

    while True:
    # ask for new items
    new_item = input("> ")

    # be able to quit the app
        if new_item == 'DONE':
             break
        elif new_item == 'HELP':
        show_help()
             continue
        elif new_item == 'SHOW':
        show_list(shopping_list)
              continue
        add_to_list(shopping_list, new_item)

    show_list(shopping_list)
dima Dima
dima Dima
901 Points

I had to add a new function main that does not take any arguements, and put everything starting from line 22 in under that function

2 Answers

Steven Parker
Steven Parker
231,271 Points
Steven Parker
Steven Parker
231,271 Points

Immediately after your "def main:" you have a return, which stops execution. That line should not be there.

Also, below that you have a stray line containing "main:". It should not be there either.

Otherwise, it looks like you missed adding indentation to 3 of the original lines. Once you fix those and remove the two spurious lines you should be good.

Michael Barlow
Michael Barlow
34,317 Points
Michael Barlow
Michael Barlow
34,317 Points

Once you've fixed the above errors, you'll also need to call the main() function to run the program. Another option would be to wrap the program logic in the following:

if __name__ == '__main__':

instead of defining the main() function and calling it separately.