昨天加班被安卓端怼了,说我连培训班差生都不如,我是不是该离职?

2019-05-29 23:37:27 +08:00
 springGun

我是做 java 后端开发的,因为安卓端调用查询接口中非必要参数未做空字符串判断,问我后端为什么不将空字符串转成 null 处理?骂我连培训班的差生都不如.我是不是该离开这家公司?

12054 次点击
所在节点    问与答
122 条回复
Vegetable
2019-05-30 10:25:48 +08:00
@springGun 讲道理如果是 string 的话传""是没问题的.这个应该后端做.
passerbytiny
2019-05-30 10:25:57 +08:00
你是不是该离职不好说,但你要主动离职了,对方估计也该滚蛋了。加班时间因为接口问题吵起来,这种环境八成是“稳定”为主。
Donne
2019-05-30 10:27:19 +08:00
让安卓端来讲,可能是另一个故事了。
passerbytiny
2019-05-30 10:30:50 +08:00
@springGun #33 !!!!兄弟,我也想骂你。查询条件 null 和 "" 同时判断是常识,因为查询条件是通过 HTTP 参数传进来的。
xmx12138
2019-05-30 10:37:29 +08:00
@k9982874 不嘛,不嘛。人家不要嘛
hstdt
2019-05-30 10:39:02 +08:00
我更喜欢 null 返回空字符串给我
springGun
2019-05-30 10:44:10 +08:00
@passerbytiny 那查询一个空字符串的数据怎么办?我也没见 ali 和腾讯的接口会给你忽略掉该条件,要么返回参数检验异常,要么直接返回空的查询结果
wutiantong
2019-05-30 10:46:17 +08:00
看楼主后面的回复,好多人前面都理解错了:

并非客户端要求后端用 null 代替,而是客户端发请求时传了个空字符串,后端(楼主)没做判断。
tt67wq
2019-05-30 10:48:45 +08:00
@drush #8 秀还是你秀
aLazarus
2019-05-30 10:51:23 +08:00
还是要做判断的吧…
joyhub2140
2019-05-30 10:55:05 +08:00
空字符串或者 null 值,应该不传,让客户端采取 default 值。
jiuanzi
2019-05-30 10:56:54 +08:00
伟大都是熬出来的
passerbytiny
2019-05-30 10:59:53 +08:00
@springGun #43 你能给我找出来一个查询“非 null 但空字符串”的需求出来吗?对于你的第二句话,我觉得没必要做深入回复,一句“你去 ali 和 腾讯吧”就足够了。
janus77
2019-05-30 11:00:11 +08:00
如果你没隐瞒别的,那这个安卓有点辣鸡
你可以直接一句:这接口 ios 用着没问题就你有问题,怪我咯
huoshanhui
2019-05-30 11:00:28 +08:00
解决问题的前提是好好说话。你确定他是这种语气和措辞的话建议你先调教下他。
yvescheung
2019-05-30 11:05:33 +08:00
为什么你要离职?你应该想办法让他离职
kingzt
2019-05-30 11:08:08 +08:00
空字符串和 null 语义本来就不一样, 直接怼回去
springGun
2019-05-30 11:13:01 +08:00
@passerbytiny
elasticsearch 查询文档中字段为""的需求又不是没有
kevin1852
2019-05-30 11:14:57 +08:00
应该是 API 里的可空请求参数没有传或者传的“”,bind parameter 时自动转成了“”,然后查询的时候只判断了是不是 null,导致查询不到结果吧?
kuyuzhiqi
2019-05-30 11:16:37 +08:00
做 Android 的表示会自己用 TextUtils.isEmpty()判断,这种 Android 可以走了

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

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

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

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

© 2021 V2EX