У меня есть этот код:
numbers = range(1, 50)
for i in numbers:
if i < 20:
numbers.remove(i)
print(numbers)
но результат я получаю:
[2, 4, 6, 8, 10, 12, 14, 16, 18, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49]
Конечно, я ожидаю, что цифры ниже 20 не появятся в результатах, я предполагаю, что я делаю что-то не так с удалением.
Вы изменяете список, пока вы его перебираете. Это означает, что в первый раз через цикл, i == 1, поэтому 1 удаляется из списка. Затем цикл for переходит ко второму элементу в списке, который не равен 2, но 3! Затем это удаляется из списка, а затем цикл for переходит к третьему элементу в списке, который теперь равен 5. И так далее. Возможно, это проще визуализировать так, с помощью ^, указывающего на значение i:
[1, 2, 3, 4, 5, 6...]
^
Это состояние списка изначально; то 1 удаляется и цикл переходит ко второму элементу в списке:
[2, 3, 4, 5, 6...]
^
[2, 4, 5, 6...]
^
И так далее.
Там нет хорошего способа изменить длину списка, итерации по нему. Лучшее, что вы можете сделать, это что-то вроде этого:
numbers = [n for n in numbers if n >= 20]
или это, для изменения на месте (вещь в parens - это выражение генератора, которое неявно преобразуется в кортеж перед назначением slice):
numbers[:] = (n for in in numbers if n >= 20)
Если вы хотите выполнить операцию над n перед ее удалением, вы можете попробовать один трюк:
for i, n in enumerate(numbers):
if n < 20 :
print "do something"
numbers[i] = None
numbers = [n for n in numbers if n is not None]
Ответ @senderle - это путь!
Сказав это, чтобы еще больше проиллюстрировать вашу проблему, если вы подумаете об этом, вы всегда захотите удалить индекс 0 двадцать раз:
[1,2,3,4,5............50]
^
[2,3,4,5............50]
^
[3,4,5............50]
^
Таким образом, вы могли бы пойти с чем-то вроде этого:
aList = range(50)
i = 0
while i < 20:
aList.pop(0)
i += 1
print aList #[21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49]
Я надеюсь, что это помогает.
Приведенные ниже не плохие практики AFAIK.
РЕДАКТИРОВАТЬ (еще немного):
lis = range(50)
lis = lis[20:]
Будет делать работу также.
EDIT2 (мне скучно):
functional = filter(lambda x: x> 20, range(50))
Удаление элементов из списка прост: начните с конца списка:
li = range(1,15)
print li,'\n'
for i in xrange(len(li)-1,-1,-1):
if li[i] < 6:
del li[i]
print li
результат
[1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14]
[6, 7, 8, 9, 10, 11, 12, 13, 14]
Мне очень нравится ответ @senderle. Единственное, что я хочу добавить (я бы прокомментировал, но не могу получить правильное форматирование кода), так это то, что вы также можете уйти с боковой обработкой для своих списков, если объявите свои функции раньше времени:
def checkRemoval(item):
# some processing here
return passed_inspection # Should be True / False.
newList = [item for item in oldList if checkRemoval(item)]