对于不需要返回数据的函数,在最后一行写"return"是否是一个好习惯? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
推荐学习书目
Learn Python the Hard Way
Python Sites
PyPI - Python Package Index
http://diveintopython.org/toc/index.html
Pocoo
值得关注的项目
PyPy
Celery
Jinja2
Read the Docs
gevent
pyenv
virtualenv
Stackless Python
Beautiful Soup
结巴中文分词
Green Unicorn
Sentry
Shovel
Pyflakes
pytest
Python 编程
pep8 Checker
Styles
PEP 8
Google Python Style Guide
Code Style from The Hitchhiker's Guide
reorx
V2EX    Python

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

  •  
  •   reorx
    reorx 2011-09-28 00:08:47 +08:00 6677 次点击
    这是一个创建于 5193 天前的主题,其中的信息可能已经有所发展或是发生改变。
    12 条回复    1970-01-01 08:00:00 +08:00
    Livid
        1
    Livid  
    MOD
    PRO
       2011-09-28 00:09:51 +08:00
    没有必要一定写 return。
    ayanamist
        2
    ayanamist  
       2011-09-28 09:12:51 +08:00
    看团队的风格,有些团队是从C转过来的,要求显式返回的代码风格。
    一般是不主张在没有返回的情况下最后加一个return的。在Scala中甚至不主张显式使用return,而是把要返回的值放在最后一行
    reorx
        3
    reorx  
    OP
       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
        4
    levey  
       2011-09-28 13:56:46 +08:00
    void的我不写return。
    istef
        5
    istef  
       2011-09-28 13:58:18 +08:00
    感觉是个人风格问题,我貌似就是比较喜欢写 return 那种类型的。。。。
    chuangbo
        6
    chuangbo  
       2011-09-28 14:03:12 +08:00
    Pythonic 的话就不写,会自动返回 None
    reorx
        7
    reorx  
    OP
       2011-09-28 14:13:14 +08:00
    @chuangbo 忽然记起曾经在github上看过一种代码风格,就是全都写return None...
    huangz
        8
    huangz  
       2011-09-28 14:14:37 +08:00
    处理这样的空函数,比起pass或者return,更好的选择是显式地抛出一个异常:

    def fun():
    ____raise NotImplementedError

    这样一来起到了占位符的作用,二来可以防止你在程序中误用这个未完成的函数一些单纯依靠副作用的函数因为返回None而“骗过”测试,这样的错误并不少见。
    reorx
        9
    reorx  
    OP
       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
        10
    huangz  
       2011-09-28 15:36:04 +08:00
    @reorx

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

    我自己一般用Exception,因为可以少打点字,哈哈。/div>
    ccp0101
        11
    ccp0101  
       2011-09-28 20:20:50 +08:00
    C中我全加return。Python一般都只是一次性用用看心情。
    Numbcoder
        12
    Numbcoder  
       2011-09-28 20:29:51 +08:00
    Ruby 里不需要return,最后一行就是返回值。
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     3212 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 26ms UTC 11:30 PVG 19:30 LAX 03:30 JFK 06:30
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86