Review #1

[jerryhoangtran]

Hi, mình có xem qua bài, có review và recommend như này:

• Code cơ bản CURD, có sử dụng TCP làm transporter, sử dụng mongoose ODM + mongo
• Về coding: Dính nhiều lỗi cơ bản như là hardcode, if-else hơi rối, thiếu try catch hoặc exception handle, cái này có thể đi tham khảo thêm docs của nest để đắp thêm.
• Phần logic + business: Bài này còn đơn giản, ví dụ làm order chế thêm mấy thông tin của order ra như là product, tiền nong, địa chỉ,… nữa thì tốt hơn, thiếu validate input, cover các case sai, cover exception, ngồi ngẫm nghĩ đề rồi tự nghĩ thêm situation thêm rồi sửa cũng đc.

Suggestion:

• Bịa cho business logic phức tạp hơn
• Thêm authentication
• Suy nghĩ về những lỗ hổng của hệ thống có thể gây sai logic/dữ liệu khi user gửi request không theo expect, hay khi nhiều request đến hệ thống đồng thời
• Cài sonarlint vào IDE và follow - Nghiên cứu setup SonarCloud và xem kết quả phân tích từ tool

Một số link có thể đọc để improve: TS:

• https://eslint.org/docs/latest/rules/no-return-await#rule-details
• https://eslint.org/docs/latest/rules/no-await-in-loop
• https://eslint.org/docs/latest/rules/no-dupe-keys
• More: https://eslint.org/docs/latest/rules/#possible-problems

NodeJS:

• https://nodejs.org/en/docs/guides/dont-block-the-event-loop/
• https://snyk.io/blog/nodejs-how-even-quick-async-functions-can-block-the-event-loop-starve-io/

Microservice:

• https://interviewnoodle.com/original-sin-of-microservices-part-1-90461ddcefb
• https://lazypro.medium.com/original-sin-of-microservices-part-2-8856c0e8426d

[vihnpalm]

Mình xin cảm ơn bạn đã review qua cho mình. Mình muốn hỏi là cover các case sai ở đây cụ thể là về vấn đề gì ạ.