Apakah umum bagi siapa saja untuk mendapatkan 100+ komentar dalam ulasan kode mereka secara rutin? Saya akan mengatakan tidak. Apakah biasa bagi orang yang kualitas kodenya "meninggalkan banyak yang diinginkan" untuk mendapatkan banyak komentar, tentu saja.
Namun, itu juga tergantung pada "aturan" dari proses peninjauan kode. SEMUA ORANG memiliki ide mereka sendiri tentang bagaimana sesuatu seharusnya dilakukan. Jika proses peninjauan kode Anda memungkinkan komentar berbentuk "Anda harus melakukannya dengan cara ini, bukan seperti itu", maka Anda kemungkinan akan mendapatkan BANYAK komentar bahkan untuk kode yang memadai. Jika proses Anda dimaksudkan untuk menemukan "cacat" maka jumlah komentar harus jauh lebih kecil.
Dalam pengalaman saya, ulasan yang memungkinkan "saran" untuk metode alternatif adalah pemborosan waktu. "Saran" itu harus ditangani satu lawan satu di luar proses peninjauan. Ulasan yang cacat lebih bermanfaat karena membuat orang berfokus pada bug daripada "mengapa Anda tidak melakukannya seperti yang saya lakukan?". Ini juga lebih bermanfaat karena tidak dapat disangkal bug jika seseorang menemukannya. Dengan demikian, tidak ada perasaan terluka, tetapi kemungkinan besar rasa terima kasih sebagai gantinya.
PEMBARUAN: Dengan semua yang dikatakan, beberapa kode benar-benar buruk, meskipun cacat gratis. Dalam hal itu, komentar ulasan harus berupa komentar tunggal yang mengatakan sesuatu seperti. "Kode ini perlu dibersihkan. Silakan tunda ulasan sampai kode dibahas dengan [nama Anda di sini]." Dalam hal ini, peninjauan kode lebih lanjut harus berhenti sampai komentar diperbaiki.
UPDATE2: @ Pengguna: Apakah Anda mendiskusikan kode / desain Anda dengan salah satu dari mereka saat Anda sedang mengembangkannya sehingga Anda dapat mengimplementasikan apa yang mereka cari sebelum Anda melakukannya dengan cara Anda? Apakah Anda mengubah sesuatu tentang bagaimana Anda mengembangkan kode berdasarkan saran mereka atau tetap berpikir cara Anda baik-baik saja? Apakah Anda belajar sesuatu dari komentar mereka?
Ketika saya memimpin proyek, itu adalah tugas saya untuk bertanggung jawab atas SEMUA produk kerja. Jika saya menyetujui produk kerja maka saya mengklaim produk tersebut dapat diterima. Saya ingin memiliki reputasi untuk membangun produk yang berkualitas. Jadi, saya memiliki harapan dan tidak akan menerima kurang memuaskan. Pada saat yang sama saya mencoba mengajar dan menjelaskan alasan preferensi saya. Preferensi-preferensi itu mungkin tidak selalu ideal (terutama di mata orang lain), tetapi sebagian besar preferensi itu berasal dari pengalaman. Biasanya merupakan reaksi untuk menghindari pengulangan yang buruk. Jadi, ada beberapa "stickler" pribadi saya yang diperlukan untuk mendapatkan persetujuan saya, terlepas dari pushback.
Di sisi lain, Anda perlu mempelajari harapan yang diperlukan untuk mendapatkan produk kerja Anda disetujui. Anda dapat tidak setuju, tetapi karena Anda tampaknya tidak memiliki wewenang untuk memerintah maka pelajari apa yang diharapkan. Saya ragu bahwa tim berusaha membuat Anda gagal. Karena itu membuat mereka terlihat buruk juga. Dalam hal itu, cukup tunjukkan bahwa Anda ingin belajar (bahkan jika tidak), ambil apa yang mereka katakan dan lakukan yang terbaik untuk beradaptasi dengan preferensi mereka dan Anda mungkin akan melihat mereka mundur sedikit. Mungkin temukan yang paling tidak bisa Anda toleransi dan lihat apakah mereka akan melakukan sedikit pegangan tangan untuk mengajari Anda cara mereka. Siapa tahu, dalam prosesnya Anda bisa belajar sesuatu yang benar-benar bisa membawa keterampilan Anda ke tingkat selanjutnya.