Уместно ли использовать Инъекции свойств в базовом классе, когда зависимость требуется только в базовом классе?

Question

Пример:

public abstract class BaseControler : Controller 
{ 
    public IUnitOfWork UnitOfWork { get; set; } 
} 

public class HomeController : BaseControler 
{ 
    readonly IUserRepository _userRepository; 

    // :-) 
    public HomeController(IUserRepository userRepository) 
    { 
     _userRepository = userRepository; 
    } 
} 

Мы все знаем, что мы должны использовать Constructor Injection когда зависимость является требуется. Если это факультативно зависимость, мы можем использовать Вклад в эксплуатацию.

Но что делать, если только ваш базовый класс требует зависимости?

Если вы использовали бы Конструктор Injection, вы бы, на мой взгляд, загрязнили все производные классы.

public abstract class BaseControler : Controller 
{ 
    readonly IUnitOfWork _unitOfWork; 

    public BaseControler(IUnitOfWork unitOfWork) 
    { 
     _unitOfWork = unitOfWork; 
    } 
} 

public class HomeController : BaseControler 
{ 
    readonly IUserRepository _userRepository; 

    // :-(
    public HomeController(IUserRepository userRepository, 
     IUnitOfWork unitOfWork) : base(unitOfWork) 
    { 
     _userRepository = userRepository; 
    } 
} 

ли approperiate использовать Injection Property в базовом классе, когда зависимость требуется только в базовом классе?

Answer 1

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

Если базовый класс требует этой зависимости для правильной работы, поскольку производный класс получен из этого базового класса, он также неявно требует этой зависимости. Итак, второй пример - правильный путь. Вы не должны использовать инъекцию свойств для того, что является необходимой зависимостью.

Answer 2

Фактически ваш производный класс IS является базовым классом. Другого объекта нет. Когда вы передаете что-то в базовый конструктор, вы фактически инициализируете тот же объект. И это показывает клиентам, что ваш экземпляр зависит от:

Существует один экземпляр, который зависит от двух вещей: хранилище пользователя, и единицы работы. Вызов конструктора базового класса просто удаляет дублирование инициализации из ваших производных классов.