keyword: review checklist, source review, coding standart
Liên quan đến design:
- Có tạo flowchart không và sử dụng theo không
- Tính maintaince và sử dụng lại như thế nào
Liên quan đến xử lý lỗi:
- Thực hiện check NULL
- những đối tượng đầu vào hàm public không
- những đối tượng get ra từ 1 API, Lib,hệ thống trước khi dùng.
- Dùng Array không, có xử lý check size trước khi sử dụng không
- Có sử dụng Notification, Delegate không, có được remove khi giải phóng chưa.
- Có handle lỗi và xử lý log không
Liên quan đến performance
- Có bị duplicate code không
- Có sử dụng cùng data mà thường xuyên phải load không
- Có thể tăng performance cho việc lấy data không
- Có thể reusing object để tăng performance không
Liên quan đến convetion:
- Có bị warning không
- Class dài quá 500line không
- Method có nhiều quá 15 khối control không(if,for, while).
- Code có dùng magic number hay magic string không (sử dụng trực tiếp số, chuỗi trong chương trình)
- Các phần xử lý có comment không (lý do vì sao có xử lý này, fixbug thế nào chẳng hạn)
- Comment cho hàm và các value param, return cho hàm public đủ chưa (trong file h)
- Indent chuẩn chưa, có code thừa không, code pending phải cho vào #if 0
- Naming chuẩn chưa: BOOL có prefix, is, has... không
- Performance tốt không
Liên quan đến lưu trữ:
- Trường hợp không cần thiết thì set flag auto icound là false