同事宁愿把 500 页面丢出去也不愿意对提交的数据做验证。

2017-01-12 11:30:54 +08:00
 AdamMaggie
他说,既然他提交了错误的数据就应该给 500 的页面。明明可以对错误做统一的处理为啥还要做验证~
说服不了他,
我真的是尽力了_(:з」∠)_
11353 次点击
所在节点    程序员
88 条回复
ivvei
2017-01-12 11:32:39 +08:00
500 就是他的统一处理
hiboshi
2017-01-12 11:32:47 +08:00
他只是懒,换个时间和他说下,比如周一
imn1
2017-01-12 11:36:01 +08:00
你们上司不管这事么?总是 500 影响企业形象啊
leisure
2017-01-12 11:39:07 +08:00
错误的数据是啥?客户端不做校验么
dubuqingfeng
2017-01-12 11:40:48 +08:00
那么就有机会 SQL 注入了,以及各种 DOS 。
wenhaoy
2017-01-12 11:42:16 +08:00
能出 500 ,说明是异常了,这种代码一般都会有漏洞。看他代码,找个漏洞“自己造”个数据让系统崩溃 /误删 /清空等等,让他认识到错误。还不听,就让老板认识到错误。

如果是他经过判断,自己抛的 500 另说~
AdamMaggie
2017-01-12 11:42:21 +08:00
@leisure 客户端的检验也是很被动的,不说他们也不做。
murmur
2017-01-12 11:43:25 +08:00
能出 500 直接接住了不就完了么。。给用户显示个友好页面这么难么
old9
2017-01-12 11:43:58 +08:00
客户端提交了错误数据,不应该丢 4xx 么?
timi
2017-01-12 11:45:28 +08:00
没有测试人员吗
AdamMaggie
2017-01-12 11:46:53 +08:00
@old9 他所给的 500 ,是因为客户端提交了错误的数据,导致服务端出现了异常返回的。也就是说他并没有处理这个_(:з」∠)_
AdamMaggie
2017-01-12 11:49:30 +08:00
@murmur 都是一些很小的问题,他认为这些数据都应该是经过了客户端验证的,所以他代码里面很少会有接住的代码。然而我们的前端的小伙伴也并不主动去验证数据|・ω・`)
jarlyyn
2017-01-12 11:53:08 +08:00
告诉他 500 错误属于服务器错误,是他的工作错误。

让他记录到 Log.每个月统计 500 错误数量,作为他的 kpi 。

客户端的错误叫 4xx ,和 5 开头的没关系。
lijy91
2017-01-12 11:56:50 +08:00
按照 HTTP 标准,验证出错是应该返回 422 的

```
422 Unprocessable entity

The request was formatted correctly but cannot be processed in its current form. Often used when the specified parameters fail validation errors.
```

另外,服务端验证能更有效的保证数据的准确完整,楼主可以试试通过这个缺陷把系统搞崩溃掉来证明你是对的(慎重,同事关系也很重要)。
flyingghost
2017-01-12 12:01:28 +08:00
电源插头插反了就是应该爆掉,不应该友善的设计保护电路。
汽车撞墙上就是应该去死,不应该友善的弹出安全气囊。
红灯时候过马路就是应该被车撞死,不应该设计行人路权最大的交通规则。

不懂得设计里的防呆设计,这叫见识短。
见识短没关系可以学,但连一丁点浅显的人性化思想都没有,这叫僵硬直板的技术型思维。

但以上都是表象。
我一直认为,不懂换位思考的根本原因,就是自私。
ys0290
2017-01-12 12:06:36 +08:00
我他娘的一个门外汉都知道前端后端都要检验数据,说白了,你这同事欠 X
skywayman
2017-01-12 12:12:27 +08:00
觉得这个得分前提,大家不能一上来就"爆"他!
如果一个页面有特么 N 多输入框,每个框的值校验不一样,输出提示又各不相同,业务指导不明确,主要是开发时间紧,换了是我,也是直接 500...我得考虑我先要活下去,再来谈你们的用户体验...
cmaster
2017-01-12 12:37:06 +08:00
这代码能用?试用期都活不过吧
TonySheva
2017-01-12 12:41:47 +08:00
。。 500 页面??后端写的接口返回处理的结果就可以了啊,他保证了安全以及给出对应提示,展示的页面这些不应该是前端做??为什么要爆他捏?
jinhan13789991
2017-01-12 12:45:05 +08:00
相信我,默默做好自己的。提高自己然后跳槽。不然你会得罪人,最后也干不下去。

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

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

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

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

© 2021 V2EX