使用async、await和Task.Run()时为何出现CS1998编译警告？

核心问题 #

你遇到的CS1998警告，根源在于**Task.Run里的async lambda根本没有用到await**——你在lambda里调用GetStringAsync后直接用.Result同步阻塞了线程，整个lambda是同步执行的，加async完全多余，所以编译器警告你这个async方法没有await，会同步运行。

讲师的写法确实有问题：虽然用Task.Run把阻塞逻辑扔到了后台线程，UI不会冻结，但这不是正确的异步编程姿势，属于"假异步"，而且会浪费线程资源。

正确处理方式 #

有两种修正方向，推荐第二种真正的异步实现：

方向1：移除多余的async关键字 #

既然lambda里没有await，直接去掉async，警告就会消失，代码也能正常运行：

await Task.Run(() =>
{
    Debug.WriteLine($"Thread No. {Thread.CurrentThread.ManagedThreadId} during button 1");
    HttpClient webClient2 = new HttpClient();
    string userAgent2 = @"Mozilla/5.0 (compatible; word-word-name.name@domain.com)";
    webClient2.DefaultRequestHeaders.Add("User-Agent", userAgent2);
    string html = webClient2.GetStringAsync("http://ipv4.download.thinkbroadband.com/20MB.zip").Result;
});

但这种方式还是用了.Result同步阻塞后台线程，只是消除了警告，本质还是同步逻辑套了个Task.Run的壳。

方向2：真正的异步实现（推荐） #

把.Result换成await，这样lambda里有了await，async关键字就合理了，同时实现真正的非阻塞异步，还能消除警告：

await Task.Run(async () =>
{
    Debug.WriteLine($"Thread No. {Thread.CurrentThread.ManagedThreadId} during button 1");
    using HttpClient webClient2 = new HttpClient(); // 用using自动释放资源
    string userAgent2 = @"Mozilla/5.0 (compatible; word-word-name.name@domain.com)";
    webClient2.DefaultRequestHeaders.Add("User-Agent", userAgent2);
    string html = await webClient2.GetStringAsync("http://ipv4.download.thinkbroadband.com/20MB.zip");
});

另外补充两个最佳实践：

• 复用HttpClient：不要每次点击都创建新的HttpClient，建议在类级别声明一个静态实例，避免socket资源耗尽问题。
• 按钮点击事件的async void是允许的（因为是UI事件处理程序），但其他场景尽量用async Task。

完整修改后的MainWindow.xaml.cs示例 #

using System;
using System.Diagnostics;
using System.Net.Http;
using System.Threading;
using System.Threading.Tasks;
using System.Windows;

namespace ThreadTasks_WPF_question
{
    public partial class MainWindow : Window
    {
        // 复用HttpClient，静态实例全局唯一
        private static readonly HttpClient _httpClient = new HttpClient();

        public MainWindow()
        {
            InitializeComponent();
            var timer = new System.Windows.Threading.DispatcherTimer();
            timer.Interval = TimeSpan.FromSeconds(1);
            timer.Tick += Timer_Tick;
            timer.Start();
        }

        private void Timer_Tick(object sender, EventArgs e)
        {
            MyLabel1.Content = DateTime.Now.ToLongTimeString();
        }

        private async void MyButton1_Click(object sender, RoutedEventArgs e)
        {
            MyButton1.Content = "MyButton1 Downloading";
            Debug.WriteLine($"Thread No. {Thread.CurrentThread.ManagedThreadId} before await button 1");
            
            await Task.Run(async () =>
            {
                Debug.WriteLine($"Thread No. {Thread.CurrentThread.ManagedThreadId} during button 1");
                string userAgent = @"Mozilla/5.0 (compatible; word-word-name.name@domain.com)";
                // 避免重复添加User-Agent，先判断是否已存在
                if (!_httpClient.DefaultRequestHeaders.Contains("User-Agent"))
                {
                    _httpClient.DefaultRequestHeaders.Add("User-Agent", userAgent);
                }
                string html = await _httpClient.GetStringAsync("http://ipv4.download.thinkbroadband.com/20MB.zip");
            });

            Debug.WriteLine($"Thread No. {Thread.CurrentThread.ManagedThreadId} after await button 1");
            MyButton1.Content = "MyButton1 Done";
        }

        private async void MyButton2_Click(object sender, RoutedEventArgs e)
        {
            MyButton2.Content = "MyButton2 Downloading";
            Debug.WriteLine($"Thread No. {Thread.CurrentThread.ManagedThreadId} before await button 2");
            
            await Task.Run(async () =>
            {
                Debug.WriteLine($"Thread No. {Thread.CurrentThread.ManagedThreadId} during button 2");
                string userAgent = @"Mozilla/5.0 (compatible; word-word-name.name@domain.com)";
                if (!_httpClient.DefaultRequestHeaders.Contains("User-Agent"))
                {
                    _httpClient.DefaultRequestHeaders.Add("User-Agent", userAgent);
                }
                string html = await _httpClient.GetStringAsync("http://ipv4.download.thinkbroadband.com/20MB.zip");
            });

            Debug.WriteLine($"Thread No. {Thread.CurrentThread.ManagedThreadId} after await button 2");
            MyButton2.Content = "MyButton2 Done";
        }
    }
}

总结 #

• 讲师的写法是"同步逻辑后台执行"，不是真正的异步，async lambda属于多余写法，导致警告。
• 正确的异步做法是用await替代.Result，配合async lambda，既消除警告，又实现真正的非阻塞，还能节省线程资源。
• 养成复用HttpClient的习惯，避免不必要的资源消耗。

内容的提问来源于stack exchange，提问作者dergeophysiker