Là một kỹ sư phần mềm có 20 năm, chủ yếu làm việc về các công cụ liên quan đến an toàn (SF-PD), tôi phải nói rằng ông chủ của bạn có thể không phải là người bạn muốn trở thành ví dụ của bạn. Thiếu ý kiến là một dấu hiệu của một lập trình viên nghiệp dư tự học, người không bao giờ học được cách thực hiện công việc đúng cách, hoặc một kỹ sư thiếu kinh nghiệm. Hoặc có lẽ một kỹ sư đơn giản là không có thời gian - thời hạn và sự nhanh chóng có thể làm những điều khủng khiếp cho mã của bạn! ;) Đó chắc chắn là một mô hình chống cho mọi kỹ sư phần mềm có thẩm quyền.
Sếp của bạn có thể là một lập trình viên rất giỏi, nhưng có vẻ như anh ta không phải là một kỹ sư phần mềm giỏi. Một kỹ sư sử dụng kinh nghiệm nhóm tập thể để tránh những cạm bẫy mà người khác đã bị bắt gặp. Nhận xét hiệu quả là một phần của trải nghiệm nhóm tập thể đó đối với phần mềm, giống như phân tích ứng suất là một phần của trải nghiệm nhóm tập thể cho kỹ thuật cơ khí. Những gì được coi là nhận xét hiệu quả là trôi chảy hơn, và đó chắc chắn là thứ bạn có được từ kinh nghiệm.
Điều cơ bản nhất là các bình luận không nên nói những gì một dòng mã làm. Đôi khi, các bình luận cho biết những gì một chức năng làm là quá thừa (đặc biệt là trong C #). Nhận xét quá mức có thể chỉ là không hiệu quả (và là một con trỏ thiếu kinh nghiệm) bởi vì bạn không thể tìm thấy những thứ quan trọng trong quá khứ. Là một người mới, bạn vẫn có thể làm việc để tìm ra "cái gì" của mã, và bạn chỉ cần đọc và hiểu những gì anh ta đã làm.
Điều quan trọng cho các bình luận là họ nói TẠI SAO một dòng mã hoặc một hàm làm những gì nó làm, trong đó điều này có thể không rõ ràng. Bạn có cần thiết lập mô-đun X trước mô-đun Y không? Điều quan trọng là phải kiểm tra mã trả về để xem liệu một tệp đã được mở hay chúng ta có ý thức bỏ qua mã trả về vì điều này đã được kiểm tra ở một nơi khác? "Tại sao" của mã sẽ có liên quan đến tất cả mọi người, bất kể kinh nghiệm - và nó cũng sẽ liên quan đến anh ta trong 6 tháng, khi anh ta quên mất lý do chính đáng để làm một việc cụ thể. Nhận xét không chỉ dành cho người khác, nó cũng giúp bạn trong tương lai.
Nếu bạn muốn tránh làm phiền sếp, hãy hỏi những câu hỏi thông minh. Tập trung vào việc hỏi về "tại sao" và cố gắng tự mình tìm ra "cái gì" (trừ khi nó thực sự tối nghĩa). Không có ông chủ tốt nào sẽ bận tâm khi được hỏi những câu hỏi nếu chúng không phải là những thứ mà bạn có thể tìm thấy từ R-ing TFM. Và sẽ không có kỹ sư giỏi nào được yêu cầu làm điều gì đó sẽ giúp cuộc sống của một kỹ sư khác dễ dàng hơn, với chi phí thấp cho họ. (Chỉ cần đừng yêu cầu anh ấy điền vào các bình luận trên toàn bộ cơ sở mã!)