作者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/m.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