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

Подумать возможно интерфейс IAutofacScopeHolder не нужен #78

Open
Gankov opened this issue Jan 9, 2020 · 1 comment
Labels
Solution needed Вопрос требует принятия общего решения

Comments

@Gankov
Copy link
Member

Gankov commented Jan 9, 2020

Изначально предпологалось что этот интерфейс в первую очередь нужен для заполнения ViewModel или другого объекта его текущим скопом. И это реализовано в фабрике ViewModel ей. НО, возможно это плохой способ, и проще добавлять скоп в конструктор, тогда авто фак сам заполнить скоп. И это будет работать более универсально, нежели текущая реалзация, так как сейчас она заколинт только для ViewModel. А через конструктор для чего угодно.

Единственно для чего этот интерфейс можно было бы использовать это для получения скопа внешним классом, но мы с внедрением DI практически избавляемся от такого рода зависимостей....

Хочется просто услышать ваше мнение.

@Gankov Gankov added the Solution needed Вопрос требует принятия общего решения label Jan 9, 2020
@Gankov
Copy link
Member Author

Gankov commented Mar 1, 2020

Почти уверен что он не нужен, покрайней мере для установки. Потому что часто скоп нужен именно в конструкторах. А в этот момент установить скоп через интерфейс нет возможности. Может только если оставить для чтения, но сомнительнго....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Solution needed Вопрос требует принятия общего решения
Projects
None yet
Development

No branches or pull requests

1 participant