C # - должен ли объект нести ответственность за создание объекта истории, когда он меняет что-то вроде статуса?

Ваш метод может работать нормально, но позвольте мне предложить альтернативу.

Почему бы не добавить событие TransactionAdded в класс Account.

Затем вы можете подписаться на событие из (я предполагаю, здесь) объекта HistoryGroup, чтобы новый объект AccountHistory добавлялся каждый раз при запуске события.

ОБНОВЛЕНИЕ

Как упоминалось в комментариях, другим методом достижения цели было бы реализовать HistoryGroup интерфейс (ITransactionLogger или что-то подобное), а затем изменить Account, чтобы можно было внедрить зависимость ITransactionLogger.

Использование любого из этих маршрутов немного упрощает управление с точки зрения сложности и отладки, но не позволяет использовать несколько регистраторов, таких как события.

Это сделает ваш код немного более гибким и в то же время позволит другим потребителям, заинтересованным в событии TransactionAdded, подписаться.

В некотором смысле я согласен с ответом Джастина, но один из тегов OP - POCO; добавление события в класс Account в некотором роде приведет к отключению POCO от POCO.

Если вы работаете с АОП и другим подобным, вы можете использовать перехват (большинство фреймворков IoC, включая Unity и Castle, предлагают эту функцию) для захвата интересующих транзакций.

Преимущество перехвата состоит в том, что ваш класс Account не имеет никакой связи с классом AccountHistory, перехват легко настраивается в соответствии с любыми правилами, которые вы хотите, и его легко изменить без принудительной перекомпиляции приложения (если вы поместите AccountHistory в другую сборку с помощью обработчики перехвата). Используя перехват, вы делаете свой код более сфокусированным на бизнес-сфере, а не на том, что можно рассматривать как задачу инфраструктуры (аудит).

Опять же, это еще одна альтернатива вашему набору инструментов; если вам не нужно сериализовать POCO по сети по какой-либо причине, то реализация шаблона наблюдателя (GoF) через события, как предлагает Джастин, может быть более легким подходом.

Похоже, что так думает банда из четырех человек. Транзакции, отслеживание истории и невыполнение - все это часть контракта шаблон команды. Вы можете реализовать историю с помощью стека. Вот фрагмент соответствующего кода, включая контракт, обратите внимание, что не все методы реализованы или должны быть реализованы:

public interface ICommand
{
    void execute();
    void undo();
    void store();
    void load();
}
public class ManagerMacro : ICommand
{
    List<ICommand> Commands;
    Stack commandStack;
    /// <summary>
    /// Use in combination with AddSteps
    /// </summary>
    //public ManagerMacro()
    //{

    //}
    public ManagerMacro(List<ICommand> commands)
    {
        this.Commands = commands;
        this.commandStack = new Stack();
    }

    #region ICommand Members

    public void execute()
    {
        for (int i = 0; i < Commands.Count; i++)
        {
            commandStack.Push(Commands[i]);
            Commands[i].execute();
        }
    }

    public void undo()
    {
        for (int i = 0; i < Commands.Count; i++)
        {
            if (commandStack.Count > 0)
            {
                ICommand Command = (ICommand)commandStack.Pop();
                Command.undo();
            }
        }
    }
    public void store()
    {
        throw new NotImplementedException();
    }

    public void load()
    {
        throw new NotImplementedException();
    }
    #endregion

    public void AddSteps(Steps[] steps)
    {
        foreach (Steps step in steps)
        {
            ICommand thisStep = null;
            switch (step)
            {
                case Steps.Manager1: thisStep = new Step1(); break;
                case Steps.Manager2: thisStep = new Step2(); break;
                case Steps.Manager3: thisStep = new Step3(); break;
                case Steps.Manager4: thisStep = new Step4(); break;
            }
            this.Commands.Add(thisStep);
        }
    }
}

Обратите внимание, что я также использую заводской узор.