Theo truyền thống, chúng tôi đã thực hiện đánh giá mã trước khi cam kết, tôi đã có một cuộc tranh luận với đồng nghiệp của tôi ngày hôm nay, người thích xem xét mã sau khi cam kết.
Đầu tiên, đây là một số nền tảng,
- Chúng tôi có một số nhà phát triển có kinh nghiệm và chúng tôi cũng có những nhân viên mới với kinh nghiệm lập trình gần như bằng không.
- Chúng tôi muốn thực hiện các bước lặp nhanh và ngắn để phát hành sản phẩm của mình.
- Tất cả các thành viên trong nhóm được đặt tại cùng một trang.
Những lợi thế của việc xem xét mã trước khi cam kết tôi đã học:
- Mentor thuê mới
- Cố gắng ngăn ngừa lỗi, thất bại, thiết kế xấu ngay từ đầu trong chu kỳ phát triển
- Học hỏi từ người khác
- Sao lưu kiến thức nếu ai đó thoát
Nhưng tôi cũng đã có một số trải nghiệm tồi tệ:
- Hiệu quả thấp, một số thay đổi có thể được xem xét qua nhiều ngày
- Khó cân bằng tốc độ và chất lượng, đặc biệt là cho người mới
- Một thành viên trong nhóm cảm thấy không tin tưởng
Đối với đánh giá sau khi cam kết, tôi biết rất ít về điều này, nhưng điều tôi lo lắng nhất là nguy cơ mất kiểm soát do thiếu đánh giá. Có ý kiến gì không?
CẬP NHẬT:
- Chúng tôi đang sử dụng Perforce cho VCS
- Chúng tôi mã hóa và cam kết trong cùng một nhánh (các nhánh sửa lỗi hoặc thân cây)
- Để cải thiện hiệu quả, chúng tôi đã cố gắng chia mã thành các thay đổi nhỏ. Chúng tôi cũng đã thử một số đánh giá hộp thoại trực tiếp, nhưng không phải ai cũng tuân theo quy tắc. Đây là một vấn đề khác mặc dù.