Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IHandlingConfiguration - Single responsibility principle #7

Open
gravity00 opened this issue Jun 16, 2016 · 0 comments
Open

IHandlingConfiguration - Single responsibility principle #7

gravity00 opened this issue Jun 16, 2016 · 0 comments

Comments

@gravity00
Copy link
Owner

At the present moment, the interface IHandlingConfiguration is responsible for storing handling configurations and running the handling logic itself. This could be considered a violation of the single responsibility principle.

To resolve this question, it could be split, at least, in two interfaces:

  • IHandlingConfiguration - stores the collection of handlers and other handling properties;
  • IHandlingService - contains the logic for handling exceptions, and uses an IHandlingConfiguration to decide his behavior;

This implementation should allow a more decoupled handling logic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant