结合抽象工厂模式的异步和非异步实现

Question

我们的在线商店有一个 ASP.Net MVC 应用程序。用户必须从多种付款方式中进行选择才能购买商品。为此，我们实现了一个抽象工厂模式：

public interface IPaymentServiceFactory
{
    IPaymentService GetPaymentService(PaymentServiceEnum paymentServiceType);
}
public interface IPaymentService
{
    PaymentSettingsModel GetPaymentSettingsModel();
}

在我们的Action中使用：

public ActionResult ProcessCart(PaymentDataModel paymentData)
{
    var paymentService = _paymentServiceFactory.GetPaymentService(paymentData.PaymentServiceType);
    var paymentSettings = paymentService.GetPaymentSettingsModel();
}

当我们了解某些支付方式需要内部异步调用时，就会出现问题。例如，第三方在线支付服务方法必须通过http异步调用，以便在他们身边创建支付对象。实现：

    public class OnlinePaymentService : IPaymentService
    {
        private readonly IOnlinePaymentServiceApiClient _client;
        public async Task<PaymentSettingsModel> GetPaymentSettings()
        {
            var result =  await _client.CreatePaymentAsync();
            return result;
        }
    }

所以我们提出了一个问题：如何处理不同支付方式的异步和同步场景。我们决定让一切异步。更新代码：

public interface IPaymentService
{
    Task<PaymentSettingsModel> GetPaymentSettings();
}
public async Task<ActionResult> ProcessCart(PaymentDataModel paymentData)
{
    var paymentService = _paymentServiceFactory.GetPaymentService(paymentData.PaymentServiceType);
    var paymentSettings = await paymentService.GetPaymentSettingsModel();
}

到目前为止一切都很好，但要为所有其他支付方式实现这一点，我们不得不使用 Task.Run：

    public class CashPaymentService : IPaymentService
    {
        public async Task<PaymentSettingsModel> GetPaymentSettings()
        {
            return await Task.Run(() => new PaymentSettingsModel());;
        }
    }

据我所知，这会创建两个不同的线程来处理 Action，这可能会导致 performance issue。 有没有办法避免这样的后果？在特定情况下使用 Task.Run 真的很糟糕吗？

Answer 1

在特定情况下使用 Task.Run 真的很糟糕吗？

是的，主要是因为它不必要地使事情复杂化。

您可以使用Task.FromResult 返回一个结果为给定值的已完成任务。

这是完全同步的：

public class CashPaymentService : IPaymentService
{
    public Task<PaymentSettingsModel> GetPaymentSettings()
    {
        return Task.FromResult( new PaymentSettingsModel() );
    }
}

请注意这里缺少async - 这是可能的，因为它是一个实现细节，而不是IPaymentService 定义的一部分。