Checklist cho review code
Các công việc review code như sau:
Kiểm tra file và tên file ở Google Drive
Việc dev team quản lý source code ở github cũng hay và đúng chuyên nghiệp, nhưng đó là quá trình phát triển. Còn việc dev team phải download, và upload lên Google Drive là 1 bước coi như là backup 1 lần nữa (thay vì có rủi ro gở github như mất tài khoản, chưa checkin, ...). Download từ githut và chắc chắn đã phải build được, và gởi lên Google Drive cơi như là 1 checkpoint (mildestone).
Download và kiểm tra các thành phần
Kiểm tra các component (project) và solution (toàn bộ dự án) và phải build thành công. Với database thì phải restore được; với .net (BE), hay typescript (FE), hay với Android Studio (mobile app) thì phải build thành công.
Warning messages
Sau khi build thành công thì phải kiểm tra các warning messages. Với 600 warning messages thì rõ ràng có vấn đề (hoặc code không thât sư rõ ràng và rủi ro cao về tính ổn định?). Không thể xóa hết các warning messages nhưng các làm ít (vì dụ < 10) thì càng tốt.
Kiểm tra các thư mục đặc biệt
Các thư mục đặc biệt như config, logging, ... hay các file đặc biệt có đúng chuẩn và sắp xếp hợp lý chưa
Kiểm tra các constant cho bên thứ 3
Nếu trong source code có sử dụng các API hay protocol bên thư 3 thì phải có constant rõ ràng và gom lại 1 file hay 1 folder.
Constant nội bộ
Constant cho source code dùng nội bộ (không phải bên thứ 3) cũng phải rõ ràng và dễ chỉnh sữa để bảo trì.
Đa ngôn ngữ
Last updated