Списки словари Странность или баг?

Question

[Andrei Sayevich]

Задача была такая ...
нужно отсортировать список словарей и
оставить там только словари со страной Россия

x = "Россия"
geo_logs = [
    {'visit1': ['Москва', 'Россия']},
    {'visit2': ['Дели', 'Индия']},
    {'visit3': ['Владимир', 'Россия']},
    {'visit4': ['Лиссабон', 'Португалия']},
    {'visit5': ['Париж', 'Франция']},
    {'visit6': ['Лиссабон', 'Португалия']},
    {'visit7': ['Тула', 'Россия']},
    {'visit8': ['Тула', 'Россия']},
    {'visit9': ['Курск', 'Россия']},
    {'visit10': ['Архангельск', 'Россия']},
    {'visit11': ['Минск', 'Беларусь']},
    {'visit12': ['Париж', 'Франция']}
]

for element in geo_logs:
    for visit, country in element.items():
        if country[1] != x:
            geo_logs.remove(element)
print(geo_logs)

я написал код которой в моем представлении должен просто удалять элементы не содержащие "Россия", но у меня не удаляет то не только еще и то что содержит "Франция"

Может кот-то объяснить в чем дело и почему так выходит ?

Answer 1

[ScriptKiddo]

Нельзя делать итерацию по списку и одновременно удалять элементы. Делайте итерацию по копии и удаляйте из исходного массива.
for element in geo_logs[:]:

Comments

[Andrei Sayevich]

да, я понял )
и изначально так сделал ...
но решил "оптимизировать" и не создавать доп список...
ошибок не получил но!
странность есть толко с Францией

[Алан Гибизов]

Хитро! То—есть оно делает срез исходного списка и потом исходный уже не дергает? Красота!

[Алан Гибизов]

Andrei Sayevich, да замени Францию на Белорусию — ничего не изменится.

Answer 2

[Алан Гибизов]

Полагаю, дело в том, что вы делаете remove и меняете список. И продолжаете по нему двигаться, но он уже изменился, а вы полагаете, что он тот же. Дело не во Франции, это случайность.

Попробуйте не менять исходный список, а писать в новый — всё должно заработать.

Comments

[javedimka]

Если бы чел менял словарь во время итерации - питон Бы кинул ошибку. Он элементы из списка во время его обхода удаляет, поэтому после .remove() следующий на очереди элемент занимает место текущего и в итоге просто пропускается циклом, т.к эта «позиция» в списке уже была пройдена

[Andrei Sayevich]

Вожможно, но почему тогда удаляется Индия, Португалия, Беларусь, а франция нет ?

[Алан Гибизов]

javedimka, поясните, пожалуйста, это разве не изменение словаря?

[javedimka]

Алан Гибизов, посмотри откуда он remove делает, из списка, словарь тут не причём

[Алан Гибизов]

javedimka, точно. Не словарь, список. Глаз замылился, спать пора. Спасибо.

[Andrei Sayevich]

javedimka, окей, как это решить ?

[Алан Гибизов]

javedimka, кстати, а почему Python не ругается на изменение списка во время прохождения оного циклом? Потому что этот список как-бы уже результат работы итератора?

[javedimka]

Andrei Sayevich, например так:

res = [d for d in geo_logs if 'Россия' in list(*d.values())]

[Алан Гибизов]

Andrei Sayevich, я же сказал — в НОВЫЙ список класть элементы с Россией.

[Andrei Sayevich]

javedimka, так как я только начал изучать ... мне не совсем понятен ваше решение ( хоть он и работает отлично! )
не могли бы немного расписать его логику?

[javedimka]

Алан Гибизов, Этого я не знаю, наверное что-то со Смарт объектами словаря связано

Andrei Sayevich, просто краткая форма записи такого:

res = []
for e in geo_lang:
    if x in e.values()[0]:
        res.append(e)

[Алан Гибизов]

javedimka, кстати, а почему в list comprehension распаковку использовал, а не [0]? Только из краткости?

[javedimka]

Алан Гибизов, потому что список может быть пустым, или наоборот иметь много значений. В первом случае будет эксепшен, а в другом, возможно, нежелательный результат. В случае автора можно и нуль.