在函数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

您无需再创建一个功能你正在尝试实现。在外部创建函数更好，这样您就可以在其他函数中调用它，而不是将它的使用仅限制在写入的位置。