thắc mắc Các bác review code giúp em ạ

  • Người tạo chủ đề Người tạo chủ đề tranhungajc
  • Ngày bắt đầu Ngày bắt đầu
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 =))
 

Thống kê chủ đề

Ngày tạo
tranhungajc,
Người trả lời cuối
Yurisha,
Trả lời
21
Lượt xem
3.123
Quay lại
Lên đầu trang