Написание оболочки для метода асинхронной

Question

У меня есть класс, который TransactionOperator выставляет следующий статический метод асинхронного:

public static async Task<bool> ProcessTransactionAsync(Transaction transaction) 
{ 
    var someTransactionOperator = ...; // get appropriate operator 
    // some code here 
    bool success = await someTransactionOperator.Process(transaction); 
    // some more code 
    return bool; 
} 

Теперь я хочу, чтобы обеспечить метод экземпляра обертки в Transaction классе. Мой вопрос в том, что было бы правильным/рекомендуемым способом его написания? Я склоняюсь к # 2, потому что чувствует себя правильно, но у меня нет никаких аргументов поддержки для этого выбора.

// Option 1 
public bool ProcessAsync() 
{ 
    return TransactionOperator.ProcessTransactionAsync(this).Result; 
} 

// Option 2 
public Task<bool> ProcessAsync() 
{ 
    return TransactionOperator.ProcessTransactionAsync(this); 
} 

// Option 3 (compiler warning because there's no 'await' operator) 
public async Task<bool> ProcessAsync() 
{ 
    return TransactionOperator.ProcessTransactionAsync(this).Result; 
} 

// Option 4 
public async Task<bool> ProcessAsync() 
{ 
    return await TransactionOperator.ProcessTransactionAsync(this); 
} 

Answer 1

Вариант 2 - лучший вариант. Вариант 4 логически эквивалентен, но имеет больше накладных расходов.

Варианты 1 и 3 неправильные. Они оба блокируют синхронно (хотя опция 3 равна async, она ведет себя синхронно). Exposing synchronous wrappers for asynchronous methods is not recommended. Среди других проблем вы можете cause deadlocks (как я расскажу в своем блоге).