Ytukyg

EDIT REQUESTED:
I created two instances of the menu, one in a function and one outside the function because the initial test prompts the user to enter 5 jersey nums and the player rank, then expects a roster to be printed, then expects the user menu to prompt changes to the roster.
"Updating dictionary" I meant the roster is created in a dict{} and then the options of the user menu allow for editing the initial dict and then printing a new roster with the changes.

INITIAL POST:
I am having trouble with this code (class assignment, hw not test). The goal is a menu that allows the user to add and delete players from a roster, update their player rating, pull a list of the top x ratings, or simply print a roster. The assignment requires that the initial user input is 5 jersey numbers and their ratings, print a roster, then offer the user menu. I have solved the first 4 tasks. PROBLEM: when accessing menu items to add, delete, update, and rank the first roster that follows the initial 5 entries is being read as the answer instead of the roster that prints to the screen after changes are made. I can't figure out how to set it up so the roster after the initial setup is optional.
I'm hoping you will see an obvious error in the code that I'm missing. (The indentation looks a little funky to format for pasting in here. I have no indentation errors per the system.)

def print_menu():

  print ('nMENU')

  print ('a - Add player')

  print ('d - Remove player')

  print ('u - Update player rating')

  print ('r - Output players above a rating')

  print ('o - Output roster')

  print ('q - Quit')

  action_in = input('nChoose an option:n')



def output_roster():

  listOFjerseys = list(player_stats.keys())

  listOFratings = list(player_stats.values())

  print ('ROSTER')

  a_jersey = sorted (listOFjerseys)

  for x in a_jersey:

     print ("Jersey number: %d, Rating: %d" % (x,player_stats[x]))



player_stats = {}

entry_count = int(5)



for i in range(1,entry_count+1):

   jersey_num = int(input("Enter player %d's jersey number:n" % i))

   p_rating = int(input("Enter player %d's rating:nn" % i))

   player_stats [jersey_num] = (p_rating)



listOFjerseys = list(player_stats.keys())

listOFratings = list(player_stats.values())



print ('ROSTER')

a_jersey = sorted (listOFjerseys)

for x in a_jersey:

   print ("Jersey number: %d, Rating: %d" % (x,player_stats[x]))





    choice = "" #I think this line is unused/unnecessary



    print ('nMENU')

    print ('a - Add player')

    print ('d - Remove player')

    print ('u - Update player rating')

    print ('r - Output players above a rating')

    print ('o - Output roster')

    print ('q - Quit')

    action_in = input('nChoose an option:n')

    while action_in is True and action_in != 'q':

        if action_in == 'a':

            #Add player

            jersey_num = int(input("nEnter a new player's jerseynumber:n"))

            p_rating = int(input("Enter the player's rating:n"))

            player_stats [jersey_num] = (p_rating)

            output_roster()

            print_menu()

        elif action_in == 'd':

            #Delete player

            player = int(input("nEnter a jersey number:n"))

            del player_stats[player]

            output_roster()

            print_menu()

        elif action_in == 'u':

            #Update rating

            jersey_num = int(input("nEnter a jersey number:n"))

            n_rating = int(input("Enter a new rating for player:n"))

            player_stats [jersey_num] = (n_rating)

            output_roster()

            print_menu()

        elif action_in == 'r':

            #output > rating

            rankVal = int(input("nEnter a rating:n"))

            print ('ABOVE %dn' % rankVal)

            for m, n in player_stats.values():

                if n > rankVal:

                    print ('Jersey number: %d, Rating: %dn' % (m,n))

                    print_menu()

        elif action_in == 'o':

            #Output roster

            output_roster()

            print_menu()

    if action_in == 'q':

        #quit

        raise SystemExit()

enter image description here