2
HttpWebRequest req = (HttpWebRequest)HttpWebRequest.Create(baseurl + url);
req.Timeout = 1000 * 10;
HttpWebResponse response = (HttpWebResponse)req.GetResponse();
Stream str = response.GetResponseStream();
HtmlAgilityPack.HtmlDocument doc = new HtmlAgilityPack.HtmlDocument();
doc.Load(str);
response.Close();
string imgurl = doc.DocumentNode.SelectSingleNode("//div[@class='one-page']/a/img[@class='manga-page']").Attributes["src"].Value;
req = (HttpWebRequest)HttpWebRequest.Create(imgurl);
req.Timeout = 1000 * 10;
response = (HttpWebResponse)req.GetResponse();
str = response.GetResponseStream();
Image img = Image.FromStream(str);
response.Close();
return img;

我在一个循环中运行这段代码(使用多个线程)以下载大约 4000 张图像,它在前数百张图片中运行良好,但随后(在每次我尝试的不同时间点)它突然停止工作,并且每次调用“req.GetResponse()”会导致 TimeoutException。我不知道为什么会发生这种情况,也不知道可能出了什么问题或如何处理它。任何帮助将不胜感激。

我用来运行此函数的代码(称为 GetPage(int) 并称为 c.GetPage(t))如下:

for (int j = 0; j < 2; j++)
{
    BackgroundWorker bw = new BackgroundWorker();
    num[bw] = j;
    bgs.Add(bw);
    bw.DoWork += (object sender, DoWorkEventArgs doargs) =>
    {
        int t = -1;
        lock (lockObjForQueueOperations)
        {
            if (images.Count != 0)
                t = images.Dequeue();
        }
        if(t < 0)
        {
            doargs.Result = false;
            return;
        }
        currently[sender] = t;
        Image img;
        try { img = c.GetPage(t); }
        catch (Exception e)
        {
            lock (lockObjForQueueOperations)
            {
                images.Enqueue(t);
            }
            lock (Console.Title)
            {
                if (num[sender] == 0) Console.ForegroundColor = ConsoleColor.Cyan;
                else if (num[sender] == 1) Console.ForegroundColor = ConsoleColor.Yellow;
                Console.WriteLine("**ERR: Error fetshing page {0}, errormsg: {1}", t, e.Message);
                Console.ForegroundColor = ConsoleColor.White;
            }
            doargs.Result = true;
            Thread.Sleep(1000*2);
            return;
        }
        lock (Console.Title)
        {
            if (num[sender] == 0) Console.ForegroundColor = ConsoleColor.Cyan;
            else if (num[sender] == 1) Console.ForegroundColor = ConsoleColor.Yellow;
            Console.WriteLine("\t\tLoaded page {0} of {1}.", t + 1, c.PagesCount);
            Console.ForegroundColor = ConsoleColor.White;
        }
        string imgpath = Path.Combine(ndir, "Page " + (t + 1) + ".png");
        img.Save(imgpath, System.Drawing.Imaging.ImageFormat.Png);
        img.Dispose();
        doargs.Result = true;
    };
    bw.RunWorkerCompleted += (object sender, RunWorkerCompletedEventArgs runargs) =>
    {
        if ((bool)runargs.Result) bw.RunWorkerAsync();
        else
        {
            finnishedworkers++;
            if (finnishedworkers == 2) restetter.Set();
            bw.Dispose();
        }
    };
    bw.RunWorkerAsync();
}
4

2 回答 2

2

你有一个糟糕的设计。不要为每个请求创建线程,而是尝试调用 BeginGetResponse。该框架将处理从线程池分配线程以服务您的请求。

添加对 ServicePointManager.SetDefaultConnectionLimit (?) 不确定的调用到 100 之类的数字。

创建一个计数与连接限制计数匹配的信号量。

在调用 BeginGetResponse 的函数中,在调用之前添加对 semaphore.WaitOne() 的调用 get BeginGet...

在您的 EndGetResponse() 处理程序中,调用 semaphore.Release() 以允许下一个请求继续。

您可能正在用自己的所有线程耗尽线程池。监控你的进程,看看你是否不能执行并且总共只使用 5-10 个线程。也许您可以记录 Thread.Current.ThreadID 以查看 SAME 线程如何处理多个请求。

做这个几十亿的时间。真的。

于 2010-02-18T23:56:30.227 回答
2

HttpWebRequest 中的 Timeout 属性以毫秒为单位。目前将其设置为 10,000 只需 10 秒,根据带宽和要提取的数据的大小以及正在运行的代码的复杂性,这可能还不够。我说先尝试增加它。

于 2010-02-18T22:25:08.383 回答