Eliminar un elemento de un diccionario cuando se desconoce su clave

¿Cuál es la mejor manera de eliminar un elemento de un diccionario por valor, es decir, cuando se desconoce la clave del elemento? He aquí un enfoque simple:

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

¿Hay mejores formas? ¿Hay algún problema con la mutación (eliminación de elementos) del diccionario mientras se itera?

Tenga en cuenta que actualmente está probando la identidad del objeto ( issolo devuelve Truesi ambos operandos están representados por el mismo objeto en la memoria; este no siempre es el caso con dos objetos que se comparan igual con ==). Si está haciendo esto a propósito, entonces podría reescribir su código como

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

Pero esto puede no hacer lo que quieres:

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

Así que probablemente quieras en !=lugar de is not.

El dict.pop(key[, default])método le permite eliminar elementos cuando conoce la clave. Devuelve el valor en la clave si elimina el elemento; de lo contrario, devuelve lo que se pasa como default. Consulte los documentos .

Ejemplo:

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

Una simple comparación entre del y 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)

resultado:

0.0329667857143
0.0451040902256

Entonces, del es más rápido que pop () .

items()devuelve una lista, y es esa lista la que está iterando, por lo que mutar el dict en el ciclo no importa aquí. Si estuviera usando en su iteritems()lugar, mutar el dict en el bucle sería problemático , y también viewitems()en Python 2.7.

No puedo pensar en una mejor manera de eliminar elementos de un dictado por valor.

Construiría una lista de claves que deben eliminarse y luego las eliminaría. Es simple, eficiente y evita cualquier problema de iterar y mutar simultáneamente el 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 es el nuevo diccionario y a es su diccionario original, {'z', 'w'} son las claves que desea eliminar de un

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

Consulte también: 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')]

No hay nada de malo en eliminar elementos del diccionario mientras se itera, como ha propuesto. Tenga cuidado con varios subprocesos que utilizan el mismo diccionario al mismo tiempo, lo que puede provocar un KeyError u otros problemas.

Por supuesto, consulte los documentos en http://docs.python.org/library/stdtypes.html#typesmapping

Así es como lo haría yo.

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