Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > C/C++: Системное программирование и WinAPI > проблема с критическими секциями


Автор: Yra 7.2.2008, 16:54
Добрый день. 
Написал "прожку", которая работает коряво. 
Эта программа создаёт 2 потока. Оба потока используют одну функцию. В фунцции критическая секция.
На форме у меня одна кнопка, два окна редактирования.

по нажатии на конпку, создаётся 2 потока. дальше, функция (обработчик нажатия на кнопку) ждёт завершения
2х потоков и удаляет критическую секцию.
вот код проги:
Код

//---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop

#include "Unit1.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;

CRITICAL_SECTION cs;

int x;

DWORD WINAPI ThreadFunc(PVOID pvParam)
{
 DWORD dwRes = 0;

 EnterCriticalSection(&cs);
 x += 10;
 ((TEdit *)pvParam)->Text = IntToStr(x);
 LeaveCriticalSection(&cs);

    Sleep(0);

  return(dwRes);
}

//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
 InitializeCriticalSection(&cs);
}
//---------------------------------------------------------------------------

void __fastcall TForm1::Button1Click(TObject *Sender)
{
 DWORD ID1, ID2;

 HANDLE hT[2];

 hT[0] = CreateThread(NULL, 0, ThreadFunc, (PVOID)Edit1, 0, &ID1);
 hT[1] = CreateThread(NULL, 0, ThreadFunc, (PVOID)Edit2, 0, &ID2);

 WaitForMultipleObjects(2, hT, TRUE, 5000);

 DeleteCriticalSection(&cs);
 ShowMessage("xxx");
}
//--------------------------------------------------------------------------- 



в итоге, 1й созданный поток отрабатывает свой код. а 2й поток не приступает к работе.
если вызывать функцию ожидания так - WaitForMultipleObjects(2, hT, TRUE, INFINIT); программа просто виснит  

P.S. если код функции потока переписать так : 
Код

DWORD WINAPI ThreadFunc(PVOID pvParam)
{
 DWORD dwRes = 0;

 EnterCriticalSection(&cs);
 x += 10; 
 LeaveCriticalSection(&cs);
 
 ((TEdit *)pvParam)->Text = IntToStr(x);

 Sleep(0);

 return(dwRes);
} 


то программа работает как нодо.......

вопрос: почему? 

Автор: Lazin 7.2.2008, 17:02
программа не работает нормально из за этой строчки:
Цитата(Yra @  7.2.2008,  16:54 Найти цитируемый пост)
((TEdit *)pvParam)->Text = IntToStr(x);

то-что программа стала работать "как надо" - это просто везение, на самом деле она не работает, перестанет работать когда измениться фаза луны))
смотри документацию по Synchronize

Автор: Yra 7.2.2008, 17:15
Цитата(Lazin @  7.2.2008,  17:02 Найти цитируемый пост)
программа не работает нормально из за этой строчки:Цитата(Yra @  7.2.2008,  16:54 )((TEdit *)pvParam)->Text = IntToStr(x);


это я и так понял, что из-за этой строки.
поэтому я и спрашиваю: "почему?"

тип данных void*  можно преобразовывать к любому типу данных.....(кажеться smile )
поэтому я тем более не понимаю в чём беда. 

Автор: Lazin 7.2.2008, 17:43

компоненты VCL можно использовать только в главном потоке, потому-что они используют очередь сообщений, тоесть неявно происходит обращение к ним из разных потоков. использовать их можно только через ф-ю Synchronize.

Автор: Yra 7.2.2008, 17:55
теперь придельно ясно! спасибо!  smile 

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