Смотрю на чужой код и не могу понять логику. Вот кусок:
def process(items):
result = []
for i in range(len(items)):
if items[i] != None:
result.append(items[i])
return resultЧто они делают? В чем их проблема? Есть ли более питоничный способ написать то же самое?
Три проблемы сразу.
Первая: items[i] != None - сравнение с None через != считается плохим тоном в Python. Правильно: is not None.
Вторая: for i in range(len(items)) - это антипаттерн. Когда нужен только элемент, не надо итерировать по индексам.
Третья: весь этот цикл заменяется одной строчкой:
result = [item for item in items if item is not None]Или ещ... еще короче через filter:
result = list(filter(None.__ne__, items))Хотя первый вариант читабельнее.
filter(lambda x: x is not None, items) тоже вариант если list comprehension не нравится
Вообще говоря, прежде чем писать такой код, стоит задуматься: а зачем в списке вообще появляются None? Это обычно симптом более глубокой проблемы в архитектуре. Возможно, функция, которая генерирует items, должна сама гарантировать чистые данные. Фильтрация None постфактум - это пластырь поверх пореза.
Кстати, если хочешь убрать не только None но и пустые строки, нули и вообще все falsy значения, то просто:
result = list(filter(None, items))Но это уже другое поведение, зависит от задачи.
Вот это вот range(len(items)) мне в свое время объяснял препод на первом курсе. Говорил "так делать можно, но так не делают". Прошло десять лет, смотрю такой код каждую неделю на ревью. Каждую. Неделю.
Спасибо! Про `is not None` знал, а вот range(len()) как антипаттерн - не догадывался. Поправил.