Remova um item de um dicionário quando sua chave for desconhecida

Qual é a melhor maneira de remover um item de um dicionário por valor, ou seja, quando a chave do item é desconhecida? Esta é uma abordagem simples:

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

Existem maneiras melhores? Há algo de errado em alterar (excluir itens) do dicionário durante a iteração?

Esteja ciente de que você está testando atualmente a identidade do objeto ( isretorna apenas Truese os dois operandos forem representados pelo mesmo objeto na memória - nem sempre é o caso com dois objetos que são comparados iguais a ==). Se você estiver fazendo isso de propósito, poderá reescrever seu código como

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

Mas isso pode não fazer o que você deseja:

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

Então você provavelmente quer em !=vez de is not.

O dict.pop(key[, default])método permite remover itens quando você conhece a chave. Ele retorna o valor na chave se remover o item, caso contrário, retorna o que foi passado como default. Veja a documentação . '

Exemplo:

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

Uma comparação simples entre 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)

resultado:

0.0329667857143
0.0451040902256

Portanto, del é mais rápido que pop () .

items()retorna uma lista, e é essa lista que você está iterando, portanto, alterar o dict no loop não importa aqui. Se você estivesse usando em iteritems()vez disso, alterar o dict no loop seria problemático , e da mesma forma para viewitems()no Python 2.7.

Não consigo pensar em uma maneira melhor de remover itens de um dicionário por valor.

Eu construiria uma lista de chaves que precisam ser removidas e, em seguida, removeria. É simples, eficiente e evita qualquer problema de iteração simultânea e mutação do 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 é o novo dicionário e a é seu dicionário original, {'z', 'w'} são as chaves que você deseja remover de um

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

Verifique também: 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')]

Não há nada de errado em excluir itens do dicionário durante a iteração, como você propôs. Tenha cuidado com vários threads usando o mesmo dicionário ao mesmo tempo, o que pode resultar em um KeyError ou outros problemas.

Claro, consulte os documentos em http://docs.python.org/library/stdtypes.html#typesmapping

É assim que eu faria.

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