Vâng, tôi có xu hướng bình luận trong một số lĩnh vực chung và mỗi loại có thể được xử lý khác nhau.
Yêu cầu thay đổi. Đây là những loại thay đổi mà tôi chỉ ra rằng mã không đáp ứng các yêu cầu chức năng hoặc không hoạt động và phải được sửa trước khi được đưa vào sản xuất. Tôi có xu hướng rất thẳng thắn trong những bình luận này. Các yêu cầu nói ..., điều này không làm điều đó. Hoặc điều này sẽ thất bại nếu giá trị được gửi vào là null (đặc biệt là khi bạn biết trường hợp đó sẽ xảy ra dựa trên dữ liệu sẽ được gửi vào).
Sau đó, có "ý kiến này hoạt động nhưng đây là một cách tốt hơn để thực hiện ý kiến đó". Bạn phải nhẹ nhàng hơn trong những điều này và làm nhiều hơn một doanh số bán hàng. Tôi có thể nói rằng tôi sẽ làm điều này thay vì bởi vì nó có khả năng hoạt động tốt hơn (tôi thường xem lại mã SQL trong đó hiệu suất rất quan trọng). Tôi có thể thêm một số chi tiết về lý do tại sao nó là một lựa chọn tốt hơn giống như tôi sẽ làm khi trả lời một câu hỏi trên Stack Overflow. Tôi có thể chỉ ra rằng không cần thiết phải thay đổi mã này cho mã cụ thể này, nhưng phải xem xét sự thay đổi trong mã hóa trong tương lai. Về cơ bản với những loại bình luận này, tôi đang giáo dục những người ít kinh nghiệm về những gì có thể làm việc tốt hơn.
Sau đó, có những bình luận "điều này hoạt động nhưng chúng tôi làm mọi thứ theo cách này". Đây có lẽ cũng sẽ được yêu cầu thay đổi. Chúng sẽ bao gồm các ý kiến về các tiêu chuẩn của công ty hoặc kiến trúc mà chúng tôi mong đợi chúng sử dụng. Tôi sẽ tham khảo tài liệu tiêu chuẩn hoặc kiến trúc và bảo họ sửa theo tiêu chuẩn. Nhận xét sẽ đơn giản nhưng trung lập, do đó nó bị thiếu và do đó hoặc các tên biến không phù hợp với tiêu chuẩn đặt tên hoặc các điều tương tự của chúng tôi. Chẳng hạn, kiến trúc của chúng tôi cho các gói SSIS yêu cầu gói sử dụng cơ sở dữ liệu siêu dữ liệu của chúng tôi để lưu trữ thông tin cụ thể về gói và yêu cầu ghi nhật ký cụ thể. Gói sẽ hoạt động mà không có những thứ này nhưng chúng được yêu cầu vì lý do công ty (chúng tôi cần báo cáo về tỷ lệ nhập khẩu thành công chẳng hạn hoặc phân tích các loại lỗi chúng tôi nhận được.)
Điều duy nhất bạn không muốn làm trong các nhận xét đánh giá mã là tấn công cá nhân ai đó. Nó cũng có thể giúp nếu bạn tìm thấy một cái gì đó họ đã làm tốt và chỉ ra rằng đó là tốt. Đôi khi tôi học được điều gì đó mới từ đánh giá mã và nếu tôi đã nói với người đó.