Tiếng Anh trong Code Review

Tại sao code review cần tiếng Anh đặc biệt

Code review không chỉ là phản hồi kỹ thuật — đó là giao tiếp nhóm. Cách diễn đạt không phù hợp có thể:

❌ Có vẻ kiêu ngạo hoặc châm biếm
❌ Làm đồng nghiệp mất động lực
❌ Gây hiểu lầm (đặc biệt trong nhóm đa văn hóa)

Tin vui là: có một bộ biểu đạt chuẩn mà mọi người đều hiểu.

10 biểu đạt thiết yếu trong code review

1. Nit (vấn đề nhỏ)

“Nit” xuất phát từ “nitpick”, nhưng trong code review là từ tích cực: “nhỏ, không ảnh hưởng đến cốt lõi, nhưng đáng cải thiện”.

❌ "This variable name is bad"
✅ "Nit: could we rename this to `userCount`?"

2. LGTM (Looks Good To Me)

Từ viết tắt nghĩa là bạn đã xem xét và đồng ý merge.

3. Blocker (vấn đề chặn)

Vấn đề nghiêm trọng phải sửa trước khi merge.

Blocker: This will cause a race condition in `handleUserLogin`.

4. Nice catch (phát hiện hay)

Khi ai đó tìm ra bug hoặc vấn đề ẩn.

Nice catch! I didn't think about the edge case where `null` ...

5. Have you considered (bạn có cân nhắc không)

Đề xuất phương án thay thế mà không áp đặt.

Have you considered using `useCallback` to prevent unnecessary re-renders?

6. We should probably (chúng ta nên)

Gợi ý nhẹ nhàng, không bắt buộc nhưng được khuyến nghị.

We should probably add error handling here for the case where the API call fails.

7. Ship it (phát hành thôi)

Đồng ý không chính thức: đã xem, không có vấn đề, có thể deploy.

Ship it! 🚀

Công thức lịch sự trong code review

Khi chỉ ra vấn đề

Công thức: Ngữ cảnh → Vấn đề → Gợi ý

❌ "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?"

Khi không đồng ý

❌ "That's wrong"

✅ "I see your approach. One concern: `Math.random()` isn't cryptographically
   secure for auth tokens. Should we use `crypto.getRandomValues()`?"

Lỗi phổ biến

Biểu đạt sai	Lý do	Phiên bản tốt hơn
”This code is not good”	Quá trực tiếp, có vẻ kiêu ngạo	”Could we refactor this for clarity?"
"You must change this”	Nghe như lệnh	”We should probably change this"
"This is wrong”	Gây tổn thương	”This might cause X in scenario Y”

Bảng tham khảo nhanh

Tình huống	Biểu đạt
Đồng ý + chi tiết nhỏ	LGTM, just one nit: …
Không đồng ý về quyết định	I see your approach, but have we considered…?
Vấn đề nghiêm trọng	Blocker: This will cause…
Gợi ý nhỏ	Nit: Could we…?
Hoàn toàn đồng ý	LGTM! Ship it.