[問題] 驗證資料的處理方式

看板C_Sharp作者 (重出江湖)時間6年前 (2017/12/20 22:07), 6年前編輯推噓2(2013)
留言15則, 4人參與, 6年前最新討論串1/1
請問各位 小弟目前接手到新同事寫的code覺得有些問題 例如要verify前端送來的資料格式是否正確 同事的寫法都是自定義一個exception 然後if判斷是否滿足不合格的條件 是的話就塞錯誤訊息給exception 然後throw new exception 但我記得發生exception這對程式來說是需要額外的成本 而且這些資料的情況其實算是「已知」的情況不算例外 我看過覺得比較好的作法 是把verify的部份獨立成一個method 只要有不合格的條件就回傳一個字串的錯誤訊息 完全正確就回傳一個string.empty就好 我外面只要判斷回傳值是否為empty就好 不曉得同事這樣寫的方式是不是真的不好呢? 他目前看起來沒什麼特殊考量 就只是存脆驗證失敗就丟錯誤訊息而已 或是大家有什麼更好的方式呢? 謝謝 -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 118.233.159.254 ※ 文章網址: https://www.ptt.cc/bbs/C_Sharp/M.1513778849.A.1A5.html ※ 編輯: aoksc (118.233.159.254), 12/20/2017 22:09:24

12/20 22:24, 6年前 , 1F
除非是系統瓶頸處,否則擲回例外是很正常的用法
12/20 22:24, 1F

12/20 22:26, 6年前 , 2F
可以讓程式碼的例外處理更結構化,也比較容易debug
12/20 22:26, 2F

12/20 22:27, 6年前 , 3F
但最好是能依例外狀況不同,擲回不同的例外類別。
12/20 22:27, 3F

12/20 22:29, 6年前 , 4F
而資料處理的業務,常常會定義一個回傳的result class,
12/20 22:29, 4F

12/20 22:30, 6年前 , 5F
裡面就會放驗證結果與錯誤訊息了。如果是包含在介面定義好
12/20 22:30, 5F

12/20 22:31, 6年前 , 6F
的,當然就不會用擲回例外的方式去回報錯誤了。
12/20 22:31, 6F
他是一個失敗定義一個Exception沒錯 但就變成如果verify有10種失敗的情況 外面的method就要寫10個catch去捕捉例外 所以各位是覺得這樣寫可以看的更清楚?

12/20 22:37, 6年前 , 7F
我也都這樣寫 好處是可以把方法提取出來
12/20 22:37, 7F

12/20 22:38, 6年前 , 8F
不喜歡的話可以參考fluentvalidation套件
12/20 22:38, 8F
※ 編輯: aoksc (118.233.159.254), 12/20/2017 22:42:17

12/20 23:10, 6年前 , 9F
verify本身的工作就是分對錯,規格內的錯誤不會用擲例外來
12/20 23:10, 9F

12/20 23:10, 6年前 , 10F
回傳阿。
12/20 23:10, 10F
我現在問的就是在規格內預期可能的錯誤用exception丟的問題… ※ 編輯: aoksc (118.233.159.254), 12/20/2017 23:52:34

12/21 02:24, 6年前 , 11F
有規格就照規格走阿,還有什麼好說的= =
12/21 02:24, 11F

12/21 10:34, 6年前 , 12F
回傳統一格式的物件,內容包含狀態碼,回傳資料,業務邏
12/21 10:34, 12F

12/21 10:34, 6年前 , 13F
輯錯誤訊息等欄位; 既然錯誤可預期,就可以用判斷式處理
12/21 10:34, 13F

12/21 10:34, 6年前 , 14F
後將結果加到回傳物件,不用丟exception
12/21 10:34, 14F

12/29 07:17, 6年前 , 15F
就寫一個 before & after method
12/29 07:17, 15F
文章代碼(AID): #1QEcwX6b (C_Sharp)