Является ли вызов другого кода нарушением (SOLID) Принципов единой ответственности (SRP)?

Хороший вопрос, заголовок немного вводит в заблуждение (маловероятно, что вы сможете создать приложение без «вызова другого кода»). Помните, что принципы SOLID - это скорее рекомендации, чем абсолютные правила, которым необходимо следовать; если вы доведете SRP до логического завершения, вы получите по одному методу на класс. Способ минимизировать влияние сквозных проблем - создать максимально простой в использовании фасад. В ваших примерах вы сделали это хорошо - каждая проблема пересечения использует только одну линию.

Другой способ добиться этого - использовать AOP, что возможно в C # с помощью PostSharp или через Перехват IoC

Нет ничего плохого в том, что метод класса координирует действия некоторых других классов, не нарушая SRP. Вы нарушите его, если логика этих классов была частью OrderShipper.

Я не уверен, что делает PerformanceProfiler, но это единственный компонент, который там выглядит странно.

Сделаем его более заметным, преобразовав класс в команду:

// Command pattern
public class ShipOrder
{
    ITransactionFactory _transactionFactory;


    public OrderShipper(ITransactionFactory factory)
    {
        if (factory == null) throw new ArgumentNullException("factory");

        _transactionFactory = factory;
    }

    [PrincipalPermission(Roles = "User")]
    public void Execute(Order order) 
    {
        if (order == null) throw new ArgumentNullException("order");

        using (new PerformanceProfiler()) 
        {
            HandleWithRetries(() => ShipOrderInTransaction(order));
        }
    }

    private void ShipOrderInTransaction(Order order) 
    {
        using (var transaction = _transactionFactory.Create()) 
        {
            ShipOrderInternal(order);

            transaction.Commit();
        }            
    }

    protected void ShipOrderInternal(order) 
    {
        // bussiness logic which is divided into different protected methods.
    }
}

Следовательно, вы можете вызвать его, используя:

var cmd = new ShipOrder(transactionFactory);
cmd.Execute(order);

Это довольно солидно.

Да, это нарушает SRP, по крайней мере, в соответствии с названием класса.

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

Вы отвечаете сами, он должен содержать только логику доставки заказа. И он не должен быть статичным (почему он статичный ?!).

Решение, предоставляемое @jgauffin, возможно, хотя я не совсем уверен, что OrderShipper должен знать о транзакции или должен быть только ее частью. Также профилировщик производительности IMO не имеет места в этом классе. Но имея только эту информацию, я не могу предложить решение. Однако профилирование - это комплексная проблема, и, возможно, лучше было бы обрабатывать его вне этого класса, возможно, с помощью атрибута.

Кстати, используя подход, управляемый сообщениями (как предлагает jgauffin), он должен позволить инфраструктуре обеспечивать профилирование и поддержку надежности (HandleWithRetries)