У меня есть простое приложение со следующим кодом:
FileInfo[] files = (new DirectoryInfo(initialDirectory)).GetFiles();
List<Thread> threads = new List<Thread>(files.Length);
foreach (FileInfo f in files)
{
Thread t = new Thread(delegate()
{
Console.WriteLine(f.FullName);
});
threads.Add(t);
}
foreach (Thread t in threads)
t.Start();
Скажем, в директории 'I = initialDirectory' у меня есть 3 файла. Затем это приложение должно создавать 3 потока, причем каждый поток печатает одно из имен файлов; однако вместо этого каждый поток будет печатать имя последнего файла в массиве "файлы".
Почему это? Почему текущая переменная 'f' не получает правильную настройку в анонимном методе?
Анонимный метод сохраняет ссылку для переменной в закрывающем блоке - не фактическое значение переменной.
К тому времени, когда методы фактически выполняются (при запуске потоков) f было назначено указать на последнее значение в коллекции, поэтому все 3 потока печатают это последнее значение.
Вот некоторые интересные статьи об анонимных методах в С# и код, который будет сгенерирован компилятором:
http://blogs.msdn.com/oldnewthing/archive/2006/08/02/686456.aspx
http://blogs.msdn.com/oldnewthing/archive/2006/08/03/687529.aspx
http://blogs.msdn.com/oldnewthing/archive/2006/08/04/688527.aspx
Я думаю, если бы вы сделали:
foreach (FileInfo f in files)
{
FileInfo f2 = f; //variable declared inside the loop
Thread t = new Thread(delegate()
{
Console.WriteLine(f2.FullName);
});
threads.Add(t);
}
это будет работать так, как вы этого хотели.
Это потому, что f.FullName является ссылкой на переменную, а не на значение (как вы пытались ее использовать). К тому моменту, когда вы на самом деле запускаете потоки, f.FullName увеличивалось до конца массива.
Во всяком случае, зачем повторять это здесь дважды?
foreach (FileInfo f in files)
{
Thread t = new Thread(delegate()
{
Console.WriteLine(f.FullName);
});
threads.Add(t);
t.Start();
}
Однако это по-прежнему не так, и, возможно, даже хуже, поскольку у вас теперь есть условие гонки, чтобы увидеть, какой поток идет быстрее: запись элемента консоли или итерация в следующую FileInfo.
Это потому, что базовый код для итератора (foreach) уже "переименован" во все значения в списке до начала потоков... Поэтому, когда они начинаются, значение, "указанное" итератором, является последним в списке...
Начните поток внутри итерации вместо этого.
foreach (FileInfo f in files)
{
string filName = f.FullName;
Thread t = new Thread(delegate()
{ Console.WriteLine(filName); });
t.Start();
}
Я не считаю, что гонка возможна здесь, так как нет доступной общей памяти из всех потоков.
Следующее будет работать.
Thread t = new Thread(delegate()
{
string name = f.Name;
Console.WriteLine(name);
});