Доступ к контролу Windows Forms из другого потока (C#)?

Question

[belk]

Пробую писать программу, основной код которой будет выполняться в другом потоке и добавлять данные в ListBox по ходу работы.


Log — объект типа ListBox.


Создание потока:

private void bGo_Click(object sender, EventArgs e)
{
    ...
    Log.Items.Add("Старт");
    Thread t = new Thread(Download);
    t.Start();
    t.Join();
    ...
}

Работа потока:

void Download()
{
    for (int i = 0; i <= 2; i ++)
    {
        for (int j = 0; j <= 1000; j += 100)
        {
            Thread.Sleep(100);
            //реальный код слал запросы в сеть, 
            //т.е. большую часть времени простаивал
            WriteToLog("test");
        }
    }
}

Работа с ListBox'ом:

void WriteToLog(string message)
{
    if (Log.InvokeRequired)
    {
        Log.BeginInvoke(new Action<string>((s)
            => Log.Items.Add(s)), message);
    }
    else
    {
        Log.Items.Add(message);
    }
}

В результате выводится «Старт», затем трёхсекундная пауза и 30 «test», хотя всё это должно выводиться по ходу работы. Если заменить BeginInvoke на Invoke, программа зависает на этой строке на неопределённое время. Как сделать нормальный вывод?

Comments

[VenomBlood]

Уточните используемую версию .net framework.

[belk]

Сначала хотел 2.0 для максимальной совместимости, теперь перешёл на 4.0.

Answer 1

[VenomBlood]

Раз используется .net 4.0 — то вместо потоков лучше использовать задачи:

Task task = new Task(Download);
task.ContinueWith(CodeAfterJoin);
task.Start();

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

Answer 2

[Vitali Borovik]

Толковое описание потоков тут www.rsdn.ru/article/dotnet/CSThreading1.xml

>Если заменить BeginInvoke на Invoke, программа зависает на этой строке на неопределённое время. Как сделать нормальный вывод?

Потому что вы используете t.Join();

Comments

[belk]

Не приходит в голову, как красиво обойтись без t.Join();, не перенося в Download() код после t.Join();.

[VenomBlood]

Рассмотрите использование классов Task и Task вместо Thread. Это более высокоуровневая абстракция для выполнения асинхронных задач, которая, в том числе, поддерживает continuations.

[VenomBlood]

Task и Task<T> уж в 2011 году можно было и сделать умный парсер.

Answer 3

[afsherman]

Как уже сказали выше, с Join() ничего работать не будет, т.к. вы ждёте завершения потока.
А вообще посмотрите в сторону асинхронного программирования и класса BackgroundWorker.
msdn.microsoft.com/en-us/library/ms228969.aspx

Answer 4

[antoo]

stackoverflow.com/questions/7461239/why-does-form-freeze

Comments

[belk]

Добавил секундную задержку на каждой итерации, то же самое. Не может GUI настолько медленно отрисовываться.

Answer 5

[Desiderata]

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

delegate void WriteToLogDelegate(string message);
void WriteToLog(string message)
{
    if (Log.InvokeRequired)
    {
         var writeToLog = new WriteToLogDelegate(WriteToLog);
         Log.Invoke(writeToLog, message);
    }
    else
    {
        Log.Items.Add(message);
    }
}

Comments

[Desiderata]

Извиняюсь, не увидел t.Join();

Answer 6

[Vitali Borovik]

private delegate void UpdateListBox(string test);
static object locker = new object();

private void UpLB(string test)
{
lock (locker)
{
Log.Items.Add(test);
}
}

//После чего вызываете
this.Invoke(new UpdateListBox(this.UpLB),new object[] { m.Message});

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

Comments

[belk]

Попробовал

delegate void DWriteToLog(string message);
static object locker = new object();

private void bGo_Click(object sender, EventArgs e)
{
	... //работа с формой
	Log.Items.Add("Загрузка id");
	Thread t = new Thread(Download);
	t.Start();
	t.Join();
	... //работа с формой
}

void Download()
{
	for (int i = 0; i <= 2; i++)
	{
		for (int j = 0; j <= 1000; j += 100)
		{
			... //запрос в сеть
			WriteToLog("test");
		}
	}
}

void WriteToLog(string message)
{
	this.Invoke(new DWriteToLog(UpLB), new object[] { message });
}

void UpLB(string message)
{
	lock (locker)
	{
		Log.Items.Add(message);
	}
}

, всё абсолютно так же (фриз).

[VenomBlood]

Пока вы используете Join() в потоке UI — ничего работать не будет, т.к. вы ждете завершения работы потока добавления элементов, и на это время поток UI заблокирован.

[VenomBlood]

Я даже напишу больше — блокирование потока UI — это очень-очень плохая идея.