Где DI вписывается в CQS?

Question

Должны ли вы прибегать к внедрению свойства всякий раз, когда команде требуется зависимость?

Скажите, у меня есть следующая команда.

public class AddReviewCommand
{
    private ISession _session;
    private readonly string _reviewSummary;

    public AddReviewCommand(string reviewSummary)
    {
        _reviewSummary = reviewSummary;
    }

    public void Execute()
    {
        var review = new Review
            {
                AddedBy = Environment.Username,
                AddedDateTime = DateTime.Now,
                ReviewSummary = _reviewSummary
            };

            _session.Save(review);
    }
}

Является ли единственный способ ввести ISession путем инъекции свойства?

Answer 1

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

public class AddReviewCommand
{
    private ISession _session;
    private readonly string _reviewSummary;

    public AddReviewCommand(string reviewSummary, ISession session)
    {
        _reviewSummary = reviewSummary;
        _session = session;
    }

    public void Execute()
    {
        var review = new Review
        {
            AddedBy = Environment.Username,
            AddedDateTime = DateTime.Now,
            ReviewSummary = _reviewSummary
        };

        _session.Save(review);
    }
}