在函數python中定義函數是否很好

Question

下面是我在一個項目中編寫的代碼。

哪一個是一個更好的寫蟒蛇？

def get_list_of_university_towns(): 
    .... 
    def parse_state(item): 
     return re.sub('\[edit\]', '', item) 
    uni_towns['State'] = uni_towns['State'].apply(parse_state) 
    return uni_towns 

或者：

def parse_state(item): 
    return re.sub('\[edit\]', '', item) 
def get_list_of_university_towns(): 
    .... 
    uni_towns['State'] = uni_towns['State'].apply(parse_state) 
    return uni_towns 

這種 「parse_state（項目）」 功能只在調用一次 「get_list_of_university_towns（）」，將再次從未使用過。我個人認爲在函數內定義它會更容易理解。但是，我在其他人的項目中幾乎看不到這種代碼。

所以，我應該怎麼寫這段代碼？

Answer 1

Pythonic在函數內寫funciotns

是的。實際上，爲了不污染模塊命名空間，它在外部做了更多Pythonic。

我應該如何寫這篇文章的代碼？

在其他函數中使用函數定義的選項有效。另一種Python的方式是使用匿名lambda函數：

def get_list_of_university_towns(): 
    .... 
    uni_towns['State'] = uni_towns['State'].apply(lambda item: re.sub('\[edit\]', '', item)) 
    return uni_towns 

優化

因爲它已被sugested，而現在，你說這是熊貓的數據幀是什麼意思，該函數將被調用一次以上，您應該編譯expresion或使用str.replace()代替re.sub()：

def get_list_of_university_towns(): 
    .... 
    uni_towns['State'] = uni_towns['State'].apply(lambda item: item.replace('[edit]', '')) 
    return uni_towns 

Answer 2

兩者都是罰款。第一個更清潔，因爲您不會污染模塊名稱空間，而其名稱不會在別處使用。

與第一表格應該是稍快爲parse_state是一個局部變量，當它被調用時，如果它是一個循環內使用尤其重要。在另一個函數中定義函數沒有運行時成本。在運行時，這是一個簡單的任務。

但是，如果它是一個循環內使用，你也應該在編譯的模塊範圍內的正則表達式：

_state_re = re.compile('\[edit\]') 
def get_list_of_university_towns(): 
    ... 
    def parse_state(item): 
     return _state_re.sub('', item) 
    uni_towns['State'] = uni_towns['State'].apply(parse_state) 
    return uni_towns 

Answer 3

我Adirio的答案達成一致，如果需要的功能。

或者，你可以考慮功能是否真的有必要，如果它只能使用一次。 如果可以遍歷uni_towns [「國家」]，這將達到相同的，在我看來更具可讀性：

def get_list_of_university_towns(): 
    .... 
    for state in uni_towns['State']: 
     state = re.sub('\[edit\]', '', state) 
    return uni_towns 

Answer 4

您無需再創建一個功能你正在嘗試實現。在外部創建函數更好，這樣您就可以在其他函數中調用它，而不是將它的使用僅限制在寫入的位置。