6

C# AsyncでWEB画像を複数枚ダウンロードする時の非同期の書き方ってこちらで合っていますか?
for文がどうも変だと思うのですが。

private async void DownloadImage(string []uri, string path)
{
    HttpClient httpClient = new HttpClient();

    for (int i = 0; i < uri.Length; i++)
    {
        HttpResponseMessage res = await httpClient.GetAsync(uri[i], HttpCompletionOption.ResponseHeadersRead);

        using (var fileStream = File.Create(path+(i+1)+".jpg"))
        {
            using (var httpStream = await res.Content.ReadAsStreamAsync())
            {
                httpStream.CopyTo(fileStream);
                fileStream.Flush();
            }
        }
    }
 }

1 件の回答 1

8
+50

awaitの時点でその操作の完了を待ってしまいます。
並列にダウンロードしたい場合は、1つ1つのダウンロードを行うTaskを用意しそれをTask.WhenAll()で待つ必要があります。

static async Task DownloadImage(HttpClient httpClient, string uri, string path) {
    using (var res = await httpClient.GetAsync(uri, HttpCompletionOption.ResponseHeadersRead))
    using (var fileStream = File.Create(path))
    using (var httpStream = await res.Content.ReadAsStreamAsync())
        await httpStream.CopyToAsync(fileStream);
}

static async Task DownloadImage(string[] uris, string path) {
    using (var httpClient = new HttpClient())
        await Task.WhenAll(uris.Select((uri, i) => DownloadImage(httpClient, uri, string.Format("{0}{1}.jpg", path, i + 1))));
}

こまごまと指摘があります。

  • 全てのダウンロードの完了を知るためにも戻り値はTaskの方がいいでしょう。
  • ReadAsStreamAsync()は読み取り開始でしかなくダウンロード全体に関してはCopyTo()部分もCopyToAsync()にすべきです。
  • IDisposableに関してはusingなどで全てDispose()しておくべきです。
  • FileStreamusingしてあればFluash()不要です。
  • usingをネストする際、括弧を書かなければインデントが深くなりません。
1
  • usingのこのような使い方が存じませんでした。 Taskは勉強不足でした。大変助かりました。 Commented 2015年8月3日 6:51

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。