Trả lời những câu hỏi thường gặp về code review
(brunch.co.kr/@cleancode)"Tôi biết code review là tốt, nhưng không có thời gian. Ngoài review còn rất nhiều việc phải làm."
- Dưới đây là phần tổng hợp câu trả lời cho câu hỏi trên, câu mà tác giả giảng dạy về code review nhận được nhiều nhất
- Hãy để tác giả (người viết PR) chủ động nỗ lực để thời gian cần cho code review là ít nhất
- Giống như họp stand-up 10 phút mỗi sáng, hãy dành thời gian review cố định: 30 phút vào buổi sáng và 30 phút sau bữa trưa
- Chất lượng và năng suất:
Nếu đầu tư ở giai đoạn đầu, có thể giảm đáng kể chi phí phát sinh về sau, đồng thời giảm chi phí thay đổi trong tương lai, từ đó dẫn đến cải thiện năng suất - Khác:
Nếu thiếu thời gian, hãy bắt đầu ít nhất từ những phần mang tính chí mạng như bug/sự cố, rồi dần mở rộng phạm vi.
Hãy để tổ chức ghi nhận nỗ lực dành cho review như một thành quả
"Tôi mong chúng ta thực hiện code review như một hoạt động chia sẻ giúp tăng trưởng mà chúng ta có thể làm ngay bây giờ, đồng thời là một phương tiện để nâng cao năng suất thông qua cải thiện chất lượng."
2 bình luận
Cần có một văn hóa công nhận thời gian code review là thời gian làm việc.
Nếu công việc bị giao xuống theo kiểu top-down với lịch trình cực kỳ sít sao thì không có cách nào xoay xở được. Hoặc là chốt sẵn due date rồi giao việc như vậy.
Trong trường hợp những công ty như vậy, khả năng cao văn hóa tổng thể mang tính top-down. Khi chỉ cần điều chỉnh một đầu việc thì tương đối dễ thay đổi, nhưng với bầu không khí chung thì rất khó vì đó là chuyện ban lãnh đạo phải thật sự quyết tâm.
Nếu không thể thảo luận về khối lượng công việc, thì hãy thử thay đổi văn hóa đó (thông qua quá trình thuyết phục bằng việc làm thêm giờ), nếu sau một tháng vẫn không thay đổi thì về mặt hiệu quả, tốt hơn là tìm một tổ chức/công ty/ngành khác. Nếu cố gắng hơn nữa, rồi bạn sẽ nhận ra chính mình sụp đổ từ bên trong.
(Ý ở “ngành” là để phân biệt SI, startup và doanh nghiệp lớn. Tôi cảm thấy công việc họ làm, cách làm việc và quan hệ giữa các thành viên trong nhóm hoàn toàn khác nhau.)