Rimuove un elemento da un dizionario quando la sua chiave è sconosciuta

Qual è il modo migliore per rimuovere un elemento da un dizionario in base al valore, ad esempio quando la chiave dell'elemento è sconosciuta? Ecco un approccio semplice:

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

Esistono modi migliori? C'è qualcosa di sbagliato nel mutare (eliminare elementi) dal dizionario durante l'iterazione?

Tieni presente che stai attualmente testando l'identità dell'oggetto ( isrestituisce solo Truese entrambi gli operandi sono rappresentati dallo stesso oggetto in memoria - questo non è sempre il caso di due oggetti che si confrontano allo stesso modo ==). Se lo fai apposta, potresti riscrivere il codice come

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

Ma questo potrebbe non fare quello che vuoi:

>>> 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'}

Quindi probabilmente vuoi !=invece di is not.

Il dict.pop(key[, default])metodo ti consente di rimuovere gli elementi quando conosci la chiave. Restituisce il valore alla chiave se rimuove l'elemento altrimenti restituisce ciò che è stato passato come default. Vedi i documenti . "

Esempio:

>>> 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']

Un semplice confronto tra del e 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)

risultato:

0.0329667857143
0.0451040902256

Quindi, del è più veloce di pop () .

items()restituisce una lista, ed è quella lista che stai iterando, quindi cambiare il dict nel ciclo non ha importanza qui. Se invece si stesse usando iteritems(), la modifica del dict nel ciclo sarebbe problematico , e lo stesso accade viewitems()in Python 2.7.

Non riesco a pensare a un modo migliore per rimuovere elementi da un dettato in base al valore.

Creerei un elenco di chiavi che devono essere rimosse, quindi le rimuoverò. È semplice, efficiente ed evita qualsiasi problema di iterazione e modifica simultanea del dict.

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 è il nuovo dizionario e a è il tuo dizionario originale, {'z', 'w'} sono le chiavi che vuoi rimuovere da un

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

Controlla anche: 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')]

Non c'è niente di sbagliato nell'eliminare elementi dal dizionario durante l'iterazione, come hai proposto. Fare attenzione a più thread che utilizzano lo stesso dizionario contemporaneamente, il che potrebbe causare un KeyError o altri problemi.

Ovviamente, vedere i documenti su http://docs.python.org/library/stdtypes.html#typesmapping

Ecco come lo farei.

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