Модераторы: gambit, Partizan
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Зациклить поток правильно 
:(
    Опции темы
Jamon
Дата 11.10.2008, 16:22 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 92
Регистрация: 1.9.2007

Репутация: нет
Всего: нет



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

Код

expression = true;


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


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




PM MAIL   Вверх
Partizan
Дата 11.10.2008, 20:25 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Let's do some .NET
****


Профиль
Группа: Модератор
Сообщений: 2828
Регистрация: 19.12.2005
Где: Санкт-Петербург

Репутация: 9
Всего: 67



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


--------------------
СУВ,
       Partizan.
PM MAIL WWW ICQ Skype GTalk Jabber   Вверх
jonie
Дата 12.10.2008, 12:55 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Завсегдатай
Сообщений: 5613
Регистрация: 21.8.2005
Где: Владимир

Репутация: 5
Всего: 118



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


--------------------
Что-то не поняли? -> Напейтесь до зеленых человечков... эта сверхцивилизация Вам поможет...
PM MAIL Jabber   Вверх
Dennnis
Дата 12.10.2008, 17:15 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 145
Регистрация: 23.3.2008

Репутация: 2
Всего: 4



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

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

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

Это сообщение отредактировал(а) Dennnis - 12.10.2008, 17:16
--------------------
Get Rich or Die Tryin'
PM   Вверх
Jamon
Дата 12.10.2008, 18:50 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 92
Регистрация: 1.9.2007

Репутация: нет
Всего: нет



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

Это сообщение отредактировал(а) Jamon - 12.10.2008, 18:52
PM MAIL   Вверх
PashaPash
Дата 13.10.2008, 01:43 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1233
Регистрация: 3.1.2008

Репутация: нет
Всего: 49



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


--------------------
PM MAIL WWW   Вверх
Jamon
Дата 13.10.2008, 09:34 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 92
Регистрация: 1.9.2007

Репутация: нет
Всего: нет



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

Код

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


PM MAIL   Вверх
PashaPash
Дата 13.10.2008, 13:38 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1233
Регистрация: 3.1.2008

Репутация: нет
Всего: 49



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


--------------------
PM MAIL WWW   Вверх
Jamon
Дата 13.10.2008, 22:48 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 92
Регистрация: 1.9.2007

Репутация: нет
Всего: нет



PashaPash спасибо.
будут еще предожения?)
PM MAIL   Вверх
PashaPash
Дата 13.10.2008, 23:57 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1233
Регистрация: 3.1.2008

Репутация: нет
Всего: 49



Jamon, покажи код в ...


--------------------
PM MAIL WWW   Вверх
  
Ответ в темуСоздание новой темы Создание опроса
Прежде чем создать тему, посмотрите сюда:
mr.DUDA
THandle

Используйте теги [code=csharp][/code] для подсветки кода. Используйтe чекбокс "транслит" если у Вас нет русских шрифтов.
Что делать если Вам помогли, но отблагодарить помощника плюсом в репутацию Вы не можете(не хватает сообщений)? Пишите сюда, или отправляйте репорт. Поставим :)
Так же не забывайте отмечать свой вопрос решенным, если он таковым является :)


Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, mr.DUDA, THandle.

 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Разработка Windows Forms | Следующая тема »


 




[ Время генерации скрипта: 0.1217 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.