You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It's normally considered a bad practice to have the database model classes also being used to send data to the user in responses.
It's better - and IMHO easier to maintain - to have the database model classes with its single responsibility, and move all de/serialization specifics to DTO's (or any other acronym like VO/POJO/DO/etc used in the project).
Description
Part of #254 . Realized I was having to deal with the MongoDB specifics and also with how data is de/serialized 😥 It would have been a bit simpler should these two were separated entities.
Expected Behavior
Separate classes, separate concerns.
Actual Behavior
Possible Fix
Steps to Reproduce
NA
Context
Your Environment
Version used:
Environment name and version (e.g. Chrome 39, Java 1.8):
The text was updated successfully, but these errors were encountered:
It's normally considered a bad practice to have the database model classes also being used to send data to the user in responses.
It's better - and IMHO easier to maintain - to have the database model classes with its single responsibility, and move all de/serialization specifics to DTO's (or any other acronym like VO/POJO/DO/etc used in the project).
Description
Part of #254 . Realized I was having to deal with the MongoDB specifics and also with how data is de/serialized 😥 It would have been a bit simpler should these two were separated entities.
Expected Behavior
Separate classes, separate concerns.
Actual Behavior
Possible Fix
Steps to Reproduce
NA
Context
Your Environment
The text was updated successfully, but these errors were encountered: