对于不需要返回数据的函数,在最后一行写"return"是否是一个好习惯?

2011-09-28 00:08:47 +08:00
 reorx
6070 次点击
所在节点    Python
12 条回复
Livid
2011-09-28 00:09:51 +08:00
没有必要一定写 return。
ayanamist
2011-09-28 09:12:51 +08:00
看团队的风格,有些团队是从C转过来的,要求显式返回的代码风格。
一般是不主张在没有返回的情况下最后加一个return的。在Scala中甚至不主张显式使用return,而是把要返回的值放在最后一行
reorx
2011-09-28 13:52:21 +08:00
我是遇到了这样的情况,和同伴一起工作,预定义一些函数来统一接口:

class FooHdr:
...
def func_a(self):
"""Description & Usage"""
return

def func_b(self):
"""Description & Usage"""
return

为了保证程序能正常运行,这些函数肯定是不能留空的,那么就是加return和加pass的选择了。

我想的是,对于需要返回数据的函数,完成之后,直接在return后面加上变量就可以了,对于不需要返回的,也省去了用pass时要删掉的麻烦。总而言之是出于协作规范的考虑,不过根据Google的Python规范来看,这样做确有画蛇填足之嫌了。
levey
2011-09-28 13:56:46 +08:00
void的我不写return。
istef
2011-09-28 13:58:18 +08:00
感觉是个人风格问题,我貌似就是比较喜欢写 return 那种类型的。。。。
chuangbo
2011-09-28 14:03:12 +08:00
Pythonic 的话就不写,会自动返回 None
reorx
2011-09-28 14:13:14 +08:00
@chuangbo 忽然记起曾经在github上看过一种代码风格,就是全都写return None...
huangz
2011-09-28 14:14:37 +08:00
处理这样的空函数,比起pass或者return,更好的选择是显式地抛出一个异常:

def fun():
____raise NotImplementedError

这样一来起到了占位符的作用,二来可以防止你在程序中误用这个未完成的函数——一些单纯依靠副作用的函数因为返回None而“骗过”测试,这样的错误并不少见。
reorx
2011-09-28 14:31:16 +08:00
@huangz fantastic solution! 抛出异常确实是处理空函数的最佳方法。

看了一下NotImplementedError的定义:
“In user defined base classes, abstract methods should raise this exception when they require derived classes to override the method.”

虽然是用于父类需要重写的方法,不过在意义上和“没有部署的函数”是相近的,很合适,毕竟我们需要的就是异常而已。
huangz
2011-09-28 15:36:04 +08:00
@reorx

是的,反正只要抛出异常就行了,Python的异常里面,NotImplementedError算是比较符合占位符目的的一个了(望文见义嘛)。

我自己一般用Exception,因为可以少打点字,哈哈。
ccp0101
2011-09-28 20:20:50 +08:00
C中我全加return。Python一般都只是一次性用用看心情。
Numbcoder
2011-09-28 20:29:51 +08:00
Ruby 里不需要return,最后一行就是返回值。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/18705

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX