Review code

Mỗi giai đoạn, tức tùy theo tuần, tháng, quý, hoặc xong 1 phrase của module/project, ... đều bắt buộc phải có review code.

Việc review code bởi team leader để bảo đảm:

• code sạch sẽ rõ ràng, sau này dễ bảo trì.

• các logic đúng hướng, chứ không bi sai quá xa và sẽ mất thời gian điều chỉnh.

• không viết dư thừa, đặc biệt trong các thành viên trong 1 dự án, và mỗi dư án trong công ty.

Trước khi gởi source code cho team leader, thành viên phải bảo đảm về source code:

• fix tất cả cá warnings của source. Những lỗi này đều có thể dễ dàng được fix, và cũng không bi mất thời gian.

  Trước kia, có thành viên, vì theo quy tắc này nên trước khi gởi source code, lại cài đặt (cấu hình) cho visusla studio 'giấu' các warnings. Đây là hành động gian dối; trong khi việc fix các warning này chừng 30 phút. Những warning này, bao gồm báo biến khai báo biến không dùng, phán đoán những lỗi có thể xảy ra, ... rất có ích cho chất lượng coding.

• sắp xếp folder cho hợp lý, bao gồm:

  • cấu trúc đa tầng (3 tầng hay n tầng), rành mạch, rõ ràng về giao diện (presentation), controller (services, wrapper, routes, ...) và database (DAO, ...).

  • đối với API protocols (routes) thì sắp xếp the subfolder. Vui lòng xem thêm phần deploy với protocol và tham số.

• comment code nhiều nhất có thể:

  • theo header của từng file

  • theo từng method từng function

• dùng tool để analyze code để phán đoán và suggesion viêc code có thể được chất lượng hơn.

  • Mỗi IDE đều có tool cho mục đích này.