[吐槽] 你们遇到过给封装方法参数加在中间的坑货吗

[OMGZui]

# 这是原方法
function loadImgUseQiNiu(clickBtn,upload_token,cb){}

# 这是他改的
function loadImgUseQiNiu(clickBtn,upload_token,size,cb){}

我真是打人的心都有了，测试跟我说图片不能上传了，打开 console，报错，cb is not a function，我说怎么会有这种莫名的报错，打开改文件一看，我擦嘞，加了个参数，我说加就加呗，还加中间，此处摊手表情

[lkj007]

我觉得正常啊，cb 是一个回调，是应该放在最后面呀

[zjp]

充分体现了静态语言的优势 :doge:
改接口不沟通，不用默认参数，还有这命名都是槽点…

[murmur]

java 和 javascript 一词之差体验差了不知道哪里去了

[OMGZui]

@lkj007 一般是放最后面，但是改了居然不说的，很多地方都得用这个方法

[OMGZui]

@zjp 2333，这里用了默认参数还是得报那个错，老前辈留下来的代码了，我是怕了改代码的这家伙了，重新封装了一次，自己调用。

[gen900]

其实是没处理好，新版本的接口加上

if (typeof size === 'function') cb = size;

就可以兼容老传参了。

[odirus]

以前写动态语言我都写怕了。。。

[OMGZui]

@gen900 这是一个方法，但是那家伙就是暴力的加在了前面，然后改了他自己调用的地方，不管其它地方，真的服

[qiuyk]

多人合作还是上 ts 比较保险

[Tneciv]

多谢 Java 给了我一条生路

[CYKun]

多谢 Java 给了我一条生路

[x7395759]

多谢 Java 给了我一条生路

[willvvvvv1]

多谢 Java 给了我一条生路

[broadliyn]

多谢 Java 给了我一条生路

[eslizn]

实在不行不能 loadImgUseQiNiu(options)吗？

[scnace]

没有 CodeReview 的吗？

[h1367500190]

就算目前只传一个参数我也会这么写：
function loadImgUseQiNiu({ clickBtn, upload_token, size, cb }){}

[jmc891205]

多谢 C++ 给了我一条生路

[xrlin]

静态类型保平安，每次改 ruby 代码我都有点不放心，js 代码也是，所以新项目还是上 ts。这也是你们接口没商量好的问题吧。

[imswing]

可能习惯回掉放最后了。。。

[duan602728596]

js 表示这个锅它不想背......

[msg7086]

@xrlin 没测试写什么 Ruby （手动滑稽

[znood]

这算什么，以前有同事复写 read，write 接口，两个函数参数顺序是相反的

[xrlin]

@msg7086 是的，写 ruby 必须要有测试加持，否则项目一大就要垮了，只是需求太多了，一个人做不来，更不用想测试了。

[guokeke]

函数参数都搞成对象啊

[kemikemian]

第一个参数驼峰,第二个下划线

[orm]

多谢 Java 给了我一条生路

[zhouyg]

明显是坑货，js 不背这个锅。