วันนี้ทำการ review code นิดหน่อย
พบ code ที่น่าสนใจเยอะพอสมควร
หนึ่งในนั้นคือ code เกี่ยวกับการ mapping error code
โดยมีขั้นตอนดังนี้
- ทำการตรวจสอบว่า Exception แต่ละตัวจากนั้นกำหนด error code ให้
- ทำการดึงรายละเอียดของ error ด้วย error code จาก Database !!
โดย code ที่ทำการ review เป็นประมาณนี้
[gist id="d300141a8617d13f1d8fbb8e21e16150" file="MappingException01.java"] คำถามที่น่าสนใจคือ จะปรับปรุง code ชุดนี้อย่างไรดี ?แต่ก่อนที่ปรับปรุง code นั้นสิ่งที่ควรจะต้องรู้ก็คือ ปัญหา เนื่องจากถ้าเรามองไม่เห็นปัญหาแล้ว การปรับปรุงก็จะไร้ทิศทางอย่างมาก ดังนั้นการปรับปรุง code อยู่ที่ปัญหานี่แหละจาก code ดังกล่าวผมถามคำถามไปยังทีมพัฒนาดังนี้ ทำไมเราต้องทำการตรวจสอบด้วยว่า Exception/Error นั้นเป็นชนิดอะไร ? จึงจะกำหนด error code ให้ จากนั้นจึงทำการดึงรายละเอียดจาก Database !!! ที่สำคัญคือ จะมี Exception/Error เพิ่มขึ้นมาอีกหรือไม่ ? ตอบเลยว่ามาก ๆๆๆ แสดงว่าต้องมาเพิ่มการตรวจสอบไปเรื่อย ๆ มันไม่น่าจะเป็นทางเลือกที่ดีหรือเหมาะสมเท่าไรนัก
นี่คือปัญหาที่ได้รับจากคำถามต่าง ๆ ดังนั้นลองมาปรับปรุง code กันดูนิดหน่อย