Dobre praktyki programowania Python do wyliczania list

Question

Jestem całkiem nowy w Pythonie i programowaniu w ogóle, i zastanawiałem się, czy dobrą praktyką programowania jest pisanie długich instrukcji z wieloma operatorami logicznymi - na przykład w pętli for.

Na przykład, oto funkcja, którą stworzyłem, która pobiera wszystkie samogłoski od słowa i zwraca listę zawierającą te samogłoski.

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 

Jak widać, instrukcja if stała się bardzo długa. Czy uważa się za dobre programowanie? Jeśli tak nie jest, czy istnieje lepszy sposób na zakodowanie tej funkcji?

Answer 1

Nie, to nie jest uważane za dobre praktyki, nie zawsze są lepsze sposoby: D

if i.upper() in "AEIOU" 

Tutaj jest znacznie krótsza wersja swojej funkcji przy użyciu wyrażeń listowych:

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

Answer 2

if jest tak samo jak:

if i in "aeiouAEIOU" 

Zasadniczo masz sprawdzanie członka w zestawie.

Answer 3

Myślę, że długie stwierdzenia są trudniejsze do zrozumienia niż krótkie. Niemal zawsze istnieje sposób na robienie tego samego z krótszymi wypowiedziami. W twoim przypadku, można uprościć, jeśli tak:

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

ponieważ Python pozwala użyć operatora „in”, aby szukać jeden ciąg wewnątrz drugiego.

Ale Python pozwala również listowych, które upraszczają pętle:

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

Answer 4

będzie prawdopodobnie lepiej użyć zestawach:

VOWELS = set('aeiouAUIOU') 

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

lub bardziej naukowy:

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

(Ale pierwsze podejście jest najbardziej czytelne i jako takie jest bardziej pythonic.Również druga funkcja zwróci generato r, a nie lista w Pythonie 3.)

EDIT porównanie wydajności c in list vs 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

odczuwalna:

set('aeiouAUIOU') & set(word)