2

Целиком задача заключается в загрузке трех картинок для трех PictureBox. По условию задачи пользователь может загружать их в любой момент, поэтому я считаю, что загрузки должны выполняться асинхронно.
Сейчас я выполняю подзадачу по загрузке лишь одной картинки, и поэтому пишу следующий код:

async void DownloadFile(string link, string filename)
    {         
        await wc.DownloadFileTaskAsync(new Uri(link), filename);
    }

    private void PB1Download_Click(object sender, EventArgs e)
    {
        string filename = Directory.GetCurrentDirectory() + "\\image " + downloadsCount.ToString() + ".jpg";
        string link = TB1Link.Text;
        Task.Run(() => DownloadFile(link, filename)).Wait();
        PB1.Image = new Bitmap(filename);
    }

Выполнение этого кода выбрасывает исключение:введите сюда описание изображения

Насколько я понял, это связано с тем, что загрузка Bitmap'а в PictireBox выполняется РАНЬШЕ, чем картинка скачалась, ибо задача асинхронна. Но при этом я думал, что метод Wait поможет дождаться окончания выполнения задачи и все-таки картинка будет успешно загружена в PictureBox. Что я понимаю и делаю не так?

2
  • PictureBox.LoadAsync Commented 22 окт. 2019 в 20:08
  • а если я хочу скачать картинку на комп, а затем поместить ее в пикчербокс? тем самым не загружая два раза ее по ссылке Commented 22 окт. 2019 в 20:15

2 ответа 2

4

Попробуйте вот так

async Task DownloadFile(string link, string filename)
{         
    await wc.DownloadFileTaskAsync(new Uri(link), filename);
}

private async void PB1Download_Click(object sender, EventArgs e)
{
    string filename = Path.Combine(Directory.GetCurrentDirectory(), $"image{downloadsCount}.jpg");
    string link = TB1Link.Text;
    await DownloadFile(link, filename);
    PB1.Image = new Bitmap(filename);
}
5
  • при переписывании метода DownloadFile код не компилируется, т.к. "ожидание void невозможно" Commented 22 окт. 2019 в 20:13
  • @VanyaBelyaev вы невнимательно скопировали мой пример, попробуйте ещё раз
    – tym32167
    Commented 22 окт. 2019 в 20:16
  • проблема решена, благодарю :) а можете объяснить как это работает? Commented 22 окт. 2019 в 20:19
  • ну всё просто, у вас было async void DownloadFile(...) - эта функция ничего не возвращает, то есть вы её заапустить можете, но когда она закончит работу - вы не узнаете. А вот эта функция async Task DownloadFile(...) возвращает таск, и вы уже, зная таск, можете дождаться конца функции вот так await DownloadFile(link, filename);
    – tym32167
    Commented 22 окт. 2019 в 20:22
  • Понял, благодарю! Commented 22 окт. 2019 в 20:23
3

Верный ответ уже дан, но мне есть что сказать.

Самый простой способ загрузки изображения в PictureBox - использовате его собственные возможности. В частности, метод PictureBox.LoadAsync.

Однако, вам нужно ещё и сохранить файл локально. Поэтому этот способ в данном случае не подоходит.


В этом методе применение async/await избыточно:

async Task DownloadFile(string link, string filename)
{         
    await wc.DownloadFileTaskAsync(new Uri(link), filename);
}

Task сам по себе является awaitable объектом, поэтому код можно переписать так:

Task DownloadFile(string link, string filename)
{
    return wc.DownloadFileTaskAsync(new Uri(link), filename);
}

Вызов метода в обоих случаях одинаков:

await DownloadFile("...", "...");

Для нахождения и устранения таких ошибок можно использовать, например, AsyncFixer.


Вы хотите загружать одновременно несколько картинок. Судя по названию и по сигнатуре метода, wc - это WebClient. Но этот класс не является потокобезопасным. Поэтому нужно либо для каждой загрузки создавать своей экземпляр WebClient, либо использовать HttpClient - его экземпляр можно использовать для нескольких загрузок одновременно.

4
  • В этом методе применение async/await избыточно: спорное утверждение. Я тоже так думал, до момента, когда мне понадобился стек вызовов. В данном конкретном случае как написать без разницы, а в общем случае нельзя сказать, что простой возврат таска предпочтительней его асинхронного ожидания. Где то был вопрос на зту тему у нас, кстати, но не могу найти.
    – tym32167
    Commented 23 окт. 2019 в 8:47
  • @tym32167 - стек вызовов - ну, может быть... А так, только (ко-)(контра-)вариантность приходит в голову, зачем может понадобиться лишний await: вот тут последний комментарий Павла. Commented 23 окт. 2019 в 9:10
  • благодарю за ответ! хотелось бы узнать, почему задача разрешилась с помощью вызова "await DownloadFile(..,..)", а именно: почему вызов этого метода через Task.Run() не сработал? Commented 24 окт. 2019 в 9:18
  • @VanyaBelyaev - tym32167 в комментарии объяснил же; на мой взгляд, вполне понятно. Commented 24 окт. 2019 в 9:40

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.