这是一个创建于 2655 天前的主题，其中的信息可能已经有所发展或是发生改变。
方法中要增加参数，但是项目中这个方法很多地方用了，
这个方法所在的类也通过 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 强力的话改起来成本并不高。 |
Select Language