Khi review code. Cần tuân thủ các khía cạnh sau

1.1. Tính bảo mật(Security) (★)

• Có bao gồm thông tin mật trong code không?
• Có bao gồm đoạn code đáng ngờ nào không?

1.2. Code có Clean Chưa (★)

• Chất lượng code có ổn không?
• Có tuân thủ quy tắc coding, quy chuẩn đặt tên không?
• Có đúng là 1 file không quá 800 dòng không?
• Có đúng là 1 function không quá 40 dòng không?
• Có để lại code không cần thiết ở phần comment không?
• Có chứa thông tin không cần thiết không?

1.3. Tính dễ đọc, dễ hiểu (★)

• Code sao cho chỉ vài giây là hiểu nội dung code về cái gì.
• Đọc có thấy dễ rối không?

1.4. Hiệu suất(Performance) (★)

Có gây giảm tốc độ thao tác không?

1.5. Xác định nơi có khả năng phát sinh lỗi (★)

• Nơi phát sinh vấn đề
• Có dễ bị nhận sai, nhầm lẫn không?
• Condition có đang bị phức tạp không?
• Có nhánh condition nào dễ gây bug hay không?
• Kiểm tra giá trị trước khi dùng. Nếu giá trị là NULL
• Kiểm tra giá trị trước khi dùng. Nếu giá trị là 0
• Kiểm tra giá trị trước khi dùng. Nếu là giá trị quá lớn
• Kiểm tra giá trị trước khi dùng. Nếu là giá trị quá nhỏ
• Kiểm tra giá trị trước khi dùng. Nếu định dạng khác nhau

1.6. Dự phòng cho trường hợp phát sinh lỗi (★)

• Đặt trường hợp nếu có phát sinh lỗi (khi chạy runtime) thì phải làm sao?
• Có dùng transaction chưa?
• đặt message, log để tracking được lỗi chưa?

1.7. Tính Bảo Trì (★)

• Khi chỉnh sửa, brushup có tốn thời gian không?
• Khi chỉnh sửa, có để lại tài liệu hay comment gì không?
• Có mở rộng hoặc thêm chức năng mới được không?

1.8. Requirements, Specs (★)

• Đã đúng, đủ với requirements chưa
• Đã đúng, đủ với specs chưa