Saat meninjau kode, saya biasanya mencoba membuat rekomendasi khusus tentang cara menyelesaikan masalah. Tetapi karena waktu yang terbatas yang dapat dihabiskan seseorang untuk meninjau, ini tidak selalu bekerja dengan baik. Dalam kasus ini saya merasa lebih efisien jika pengembang sendiri yang membuat solusi.
Hari ini saya meninjau beberapa kode dan menemukan bahwa kelas jelas tidak dirancang dengan baik. Itu memiliki sejumlah atribut opsional yang hanya ditugaskan untuk objek tertentu dan dibiarkan kosong untuk yang lain. Cara standar untuk menyelesaikan ini adalah dengan membagi kelas dan menggunakan warisan. Namun dalam kasus khusus ini solusi ini tampaknya terlalu rumit. Saya sendiri tidak terlibat dalam pengembangan perangkat lunak ini dan tidak terbiasa dengan semua modul. Karena itu saya tidak merasa cukup berpengetahuan untuk membuat keputusan tertentu.
Kasus khas lain yang saya alami berkali-kali adalah saya menemukan fungsi, kelas atau nama variabel yang jelas tidak berarti atau bahkan menyesatkan, tetapi saya sendiri tidak dapat menemukan nama yang bagus.
Jadi secara umum, sebagai peninjau, apakah boleh mengatakan "kode ini cacat karena ..., lakukan secara berbeda" atau apakah Anda harus menemukan solusi spesifik?