Python buena práctica de programación para enumerar listas

Question

Soy bastante nuevo en Python y la programación en general, y me preguntaba si es una buena práctica de programación escribir sentencias largas con muchos operadores lógicos, por ejemplo, en un bucle for.

Por ejemplo, aquí hay una función que hice que obtiene todas las vocales de una palabra y devuelve una lista que contiene esas vocales.

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 

Como puede ver, la instrucción if ha sido muy larga. ¿Se considera una buena programación? Si no es así, ¿hay una mejor manera de codificar esta función?

Answer 1

No, no se considera una buena práctica, siempre hay mejores formas: D

if i.upper() in "AEIOU" 

Aquí está una versión mucho más corta de su función usando listas por comprensión:

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

Answer 2

El if es el Lo mismo que:

if i in "aeiouAEIOU" 

Básicamente está buscando miembro en un conjunto.

Answer 3

Creo que las declaraciones largas son más difíciles de entender que las breves. Casi siempre hay una manera de hacer lo mismo con declaraciones más cortas. En su caso, se puede simplificar el caso de la siguiente manera:

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

desde Python le permite usar el operador "in" para buscar una cadena dentro de otra.

Pero Python también permite que las listas por comprensión, que simplifican bucles:

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

Answer 4

probablemente sería mejor utilizar conjuntos:

VOWELS = set('aeiouAUIOU') 

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

o, más geek:

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

(Pero el primer enfoque es más legible y, como tal, es más pitónico. Además, la segunda función devolverá r, no una lista en Python 3.)

EDITAR comparación de rendimiento de 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

igual:

set('aeiouAUIOU') & set(word)