Code review tốt không phải là cuộc kiểm tra người viết mã, mà là cuộc trò chuyện giúp cả nhóm cùng hiểu và cùng sở hữu mã nguồn. Sau nhiều năm làm việc trong các nhóm khác nhau, tôi nhận ra văn hóa quan trọng hơn quy trình.
Người review nên làm gì
- Hỏi trước khi kết luận: "Mình chưa hiểu đoạn này, bạn giải thích giúp nhé?"
- Phân biệt rõ góp ý bắt buộc và gợi ý tùy chọn.
- Khen những chỗ làm tốt — điều này được nhắc đến quá ít.
Người viết mã nên làm gì
- Giữ thay đổi nhỏ, tối đa vài trăm dòng.
- Mô tả rõ vì sao thay đổi, không chỉ thay đổi cái gì.
- Tự review trước khi gửi.
Mục tiêu của review không phải là mã hoàn hảo, mà là mã mà cả nhóm đều dám chạm vào.