BAB 04

04 - AI untuk Code Review dan Refactoring

Estimasi Waktu: 35-45 menit Prasyarat: 01-ai-assisted-coding.md

Apa Itu Materi Ini?

Materi ini membahas penggunaan AI untuk code review dan refactoring. AI bisa menjadi “reviewer pertama” yang cepat sebelum code masuk ke human review, dan bisa membantu mengidentifikasi kode yang perlu di-refactor beserta strategi perbaikannya.

Tujuan Pembelajaran

Setelah mempelajari materi ini, kamu akan mampu:

Menggunakan AI untuk review kode secara efektif
Memahami aspek-aspek yang perlu direview dengan AI
Menggunakan AI untuk merencanakan dan mengeksekusi refactoring
Mengetahui keterbatasan AI dalam code review

1. AI sebagai Reviewer Pertama

Posisi AI dalam Code Review Process

┌─────────────────────────────────────────────────────────┐
│          CODE REVIEW PROCESS DENGAN AI                   │
│                                                         │
│  Developer writes code                                  │
│       │                                                 │
│       ▼                                                 │
│  AI REVIEW (cepat, comprehensive)                       │
│  - Security issues                                      │
│  - Best practices                                       │
│  - Potential bugs                                       │
│  - Performance                                          │
│       │                                                 │
│       ▼                                                 │
│  Developer fixes AI findings                            │
│       │                                                 │
│       ▼                                                 │
│  HUMAN REVIEW (context, design, business logic)         │
│  - Architecture decisions                               │
│  - Business logic correctness                           │
│  - Team conventions                                     │
│       │                                                 │
│       ▼                                                 │
│  MERGE                                                  │
└─────────────────────────────────────────────────────────┘

2. Template Prompt untuk Code Review

Template: Comprehensive Review

Review kode berikut secara menyeluruh:

```[bahasa]
[paste kode]

Review dari aspek berikut:

BUGS & CORRECTNESS
- Logic errors
- Off-by-one errors
- Null/undefined handling
- Race conditions (jika async)
- Resource leaks
SECURITY
- Input validation
- SQL injection
- XSS vulnerabilities
- Authentication/Authorization issues
- Sensitive data exposure
PERFORMANCE
- Unnecessary loops
- Memory leaks
- Inefficient algorithms
- Missing caching opportunities
MAINTAINABILITY
- Readability
- Naming conventions
- Function length
- Complexity (cyclomatic)
- DRY violations
BEST PRACTICES
- Language/framework specific
- Error handling patterns
- Logging practices

Untuk setiap issue:

Severity: [Critical/High/Medium/Low]
Line number
Explanation
Suggested fix


### Template: Security-Focused Review

Review kode ini SECARA KHUSUS untuk security vulnerabilities:

[kode]

Framework: [misal: Express.js, Django, Spring Boot]

Periksa OWASP Top 10:

Injection (SQL, NoSQL, Command)
Broken Authentication
Sensitive Data Exposure
XML External Entities
Broken Access Control
Security Misconfiguration
Cross-Site Scripting (XSS)
Insecure Deserialization
Using Components with Known Vulnerabilities
Insufficient Logging & Monitoring

Untuk setiap vulnerability yang ditemukan:

Severity (Critical/High/Medium/Low)
Type of vulnerability
How it could be exploited
How to fix it


### Template: Performance Review

Analisis performa kode berikut:

[kode]

Context:

Data volume: [misal: 100K records, growing 10% per month]
Concurrent users: [misal: 1000]
Response time target: [misal: < 200ms]

Analisis:

Big O complexity untuk setiap operasi
Bottleneck identification
Memory usage patterns
Database query efficiency
Caching opportunities

Berikan:

Performance ranking (current state 1-10)
Top 3 optimizations yang paling berdampak
Refactored code untuk setiap optimization
Expected improvement


---

## 3. Refactoring dengan AI

### Kapan Perlu Refactoring?

TANDA KODE PERLU REFACTORING: ├── Fungsi terlalu panjang (> 20-30 baris) ├── Duplikasi kode (copy-paste di banyak tempat) ├── Nama yang tidak deskriptif ├── Komentar yang menjelaskan WHAT, bukan WHY ├── Nested conditionals yang dalam (> 3 level) ├── God class/function (melakukan terlalu banyak hal) ├── Magic numbers tanpa konstanta └── Tight coupling antar komponen


### Template: Refactoring Request

Refactor kode berikut untuk meningkatkan readability dan maintainability:

[kode asli]

Tujuan refactoring:

Extract method untuk fungsi panjang
Rename variable/function yang tidak deskriptif
Hapus duplikasi
Simplify conditionals
Apply design pattern yang sesuai jika ada

Constraints:

Jangan ubah behavior (refactoring harus preserve functionality)
Jangan over-engineer - tetap simple
Preserve backward compatibility
Tetap dalam bahasa yang sama

Berikan:

Kode yang sudah di-refactor
Penjelasan setiap perubahan dan mengapa
Perbandingan sebelum/sesudah untuk bagian kunci


### Template: Design Pattern Application

Kode berikut menggunakan pendekatan prosedural yang sulit di-maintain:

[kode]

Identifikasi design pattern yang cocok dan refactor:

Identifikasi masalah structural
Rekomendasikan design pattern yang sesuai
Implement pattern-nya
Jelaskan benefit dari pattern yang dipilih
Tunjukkan bagaimana pattern ini memudahkan extension di masa depan


---

## 4. Teknik Refactoring dengan AI

### Teknik 1: Step-by-Step Refactoring

Jangan minta AI refactor semua sekaligus. Lakukan bertahap:

Prompt 1: “Identifikasi 3 masalah terbesar di kode ini dan urutkan berdasarkan dampak.”

Prompt 2: “Refactor masalah pertama saja. Jelaskan perubahan dan pastikan behavior tidak berubah.”

Prompt 3: “Sekarang refactor masalah kedua. Pastikan tidak mengganggu perbaikan pertama.”

dst.


### Teknik 2: Comparison Review

“Saya ingin membandingkan dua pendekatan untuk masalah yang sama:

Pendekatan A: [kode A]

Pendekatan B: [kode B]

Bandingkan dari:

Readability
Performance
Maintainability
Testability
Edge case handling

Rekomendasikan mana yang lebih baik dan mengapa.”


### Teknik 3: Legacy Code Modernization

Kode berikut adalah legacy code yang perlu di-modernisasi:

[kode lama]

Target: [misal: migrasi dari JavaScript ke TypeScript, atau dari callback ke async/await, atau dari class component ke hooks]

Rencanakan migrasi:

Identifikasi semua area yang perlu berubah
Buat migration plan langkah demi langkah
Untuk setiap langkah, berikan kode sebelum dan sesudah
Identifikasi risiko di setiap langkah
Berikan strategy untuk testing di setiap tahap migrasi


---

## 5. Keterbatasan AI dalam Code Review

### AI Tidak Bisa:
- Memahami business context penuh
- Menilai apakah kode memenuhi requirement bisnis
- Memahami konvensi tim yang tidak tertulis
- Menjalankan kode dan melihat performa nyata
- Memahami dependencies di seluruh codebase
- Menilai UX impact dari perubahan kode

### Best Practice: AI + Human Review

AI REVIEW: HUMAN REVIEW:

Syntax errors - Architecture decisions
Security vulnerabilities - Business logic correctness
Performance anti-patterns - UX impact
Missing error handling - Team conventions
Code style violations - Requirement fulfillment
Obvious bugs - Domain-specific knowledge
Test coverage gaps - Integration concerns

AI cepat tapi superficial Human lambat tapi contextual ↓ ↓ └─────────── COMPLEMENTARY ─────────────┘


---

## Latihan (~15 menit)

### Pilihan A: Review Kode Sendiri
Ambil kode yang pernah kamu tulis dan gunakan template Comprehensive Review. Evaluasi temuan AI - ada yang valid? Ada yang terlewat?

### Pilihan B: Refactoring Challenge
Cari kode yang perlu di-refactor (bisa dari internet atau projectmu). Gunakan AI untuk merencanakan dan mengeksekusi refactoring bertahap.

### Pilihan C: Security Audit
Ambil kode yang menangani user input dan gunakan template Security-Focused Review. Implement perbaikan yang direkomendasikan.

---

## Poin-Poin Utama

1. **AI adalah reviewer pertama** yang cepat dan comprehensive, bukan pengganti human review
2. **Review dari banyak aspek**: bugs, security, performance, maintainability, best practices
3. **Refactoring bertahap** lebih aman daripada big-bang refactor
4. **Selalu pastikan behavior tidak berubah** setelah refactoring
5. **Security review** sangat penting - AI bisa menemukan vulnerabilities yang terlewat
6. **AI + Human review** adalah kombinasi terbaik - masing-masing melengkapi kekurangan
7. **Kamu tetap bertanggung jawab** atas kode yang di-commit

---

## Informasi Materi

- **Lisensi:** CC BY-NC-SA 4.0