Скажите правильно ли я считаю

@SoltX · Регистрация: 13.03.2016

Author24 — интернет-сервис помощи студентам

Нужно найти ошибку в коде и дать объяснение преподавателю, скажите правильно ли я считаю:

В этом примере программы нет критических ресурсов. Можно не использовать объект ядра Mutex, т.к. каждый поток использует свои ресурсы. Так же есть ошибке в предоставленном коде в параметрах вызова функций CreateMutex(NULL, FALSE, FALSE) в третьем параметре нужно указать NULL и после использований функций создания объектов ядра их описатели необходимо закрывать функциями CloseHandle(hMutex) для мьютекса и CloseHandle(hT) для потоков.

C++

HANDLE hMutex[] = {CreateMutex(NULL, FALSE, FALSE), CreateMutex(NULL, FALSE, FALSE)};
 
DWORD __stdcall Process(LPVOID)
{
  int i=10; int j=20;
 
  WaitForMultipleObjects(2, hMutex, FALSE, INFINITE);
  int t = i+j;
  ReleaseMutex(hMutex);
  
  return 0;
}
 
void main()
{
  HANDLE hT[2];
  for (int t=0; t<2; t++)
     hT[t] = CreateThread(NULL, 0, Process, NULL, 0, NULL);
  return; 
}

@rao · 13.03.2016, 22:29

Еще подозрительно выглядит использование функции WaitForMultipleObjects. Она ждет получения одного из двух мьютексов, а дождавшись всегда освобождает первый элемент массива ReleaseMutex(hMutex); Надо проверять, что вернула WaitForMultipleObjects:

C++

DWORD dwEvent = WaitForMultipleObjects(...);
switch (dwEvent) 
{
    case WAIT_OBJECT_0 + 0: 
    ReleaseMutex(hMutex[0]);
    break; 
 
        case WAIT_OBJECT_0 + 1: 
    ReleaseMutex(hMutex[1]);
    break; 
...

@SoltX · 13.03.2016, 23:34 **[ТС]**

Моя версия кода программы:

C++

DWORD __stdcall Process(LPVOID)
{
  int i=10; int j=20;
  int t = i+j;
  
  return 0;
}
 
void main()
{
  HANDLE hT[2];
  for (int t=0; t<2; t++)
     hT[t] = CreateThread(NULL, 0, Process, NULL, 0, NULL);
 
  for (int z=0; z<2; z++) CloseHandle(hT[z]);
  return; 
}

	13.03.2016, 23:34