正确使用 Async/Await [关闭]

Question

这是我的 async/await 实现，但我不确定我是否正确执行。

代码按预期工作，但我希望有人检查这段小代码以指出任何错误。

public class DbUtils
{

    public static List<string> GetDataSources()
    {
        //I have removed some logic from here as it's not important to my question
        List<string> names = new List<string>();
        names = SomeClass.SomeLongSystemMethod();
        return names;
    }

    public static async Task<List<string>> GetDataSourcesAsync()
    {
        //This is a place where I have my doubts
        return await Task.Run(() =>
        {
            return GetDataSources();
        });
    }
}

public class SomeOtherClass
{
    private async void BT_RefreshServerName_Click(object sender, EventArgs e)
    {
        CB_ServerName.DataSource = await DbUtils.GetDataSourcesAsync();
    }
}

编辑：

我的新版本代码如下所示。

public class SomeOtherClass
{
    private async void BT_RefreshServerName_Click(object sender, EventArgs e)
        {
            try
            {
                BT_RefreshServerName.Enabled = false;

                //.ConfigureAwait(true) (this is a default) is to attempt to
                //marshal the continuation back to the original context as the
                //result is used to update UI.                    
                CB_ServerName.DataSource = await Task.Run(
                 () => DbUtils.GetDataSources()).ConfigureAwait(true);
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message, "Exception", MessageBoxButtons.OK);
            }
            finally
            {
                BT_RefreshServerName.Enabled = true;
            }
        }
    }
}

public class DbUtils
{

    public static List<string> GetDataSources()
    {
        //Some logic

        List<string> names = new List<string>();
        names = SomeClass.SomeLongSystemMethod();
        return names;
    }
}

Answer 1

GetDataSourcesAsync 的实现被称为“异步优于同步”，并且通常被认为是一种反模式（尽管公平地说，它并不像“同步优于异步”那么糟糕）。至少它会起作用，但如果你实际上没有真正的async 操作，那么使用工作线程（通常是线程池）和Invoke 并没有什么区别/BeginInvoke 取回 UI 线程（进行 UI 更新）。

如果这是问题，它会起作用。

现在，如果GetDataSourcesAsync 实际使用了*Async 数据访问方法； 然后这将是很棒的代码。在那种情况下，你不需要Task.Run，顺便说一句。

另外：为您的事件处理程序添加一些异常处理。