1

Мне нужно сделать так, чтобы в цикле 5 методов параллельно работали(они качают картинки)

Если я делаю цикл без Task/Thread все прекрасно работает, но не выполняет мою задачу. Если же я включаю в код Task/Thread, то цикл неведомым образом, сразу скачивает 5 элемент. P.S без тасков все четко работает. Вопрос, почему происходить данная ситуация?

Task:
for(int countfilms = 0; countfilms < 5; countfilms++)
{
      await Task.Factory.StartNew(async () =>
      {
          string namefilm = Regex.Replace(document.QuerySelectorAll("div.sect-cont.sect-items.clearfix div.th-title")[countfilms].TextContent, @"\s+", " ");
          string link = await Bing.CheckAndSend(namefilm, new ImageElement());
          TakePicture(DownloadPicture(in link, in countfilms));
      });
}
Thread:
for(int countfilms = 0; countfilms < 5; countfilms++) 
{
     new Thread(async() =>
     {
        string namefilm = Regex.Replace(document.QuerySelectorAll("div.sect-cont.sect-items.clearfix div.th-title")[countfilms].TextContent, @"\s+", " ");
        string link = await Bing.CheckAndSend(namefilm, new ImageElement());
        TakePicture(DownloadPicture(in link, in countfilms));
     }).Start();
}

TakePicture- Ивент(Делегат)

7
  • 3
    Task != Thread. Зачем вы вообще тут создаете потоки? Task: - я надеюсь это у вас не в коде? Regex.Replace - уверены? Ну а ответ, скорей всего этот.
    – EvgeniyZ
    21 июн 2021 в 19:42
  • Нет таск не в когде. Потоки нужны для того, чтобы если попалась большая картинка, она не задерживала маленькие, пока она скачиваться будет
    – Houl
    21 июн 2021 в 19:44
  • 1
    Да я знаю что таском. Мне подсказали, раз с таском не работает, попробуй треад. Ваш ответ помог, все работает, спасибо
    – Houl
    21 июн 2021 в 19:49
  • 3
    Странный у вас там советчик... Если не работает одно, значит надо это бросать и переходить сразу на другое? Такое себе решение... Не работает - значит делаете что-то не так. Потоки вас только замедлят, вот и вся суть)
    – EvgeniyZ
    21 июн 2021 в 19:53
  • 1
    @aepot вам тоже спасибо, я таск фактори писал, т.к читал старую, либо плохую статью
    – Houl
    21 июн 2021 в 21:33

1 ответ 1

0

Не надо никаких Task.Factory...и старайтесь не выполнять одну и ту же операцию в цикле, когда ее выполнить можно 1 раз.

Нужно например выполнить много запросов, но чтобы одновременно были активны только 5.

var nodes = document.QuerySelectorAll("div.sect-cont.sect-items.clearfix div.th-title");
using var semaphore = new SemaphoreSlim(5);
List<Task> tasks = new List<Task>();
for (int i = 0; i < nodes.Count; i++)
{
    await semaphore.WaitAsync();
    int id = i; // это исправляет проблему захвата счетчика цикла, чтобы не было 5 5 5 5 5
    tasks.Add(((Func<Task>)(async () =>
    {
        try
        {
            string namefilm = Regex.Replace(nodes[id].TextContent, @"\s+", " ");
            string link = await Bing.CheckAndSend(namefilm, new ImageElement());
            TakePicture(DownloadPicture(in link, in id)); // здесь сомнение, почему эти методы синхронные? Они очень быстро отрабатывают?
        }
        finally
        {
            semaphore.Release();
        }
    }))());
}
await Task.WhenAll(tasks);

Асинхронное программирование. При использовании асинхронного программирования про существование Thread лучше забыть.

10
  • Кхм, вопрос не про то как правильно ведь, а почему стартует с 5-го элемента, а это, дубликат (ссылка в первом комментарии)... Также человек хочет не "5 первых", а "по 5", судя по коду там запросы в поисковик идут, а он может сотню запросов заблокировать, нужен семафор (что вроде тоже было, даже у Вас и можно было бы закрыть дублем, если думать в этом ключе).
    – EvgeniyZ
    21 июн 2021 в 20:27
  • @EvgeniyZ спасибо, дописал.
    – aepot
    21 июн 2021 в 20:38
  • @EvgeniyZ я с телефона, завтра дубли поищу, если автор ответ не примет.
    – aepot
    21 июн 2021 в 20:48
  • На конкретно поставленный вопрос "почему с 5-го" ответ здесь, автор сказал "спасибо, ответ помог".
    – EvgeniyZ
    21 июн 2021 в 20:50
  • @EvgeniyZ я видел, да. То что у него где-то заработало, это не значит что он написал тоже самое, что и я.
    – aepot
    21 июн 2021 в 20:53

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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