Delete value from Dictionary

Question

players = {
        "nome": "",
        "score": [] 
    }

cancel = False
player_list = []

while (True):
    name = input("Insert player name: ")

    player_list.append({
        "Player Name": name
    })

    cont = input("Do you wish to add another player? (Y/N)")
    if cont == "N":
        break;

print("Player List: ", player_list)

while (True):
    eliminate_player = input("Select player to delete: ")

    player_list.pop({
        "Player Deleted": eliminate_player
    })    

print ("Updated List", player_list)

Hello everyone! Im having some trouble deleting an entry from my player list using user input. Im getting the error:'dict' object cannot be interpreted as an integer but I can't seem to find what is wrong because Im using similar method I used to add players and it worked fine.

Any ideas?

`list.pop(n)` requires an integer to remove the given index from the list. `list.remove(x)` takes an item to find and remove. — N Chauhan, Dec 23 '19 at 18:42
What is `jogador_elminado`? Should that be `eliminate_player`? — Barmar, Dec 23 '19 at 18:43
The dictionary you use in `.remove()` has to have the same key and value as the dictionary in the list. You can't change `Player Name` to `Player Deleted` when calling `remove()`. — Barmar, Dec 23 '19 at 18:45
@Barmar yeah it is, sorry. I translated it from my language and missed it. — lordvader, Dec 23 '19 at 18:47
What is the `players` dictionary for? You never use it. Also `cancel`. — Barmar, Dec 23 '19 at 18:48

score 4 · Answer 1 · answered Dec 23 '19 at 18:48

4

Use .remove() rather than .pop(). And the dictionary there has to have the same structure as the dictionaries in the list.

player_list.remove({"Player name": eliminate_player})

answered Dec 23 '19 at 18:48

Barmar

741,623
53
500
612

That seems to work. How can I stop it after deleting one player and then show the updated list? Sorry if this is a basic question but Im still learning. – lordvader Dec 23 '19 at 18:53
Do it the same way you enter the players into the list, by checking for `N` and breaking out of the loop. – Barmar Dec 23 '19 at 20:13

score 2 · Answer 2 · answered Dec 23 '19 at 18:54

2

pop() takes an index as argument. You can use remove() instead.

answered Dec 23 '19 at 18:54

Devansh Soni

771
5
16

abhiarora · Accepted Answer · 2019-12-23T19:10:22.793

You are not using the right method. pop(n) requires an integer (index) to remove the given index from the list. You should use remove().

remove() is an inbuilt function that removes a given object from the list. It does not return any value.

Syntax:

list_name.remove(obj)

Try this:

player_list.remove({"Player name": eliminate_player})

If the element doesn't exist, it throws ValueError: list.remove(x): x not in list exception. Wrap it in try catch clause in Python.

Note:

remove() removes the first occurrence of the object from the list.

To remove all occurence of your Player:

while (player_list.count({"Player name": eliminate_player})): 
    player_list.remove({"Player name": eliminate_player})

See this to understand the difference between pop, remove and del on lists.

How can I stop it after deleting one player and then show the updated list?

You can use count method to break from the while loop:

while (True):
    eliminate_player = input("Select player to delete: ")

    di = {"Player name": eliminate_player}
    if player_list.count(di):
        player_list.pop(di)
        break

Delete value from Dictionary

3 Answers3