Список за пределами цикла

Question

im пытается сделать функцию, которая просматривает список и удаляет все нечетные числа. У меня проблема, когда моя петля выходит за границы, и я не уверен, как ее исправить, может ли кто-нибудь помочь? Благодаря

def isEven(number): 
    if(number % 2) == 0 and (number != 0): 
     return True 
    else: 
     return False 

def only_even(intList): 
    for i in range(0, len(intList)): 
     if(isEven(intList[i])) == False: 
      del intList[i] 
    return intList 

print(only_even([5, 8, 17, 6, 0, 20, 3])) 

Answer 1

при использовании del, вы изменяете длину списка, но 0 Циклпытается перебрать исходную длину списка, поэтому, в конце концов, он выйдет за пределы списка. То есть len(intList) не оценивается каждый раз for.

Решение этой проблемы было бы создать новый список из старого списка и вернуться, что, например, так:

def only_even(intList): 
    new_list = [] 
    for i in range(0, len(intList)): 
     if isEven(intList[i]) == True: 
      new_list.append(intList[i]) 
    return new_list 

Теперь ваш оператор печати будет работать:

>>> print(only_even([5, 8, 17, 6, 0, 20, 3])) 
[8, 6, 20] 

Answer 2

Причина список выходит за пределы, потому что вы перебор в диапазоне от исходного размера в то время как сокращение списка!

def only_even(intList): 
    intList[:] = [i for i in intList if isEven(i)] 
    return intList 

Я делаю назначение ломтика [:] сохранить точную семантику мутирует и возвращает список. Если вы просто хотите, чтобы вернуть список его еще короче

def only_even(intList): 
    return [i for i in intList if isEven(i)] 

Вы также можете упростить ваши даже функция, так как она уже имеет значение True/False

def isEven(number): 
    return number % 2 == 0 and number != 0 # Why isn't 0 even? 

Answer 3

def only_even(intList): 
    intList[:] = filter(isEven, intList) 
    return intList 

Или, если вы не «т нужно изменить список на месте:

def only_even(intList): 
    return list(filter(isEven, intList))