class test
{
public List<Thread> threads = new List<Thread>();
public int nThreads = 0;
public int maxThreads = 5;
public void DoWork(object data)
{
string message = (string)data;
//MessageBox.Show(message);
}
public void CreateThread(object data)
{
if (nThreads >= maxThreads)
return;
Thread newThread = new Thread(DoWork);
threads.Add(newThread);
newThread.IsBackground = true;
newThread.Start(data);
nThreads++;
}
public void WindUpThreads()
{
//MessageBox.Show("count: " + nThreads.ToString());
for(int i = 0; i < threads.Count; i++)
{
if (threads[i].IsAlive == false)
{
threads[i].Abort();
threads.RemoveAt(i);
//MessageBox.Show("removing at " + i.ToString());
}
}
nThreads = threads.Count;
}
}
Мне нужно в методе DoWork
изменять данные в listbox
, но я не хочу привязывать логику к элементам управления. Как лучше поступить в таком случае?
В общем я решил пойти самым простым путем и передал делегат в тред:
public delegate void TestDeleg(string message);
public class Data
{
public TestDeleg deleg;
public string mess;
}
public void DoWork(object data)
{
Data d = (Data)data;
d.deleg(d.mess);
//string message = (string)data;
//MessageBox.Show(d.mess);
return;
}
В цикле теперь передаю класс с запакованным данными:
Data d = new Data();
d.deleg = AddItem;
d.mess = strings[counter];
thTest.CreateThread((object)d);
Ну и сам метод, который добавляет строку в листбокс:
public void AddItem(string message)
{
//listBox1.Items.Add(message);
if(InvokeRequired)
listBox1.Invoke( (Action)( () => { listBox1.Items.Add(message); } ) );
else
listBox1.Items.Add(message);
}
Теперь почему-то цикл никогда не заканчивается и треды не завершаются, метод, который должен очищать треды почему-то не срабатывает.
UPD: Кажется, причина, по которой интерфейс не обновлялся, выяснилась. Основная часть моей программы была такой:
while (flag == true)
{
if (counter >= dataCount)
{
flag = false;
}
while (thTest.nThreads < thTest.maxThreads)
{
if (flag == false)
break;
thTest.CreateThread(strings[counter]);
counter++;
}
thTest.WindUpThreads();
if (flag == false)
{
do
{
thTest.WindUpThreads();
} while (thTest.nThreads != 0);
}
}
И запускался этот код по нажатию на кнопку. Получается, что цикл крутился в основном потоке и поэтому ui не принимал сообщения об изменении. Когда я вынес уже этот код в отдельный поток (получается один поток создает другие и управляет ими), то все стало на свои места.
listBox1.Invoke( (Action)( () => { listBox1.Items.Add(message); } ) );
-- так у вас не те данные могут оказаться в listbox. чтобы такого не было, в listBox1.Invoke надо передать делегат на AddItem и message. посмотрите мой ответ ниже.