Khi xem xét mã, tôi thường cố gắng đưa ra các khuyến nghị cụ thể về cách giải quyết các vấn đề. Nhưng do thời gian có hạn mà người ta có thể dành để xem xét, điều này không phải lúc nào cũng hoạt động tốt. Trong những trường hợp này, tôi thấy hiệu quả hơn nếu nhà phát triển tự đưa ra giải pháp.
Hôm nay tôi đã xem lại một số mã và thấy rằng một lớp rõ ràng không được thiết kế tốt. Nó có một số thuộc tính tùy chọn chỉ được gán cho một số đối tượng nhất định và để trống cho các đối tượng khác. Cách tiêu chuẩn để giải quyết điều này sẽ là tách lớp và sử dụng kế thừa. Tuy nhiên trong trường hợp cụ thể này, giải pháp này dường như quá phức tạp. Bản thân tôi không tham gia vào việc phát triển phần mềm này và không quen thuộc với tất cả các mô-đun. Vì vậy, tôi không cảm thấy đủ hiểu biết để đưa ra quyết định cụ thể.
Một trường hợp điển hình khác mà tôi đã trải qua nhiều lần là tôi thấy một hàm rõ ràng là vô nghĩa hoặc thậm chí sai lệch, tên lớp hoặc tên biến nhưng bản thân tôi không thể đưa ra một cái tên hay.
Vì vậy, nói chung, với tư cách là một nhà phê bình, có ổn không khi nói "mã này bị sai sót bởi vì ..., làm khác đi" hoặc bạn phải đưa ra một giải pháp cụ thể?