Có ai thường nhận được hơn 100 bình luận trong các bài đánh giá mã của họ trên cơ sở thường xuyên không? Tôi sẽ nói không. Có phải thông thường đối với những người có chất lượng mã "để lại nhiều mong muốn" để nhận được nhiều ý kiến, hoàn toàn.
Tuy nhiên, nó cũng phụ thuộc vào "quy tắc" của quy trình xem xét mã. MỌI NGƯỜI có ý tưởng riêng của họ về cách một cái gì đó nên được thực hiện. Nếu quy trình xem xét mã của bạn cho phép nhận xét ở dạng "Bạn nên thực hiện theo cách này thay vì cách đó", thì bạn có thể sẽ nhận được rất nhiều bình luận ngay cả khi có mã đầy đủ. Nếu quy trình của bạn nhằm tìm "lỗi" thì số lượng bình luận sẽ nhỏ hơn nhiều.
Theo kinh nghiệm của tôi, các đánh giá cho phép "đề xuất" cho các phương pháp thay thế là lãng phí thời gian. Những "gợi ý" đó nên được xử lý từng cái một ngoài quy trình xem xét. Đánh giá khiếm khuyết sẽ hữu ích hơn khi chúng khiến mọi người tập trung vào các lỗi thay vì "tại sao bạn không làm điều đó như tôi sẽ làm?". Nó cũng hữu ích hơn vì không thể phủ nhận một lỗi nếu ai đó tìm thấy. Vì vậy, không có cảm giác tổn thương nhưng có thể biết ơn thay vào đó.
CẬP NHẬT: Với tất cả những gì đã nói, một số mã chỉ đơn giản là xấu, ngay cả khi lỗi miễn phí. Trong trường hợp đó, nhận xét đánh giá nên là một nhận xét duy nhất có nội dung giống như vậy. "Mã này cần được dọn sạch. Vui lòng hoãn đánh giá cho đến khi mã được thảo luận với [tên của bạn ở đây]." Trong trường hợp đó, việc xem xét lại mã sẽ dừng lại cho đến khi nhận xét được khắc phục.
CẬP NHẬT2: @ Người dùng: Bạn có thảo luận về mã / thiết kế của bạn với một trong số họ trong khi bạn đang phát triển nó để bạn có thể thực hiện những gì họ đang tìm kiếm trước khi bạn tiến xa theo cách của mình không? Bạn có thay đổi bất cứ điều gì về cách bạn đang phát triển mã dựa trên đề xuất của họ hoặc tiếp tục nghĩ theo cách của bạn là tốt? Bạn có học được gì từ ý kiến của họ không?
Khi tôi là người dẫn đầu trong một dự án, công việc của tôi là chịu trách nhiệm về TẤT CẢ các sản phẩm công việc. Nếu tôi phê duyệt một sản phẩm làm việc thì tôi khẳng định sản phẩm đó là chấp nhận được. Tôi muốn có một danh tiếng để xây dựng sản phẩm chất lượng. Vì vậy, tôi có kỳ vọng và sẽ không chấp nhận ít hơn thỏa đáng. Đồng thời tôi cố gắng dạy và giải thích lý do cho sở thích của tôi. Những sở thích đó có thể không phải lúc nào cũng lý tưởng (đặc biệt là trong mắt người khác), nhưng hầu hết những sở thích đó đều đến từ kinh nghiệm. Thường là một phản ứng để tránh lặp lại những cái xấu. Vì vậy, có một vài "sticklers" cá nhân của tôi là cần thiết để có được sự chấp thuận của tôi, bất kể đẩy lùi.
Mặt khác, bạn cần học những kỳ vọng cần thiết để sản phẩm công việc của bạn được phê duyệt. Bạn có thể không đồng ý, nhưng vì bạn dường như không có quyền cai trị quá mức, sau đó tìm hiểu những gì được mong đợi. Tôi nghi ngờ rằng nhóm đang cố gắng làm cho bạn thất bại. Vì điều đó làm cho họ trông cũng xấu. Về vấn đề đó, chỉ cần chứng minh rằng bạn rất ham học (ngay cả khi bạn không), hãy nói những gì họ nói và cố gắng hết sức để thích nghi với sở thích của họ và bạn sẽ có thể thấy họ lùi lại khá nhiều. Có thể tìm một người mà bạn ít nhất có thể chịu đựng được và xem liệu họ sẽ làm một chút nắm tay để dạy bạn cách của họ. Ai biết được, trong quá trình bạn có thể học được điều gì đó thực sự có thể đưa kỹ năng của bạn lên một tầm cao mới.