# 这是原方法
function loadImgUseQiNiu(clickBtn,upload_token,cb){}
# 这是他改的
function loadImgUseQiNiu(clickBtn,upload_token,size,cb){}
我真是打人的心都有了,测试跟我说图片不能上传了,打开 console,报错,cb is not a function,我说怎么会有这种莫名的报错,打开改文件一看,我擦嘞,加了个参数,我说加就加呗,还加中间,此处摊手表情
1
lkj007 2018 年 4 月 2 日
我觉得正常啊,cb 是一个回调,是应该放在最后面呀
|
2
zjp 2018 年 4 月 2 日 via Android 充分体现了静态语言的优势 :doge:
改接口不沟通,不用默认参数,还有这命名都是槽点… |
3
murmur 2018 年 4 月 2 日
java 和 javascript 一词之差体验差了不知道哪里去了
|
6
gen900 2018 年 4 月 2 日
其实是没处理好,新版本的接口加上
if (typeof size === 'function') cb = size; 就可以兼容老传参了。 |
7
odirus 2018 年 4 月 2 日
以前写动态语言我都写怕了。。。
|
9
qiuyk 2018 年 4 月 2 日
多人合作还是上 ts 比较保险
|
10
Tneciv 2018 年 4 月 2 日
多谢 Java 给了我一条生路
|
11
CYKun 2018 年 4 月 2 日 via Android
多谢 Java 给了我一条生路
|
12
quickma 2018 年 4 月 2 日
多谢 Java 给了我一条生路
|
13
willvvvvv1 2018 年 4 月 2 日
多谢 Java 给了我一条生路
|
14
broadliyn 2018 年 4 月 2 日
多谢 Java 给了我一条生路
|
15
eslizn 2018 年 4 月 2 日
实在不行不能 loadImgUseQiNiu(options)吗?
|
16
scnace 2018 年 4 月 2 日 via Android
没有 CodeReview 的吗?
|
17
h1367500190 2018 年 4 月 2 日
就算目前只传一个参数我也会这么写:
function loadImgUseQiNiu({ clickBtn, upload_token, size, cb }){} |
18
jmc891205 2018 年 4 月 2 日
多谢 C++ 给了我一条生路
|
19
xrlin 2018 年 4 月 2 日 via iPhone
静态类型保平安,每次改 ruby 代码我都有点不放心,js 代码也是,所以新项目还是上 ts。这也是你们接口没商量好的问题吧。
|
20
imswing 2018 年 4 月 2 日 via iPhone
可能习惯回掉放最后了。。。
|
21
duan602728596 2018 年 4 月 2 日 via iPhone
js 表示这个锅它不想背......
|
23
znood 2018 年 4 月 3 日 via iPhone
这算什么,以前有同事复写 read,write 接口,两个函数参数顺序是相反的
|
25
guokeke 2018 年 4 月 3 日 via Android
函数参数都搞成对象啊
|
26
kemikemian 2018 年 4 月 3 日
第一个参数驼峰,第二个下划线
|
27
orm 2018 年 4 月 3 日
多谢 Java 给了我一条生路
|
28
zhouyg 2018 年 4 月 3 日
明显是坑货,js 不背这个锅。
|