Можете ли это выражение понимания списка Python быть упрощенным?

Question

input = "foo ,,bar ,baz," 
tags = [x.strip() for x in input.split(',') if len(x.strip()) > 0] 

Желаемый результат, очевидно, является списком, в котором нет пустых строк.

Вопрос в духе микро оптимизации; есть ли способ не strip() кандидат x дважды, то есть один раз для теста и один раз для добавления?

Чтобы перефразировать, вы можете создать значение в выражениях, которые могут быть добавлены в список, без выполнения работы дважды?

Answer 1

Создание новой строки всегда будет дороже, чем сканирование. x.isspace() вернется после того, как первый не символ пробела встречается

tags = [x.strip() for x in input.split(',') if x and not x.isspace()] 

Answer 2

text = 'foo ,,bar ,baz,' 

(я использую text, а не input, как input это имя встроенной команды. Избегайте затенения встроенных команд.)

Прежде всего, len(x.strip()) > 0 можно записать просто (и более эффективно) в качестве x.strip().

tags = [x.strip() for x in text.split(',') if x.strip()] 

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

tags = [x for x in (x.strip() for x in text.split(',')) if x] 

Если вы действительно хотите, вы могли бы даже сделать это функционально ...

tags = filter(bool, map(lambda x: x.strip(), text.split(','))) 

показатели эффективности:

>>> from timeit import timeit 
>>> timeit(lambda: [x.strip() for x in text.split(',') if x.strip()]) 
1.9443869590759277 
>>> timeit(lambda: [x for x in (x.strip() for x in text.split(',')) if x]) 
2.1135239601135254 
>>> timeit(lambda: filter(bool, map(lambda x: x.strip(), text.split(',')))) 
2.52907395362854 

Как вы видите, первое быстрее.

Answer 3

это тоже работает ...

text = "foo ,,bar ,baz," 
text.replace(',',' ').split()