public function addAction() { return parent::addAction(); }
在我看来
除了恶心我找方法的时候 IDE 给我一堆一样的提示没有任何意义。
![]() | 1 hibobby 2017-12-25 15:45:15 +08:00 本来打算重写的吧 |
2 HowToMakeLove 2017-12-25 16:11:09 +08:00 看你怎么理解吧 |
![]() | 3 chairuosen 2017-12-25 16:12:55 +08:00 增加代码行数。。 |
![]() | 4 zjsxwc 2017-12-25 16:45:49 +08:00 addAction 看起来是机器生产的代码就没毛病. 一般人也不会去(懒的去)多写这种代码 |
![]() | 5 forestyuan 2017-12-25 16:55:17 +08:00 说明公司没有 code review 啊 |
6 zj299792458 2017-12-25 17:01:08 +08:00 为了提醒你 parent 还有这个方法可用,别自己再写一个了 |
7 gloryLi 2017-12-25 17:38:53 +08:00 应该是想重写的把。。。看的有点尴尬。。。 |
8 Mitt 2017-12-25 17:40:42 +08:00 可能缺个 // TODO |
![]() | 9 lwbjing 2017-12-25 17:40:54 +08:00 意义是今天至少写了两行代码了。。 |
![]() | 10 ihuotui 2017-12-25 20:26:31 +08:00 via iPhone 单看一个函数没有意义,看整体类,是不是继承的,抽象的。在 java 语言中 |
11 lihongjie0209 2017-12-25 20:37:02 +08:00 为以后的扩展留下的? |
![]() | 12 ZyZyZzz 2017-12-25 20:43:24 +08:00 via Android 醒醒,这题根本信息不足好嘛 为什么不能是装饰者模式中的一层呢? |
![]() | 13 changnet 2017-12-25 20:53:39 +08:00 via Android 抛开业务谈就没有任何意义。也许之前还有几行逻辑代码,后来需要变了改没了。也许是业务有需求预留的。也许只是工具自动产生的。也许是改某个 bug,强制调 parent。也许是你们的框架为了方便维护,这样要求的...... |
![]() | 14 t2doo 2017-12-25 21:04:28 +08:00 哎呀,看了楼主的这个贴子,我准备把我瞎写的代码都重构了,不然给接手的人看到,得疯 |