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
Interesting project! Have a couple of questions concerning some choices in the project.
Placing the IAppLogger interface in the Domain layer does not seem aligned with the typical responsibilities of the Domain layer in Clean Architecture. The Domain layer is primarily responsible for encapsulating the core business logic and domain entities. Logging is generally considered an infrastructure concern, and exposing logging interfaces in the Domain layer could violate the separation of concerns.
Why did you place that interface in the Core/Domain layer?
Also, Application Layer usually defines interfaces that are implemented by the outer layers, so why are repository interfaces inside the Core Layer and not the Application Layer? You seem to implement them in the Infrastructure layer as recommended so I'm not getting this choice.
Thank you
The text was updated successfully, but these errors were encountered:
HybridSolutions
changed the title
IAppLogger Interface placement according to Clean Architecture
Class placement issues according to Clean Architecture
Dec 14, 2023
Hi,
Interesting project! Have a couple of questions concerning some choices in the project.
Placing the IAppLogger interface in the Domain layer does not seem aligned with the typical responsibilities of the Domain layer in Clean Architecture. The Domain layer is primarily responsible for encapsulating the core business logic and domain entities. Logging is generally considered an infrastructure concern, and exposing logging interfaces in the Domain layer could violate the separation of concerns.
Why did you place that interface in the Core/Domain layer?
Also, Application Layer usually defines interfaces that are implemented by the outer layers, so why are repository interfaces inside the Core Layer and not the Application Layer? You seem to implement them in the Infrastructure layer as recommended so I'm not getting this choice.
Thank you
The text was updated successfully, but these errors were encountered: