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

Be explicit about failing assumptions #2

Open
corstian opened this issue Dec 13, 2023 · 0 comments
Open

Be explicit about failing assumptions #2

corstian opened this issue Dec 13, 2023 · 0 comments

Comments

@corstian
Copy link
Member

Ran into a practical issue with a custom implementation of the IAggregateHandlerFactory. As the Instantiate method had been an explicit interface implementation, the default interface method as implemented on the IAggregateHandlerFactory was unable to use reflection to direct the call to the implementation.

The action point here is to be more explicit about assumptions, and explicitly throw an exception when those are broken. Specifically scoped to operations involving reflection.

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

No branches or pull requests

1 participant