Передача переменной в метод - объясните логику (код внутри)

@Suppir · Регистрация: 08.08.2011

Студворк — интернет-сервис помощи студентам

Добрый день!

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading;
using System.Threading.Tasks;
 
namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            for (int c = 1; c <= 10; c++)
            {
                ThreadPool.QueueUserWorkItem(x =>
                {
                    WriteLog((object)c);
                });
            }
            Thread.Sleep(5000);
        }
 
        static private void WriteLog(object c)
        {
            Console.WriteLine(c);
        }
 
    }
}

Объясните, почему метод WriteLog пишет не числа от 1 до 10, а:

6
10
10
10
10
6
10
10

Как нужно передавать переменные в метод WriteLog, чтобы писались именно числа от 1 до 10 (не важно в каком порядке).

@insite2012 · 13.12.2014, 17:50

Suppir, эффект, конечно, интересный получается... У меня выводит все числа по 11)))))
Я думаю, все дело в самом пуле потоков, ведь поставив метод в очередь пула, это еще не гарантирует, что поток запустится немедленно. Мы лишь просим планировщик потоков запустить метод, когда это станет возможным.
А поскольку все потоки разделяют локальные переменные (в данном случае с), то к моменту запуска потока она становится совершенно другой. Это одна из опасностей применения таких переменных в анонимных методах.
Если вы хотите использовать именно пул (хотя это можно использовать и с другими типами запуска потоков), то тогда просто остановите цикл и не запускайте следующий поток, пока тот что был запущен раньше не выполнится. Примерно таким вот образом.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
using System.Threading;
 
namespace ConsoleApplication69 {
    class Program {
        static AutoResetEvent wh = new AutoResetEvent(false);
 
        static void Main(string[] args) {
            for (int c = 1; c <= 10; c++) {
                ThreadPool.QueueUserWorkItem((x) => {
                    WriteLog(c);
                });
                wh.WaitOne();
            }
            Console.ReadLine();
        }
 
        static private void WriteLog(object c) {
            Console.WriteLine(c);
            wh.Set();
        }
 
    }
}

@Suppir · 13.12.2014, 18:14 **[ТС]**

Да, правильно сказали, что переменная "c" успевает измениться. Я нашел такое решение, оно работает:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading;
using System.Threading.Tasks;
 
namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            for (int c = 1; c <= 10; c++)
            {
                int j = new int();
                j = c; 
                ThreadPool.QueueUserWorkItem(x =>
                {
                    WriteLog((object)j);
                });
            }
            Thread.Sleep(5000);
        }
 
        static private void WriteLog(object j)
        {
            Console.WriteLine(j);
        }
 
    }
}

Psilon · 13.12.2014, 19:07

Suppir, правильно нашли, нужно просто скопировать в локальную переменную:

C#
1
2
3
4
5
6
7
8
9
10
11
12
        static void Main(string[] args)
        {
            for (int c = 1; c <= 10; c++)
            {
                int local = c;
                ThreadPool.QueueUserWorkItem(x =>
                {
                    WriteLog((object)local);
                });
            }
            Thread.Sleep(5000);
        }

Добавлено через 48 секунд
кстати советую поставить решарпер, он такие вещи сразу помогает выявить (и объясняет, кстати, тоже)

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение/ Перевод https:/ / **********/ gallery/ thinkpad-x220-tablet-porn-gzoEAjs . . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .