作者aoksc (重出江湖)
看板C_Sharp
标题[问题] 验证资料的处理方式
时间Wed Dec 20 22:07:26 2017
请问各位
小弟目前接手到新同事写的code觉得有些问题
例如要verify前端送来的资料格式是否正确
同事的写法都是自定义一个exception
然後if判断是否满足不合格的条件
是的话就塞错误讯息给exception
然後throw new exception
但我记得发生exception这对程式来说是需要额外的成本
而且这些资料的情况其实算是「已知」的情况不算例外
我看过觉得比较好的作法
是把verify的部份独立成一个method
只要有不合格的条件就回传一个字串的错误讯息
完全正确就回传一个string.empty就好
我外面只要判断回传值是否为empty就好
不晓得同事这样写的方式是不是真的不好呢?
他目前看起来没什麽特殊考量
就只是存脆验证失败就丢错误讯息而已
或是大家有什麽更好的方式呢?
谢谢
--
※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 118.233.159.254
※ 文章网址: https://webptt.com/cn.aspx?n=bbs/C_Sharp/M.1513778849.A.1A5.html
※ 编辑: aoksc (118.233.159.254), 12/20/2017 22:09:24
1F：→ Litfal: 除非是系统瓶颈处，否则掷回例外是很正常的用法 12/20 22:24
2F：→ Litfal: 可以让程式码的例外处理更结构化，也比较容易debug 12/20 22:26
3F：→ Litfal: 但最好是能依例外状况不同，掷回不同的例外类别。 12/20 22:27
4F：→ Litfal: 而资料处理的业务，常常会定义一个回传的result class， 12/20 22:29
5F：→ Litfal: 里面就会放验证结果与错误讯息了。如果是包含在介面定义好 12/20 22:30
6F：→ Litfal: 的，当然就不会用掷回例外的方式去回报错误了。 12/20 22:31
他是一个失败定义一个Exception没错
但就变成如果verify有10种失败的情况
外面的method就要写10个catch去捕捉例外
所以各位是觉得这样写可以看的更清楚？
7F：推 vi000246: 我也都这样写 好处是可以把方法提取出来 12/20 22:37
8F：→ vi000246: 不喜欢的话可以参考fluentvalidation套件 12/20 22:38
※ 编辑: aoksc (118.233.159.254), 12/20/2017 22:42:17
9F：→ Litfal: verify本身的工作就是分对错，规格内的错误不会用掷例外来 12/20 23:10
10F：→ Litfal: 回传阿。 12/20 23:10
我现在问的就是在规格内预期可能的错误用exception丢的问题…
※ 编辑: aoksc (118.233.159.254), 12/20/2017 23:52:34
11F：→ Litfal: 有规格就照规格走阿，还有什麽好说的= = 12/21 02:24
12F：推 t64141: 回传统一格式的物件，内容包含状态码，回传资料，业务逻 12/21 10:34
13F：→ t64141: 辑错误讯息等栏位; 既然错误可预期，就可以用判断式处理 12/21 10:34
14F：→ t64141: 後将结果加到回传物件，不用丢exception 12/21 10:34
15F：→ disabledman: 就写一个 before & after method 12/29 07:17