方法中要增加参数,但是项目中这个方法很多地方用了,大家会怎么做?

2017-08-16 11:07:56 +08:00
 guyeuro

方法中要增加参数,但是项目中这个方法很多地方用了,

这个方法所在的类也通过 dubbo 暴露给其他服务了

大家怎么做?

5122 次点击
所在节点    程序员
45 条回复
maowu
2017-08-16 14:33:03 +08:00
@rocksolid 既然需要调用新方法,那调用方本身也就有了修改的需求不是?
Morriaty
2017-08-16 15:23:02 +08:00
@liyu4 go 既不支持重载,又不支持函数默认参数
rocksolid
2017-08-16 15:26:36 +08:00
@maowu 我觉得楼主就是这个需求 不想修改调用方
SingleX
2017-08-16 15:29:47 +08:00
看大家回复,终于明白为什么程序员会撕逼,会跟 PM 掐架了。这需求理解的
dallaslu
2017-08-16 15:47:30 +08:00
1. 直接重载出一个新方法,保留原有方法,将原有方法的实现转移到新方法中。
2. 旧方法不需保留时,可以重构( Change Method Signature )方法,并做好参数为 null 的处理。项目中引用该方法的代码会由 IDE 自动修改。
maowu
2017-08-16 16:02:46 +08:00
@rocksolid 那重载挺合适的呀,为什么不行呢?
reus
2017-08-16 16:05:38 +08:00
参数全都传一个 json !爱怎么加怎么加!
UncleRiver
2017-08-16 16:13:07 +08:00
同 @ahhui
popbones
2017-08-16 16:25:52 +08:00
@Morriaty golang 的好处是直接改,编译报错,强迫调用者升级或者锁定旧版本
junbaor
2017-08-16 17:26:31 +08:00
@reus 这几天组里也有人主张 api 参数全传 json , 不太明白这是什么想法,java 静态强类型语言非要引入弱类型那一套不利于重构的东西。
eloah
2017-08-16 17:28:01 +08:00
def old_function(a, b):
new_function(a, b, c)

def new_function(a, b, c):
pass

这样就可以了
fan123199
2017-08-16 17:35:57 +08:00
如果你用 itellij,有个重构 method 的功能,可以加上默认参数。
tempdban
2017-08-16 17:50:06 +08:00
@SuperMild 兄弟,点不在这里,golang 是能写出来可变参数的
SuperMild
2017-08-16 18:15:31 +08:00
@tempdban 可以是可以,但是意义不一样,通常在 golang 里看到一个函数有可变参数,一般就认为可以随意传一堆参数进去,但楼主的情况那个函数的个数毕竟还是希望固定数量的,毕竟静态类型语言的优势不想轻易丢掉,参数数量和类型最好还是尽量确定。
annielong
2017-08-16 18:16:29 +08:00
没影响啊,如果老项目中的函数都要增加参数,那就重构函数,反正老项目都要改,如果仅仅新的用,那就单独写个新函数
sampeng
2017-08-16 18:19:40 +08:00
@ahhui 一般我也采用这种方式。不影响原有功能。其实和重载是一个思路
current
2017-08-16 19:01:52 +08:00
oldFunc
oldFuncEx
充分暴露软粉本质
iFlicker
2017-08-16 19:36:49 +08:00
瞬间想到了装饰者模式 然后想到了 Kotlin。。。然后才反应过来是方法加参
owlsec
2017-08-16 21:50:22 +08:00
还是 @ahhui 的方法最好。
JamesRuan
2017-08-16 23:48:10 +08:00
先看需求是不是伪需求,如果不是,那么该重构就重构。

我不知道 IDE 怎么做,我自己都是 grep,然后用 vim 的:%s/old/new/gc 一个一个改。

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

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

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

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

© 2021 V2EX