Удалить элемент из словаря, если его ключ неизвестен

Как лучше всего удалить элемент из словаря по значению, т.е. когда ключ элемента неизвестен? Вот простой подход:

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

Есть способы лучше? Что-то не так с изменением (удалением элементов) из словаря при его повторении?

Имейте в виду, что в настоящее время вы проверяете идентичность объекта ( isвозвращается только в том Trueслучае, если оба операнда представлены одним и тем же объектом в памяти - это не всегда происходит с двумя объектами, которые сравниваются с равными ==). Если вы делаете это специально, то можете переписать свой код как

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

Так что вы, вероятно, захотите !=вместо is not.

dict.pop(key[, default])Метод позволяет удалять элементы , когда вы знаете , ключ. Он возвращает значение ключа, если он удаляет элемент, иначе он возвращает то, что было передано как default. См. Документацию » .

Пример:

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

Простое сравнение между del и 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)

результат:

0.0329667857143
0.0451040902256

Итак, del быстрее, чем pop () .

items()возвращает список, и это тот список, который вы повторяете, поэтому изменение dict в цикле здесь не имеет значения. Если бы вы использовали iteritems()вместо этого, изменение dict в цикле было бы проблематичным , как и viewitems()в Python 2.7.

Я не могу придумать лучшего способа удалить элементы из dict по значению.

Я бы составил список ключей, которые нужно удалить, а затем удалил бы их. Это просто, эффективно и позволяет избежать проблем с одновременным повторением и изменением 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 - новый словарь, a - ваш исходный словарь, {'z', 'w'} - ключи, которые вы хотите удалить из

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

Также проверьте: 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')]

Нет ничего плохого в удалении элементов из словаря во время итерации, как вы предложили. Будьте осторожны с несколькими потоками, использующими один и тот же словарь одновременно, что может привести к ошибке KeyError или другим проблемам.

Конечно, смотрите документацию на http://docs.python.org/library/stdtypes.html#typesmapping

Вот как бы я это сделал.

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