Ketika saya melakukan review kode, saya cenderung hanya memiliki monolog yang sedang berjalan, jadi ketika saya memahami apa yang saya baca akan ada banyak "Ok, saya mengerti apa yang terjadi .. Bagus terhubung dengan ini dan memanggil itu, baiklah .. dan bagian itu tergantung pada keduanya baik - baik saja. "
Saya pikir dengan cara ini bukan "oo la la ini begitu hebat!", Itu bisa menjadi kode membosankan sepele, tetapi mendengar orang lain benar-benar mengurai dan menunjukkan pemahaman tentang apa yang Anda tulis adalah bentuk umpan balik positif di dalam dan dari dirinya sendiri, umpan baliknya adalah "Kode ini masuk akal", ketika saya menemukan bagian-bagian yang saya tidak mengerti, saya meminta penjelasan dan ketika saya memahaminya berseru, "Ah, saya mengerti".
Saya pikir transfer pemahaman yang sederhana adalah pujian untuk insinyur lain karena kita semua ingin kode kita dipahami oleh orang lain, itu memberikan bentuk validasi implisit.
Yang mengatakan, jika Anda melihat bagian dari kode yang karakteristik baik atau positif (bahkan kode sepele yang membosankan dapat menjadi baik jika itu bentuk minimal dari dirinya sendiri) Saya pasti cenderung menyatakan karakteristik itu, sekali lagi saya tidak menganggapnya sebagai "Wow Bagus!" sama seperti "Saya melihat ini adalah implementasi minimal" atau "Ok, algoritma yang kompleks ini memiliki banyak komentar", fokus pada atribut kode tidak begitu banyak sifatnya baik atau buruk.
Setiap kali Anda mengaitkan "kebaikan" atau "kejahatan" dengan kode dalam tinjauan kode untuk menghindari membuat insinyur merasa diremehkan atau dipegang di atas alas tidak mengatakan sesuatu itu baik atau buruk, tetapi lebih baik membicarakan sebab dan akibat dari kode mereka.
"Ok bagian ini masuk akal, ah ada angka ajaib di sini, arti dari nilai itu mungkin tidak dipahami dengan baik oleh insinyur berikutnya untuk menyentuh ini"
"Aku tahu kamu punya wadah DI di sini ok jadi kamu akan kehilangan kopling dengan repositori itu"
"Ah ada kamus statis di sini, jika banyak utas menyentuh kamus itu kita bisa mengalami beberapa kondisi lomba"
Perhatikan, saya tidak mengatakan sesuatu yang baik atau buruk, tetapi apakah insinyur harus mengubahnya atau tidak akan dipahami oleh insinyur yang kodenya sedang ditinjau. Jelas Anda harus mengakhiri tinjauan kode dengan yay atau nay, tetapi mengumpulkan pernyataan ini selama itu akan melunakkan nay sebagai penjelasan telah dibuat dalam bentuk sebab dan akibat pernyataan ketika Anda memberi tahu mereka "Saya ingin angka-angka ajaib diperbaiki sebelum memeriksa ini di ".