Lewati ke konten

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”

“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"

Singkatan, menandakan Anda sudah selesai review dan setuju untuk merge.

LGTM! 👍

Atau lebih formal:

LGTM. Great implementation.

Masalah serius, harus diperbaiki sebelum merge.

Blocker: This will cause a race condition in `handleUserLogin`.
Here's why: ...

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`?

Digunakan untuk persetujuan sebagian.

This part looks good. One question about line 42: why are we...?

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?

Persetujuan paling kasual, menandakan sudah diperiksa, tidak ada masalah, boleh diluncurkan.

Ship it! 🚀

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?"
❌ "That's wrong"
✅ "I see your approach. One concern: `Math.random()` isn't cryptographically
secure for auth tokens. Should we use `crypto.getRandomValues()`?"

Pikiran Anda dalam bahasa ibu → Tiga opsi bahasa Inggris → Pilih yang paling tepat

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."
Ekspresi SalahAlasanVersi 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”
SituasiEkspresi
Setuju + masalah kecilLGTM, just one nit: …
Tidak setuju dengan keputusanI see your approach, but have we considered…?
Masalah seriusBlocker: This will cause…
Saran kecilNit: Could we…?
Setuju + ingin belajarNice! Could you explain why you chose…?
Tidak yakinI’m not sure about this. Could we discuss?
Sepenuhnya setujuLGTM! Ship it.