Объясните как работает в данном случае программа?

Question

[LGNadin]

Хочу удалить элемент списка, если его длина равна = 0. Вроде все удаляет, но никак не пойму как работает счетчик i, почему после удаления элемента списка '-' он (i) все равно принимает в конце значение 4, хотя список стал короче и последнее значение у i должно = 3

lst = ['Hello?', 'my,', '-', 'friend!', 'hoy']
i = 0
for i in range(len(lst)):
    if len(lst[i]) == 1:
        lst.pop(i)
print(lst)

Comments

[Alex Maximovich]

Я конечно не разработчик на питон, но все просто и логично; Итерируемся по массиву lst. Если длинна строки, что является элементом массива = 1 => удаляем ее из массива. На выходе должно получиться что-то вроде этого: ['Hello?', 'my,', 'friend!', 'hoy'] Но тут косяк будет с индексами, так как мы изменяем исходную коллекцию и потом пытаемся получить элемент по индексу, который уже будет выходить за ее пределы.

Как вариант можно изменить вот так:

lst = ['Hello?', 'my,', '-', 'friend!', 'hoy']
i = 0
indexies = []
for i in range(len(lst)):
    if len(lst[i]) == 1:
        indexies.append(i);

for i in range(len(indexies)):
    lst.pop(indexies[i])
print(lst)

[Sand]

У Вас действительно это работает?

[LGNadin]

Sand, конечно же нет)

[pfemidi]

А разве корректно изменять коллекцию, по которой итерируешься? Всегда думал что в подобных случаях надо в процессе итераций по исходной коллекции создавать временную, а после окончания цикла присваивать временную исходной. Я неправ?

[LGNadin]

pfemidi, я только учусь. Не судите строго)

Answer 1

[zexer]

Реализовать удаление элементов с длиной равной единице лучше так
[i for i in list if len(i) !=1]

Answer 2

[Дмитрий Шицков]

range(len(lst)) вычисляется один раз, при заходе в цикл, а не вычисляется при каждой итерации.

Comments

[LGNadin]

Спасибо, я думал что после изменения списка поменялась длина и соответственно range

[pfemidi]

Дмитрий Шицков, но после lst.pop(i) индексы ведь меняются. А range(len(lst)) остаётся прежним, вычисленным единожды. Как это вообще может работать? Не понимаю.

Answer 3

[kurrbanov]

После того, как вы внутри range указали длину списка, он сохраняет изначальную его длину, переменная внутри range не изменяется.
Простыми словами, он запомнил, что len(lst) = 5 и дальше не будет изменять этот параметр.

В вашем случае лучше длину списка задать изначально и задать условие выхода:

n = len(lst)

for i in range(n):
    if i == n:
        break
    if len(lst[i]) == 1:
        lst.pop(i)
        n -= 1

Comments

[LGNadin]

Спасибо за подробное объяснение!

[Wataru]

LGNAdin, Мне кажется, тут через while будет читабельнее:

n = len(lst)
while i < n:
    if len(lst[i]) == 1:
        lst.pop(i)
        n -= 1
    else:
        i += 1

А код сверху, кстати некоректно работает, если надо удалять 2 элемента подряд - второй будет пропущен, потому что после удаления i-ого элемента, следующий съедет на его место, но цикл for все-равно увеличит счетчик i.

[kurrbanov]

Илья Николаевский, согласен с вами, признаю свою ошибку(

Answer 4

[soremix]

Потому что ваш цикл выглядит как
for i in range(5):
Попробуем посчитать
0
1
2
3
4