목록 열거를위한 파이썬 좋은 프로그래밍 연습

Question

저는 파이썬과 프로그래밍에 익숙하지 않습니다. 많은 논리 연산자를 사용하여 긴 명령문을 작성하는 것이 좋은 프로그래밍 방법인지 궁금합니다. 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 

파이썬은 다른 하나의 내부 문자열을 검색하기 위해 "에서"연산자를 사용 할 수 있기 때문이다.

그러나 파이썬은 또한 루프를 단순화 지능형리스트를 수 있습니다 :

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) 

을 (하지만 첫 번째 접근법은 가장 읽기 쉽고 더 파이썬 적이기 때문에 두 번째 함수는 generato R, 파이썬 3에서하지리스트)

편집 c in set 대 c in list의 성능 비교 :

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) 

당신은 [`enumerate`] (HTTP를 사용한다