1

У меня есть следующий простейший скрипт

inp = input().split()
print('inp', inp)
l = []
for elem in inp:
    l.append(int(elem))

print('initial: ', l)

for el in l:
    print(el)
    if el % 2 != 0:
        l.remove(el)

print('after removing odd numbers: ', l)

Первый инпут (строка):

1 2 3 4 5 6 7

вывод:

inp ['1', '2', '3', '4', '5', '6', '7']
initial:  [1, 2, 3, 4, 5, 6, 7]
1
3
5
7
after removing odd numbers:  [2, 4, 6]

Второй инпут (строка):

10 8 5 3 1

вывод:

inp ['10', '8', '5', '3', '1']
initial:  [10, 8, 5, 3, 1]
10
8
5
1
after removing odd numbers:  [10, 8, 3]

Скриншот здесь.

1й ввод/вывод

1) Почемы скрипт выводит только нечетные элементы? Он должен выводить все элементы.


2й ввод/вывод

1) Почему в выводе нет числа 3, только 10, 8, 5 и 1?

2) И, следовательно, почему конечный список содержит число 3?

Благодарю!

0

2 ответа 2

4

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

Покажу наглядно на этом коде:

a = [1, 2, 3, 4, 5]
for x in a:
    a.remove(x)

Первая итерация до удаления — берётся элемент с индексом 0:

[1, 2, 3, 4, 5]
 ^ - текущий элемент

Первая итерация после удаления (обращаю внимание, что значение переменной x никто не поменяет):

[2, 3, 4, 5]
 ^

Упс, мы неявно поменяли текущий элемент. Хотя в переменной x всё ещё содержится единица, по индексу 0 уже находится совсем другой элемент.

Вторая итерация — берётся элемент с индексом 1:

[2, 3, 4, 5]
    ^

Упс! Проскочили.


Как лечить?

Никогда не менять список, по которому проходитесь циклом. Например, сделать его копию и запустить цикл уже по копии:

l_copy = l[:]
for el in l_copy:
    print(el)
    if el % 2 != 0:
        l.remove(el)

Для лучшего понимания работы цикла ещё пример:

a = [0, 0, 0, 0]
i = 1
for x in a:
    a.insert(i, 7)
    print(x)
    i += 1

Этот код отпечатает сперва 0, а потом бесконечно будет выдавать семёрки :)

a = [0, 1, 2, 3]
i = 0
for x in a:
    a.insert(i, 7)
    print(x)
    i += 1

А вот так семёрок не будет, а будут бесконечные нули (мы постоянно смещаем его вправо новыми семёрками и он постоянно оказывается следующим элементом по индексу).

2
  • Спасибо, теперь все стало на свои места! 20 мая 2016 в 7:47
  • бесконечно будет выдавать семёрки только если у нас будет бесконечно памяти 26 сен 2017 в 10:32
1

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

a = [1, 2, 3, 4, 5]
b = [el for el in a if el%2 == 0]
print (b)

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.