方法中要增加参数,但是项目中这个方法很多地方用了,
这个方法所在的类也通过 dubbo 暴露给其他服务了
大家怎么做?
1
amon 2017-08-16 11:12:01 +08:00
重载?
|
2
Morriaty 2017-08-16 11:14:38 +08:00 1
这就体现函数默认参数的优越性了,当然,如果是 golang,就当我没说。
|
5
yibin001 2017-08-16 11:28:22 +08:00
如果当时的参数是一个对象的话,可以通过增加属性来解决。
如果是基本类型就。。。 |
6
littleylv 2017-08-16 11:34:53 +08:00
php 一般就加默认值
|
7
Qlccks2 2017-08-16 11:35:07 +08:00
原 method(param1~paramN){
掏空 新 method(param1~paramN,null) } 新 method(param1~paramN,new param){ 原方法实现 } 不知道是否可行,尽量避免吧。 |
8
kaulie 2017-08-16 11:36:33 +08:00
1. 有没有可能通过其他参数附带传进来;
2. 考虑下参数的动态绑定 |
9
SuperMild 2017-08-16 11:42:46 +08:00
原来已经使用该函数的代码,不需要使用新增参数吧?那可以考虑增加一个函数给新写的代码用。
|
10
liuzhen 2017-08-16 11:55:04 +08:00
|
11
0TSH60F7J2rVkg8t 2017-08-16 12:56:17 +08:00 7
我个人会采取这种方式来修改接口声明:
假设原函数: method(param1, param2) 首先,新增函数: method2(param1, param2, param3) 然后让原函数 method() 直接内部直接 call method2,并添加默认值的 param3 传递。 开发文档和源码里标记 method 为 deprecated。并注明将在数个版本后彻底移除该函数,建议使用 method2 替代。 |
13
SuperMild 2017-08-16 13:21:49 +08:00
@liyu4 有些动态语言,比如 JavaScript,函数的参数是不固定数量的,同一个函数,调用的时候塞两个参数给它也行,塞三个参数给它也行。
|
14
zj299792458 2017-08-16 13:25:31 +08:00 via iPhone
Java:重载,c:写个和原来一样的方法,以默认参数调用新方法。
|
15
rocksolid 2017-08-16 13:33:51 +08:00
说重载的估计没理解楼主意思,如果你在 100 个地方调用了这个函数,重载还是要在 100 个地方修改一遍。楼主估计是只想改这函数不想动调用的地方
|
16
misaka19000 2017-08-16 13:53:05 +08:00
@rocksolid #15 重载为什么要改调用函数?
|
17
Shura 2017-08-16 13:53:50 +08:00 via Android
|
19
rocksolid 2017-08-16 13:58:03 +08:00
@misaka19000 楼主不是说了么,原来的地方需要使用增加参数的新函数
|
24
SingleX 2017-08-16 15:29:47 +08:00
看大家回复,终于明白为什么程序员会撕逼,会跟 PM 掐架了。这需求理解的
|
25
dallaslu 2017-08-16 15:47:30 +08:00
1. 直接重载出一个新方法,保留原有方法,将原有方法的实现转移到新方法中。
2. 旧方法不需保留时,可以重构( Change Method Signature )方法,并做好参数为 null 的处理。项目中引用该方法的代码会由 IDE 自动修改。 |
27
reus 2017-08-16 16:05:38 +08:00
参数全都传一个 json !爱怎么加怎么加!
|
28
UncleRiver 2017-08-16 16:13:07 +08:00
同 @ahhui
|
30
junbaor 2017-08-16 17:26:31 +08:00
@reus 这几天组里也有人主张 api 参数全传 json , 不太明白这是什么想法,java 静态强类型语言非要引入弱类型那一套不利于重构的东西。
|
31
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 这样就可以了 |
32
fan123199 2017-08-16 17:35:57 +08:00
如果你用 itellij,有个重构 method 的功能,可以加上默认参数。
|
34
SuperMild 2017-08-16 18:15:31 +08:00
@tempdban 可以是可以,但是意义不一样,通常在 golang 里看到一个函数有可变参数,一般就认为可以随意传一堆参数进去,但楼主的情况那个函数的个数毕竟还是希望固定数量的,毕竟静态类型语言的优势不想轻易丢掉,参数数量和类型最好还是尽量确定。
|
35
annielong 2017-08-16 18:16:29 +08:00
没影响啊,如果老项目中的函数都要增加参数,那就重构函数,反正老项目都要改,如果仅仅新的用,那就单独写个新函数
|
37
current 2017-08-16 19:01:52 +08:00
oldFunc
oldFuncEx 充分暴露软粉本质 |
38
iFlicker 2017-08-16 19:36:49 +08:00
瞬间想到了装饰者模式 然后想到了 Kotlin。。。然后才反应过来是方法加参
|
40
JamesRuan 2017-08-16 23:48:10 +08:00
先看需求是不是伪需求,如果不是,那么该重构就重构。
我不知道 IDE 怎么做,我自己都是 grep,然后用 vim 的:%s/old/new/gc 一个一个改。 |
41
cncqw 2017-08-17 04:28:47 +08:00 via iPhone
php 加默认值 ++1
|
42
cncqw 2017-08-17 04:29:56 +08:00 via iPhone
或者直接在那个方法下面复制粘贴,命名为 function2
|
43
yidinghe 2017-08-17 08:38:13 +08:00 via Android
写一个同名的新方法即可。
|
44
xrlin 2017-08-17 09:32:19 +08:00 via iPhone
重载
|
45
weakish 2017-08-17 14:33:57 +08:00
既然暴露出去了,那就是公开的 API, 除非有非常非常强烈的理由,否则改不了了。
public 要严格控制,拿不准就别 public. public 的接口、类、方法就好好设计。 私有的就随意多了,以后随时可以改。IDE 强力的话改起来成本并不高。 |