🔍 Code Review

Etkili kod inceleme ve kalite kontrol rehberi.

---

📋 İçindekiler

1. Review Checklist
2. Code Smell Detection
3. PR Best Practices
4. Review Comments

---

1. Review Checklist

Fonksiyonellik

- [ ] Kod istenen işi yapıyor mu?
- [ ] Edge case'ler handle ediliyor mu?
- [ ] Error handling yeterli mi?
- [ ] Input validation var mı?

Kod Kalitesi

- [ ] DRY prensibi uygulanmış mı?
- [ ] Single Responsibility takip ediliyor mu?
- [ ] Naming conventions tutarlı mı?
- [ ] Magic numbers/strings yok mu?

Güvenlik

- [ ] SQL injection riski var mı?
- [ ] XSS riski var mı?
- [ ] Sensitive data expose edilmiş mi?
- [ ] Authentication/authorization doğru mu?

Performans

- [ ] N+1 query problemi var mı?
- [ ] Gereksiz re-render var mı?
- [ ] Memory leak riski var mı?
- [ ] Büyük dosya/data handling doğru mu?

---

2. Code Smell Detection

Yaygın Code Smell'ler

Smell	Açıklama	Çözüm
Long Method	>20 satır fonksiyon	Extract Method
Large Class	>300 satır class	Extract Class
Long Parameter List	>3 parametre	Parameter Object
Duplicate Code	Tekrarlayan bloklar	Extract Method/Class
Dead Code	Kullanılmayan kod	Sil
Magic Numbers	Açıklamasız değerler	Constants
Deep Nesting	>3 seviye if/loop	Early return, Extract
God Class	Her şeyi yapan class	Single Responsibility

Algılama Komutları

# ESLint complexity check
npx eslint . --rule 'complexity: ["error", 10]'

# SonarQube
sonar-scanner

# Code coverage
npm run test:coverage

---

3. PR Best Practices

İdeal PR Boyutu

• Küçük: <200 satır (ideal)
• Orta: 200-400 satır
• Büyük: >400 satır (bölünmeli)

PR Açıklama Template

## Özet
Kısa açıklama

## Değişiklik Tipi
- [ ] Bug fix
- [ ] Yeni özellik
- [ ] Refactoring
- [ ] Breaking change

## Test
- Test X yapıldı
- Test Y sonucu: başarılı

## Screenshots (UI değişikliği varsa)

Commit Messages

feat: Add user authentication
fix: Resolve memory leak in cache
refactor: Extract validation logic
docs: Update API documentation
test: Add unit tests for user service
chore: Update dependencies

---

4. Review Comments

Etkili Yorum Yazma

❌ Kötü: "Bu yanlış"
✅ İyi: "Bu yaklaşım X durumunda hata verebilir. Y alternatifini düşünebilir misin?"

❌ Kötü: "Bunu değiştir"
✅ İyi: "suggestion: Bu fonksiyon extract edilse okunabilirlik artar"

Yorum Prefixleri

Prefix	Anlam
blocking:	Merge edilemez, düzeltilmeli
suggestion:	Öneri, isteğe bağlı
question:	Açıklama gerekiyor
nitpick:	Minor, önemsiz
praise:	İyi iş!

---

Code Review v1.1 - Enhanced

🔄 Workflow

Kaynak: Google Engineering Practices

Aşama 1: Triage (Ön Kontrol)

• CI Checks: Testler geçmiş mi? Lint hatası var mı?
• Scope: PR çok mu büyük? (>400 satır ise bölmesini iste).
• Description: "Ne" ve "Neden" açıkça anlatılmış mı?

Aşama 2: Deep Dive

• Logic: Kodun algoritması doğru ve verimli mi?
• Architecture: Mevcut mimari desenlere uyuyor mu?
• Test: Yeni özellikler için test yazılmış mı?

Aşama 3: Feedback

• Comments: Yapıcı, nazik ve net yorumlar yaz (suggestion:, question:).
• Decision: Approve, Request Changes veya Comment.

Kontrol Noktaları

Aşama	Doğrulama
1	PR açıklaması boş ise reddedildi mi?
2	Breaking change varsa versiyonlamaya dikkat edildi mi?
3	Yorumlar kişiye değil koda mı yönelik?