Расширения для внедрения зависимостей #256
Conversation
|
SonarCloud Quality Gate failed. |
| /// Получить именованную зависимость с помощью IUnityContainer (IUnityContainer должен быть зарегистрирован в IServiceProvider). | ||
| /// </summary> | ||
| /// <typeparam name="T">Тип.</typeparam> | ||
| /// <param name="serviceProvider">IServiceProvider.</param> |
There was a problem hiding this comment.
"Тип.
/// IServiceProvider."
Непонятные комментарии.
| using Unity; | ||
|
|
||
| /// <summary> | ||
| /// Расширения для внедрения зависимостей. |
There was a problem hiding this comment.
Формально в новых классах нужно стараться делать комментарии на английском.
|
|
||
| if (unityContainer == null) | ||
| { | ||
| throw new InvalidOperationException($"Невозможно получить именнованную зависимость - IUnityContainer не зарегистрирован в этом ServiceProvider."); |
There was a problem hiding this comment.
Лучше на английском.
| } | ||
| else | ||
| { | ||
| return (unityContainer as IUnityContainer).Resolve<T>(name); |
There was a problem hiding this comment.
А вот тут некоторая особенность. Дело в том, что насколько я поняла, в описании IServiceProvider в методе GetService если зависимость не зарегана, то должно вернуться null. UnityFactory в таком раскладе вернёт exception.
Но опять же, метод isregistered не совсем удачен для проверки, потому что может быть что-то не зарегано, но имеющихся сведений может хватить, чтобы зарезолвить.
| /// <param name="name">Имя зависимости.</param> | ||
| /// <returns>Разрешенная зависимость.</returns> | ||
| /// <exception cref="InvalidOperationException">Ошибка получения именованной зависимости.</exception> | ||
| public static object GetService<T>(this IServiceProvider serviceProvider, string name) |
There was a problem hiding this comment.
Обязательно нужны тесты:
- Ничего не зарегано.
- Зарегано всё.
- Зареганы только параметры конструктора, а сам тип нет.
- Именованные\неименованные резолвы.
There was a problem hiding this comment.
Тесты реализованы здесь
turbcool
force-pushed
the
feature-di-extension
branch
from
775f599 to
384e52d
Compare
|
SonarCloud Quality Gate failed.
|
Добавлена возможность резолвить именованные зависимости через
IServiceProvider, используя внутри зарегистрированныйIUnityContainer.