🔍 Code Review

🔍 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

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

---

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ı