C#中使用Task.Run并行发起HTTP请求获取计算机名称的实现是否正确？

你的代码功能上是正确的，能完成并行请求并返回填充好的Computer列表，但存在几个可以优化的点，尤其是Task.Run的使用并不恰当，下面详细说明：

原实现的问题 #

1. 不必要的Task.Run包装：GetName本身就是异步方法，异步IO操作（HTTP请求）不需要占用线程池线程，用Task.Run包裹它会额外消耗线程池资源，完全没必要。
2. 同步阻塞风险：Task.WaitAll会阻塞当前调用线程，在UI线程、ASP.NET等有同步上下文的环境中，可能引发死锁问题。
3. 代码冗余：先创建Computer对象再异步填充Name的写法，不如直接通过异步任务生成完整对象来得简洁。

更优的异步实现方案 #

遵循.NET异步编程规范，把方法改为异步非阻塞模式，同时简化逻辑：

public class Computer 
{
    public string IPAddress { get; set; }
    public string Name { get; set; }
}

public class MyClass 
{
    // 静态HttpClient是推荐用法，线程安全且避免套接字资源耗尽
    private static readonly HttpClient _client = new HttpClient();

    // 遵循异步命名规范，添加Async后缀
    public async Task<List<Computer>> GetComputersAsync(List<string> ipAddresses)
    {
        // 为每个IP生成创建Computer的异步任务
        var computerTasks = ipAddresses.Select(async ip => new Computer
        {
            IPAddress = ip,
            Name = await GetNameAsync(ip)
        });

        // 非阻塞等待所有任务完成，再转为列表返回
        return (await Task.WhenAll(computerTasks)).ToList();
    }

    private async Task<string> GetNameAsync(string ipAddress)
    {
        var response = await _client.GetAsync($"http://{ipAddress}/api/name");
        // 建议添加响应检查，确保请求成功
        response.EnsureSuccessStatusCode();
        return await response.Content.ReadAsStringAsync();
    }
}

优化点说明 #

• 移除Task.Run：直接await异步IO方法，避免不必要的线程开销，充分利用异步编程的优势。
• 异步非阻塞：用await Task.WhenAll替代Task.WaitAll，不会阻塞调用线程，避免死锁风险，同时符合现代.NET异步编程模型。
• 简洁的逻辑：通过Select直接生成每个Computer的异步任务，代码更紧凑，逻辑更清晰。
• 响应检查：添加response.EnsureSuccessStatusCode()，在请求失败时抛出异常，便于后续错误处理（你原代码移除了错误处理，实际项目中建议保留）。
• HttpClient规范：静态HttpClient的用法是正确的，但可以根据需求配置超时、默认请求头等参数。

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