Dưới đây là bảng checklist hành động 15 thực hành cốt lõi – được Việt hóa và tối ưu cho lập trình viên .NET (C#, WebAPI, Dapper, EF, RabbitMQ, WinForms) như bạn.
Bảng này giúp bạn và team tự đánh giá chất lượng code mỗi tuần, giống cách các senior developer ở các công ty lớn thực hiện.
| # | Thực hành | Mục tiêu chính | Cách áp dụng thực tế trong .NET | Đánh giá tuần này (✓/✗) |
|---|---|---|---|---|
| 1 | Viết code dễ dự đoán (Predictable) | Hành vi code rõ ràng, không side-effect ẩn | Tránh việc function GetData() lại vừa cập nhật DB; tách riêng Query và Command (CQRS pattern) | |
| 2 | Hàm nhỏ, đơn nhiệm (Single Responsibility) | Code dễ đọc, dễ test | Nếu hàm dài >30 dòng → tách logic ra service hoặc helper riêng | |
| 3 | Hiểu rõ “tại sao” | Không chỉ làm theo tutorial | Khi thêm logic → ghi chú rõ lý do trong comment XML hoặc Git commit message | |
| 4 | Viết code cho người khác đọc | Tăng tính bảo trì | Đặt tên rõ ràng, viết summary <summary> cho method/class | |
| 5 | Refactor định kỳ | Giảm nợ kỹ thuật | Dành 10–20% thời gian mỗi sprint dọn code: xóa code chết, gom logic trùng | |
| 6 | Viết unit test / integration test | Tự tin refactor | Sử dụng xUnit hoặc NUnit, test service trước khi gắn vào controller | |
| 7 | Đọc code người khác | Học cách tổ chức tốt hơn | Review pull request của đồng nghiệp mỗi tuần 1–2 lần | |
| 8 | Giữ mọi thứ đơn giản (KISS) | Code dễ hiểu hơn | Tránh generic phức tạp không cần thiết, hoặc over-abstract (ví dụ base class quá nặng) | |
| 9 | Không tối ưu sớm | Tập trung rõ ràng trước, hiệu năng sau | Chỉ optimize khi profiling thấy bottleneck thật | |
| 10 | Thiết kế mở rộng được | Giảm công sức sửa đổi | Dùng interface, DI, cấu hình từ appsettings.json, áp dụng Open/Closed Principle | |
| 11 | Đặt tên có ý nghĩa | Giúp hiểu nhanh | Tên biến thể hiện mục đích, ví dụ: is2FAEnabled, dailyTransactions, không viết tắt kiểu trnDly | |
| 12 | Code Review chất lượng | Giữ chuẩn chung của team | Áp dụng checklist: Naming, Null check, Exception, Log, Async/Await chuẩn | |
| 13 | Log & xử lý lỗi chuẩn hóa | Dễ điều tra sự cố | Dùng LogHelper.Warning / LogHelper.Info, log context (IP, userId, controller) rõ ràng | |
| 14 | Không phụ thuộc framework | Tăng khả năng chuyển đổi | Hiểu rõ RabbitMQ, Dapper, EFCore làm gì phía sau → không chỉ dùng qua wrapper | |
| 15 | Học hỏi & mentoring | Nâng tầm team | Chia sẻ mini session: “Code smell trong dự án tuần này”, hoặc “Refactor mẫu thực tế” |
🧩 Mẫu đánh giá định kỳ (team dùng hàng tuần)
| Nhóm | Điểm tối đa | Điểm đạt được | Ghi chú |
|---|---|---|---|
| Readability & Predictability | 20 | ||
| Structure & Maintainability | 25 | ||
| Testing & Quality | 20 | ||
| Logging & Error Handling | 15 | ||
| Collaboration & Review | 20 |
👉 Tổng điểm ≥ 80/100 → “Senior Level Code Quality”.
💬 Gợi ý thêm cho dự án của bạn
- Tích hợp checklist này vào CI/CD pipeline (ví dụ dùng GitHub Actions hoặc Azure DevOps) → hiển thị báo cáo quality score mỗi lần merge.
- Lưu checklist này trong Notion/Jira → team tick trực tiếp theo sprint.
- Kết hợp SonarQube hoặc Roslyn Analyzer để tự động nhắc code smell, duplication, naming rule.
