Usuń element ze słownika, gdy jego klucz jest nieznany

Jaki jest najlepszy sposób usunięcia pozycji ze słownika według wartości, tj. Gdy klucz pozycji jest nieznany? Oto proste podejście:

for key, item in some_dict.items():
    if item is item_to_remove:
        del some_dict[key]

Czy są lepsze sposoby? Czy jest coś złego w mutowaniu (usuwaniu elementów) ze słownika podczas iteracji?

Należy pamiętać, że aktualnie testujesz tożsamość obiektu ( iszwraca tylko Truewtedy, gdy oba operandy są reprezentowane przez ten sam obiekt w pamięci - nie zawsze tak jest w przypadku dwóch obiektów, które porównują równe z ==). Jeśli robisz to celowo, możesz przepisać swój kod jako

some_dict = {key: value for key, value in some_dict.items() 
             if value is not value_to_remove}

Ale to może nie zrobić tego, co chcesz:

>>> some_dict = {1: "Hello", 2: "Goodbye", 3: "You say yes", 4: "I say no"}
>>> value_to_remove = "You say yes"
>>> some_dict = {key: value for key, value in some_dict.items() if value is not value_to_remove}
>>> some_dict
{1: 'Hello', 2: 'Goodbye', 3: 'You say yes', 4: 'I say no'}
>>> some_dict = {key: value for key, value in some_dict.items() if value != value_to_remove}
>>> some_dict
{1: 'Hello', 2: 'Goodbye', 4: 'I say no'}

Więc prawdopodobnie chcesz !=zamiast is not.

dict.pop(key[, default])Metoda pozwala na usuwanie elementów, gdy wiesz klucz. Zwraca wartość z klucza, jeśli usuwa element, w przeciwnym razie zwraca wartość przekazaną jako default. Zobacz dokumentację .

Przykład:

>>> dic = {'a':1, 'b':2}
>>> dic
{'a': 1, 'b': 2}
>>> dic.pop('c', 0)
0
>>> dic.pop('a', 0)
1
>>> dic
{'b': 2}

a = {'name': 'your_name','class': 4}
if 'name' in a: del a['name']

Proste porównanie między del i pop () :

import timeit
code = """
results = {'A': 1, 'B': 2, 'C': 3}
del results['A']
del results['B']
"""
print timeit.timeit(code, number=100000)
code = """
results = {'A': 1, 'B': 2, 'C': 3}
results.pop('A')
results.pop('B')
"""
print timeit.timeit(code, number=100000)

wynik:

0.0329667857143
0.0451040902256

Tak więc del jest szybsze niż pop () .

items()zwraca listę i to jest ta lista, którą iterujesz, więc mutowanie dyktowania w pętli nie ma tutaj znaczenia. Gdybyś iteritems()zamiast tego używał , mutowanie dyktowania w pętli byłoby problematyczne , podobnie jak viewitems()w Pythonie 2.7.

Nie mogę wymyślić lepszego sposobu na usuwanie pozycji z dyktu według wartości.

Zbudowałbym listę kluczy, które trzeba usunąć, a następnie je usunąłem. Jest to proste, wydajne i pozwala uniknąć problemów związanych z jednoczesnym iterowaniem i mutowaniem dyktowania.

keys_to_remove = [key for key, value in some_dict.iteritems()
                  if value == value_to_remove]
for key in keys_to_remove:
    del some_dict[key]

c to nowy słownik, a a to Twój oryginalny słownik, {'z', 'w'} to klucze, które chcesz usunąć z

c = {key:a[key] for key in a.keys() - {'z', 'w'}}

Sprawdź również: https://www.safaribooksonline.com/library/view/python-cookbook-3rd/9781449357337/ch01.html

y={'username':'admin','machine':['a','b','c']}
if 'c' in y['machine'] : del y['machine'][y['machine'].index('c')]

Nie ma nic złego w usuwaniu elementów ze słownika podczas iteracji, jak zaproponowałeś. Uważaj na wiele wątków używających tego samego słownika w tym samym czasie, co może spowodować błąd KeyError lub inne problemy.

Oczywiście zapoznaj się z dokumentacją pod adresem http://docs.python.org/library/stdtypes.html#typesmapping

Tak bym to zrobił.

for key in some_dict.keys():
    if some_dict[key] == item_to_remove:
        some_dict.pop(key)
        break