我的多线程 HttpClient 有什么问题吗？

Question

所以在涉足 Java 和 HttpClient 之后，我决定过渡到 C# 以尝试在提高速度的同时降低内存使用量。我一直在阅读这里的成员提供的大量关于异步与多线程的文章。看来多线程将是更好的方向。

我的程序将访问服务器并一遍又一遍地发送相同的请求，直到检索到 200 代码。原因是因为在高峰时段交通非常繁忙。它使服务器很难到达并引发 5xx 错误。代码非常简单，我还不想添加循环以帮助查看者简化。

我觉得我正朝着正确的方向前进，但我想与社区联系以改掉任何不良习惯。再次感谢您的阅读。

版主注意：我要求检查我的代码是否存在差异，我很清楚多线程 HttpClient 是可以的。

using System;
using System.Net.Http;
using System.Collections.Generic;
using System.Net.Http.Headers;
using System.Threading;

namespace MF
{
    class MainClass
    {
        public static HttpClient client = new HttpClient();
        static string url = "http://www.website.com";

        public static void getSession() {
            StringContent queryString = new StringContent("{json:here}");

            // Send a request asynchronously continue when complete
            var result = client.PostAsync(new Uri(url), queryString).Result;

            // Check for success or throw exception
            string resultContent = result.Content.ReadAsStringAsync().Result;

            Console.WriteLine(resultContent);
        }
        public static void Run() 
        {
            getSession ();
          // Not yet implemented yet..
          //doMore ();
        }

        public static void Main (string[] args)
        {
            Console.WriteLine ("Welcome!");

            // Create the threads
            ThreadStart threadref1 = new ThreadStart(Run);
            ThreadStart threadref2 = new ThreadStart(Run);
            Console.WriteLine("In the main: Creating the threads...");
            Thread Thread1 = new Thread(threadref1);
            Thread Thread2 = new Thread(threadref1);

            // Start the thread
            Thread1.Start();
            Thread2.Start();
        }
    }
}

另外，我不确定这是否重要，但我正在我的 MacBook Pro 上运行它，并计划在我的 BeagleBoard 上运行它。

Answer 1

如果我是你，我会这样做。我会尽可能多地利用异步，因为它比使用线程更有效（您很可能不必一直进行上下文切换，这很昂贵）。

class MainClass
{
    public static HttpClient client = new HttpClient();
    static string url = "http://www.website.com";

    public static async Task getSessionAsync()
    {
        StringContent queryString = new StringContent("{json:here}");

        // Send a request asynchronously continue when complete
        using (HttpResponseMessage result = await client.PostAsync(url, queryString))
        {
            // Check for success or throw exception
            string resultContent = await result.Content.ReadAsStringAsync();
            Console.WriteLine(resultContent);
        }
    }

    public static async Task RunAsync()
    {
        await getSessionAsync();
        // Not yet implemented yet..
        //doMore ();
    }

    public static void Main(string[] args)
    {
        Console.WriteLine("Welcome!");

        const int parallelRequests = 5;
        // Send the request X times in parallel
        Task.WhenAll(Enumerable.Range(1, parallelRequests).Select(i => RunAsync())).GetAwaiter().GetResult();

        // It would be better to do Task.WhenAny() in a while loop until one of the task succeeds
        // We could add cancellation of other tasks once we get a successful response
    }
}

请注意，我确实同意@Damien_The_Unbeliever：如果服务器在重负载下出现问题，您不应该添加不必要的负载（执行 X 次相同的请求）并导致服务器问题。理想情况下，您会修复服务器代码，但我可以理解它不是您的。

Answer 2

以下方法是线程安全的：

CancelPendingRequests
DeleteAsync
GetAsync
GetByteArrayAsync
GetStreamAsync
GetStringAsync
PostAsync
PutAsync
SendAsync

更多细节：

• Is HttpClient safe to use concurrently?
• http://blogs.msdn.com/b/henrikn/archive/2012/08/07/httpclient-httpclienthandler-and-httpwebrequesthandler.aspx

Answer 3

弄乱标题不是线程安全的。

例如，将 OAuth 访问令牌换成新的不是线程安全的。现在使用 Xamarin 移动应用程序面对这个问题。我有一个崩溃报告，其中一个线程正在修改标头，而另一个线程正在尝试请求。

Answer 4

阅读HttpClient的文档：

此类型的任何公共静态（在 Visual Basic 中为共享）成员都是线程安全的。不保证任何实例成员都是线程安全的。

不要冒险。有一个单独的 HTTP 客户端 per 线程。

您似乎正在阻塞等待回复的线程，并且您正在从一个不做任何额外操作的线程执行此操作。那为什么还要打扰 async/await 呢？您可以使用简单的阻塞调用。

另外 - 您的程序现在在启动线程后立即结束。在从 main 返回之前，您可能希望等待线程完成。您可以在程序结束时通过以下代码执行此操作：

Thread1.Join();
Thread2.Join();

基于 cmets 的更新：

1. 决定您要发出多少并行请求 - 这将是您的线程数。
2. 使用 ManualResetEvent 使主线程等待信号。
3. 在每个线程中不断提交您的请求。一旦您得到等待的答案 - 发出 ManualResetEvent 信号并允许您的主函数返回。