refactoring help, anyone?

Question

I'm only being asked to put the last chunk of code which calls all of the above functions into one function called "main". Could anyone tell me if there is something to do here other than define "main" and adjust the position of any blocks so that they fit "inside" of it, or if I've failed to do that properly? Thank you to anyone who has any advice. I know it's a newb problem.

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():
    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)

Answer 1 · 2016-02-26T06:56:16Z

February 26, 2016 6:56am

You are correct that the challenge is simply defining the main() function and shifting everything over evenly. You are also correct that you have indentation issues. Look at the original code from the challenge. Make your code follow the same relative indentation pattern

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)

Welcome to the Treehouse Community

Looking to learn something new?

douglas gray

douglas gray

refactoring help, anyone?

1 Answer

Chris Freeman

Chris Freeman

Robin Johannessen

Robin Johannessen