Python хорошая практика программирования для перечисления списков

Question

Я довольно новичок в Python и программировании вообще, и мне было интересно, хорошо ли это практика программирования для написания длинных операторов со многими логическими операторами - например, в цикле for.

Например, вот функция, которую я создал, которая получает все гласные от слова и возвращает список, содержащий эти гласные.

def getVowels(word): 
    vowel_list = [] 
    index = 0 
    for i in word: 
     if i == "a" or i == "e" or i == "i" or i == "o" or i == "u" or i == "A" or i == "E" or i == "I" or i == "O" or i == "U": 
      vowel_list.append(word[index]) 
     index += 1 
    return vowel_list 

Как вы можете видеть, инструкция if получила очень много времени. Это считается хорошим программированием? Если это не так, есть ли лучший способ кодировать эту функцию?

Answer 1

Нет, это не считается хорошей практикой, всегда есть более эффективные способы: D

if i.upper() in "AEIOU" 

Здесь гораздо более короткая версия вашей функции с помощью списковых:

def get_vowels(word): 
    vowels = "AEIOU" 
    return [c for c in word if c.upper() in vowels] 

Answer 2

if является такие как:

if i in "aeiouAEIOU" 

В основном вы проверяете член в наборе.

Answer 3

Я думаю, что длинные заявления сложнее понять, чем короткие. Почти всегда есть способ сделать то же самое с более короткими утверждениями. В вашем случае, вы можете упростить, если так:

def getVowels(word): 
    vowel_list = [] 
    for i in word: 
     if i in "aeiouAEIOU": 
      vowel_list.append(i) 
    return vowel_list 

поскольку Python позволяет использовать «в» оператор для поиска одной строки внутри другой.

Но Python позволяет списковых, упрощающие петли:

def getVowels(word): 
    return [i for i in word if i in "aeiouAEIOU"] 

Answer 4

бы, вероятно, будет лучше использовать наборы:

VOWELS = set('aeiouAUIOU') 

def get_vowels(word): 
    return [c for c in word if c in VOWELS] 

или, более вызывающим:

def get_vowels(word): 
    return filter(VOWELS.__contains__, word) 

(Но первый подход является наиболее читаемым и, как таковой, более питоническим. Кроме того, вторая функция вернет генерато г, а не список в Python 3.)

EDIT сравнение производительности c in list против c in set:

import timeit 

VOWELS = 'aeiouAEIOU' 
VOWSET = set(VOWELS) 
SAMPLE = 'asflasrjoperugASDFAROUAoarfpeoriugargagadropgue' 

def get_vowels(word, vowels): 
    return [c for c in word if c in vowels] 

print timeit.timeit('get_vowels(SAMPLE, VOWELS)', 
        'from __main__ import VOWELS, SAMPLE, get_vowels') 
#^prints 10.0739870071 
print timeit.timeit('get_vowels(SAMPLE, VOWSET)', 
        'from __main__ import VOWSET, SAMPLE, get_vowels') 
#^prints 9.43965697289 

Answer 5

Как:

set('aeiouAUIOU') & set(word)