Использование как операции с базой данных, так и внешнего метода в единице работы

В Unit of Work вы управляете transactional операциями. В области Unit of Work вы запускаете некоторую бизнес-логику, и когда все операции готовы, вызываются операции Complete или SaveChanges. Эта операция дает вам шанс, что все ваши операции завершатся успешно или все ваши операции будут отменены. В этом случае код не работает как единое целое. Есть две отдельные операции: операции вставки и операции увеличения веб-службы.

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

В качестве решения вы рассматриваете изменение своего подхода.

1) Операция вызова WebService может быть обернута и преобразована как транзакционная операция. Я предлагаю инструмент, который называется Hangfire. Он сохраняет имя операции и параметры в БД, а транзакция завершает чтение БД и запускает зарегистрированные функции. Вы можете сохранить entity и call webservice operation execution в БД как одну операцию.

2) Вы можете сохранить entity и опубликовать user-created event или increment-user-count command. Ваш наблюдатель/потребитель воспользовался этим событием/командой и выполнил вызов веб-API.

Оба решения в конечном итоге дают согласованность.

Из документации по Unit Of Работа:

Если метод выдает исключение, транзакция откатывается и соединение уничтожается. Таким образом, метод единицы работы является атомарным (единица работы).

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

Вы можете использовать метод SaveChanges или SaveChangesAsync для текущей единицы работы.

Обратите внимание, что если текущая единица работы является транзакционной, все изменения в транзакции откатываются в случае возникновения исключения. Даже сохраненные изменения!

Итак, сгенерируйте исключение перед вызовом Complete для отката:

try
{
    using (var unitOfWork = _unitOfWorkManager.Begin(TransactionScopeOption.RequiresNew))
    {
        _personRepository.Insert(person);

        // Save changes
        _unitOfWorkManager.Current.SaveChanges();

        var result = externalWebService.IncrementPeopleCount();
        if (result != 200)
        {
            // Rollback
            throw new MyExternalWebServiceException("Unable to increment people count!");
        }

        // Commit
        unitOfWork.Complete();
    }
}
catch (MyExternalWebServiceException)
{
    // Transaction rolled back, propagate exception?
    throw;
}

MyExternalWebServiceException может наследовать UserFriendlyException для отображения пользователю:

public class MyExternalWebServiceException : UserFriendlyException
{
    public MyExternalWebServiceException(string message)
        : base(message)
    {
    }
}

Что делает вызов externalWebService? Синхронизируется ли это с другим репозиторием данных? Или это обновление элемента пользовательского интерфейса?

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

Я мог бы привести аналогичный аргумент в отношении синхронизации двух наборов данных.

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

В данном конкретном случае я не вижу причин, по которым целостность данных вашего личного репозитория должна зависеть от успеха второго вызова.