winapi, синхронизация потоков

[veniside]

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

Скорей всего, все ваши WaitForSingleObject() возвращают WAIT_ABANDONED:

The specified object is a mutex object that was not released by the thread that owned the mutex object before the owning thread terminated. Ownership of the mutex object is granted to the calling thread and the mutex state is set to nonsignaled.

И ещё, вместо создания 300 потоков, разумнее было бы перенести цикл внутрь потоков.

[veniside]

- -

[LockHeart]

Пытаюсь решить, на первый взгляд, несложную задачу... нужно создать 3 потока, каждый из которых записывает свое число в файл (1, 2, 3), и при помощи мьютексов синхронизировать их.

В WinMain создаю три мьютекса, затем 3 потока (потоки завернуты в цикл For, просто для того, чтобы в файл записывалось несколько итераций)

hMutex1 = CreateMutex(NULL, FALSE, NULL);
    hMutex2 = CreateMutex(NULL, TRUE, NULL);
	hMutex3 = CreateMutex(NULL, TRUE, NULL);

for (i=0; i<100; i++)
{
 aThread[0] = CreateThread( 
					 NULL,       
					 0,         
					 (LPTHREAD_START_ROUTINE) func1,
					 NULL,      
					 0,       
					 &ThreadID);

aThread[1] = CreateThread( 
					 NULL,       
					 0,          
					 (LPTHREAD_START_ROUTINE) func2, 
					 NULL,       
					 0,         
					 &ThreadID); 

aThread[2] = CreateThread( 
					 NULL,       
					 0,          
					 (LPTHREAD_START_ROUTINE) func3, 
					 NULL,       
					 0,         
					 &ThreadID); 


		WaitForMultipleObjects(3, aThread, TRUE, INFINITE);

}
Сами функции для потоков имеют следующий вид:

DWORD WINAPI func1()
{
WaitForSingleObject(hMutex1, INFINITE);
	OutFile << a;
	ReleaseMutex(hMutex2);
return 0;
}

DWORD WINAPI func2()
{
WaitForSingleObject(hMutex2, INFINITE);
	OutFile << b;
	ReleaseMutex(hMutex3);
return 0;
}

DWORD WINAPI func3()
{
WaitForSingleObject(hMutex3, INFINITE);
	OutFile << c;
	ReleaseMutex(hMutex1);
return 0;
}

По какой-то, пока не понятной мне причине, записанные в файл данные имеют вид не 123123123, а представляют собой совершенно случайную последовательность из этих цифр(213213231213..). Подскажите, пожалуйста, что я делаю не так, почему потоки не синхронизируются?)

[LockHeart]

Сообщение от veniside

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

И ещё, вместо создания 300 потоков, разумнее было бы перенести цикл внутрь потоков.

Спасибо) Попробовал сделать все то же самое, но через CreateEvent, теперь работает)

Цикл внутрь потоков перенести, наверное, не получится... Если рамки задачи немного расширить, то там нужно, чтобы данные выводились еще и на экран, причем после каждой итерации обновлялись. Поэтому в винмейне пришлось сделать такой цикл, а после WaitForMultipleObjects юзать UpdateWindow...