Code Smells – Nhận Diện & Làm Sạch Code Ngay Từ Đầu

Một bài tổng quan giúp bạn hiểu rõ Code Smells là gì, vì sao chúng nguy hiểm, và cách nhận diện những dấu hiệu phổ biến trong code để refactor kịp thời.

#Code Smells

Code smells là những dấu hiệu cảnh báo rằng code của bạn đang có vấn đề về thiết kế hoặc cấu trúc — và cần được refactor (tái cấu trúc).

Khi xử lý sớm các code smells, bạn sẽ giữ được codebase gọn gàng, dễ mở rộng và dễ bảo trì hơn. Ngược lại, nếu bỏ qua, chúng sẽ tích tụ và khiến dự án trở nên khó kiểm soát.

⚠️ Hậu quả khi không refactor thường xuyên:

• Code ngày càng khó đọc, khó hiểu
• Việc thêm tính năng mới trở nên chậm chạp
• Dễ phát sinh bug khi chỉnh sửa
• Tệ nhất: phải viết lại toàn bộ hệ thống

👉 Kết luận: Hãy xử lý code smells khi chúng còn nhỏ — đừng để chúng trở thành “technical debt” khổng lồ.

#Bloaters

Bloaters là những đoạn code, method hoặc class bị “phình to” quá mức theo thời gian.

📌 Đặc điểm:

• Class quá nhiều responsibility
• Method quá dài, nhiều logic
• Code bị nhồi nhét, vá víu liên tục

📉 Vấn đề gây ra:

• Khó đọc, khó maintain
• Khó debug
• Khó tách logic

💡 Hiểu đơn giản: Ban đầu code rất gọn, nhưng qua thời gian bị thêm thắt liên tục → trở thành một khối lớn khó kiểm soát.

#Long Method

Long Method là khi một method chứa quá nhiều dòng code.

📌 Dấu hiệu nhận biết:

• Method dài hơn ~10 dòng
• Có nhiều if/else, loop lồng nhau
• Làm nhiều hơn một nhiệm vụ

📉 Vấn đề gây ra:

• Khó đọc
• Khó test
• Dễ gây bug khi sửa

💡 Best practice: Một method nên chỉ làm một việc duy nhất và làm tốt việc đó.

👉 Nếu thấy method quá dài → hãy tách nhỏ thành các function rõ ràng hơn.

#God Object

God Object là một class “làm tất cả mọi thứ” trong hệ thống.

📌 Dấu hiệu nhận biết:

• Class rất lớn
• Chứa nhiều logic không liên quan
• Bị phụ thuộc bởi nhiều nơi

📉 Vấn đề gây ra:

• Khó test
• Khó maintain
• Dễ gây side effects khi sửa

💡 Hiểu đơn giản: “Biết quá nhiều, làm quá nhiều” = God Object.

👉 Vi phạm nguyên tắc: Single Responsibility Principle (SRP)

#Duplicate Code

Duplicate Code là khi cùng một logic bị lặp lại ở nhiều nơi.

📌 Dấu hiệu nhận biết:

• Copy-paste code
• Nhiều đoạn code giống nhau ở các file khác nhau

📉 Vấn đề gây ra:

• Khó maintain
• Dễ quên update khi sửa
• Tăng nguy cơ bug

💡 Best practice:

• Tách thành function
• Tạo reusable component
• Áp dụng DRY (Don’t Repeat Yourself)

#Primitive Obsession

Primitive Obsession là việc lạm dụng các kiểu dữ liệu cơ bản (string, int, array) để biểu diễn logic phức tạp.

📌 Dấu hiệu nhận biết:

• Dùng string cho email, phone, address
• Dùng array thay vì object có structure rõ ràng

📉 Vấn đề gây ra:

• Thiếu rõ ràng
• Dễ truyền sai dữ liệu
• Khó validate

💡 Giải pháp: Tạo các Value Object hoặc class riêng:

• Email
• Money
• UserProfile

👉 Giúp code rõ ràng, an toàn và đúng domain hơn.

#Tổng kết

👉 Chìa khóa: Refactor sớm – refactor thường xuyên – refactor có chiến lược.

✨ Clean code không phải là viết lại từ đầu — mà là cải thiện liên tục từng chút một.

👉 Ở bài tiếp theo, chúng ta sẽ bắt đầu với: Long Method