thắc mắc Các bác review code giúp em ạ
- Thread starter tranhungajc
- Start date
Yurisha
Senior Member
1. vi phạm S trong SOLID principal, filter này thực hiện quá nhiều logic về cơ bản chỉ có quan hệ phụ thuộc tuyến tính, hoàn toàn phù hợp với việc tách nhỏ ra và sắp xếp vào filter chain. Vi phạm S nên code smell kinh, logic flow loạn cả lên.
2. ObjectMapper là 1 object khởi tạo khá tốn kém và tôi đoán dùng ở rất nhiều chỗ khác, tại sao không inject?
3. đoạn đọc map để vẽ menu rất thiếu null check
4. 1 cái nhỏ tôi thường không cho pass khi review code là cộng chuỗi khi log
2. ObjectMapper là 1 object khởi tạo khá tốn kém và tôi đoán dùng ở rất nhiều chỗ khác, tại sao không inject?
3. đoạn đọc map để vẽ menu rất thiếu null check
4. 1 cái nhỏ tôi thường không cho pass khi review code là cộng chuỗi khi log
