Tôi sẽ không gọi mình là một siêu sao, nhưng là một người tương đối có kinh nghiệm. Tôi cố gắng giữ chất lượng mã ở mức cao và luôn tìm cách cải thiện phong cách mã hóa của mình, cố gắng làm cho mã hiệu quả, dễ đọc và nhất quán cũng như khuyến khích nhóm làm theo mô hình & phương pháp để đảm bảo tính nhất quán. Tôi cũng hiểu sự cần thiết của sự cân bằng giữa cả chất lượng và tốc độ.
Để đạt được điều này, tôi đã giới thiệu cho nhóm của mình khái niệm về đánh giá ngang hàng. Hai ngón tay cái lên trong github pull-request để hợp nhất. Tuyệt vời - nhưng không phải theo ý kiến của tôi mà không có tiếng nấc.
Tôi thường thấy các bình luận đánh giá ngang hàng từ các đồng nghiệp giống như -
- Sẽ là tốt để thêm một không gian sau
<INSERT SOMETHING HERE>
- Dòng bổ sung không mong muốn giữa các phương thức
- Dừng hoàn toàn nên được sử dụng ở cuối các bình luận trong docblocks.
Bây giờ theo quan điểm của tôi - người đánh giá nhìn bề ngoài về tính thẩm mỹ của mã - và không thực sự thực hiện đánh giá mã. Đánh giá mã mỹ phẩm đến với tôi như tâm lý kiêu ngạo / tinh hoa. Nó thiếu chất, nhưng bạn thực sự không thể tranh luận quá nhiều với nó bởi vì người đánh giá là đúng về mặt kỹ thuật . Tôi muốn nhìn thấy ít hơn các loại đánh giá ở trên, và nhiều đánh giá như sau:
- Bạn có thể giảm độ phức tạp theo chu kỳ bằng cách ...
- Thoát sớm và tránh nếu / khác
- Tóm tắt truy vấn DB của bạn vào một kho lưu trữ
- Logic này không thực sự thuộc về nơi này
- Đừng lặp lại chính mình - trừu tượng và tái sử dụng
- Điều gì sẽ xảy ra nếu
X
được thông qua như là một đối số cho phương thứcY
? - Đâu là bài kiểm tra đơn vị cho việc này?
Tôi thấy rằng luôn luôn có cùng loại người đưa ra các loại đánh giá mỹ phẩm, và cùng loại người mà theo tôi đưa ra đánh giá ngang hàng "Chất lượng & Logic".
Điều gì (nếu có) là cách tiếp cận chính xác để đánh giá ngang hàng. Và tôi có đúng không khi thất vọng với cùng một người về cơ bản lướt qua mã tìm lỗi chính tả & lỗi thẩm mỹ hơn là lỗi mã thực tế?
Nếu tôi đúng - làm thế nào tôi có thể khuyến khích các đồng nghiệp thực sự tìm kiếm các lỗi trong mã một cách cân bằng với gợi ý chạm vào mỹ phẩm?
Nếu tôi không chính xác - xin hãy khai sáng cho tôi. Có bất kỳ quy tắc nào cho những gì thực sự cấu thành một đánh giá mã tốt không? Tôi đã bỏ lỡ quan điểm của các đánh giá mã là gì?
Từ quan điểm của tôi - xem xét mã là về trách nhiệm chung cho mã. Tôi sẽ không cảm thấy thoải mái khi đưa ngón tay cái lên mã mà không giải quyết / kiểm tra logic, khả năng đọc và chức năng. Tôi cũng sẽ không bận tâm đến việc hợp nhất một đoạn mã vững chắc nếu tôi nhận thấy ai đó đã bỏ qua một điểm dừng hoàn toàn trong khối tài liệu.
Khi tôi xem lại mã, tôi có thể chi tiêu trong khoảng 15-45 phút trên 500 Loc. Tôi không thể tưởng tượng những đánh giá nông cạn này mất hơn 10 phút bao giờ nếu đó là độ sâu của đánh giá mà họ đang thực hiện. Hơn nữa, bao nhiêu giá trị là ngón tay cái từ người đánh giá nông? Chắc chắn điều này có nghĩa là tất cả các ngón tay cái không có trọng lượng bằng nhau và có thể cần phải có quá trình xem xét 2 lần. Một ngón tay cái để đánh giá sâu và ngón tay cái thứ 2 cho "đánh bóng"?