Kod İncelemesi (Code Review) Yaparken Nelere Dikkat Ediyorum

Code review, yalnızca hata avlama süreci değil; bilgi paylaşımı ve ortak standart oluşturma fırsatıdır. Zamanla oluşturduğum kontrol listesini paylaşıyorum.

Önce Doğruluk, Sonra Stil

Bir PR'ı incelerken ilk sorduğum soru: "Bu kod, amaçlanan işi doğru yapıyor mu?" Değişken isimlendirme veya formatlama gibi stil tercihleri, otomatik araçlara (linter, formatter) bırakılmalı; insan gözü mantık hatalarına odaklanmalı.

Edge Case'leri Sormak

• Boş liste geldiğinde ne olur?
• Ağ isteği başarısız olursa kullanıcı ne görür?
• Eşzamanlı iki istek aynı kaydı güncellemeye çalışırsa?

Bu sorular, çoğu zaman yazarın da düşünmediği senaryoları ortaya çıkarır.

Testlerin Varlığından Çok Neyi Test Ettiğine Bakmak

Yüksek test kapsamı yanıltıcı olabilir. Testlerin gerçekten kritik iş mantığını mı yoksa yalnızca "kolay" kod yollarını mı kapsadığına bakmak daha değerli.

Küçük PR'lar, Hızlı Geri Bildirim

500 satırlık bir PR'ı derinlemesine incelemek pratik olarak imkansız. Küçük, odaklı PR'lar hem incelemeyi kolaylaştırır hem de hataların erken yakalanmasını sağlar.

Yorumlarda Ton Önemli

"Bu yanlış" yerine "Şu senaryoda sorun yaşanabilir mi?" gibi soru odaklı geri bildirim, savunmacı tepkiler yerine yapıcı bir tartışma başlatıyor.

Sonuç

İyi bir code review kültürü, ekip içinde güven inşa eder ve bilgi tek bir kişide birikmez. Bu süreç ne kadar özenli yürütülürse, üretim ortamındaki sürprizler o kadar azalıyor.