异步匿名无参数方法作为方法参数

Question

我有一个班级用作我所有ViewModels 的父级。它包含一个用于调用其他方法的特定方法，并在错误时显示加载消息和消息框（主要）：

public class BaseViewModel
{
    async public void Try(Func<Task> action, string errorMessage = null, string waitMessage = null)
    {
        try
        {
            if (waitMessage != null)
                ShowLoading(waitMessage);
            await action();
        }
        catch (Exception e)
        {
            ShowError(errorMessage, e);
        }
        finally
        {
            HideLoading();
        }
    }
}

它是异步的，所以我的ShowLoading 可以是动画之类的。

1. 是否正确实施？

它将始终获得匿名 (lambda) 无参数 Tasks。我的主要问题是如何实际构建这些Tasks。假设我在ViewModelBase 的孩子中有一个Command，它在执行时调用以下方法：

private void OnMyCommandExecute()
{
    Try(() =>
    {
        Thread.Sleep(5000);
    }, "error", "please wait");
}

它无法编译，因为Not all code paths return a value in lambda expression of type 'System.Func<System.Threading.Tasks.Task>'。很明显，因为我们await 这个Func。这就引出了第二个问题：

1. 在此示例中，我应该在 Try 调用中添加什么内容才能使其正常工作？

我尝试了一些非常丑陋的东西，我真的希望答案会有所不同，否则可读性会很差：

Try(async () =>
{
    return await Task.Factory.StartNew(() =>
    {
        SharePointService.Connect(Connection);
        IsConnected = true;
    });
}

它不会编译，但在这一点上，最好是这样。 return 上的错误：Since 'System.Func<System.Threading.Tasks.Task>' is anasyncmethod that returns 'Task', a return keyword must not be followed by an object expression. Did you intend to return 'Task<T>'?

Answer 1

我应该在这个示例中的 Try 调用中添加什么内容才能正常工作？

您需要通过添加（令人惊讶的）async 来创建 lambda 表达式 async：

Try(async () =>
{
    Thread.Sleep(5000);
}, "error", "please wait");

然而，虽然这将使您能够创建一个async 委托，但它实际上并没有什么异步的（它用Thread.Sleep 阻塞调用线程）。如果这只是一个例子，那么：

Try(async () =>
{
    await Task.Delay(5000);
}, "error", "please wait");

更好。如果不是，请不要使用async。

是否正确实施？

不是真的。 async void 几乎总是应该避免（除非在 UI 事件处理程序中）。请改用async Task，并确保在某个时间点对返回的任务进行await，以确保操作完成而没有任何异常。

Answer 2

Try 接受一个返回Task 的方法。在您的第一个示例中，您提供的方法是 void。

在您的第二个示例中，您提供了一个返回 Task<Task> 的方法，但尝试在需要 Task（非泛型）的上下文中使用它。

如果您想使用非异步 lambda，则只需让该 lambda 返回您要使用的 Task：

Try(()=>Task.Factory.StartNew(() =>
    {
        SharePointService.Connect(Connection);
        IsConnected = true;
    }));

如果你想使用async lambda，那么你需要等待任务而不返回它：

Try(async () => await Task.Factory.StartNew(() =>
    {
        SharePointService.Connect(Connection);
        IsConnected = true;
    }));

请注意，在这里使用异步 lambda 并没有真正的目的。这两个 sn-ps 将执行相同的操作，但第二个增加了一些额外的代码膨胀开销以及运行时实际上不需要的整个状态机。

Answer 3

为了让Try 尽可能透明，我最终做到了这一点。

async public Task Try(Action action, string errorMessage = null, string waitMessage = null)
{
    try
    {
        if (waitMessage != null)
        {
            ShowLoading(waitMessage);
            await Task.Factory.StartNew(() => action());
        }
        else
            action();
    }
    catch (Exception e)
    {
        ShowError(errorMessage, e);
    }
    finally
    {
        HideLoading();
    }
}

因此，调用时不必使用Task.Factory.StartNew 或async/await：

Try(() =>
{
    Thread.Sleep(5000);
}, "error", "please wait");