Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Разработка Windows Forms > Зациклить поток правильно


Автор: Jamon 11.10.2008, 16:22
Делаю я 20 потоков, например. И хочу ,чтобы каждый поток, после того выполнился, начал свою работу заново. Сейчас у меня просто сделано внутри метода while(expression), где expression - булевский параметр, который объявлен на уровне класса, соответственно, когда я его выставлю в false, то все потоки на новой итерации останавливаются. 

Код

expression = true;


void ThreadWork()
{
while(expression)
{
   ....
}
}


Основная проблема - то что проц почему-то загружается на 100% при таком подходе, хотя внутри потока выполняются несколько сетевых запросов, сами по себе на проц 20 потоков не должны давать такую нагрузку .?




Автор: Partizan 11.10.2008, 20:25
а вы попробуйте внутрь блока while вставить усыпление потока эдак на полсекунды...
думаю диспетчер сразу перестанет вас мучить тем, что проц грузится )

Автор: jonie 12.10.2008, 12:55
Jamon у вас написан не потокобезопасный код. используйте lock, например. а лучше объекты синхронизации, например эвенты. а то когда-нибудь будете сильно удивляться и искать плавающие баги.

Автор: Dennnis 12.10.2008, 17:15
Цитата(Jamon @  11.10.2008,  16:22 Найти цитируемый пост)
Основная проблема - то что проц почему-то загружается на 100% при таком подходе

Согласен с Partizan smile 
Цитата(jonie @  12.10.2008,  12:55 Найти цитируемый пост)
Jamon у вас написан не потокобезопасный код. используйте lock, например. а лучше объекты синхронизации, например эвенты

В данном случае проще и эффективнее использовать класс System.Threading.Interlocked для присвоения и чтения значения переменной 'expression'.

Автор: Jamon 12.10.2008, 18:50
Поле expression я изменяю в пределах lock(this) {...}  в некотором методе (например по нажатию кнопки) -  вроде это safe-threading подход. 
Плюс была идея сделать это поле volatile, но это никак не может помочь и вопщем то то что я юзаю lock() нивелирует пользу от volatile, насколько я понял.
Плюс на мой вопрос так и не ответилиsmile
Задержка на секунду у меня там и так есть, для того чтобы из каждого потока запросы не сыпались подряд, а с некоторым delay'ем

Автор: PashaPash 13.10.2008, 01:43
Jamon, идея в том, чтобы поток спал в Thread.Sleep вообще все время, пока он не занят полезной работой. А у тебя он занят беганьем по циклу и проверками while(expression) пару миллионов раз в секунду. А lock тут не нужен скорее всего.

Автор: Jamon 13.10.2008, 09:34
PashaPash ok'ей, а как это реализовать?)
у меня написано так:

Код

while(expression)
{
Thread.Sleep(1000);
...
} 


Автор: PashaPash 13.10.2008, 13:38
Jamon, Если код в ... работает 1/20 секунды, то на 20 потоках ты получишь в среднем 100% загрузку проца.

Автор: Jamon 13.10.2008, 22:48
PashaPash спасибо.
будут еще предожения?)

Автор: PashaPash 13.10.2008, 23:57
Jamon, покажи код в ...

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)