Khi nhận code review mà bạn không đồng ý với một góp ý, bạn làm gì?

Người phỏng vấn hỏi câu này để xem bạn làm việc nhóm thế nào, không phải để xem bạn thắng tranh luận. Một câu trả lời tốt cho thấy bạn tách được cảm xúc khỏi kỹ thuật, và bạn quyết định dựa trên bằng chứng.

Bạn có thể trả lời theo bốn bước:

1. Hiểu đúng góp ý trước đã. Đọc lại, và nếu chưa rõ thì hỏi lại người review đang lo điều gì. Nhiều khi mình không đồng ý chỉ vì hiểu sai ý họ.
2. Thử điều họ đề nghị. Chạy thử thay đổi đó, chạy test. Kết quả thường tự trả lời: hoặc góp ý đúng và bạn sửa theo, hoặc nó làm hỏng một trường hợp cụ thể.
3. Trả lời bằng dữ liệu và kết thúc bằng câu hỏi. Ví dụ: "Em thử cách này rồi, 4 test từ chối bị đỏ vì '35,000' bị đọc thành 35000 mà không báo lỗi. Em muốn giữ cách cũ, anh thấy ổn không?". Câu hỏi để ngỏ khả năng người review biết điều bạn chưa biết.
4. Không xong sau hai lượt thì nói chuyện trực tiếp. Mười bình luận qua lại tốn thời gian hơn năm phút gọi nhau. Nếu vẫn chưa thống nhất, theo quy ước của nhóm hoặc nhờ một người thứ ba.

Nên nêu thêm hai ý. Một là phân biệt lỗi thật với sở thích cá nhân: với chuyện sở thích, mình nói rõ lý do của mình nhưng sẵn sàng theo cách của nhóm. Hai là góp ý đúng nhưng ngoài phạm vi thì ghi thành việc riêng, không nhồi vào pull request đang review.

Điểm cộng: kể một lần cụ thể bạn từng không đồng ý, bạn đã thử gì, và kết quả ra sao, kể cả khi người review đúng.