0

我正在尝试下载多个文件，但它没有像我希望的那样工作。谁能告诉我这个脚本有什么问题，因为我尝试了很多东西，真的不知道该怎么做。 

public static void DownloadFile(string url)
        {
            WebClient client = new WebClient();
            var name = url.Substring(url.LastIndexOf('/')).Remove(0, 1);
            foreach (var item in urls)
            {
                client.DownloadFile(item, "C:\\" + name);
            }
        }

        private void btnGo_Click(object sender, EventArgs e)
        {
            urls.Add("url1");
            urls.Add("url2");
            urls.Add("url3");
            Parallel.ForEach(urls,
               new ParallelOptions { MaxDegreeOfParallelism = 10 }, 
               DownloadFile);
        }

using (var sr = new StreamReader(HttpWebRequest.Create(url).GetResponse().GetResponseStream()))
            {
                using (var sw = new StreamWriter(url.Substring(url.LastIndexOf('/'))))
                {
                    sw.Write(sr.ReadToEnd());
                }
            }
4

2 回答 2

5

我会用 aSystem.Net.HttpWebRequest代替。

这就是代码的样子：

private List<string> urls = new List<string>();

private void btnGo_Click(object sender, EventArgs e)
{
    urls.Add("http://199.91.152.106/ua0p3fbc5nlg/gg2w2fq4ljc1nnd/MicroCraft_Beta.zip");
    Parallel.ForEach(urls, new ParallelOptions { MaxDegreeOfParallelism = 10 }, DownloadFile);
}

public static void DownloadFile(string url)
{
    var req = (HttpWebRequest)WebRequest.Create(url);
    var name = url.Substring(url.LastIndexOf('/') + 1);
    using (var res = (HttpWebResponse)req.GetResponse())
    using (var resStream = res.GetResponseStream())
    using (var fs = new FileStream("C:\\" + name, FileMode.Create, FileAccess.Write, FileShare.None))
    {
        // Save to file
        var buffer = new byte[8 * 1024]; // 8 KB buffer
        int len; // Read count
        while ((len = resStream.Read(buffer, 0, buffer.Length)) > 0)
            fs.Write(buffer, 0, buffer.Length);
    }
}

因为您在评论中告诉我的 URL 没有使用 HTTP 协议的正确实现。您必须将其添加到您的配置文件中才能使其正常工作（App.config 或 Web.config，取决于它是 ASP.Net 站点还是离线应用程序）：

<system.net>
    <settings>
        <httpWebRequest useUnsafeHeaderParsing="true" />
    </settings>
</system.net>

至于您在评论中所说的名称冲突问题，应该通过将您的名称更改var name = url.Substring(url.LastIndexOf('/')).Remove(0, 1);为其他内容来解决。

如果你想有一个增量文件名，你可以使用这个：

// Inside your class:
private static int counter = 0;

// In your method:
var name = "file" + System.Threading.Interlocked.Increment(ref counter) + ".html";
于 2012-07-11T01:35:55.997 回答
1

您正在将所有文件下载到您的 DownloadFile 代码中的同一文件中，该代码假定对该函数的单次调用会下载所有文件。

修复：

选项 1：不要使用Parallel.ForEach，只需调用 DownloadFile 一次。为每次下载指定唯一的文件名。即通过使用您正在下载的 URL 的一部分或只是使用随机/临时文件名。

像这样的东西（假设网址是某种IEnumerable<string>）

foreach (var item in urls)
{
   var name = item.Substring(item.LastIndexOf('/')).Remove(0, 1);
   client.DownloadFile(item, "C:\\" + name);
}

选项 2：使用Parallel.ForEach但更改 DownloadFile 代码以仅下载单个文件：

public static void DownloadFile(string url)
{
    WebClient client = new WebClient();
    var name = url.Substring(url.LastIndexOf('/')).Remove(0, 1);
    client.DownloadFile(url, "C:\\" + name);
}
于 2012-07-11T01:47:55.613 回答