2

Начал изучать работу с Task в C#, т.к. есть у меня задача, где в цикле делаются достаточно длительные запросы через Web API к сторонним сервисам, а потом результаты заносятся в базу данных. Хотел распараллелить выполнение этих запросов через Tasks, но оказалось не все так просто. Даже простейшая тестовая задача по передаче параметров в Task поставила в тупик.

static void Main(string[] args)
{
    List<Task> tasks = new List<Task>();

    Console.WriteLine("Start");

    for (int i = 0; i < 3; i++)
    {
        tasks.Add(Task.Run(() => printIdx(i)));
    }

    Console.WriteLine("Begin waiting");
    Task.WaitAll(tasks.ToArray());
    Console.WriteLine("End waiting");
}

static void printIdx(int j)
{
    Console.WriteLine(j + "\tstart");
    System.Threading.Thread.Sleep(500);
    Console.WriteLine(j + "\tend");
}

Результат выполнения такой:

Start
Begin waiting
3 start
3 start
3 end
3 start
3 end
3 end
End waiting

Получается, что фактическая передача значения параметра происходит уже после окончания цикла. Подскажите, как правильно передавать параметры в таком случае?

5
  • У вас проблема в замыкании, а не в Task 30 авг 2017 в 14:34
  • Это понятно, что с Task все хорошо, мне не понятно, что я упускаю?
    – Adva-R
    30 авг 2017 в 14:42
  • 2
    Вам уже ответили, нужна переменная, которую извне никто не меняет 30 авг 2017 в 14:45
  • Когда таск фактически стартует, в i уже лежит другое значение 30 авг 2017 в 14:46
  • Если вы работаете с Task'ами, переходите на async/await, и используйте async Main.
    – VladD
    30 авг 2017 в 19:29

1 ответ 1

5

Сделайте копию вашей переменной

for (int i = 0; i < 3; i++)
{
    var t = i;
    tasks.Add(Task.Run(() => printIdx(t)));
}
7
  • Спасибо, помогло! Только непонятно почему нужно делать копию переменной?
    – Adva-R
    30 авг 2017 в 14:39
  • 1
    @Advarus i меняется с каждым циклом, и на момент запуска тасков становится равна 3. t не меняется извне и на момент запуска тасков каждый из тасков будет ссылаться на отдельное локальное значение. Не претендую на академичность)
    – RusArt
    30 авг 2017 в 14:45
  • 1
    @Advarus, Почитайте про замыкания.
    – trydex
    30 авг 2017 в 14:54
  • спасибо за ответы, буду углублять знания по замыканиям :)
    – Adva-R
    30 авг 2017 в 15:02
  • @maxwell, спасибо за ссылку, там как раз описан подобный случай
    – Adva-R
    30 авг 2017 в 15:10

Ваш ответ

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

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