Как исправить код для решения задачи?

Question

[Илья]

Знатоки, подскажите, пожалуйста, где у меня ошибка в коде...
Суть задачи: Написать функцию modify_list(l), которая принимает на вход список целых чисел, удаляет из него все нечётные значения, а чётные нацело делит на два. Функция не должна осуществлять ввод/вывод информации, требуется только изменение переданного списка.
Мой код:

def modify_list(l):
    for i in range(len(l)):
        if i % 2 == 1:
            l.remove(l[i])
        elif i % 2 == 0:
            i = l.append(l[i//2])
    return (l)

Просьба не присылать другие решения, а исправить именно данный фрагмент.

Comments

[MinTnt]

Вообще, странная немного задача, "изменять изначальный список". Обычно так не делают, и в случае чего создают копию...

[MinTnt]

Стоп. Не понял

Функция не должна ничего возвращать, требуется только изменение переданного списка.

А это тогда....

return (l)

?

[Илья]

MinTnt, видимо подразумевается отсутствие input и output. А return просто возвращает новое значение в функцию, которую впоследствии можно будет вызвать в программе, если я все правильно понимаю. P.S условие задачи скопипастил, а автор заданий не очень ясно их формулирует...

[MinTnt]

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

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

Отредактируй текст вопроса, оберни код тегом code.

[Илья]

Алан Гибизов, Новую функцию на Хабре для себя открыл )) Спасибо за подсказку)

Answer 1

[Vindicar]

Нужно быть осторожным при переборе индексов списка и его одновременном изменении. Если ты удалил элемент - индексы последующих "съедут" влево, к нулю, и при обычном переборе ты пропустишь элемент сразу за удалённым.
Хорошее решение - перебирать элементы в обратном порядке, тогда "съедут" индексы уже обработанных элементов.

for i in range(len(l)-1, -1, -1):
  if l[i] % 2 == 0:
    l[i] = l[i] / 2
  else:
    del l[i]

Comments

[Илья]

(len(l)-1, -1, -1) а здесь мы, получается, делаем перебор в диапазоне, где левая граница: -1; правая граница: -1; и шаг: -1?

[Vindicar]

Илья, не совсем.
Начало диапазона - длина l минус 1, т.е. последний элемент.
Шаг диапазона - минус 1, т.е. перебираем от больших к меньшим.
Конец диапазона - минус 1, но так как конец не включается в диапазон, то фактически остановимся на предыдущем значении, т.е. 0.

Просто введи в консоль питона list(range(10, -1, -1)), и увидишь.

[Илья]

Vindicar, Понятно, спасибо за разъяснения.