Bahasa Inggris dalam Code Review
Mengapa Code Review Membutuhkan Bahasa Inggris Khusus
Section titled “Mengapa Code Review Membutuhkan Bahasa Inggris Khusus”Code review bukan sekadar umpan balik teknis — ini adalah komunikasi tim. Pilihan kata yang tidak tepat dapat:
- Terkesan sombong atau sarkastis
- Melukai motivasi rekan kerja
- Menimbulkan kesalahpahaman (terutama di tim lintas budaya)
- Membuat orang berpikir Anda “mencari-cari kesalahan” daripada membantu dengan tulus
Kabar baiknya: ada serangkaian ekspresi standar yang dipahami semua orang.
10 Ekspresi Code Review yang Wajib Dipelajari
Section titled “10 Ekspresi Code Review yang Wajib Dipelajari”1. Nit (Masalah kecil)
Section titled “1. Nit (Masalah kecil)”“Nit” berasal dari “nitpick” (mencari-cari kesalahan kecil), tetapi dalam code review ini adalah istilah positif, berarti “ini sangat kecil, tidak mempengaruhi keseluruhan, tapi layak diperbaiki”.
❌ "This variable name is bad"✅ "Nit: could we rename this to `userCount`?"✅ "Minor: I'd prefer `userCount` for clarity"2. LGTM (Looks Good To Me)
Section titled “2. LGTM (Looks Good To Me)”Singkatan, menandakan Anda sudah selesai review dan setuju untuk merge.
LGTM! 👍Atau lebih formal:
LGTM. Great implementation.3. Blocker (Masalah pemblokir)
Section titled “3. Blocker (Masalah pemblokir)”Masalah serius, harus diperbaiki sebelum merge.
Blocker: This will cause a race condition in `handleUserLogin`.Here's why: ...4. Nice catch (Pengamatan bagus)
Section titled “4. Nice catch (Pengamatan bagus)”Ketika orang lain menemukan bug, masalah performa, atau kebutuhan tersembunyi, Anda mengonfirmasi mereka benar.
Nice catch! I didn't think about the edge case where `null` ...5. Could you elaborate (Bisakah Anda jelaskan lebih detail)
Section titled “5. Could you elaborate (Bisakah Anda jelaskan lebih detail)”Ketika Anda tidak mengerti mengapa ditulis demikian, atau ingin memahami keputusan desain.
Could you elaborate on why we're using `Map` instead of `Object`?6. Looks good (Setuju dengan bagian ini)
Section titled “6. Looks good (Setuju dengan bagian ini)”Digunakan untuk persetujuan sebagian.
This part looks good. One question about line 42: why are we...?7. We should probably (Saran)
Section titled “7. We should probably (Saran)”Saran yang lembut, tidak wajib tapi direkomendasikan.
We should probably add error handling here for the case where the API call fails.8. Have you considered (Pertanyaan terbuka)
Section titled “8. Have you considered (Pertanyaan terbuka)”Mengajukan alternatif tanpa langsung mengatakan “harus begini”. Memberikan ruang berpikir.
Have you considered using `useCallback` to prevent unnecessary re-renders?9. Ship it (Luncurkan saja)
Section titled “9. Ship it (Luncurkan saja)”Persetujuan paling kasual, menandakan sudah diperiksa, tidak ada masalah, boleh diluncurkan.
Ship it! 🚀Rumus Kesopanan Code Review
Section titled “Rumus Kesopanan Code Review”Saat Menunjukkan Masalah
Section titled “Saat Menunjukkan Masalah”Rumus: Context → Issue → Suggestion
❌ "This code is inefficient"
✅ "In `processUserData`, iterating `O(n²)` might be slow with large datasets. Could we use a `Set` to get `O(n)` instead?"Saat Berbeda Pendapat
Section titled “Saat Berbeda Pendapat”❌ "That's wrong"
✅ "I see your approach. One concern: `Math.random()` isn't cryptographically secure for auth tokens. Should we use `crypto.getRandomValues()`?"Mode Express dalam Code Review
Section titled “Mode Express dalam Code Review”Pikiran Anda dalam bahasa ibu → Tiga opsi bahasa Inggris → Pilih yang paling tepat
Skenario: Menunjukkan bug
Section titled “Skenario: Menunjukkan bug”Anda ingin mengatakan: “Ada race condition di sini, beberapa request akan bentrok”
Buka DevGlish Express Mode:
Input: "Ada race condition di sini, beberapa request akan bentrok"
Basic (Direct): "There is a race condition here, multiple requests will conflict"
Intermediate (Natural): "Multiple concurrent requests could cause a race condition"
Native (Idiomatic): "Concurrent requests might race here. Consider using a lock or queue."Kesalahan Umum Code Review
Section titled “Kesalahan Umum Code Review”Interferensi Umum Penutur Non-Inggris
Section titled “Interferensi Umum Penutur Non-Inggris”| Ekspresi Salah | Alasan | Versi Lebih Baik |
|---|---|---|
| ”This code is not good” | Terlalu langsung, terkesan sombong | ”Could we refactor this for clarity?" |
| "You must change this” | Terdengar seperti perintah | ”We should probably change this" |
| "This is wrong” | Menyakitkan | ”This might cause X in scenario Y" |
| "The code of the function is bad” | Penggunaan “of” berlebihan | ”The function could be clearer” |
Referensi Cepat
Section titled “Referensi Cepat”| Situasi | Ekspresi |
|---|---|
| Setuju + masalah kecil | LGTM, just one nit: … |
| Tidak setuju dengan keputusan | I see your approach, but have we considered…? |
| Masalah serius | Blocker: This will cause… |
| Saran kecil | Nit: Could we…? |
| Setuju + ingin belajar | Nice! Could you explain why you chose…? |
| Tidak yakin | I’m not sure about this. Could we discuss? |
| Sepenuhnya setuju | LGTM! Ship it. |