WIN32 线程程序问题

Question

这是我第一次处理线程。

当我在没有GetCurrentThreadId() 函数的情况下运行程序时，它的执行没有任何问题。

当我添加那行代码时，它仍然会执行，但一旦到达末尾就会崩溃。这是为什么呢？

#include <Windows.h>
#include <stdio.h>
#include <conio.h>


static int tix[500];
static int done = 0;
HANDLE ghSemaphore;

DWORD WINAPI ThreadFunction();

int main(void)
{
    DWORD threadID1, threadID2, threadID3, threadID4;
    HANDLE hThread1, hThread2, hThread3, hThread4;

    for (int i = 0; i < 500; i++) //initialize array
    {
        tix[i] = 0;
    }

    ghSemaphore = CreateSemaphore(NULL, 1, 10, NULL);

    hThread1 = CreateThread(NULL, 0, ThreadFunction, NULL, 0, &threadID1);
    hThread2 = CreateThread(NULL, 0, ThreadFunction, NULL, 0, &threadID2);
    hThread3 = CreateThread(NULL, 0, ThreadFunction, NULL, 0, &threadID3);
    hThread4 = CreateThread(NULL, 0, ThreadFunction, NULL, 0, &threadID4);

    //printf("The thread ID: %d.\n", threadID1);
    //printf("The thread ID: %d.\n", threadID2);
    //printf("The thread ID: %d.\n", threadID3);
    //printf("The thread ID: %d.\n", threadID4);

    if (done = 1)
    {
        CloseHandle(hThread1);
        CloseHandle(hThread2);
        CloseHandle(hThread3);
        CloseHandle(hThread4);
    }

    for (int j = 0; j < 500; j++)
    {
        if (tix[j] = 0)
        {
            printf("not sold");
        }
        else if (tix[j] = 1)
        {
            printf("sold");
        }
    }

    return 0;
}

DWORD WINAPI ThreadFunction()
{
    WaitForSingleObject(ghSemaphore, 0);

    printf("current thread running : %d\n", GetCurrentThreadId());

    int i = 0;
    if (done != 0) // if loop to test wether or not the array is full
    {
        while (tix[i] = 1) //traverse the array to find a open spot
        {
            i++;
        }
        tix[i] = 1;

    }
    if (i == 499) //if i is 499, set test variable to 1
    {
        done = 1;
        return 0;
    }

    ReleaseSemaphore(ghSemaphore, 1, NULL);
}

Answer 1

您的线程函数的签名不正确。线程接受一个PVOID 上下文参数：

DWORD WINAPI ThreadProc(
  _In_ LPVOID lpParameter
);

您的线程可以在不释放信号量的情况下退出。此外，由于您使用大于线程数量的值对其进行了初始化，并且从不检查WaitForSingleObject 结果，因此不提供同步，并且多个线程将以不一致的方式修改共享缓冲区。更糟糕的是 - 没有什么比 ThreadFunction 更早地阻止您的程序主线程退出。

在你的线程函数末尾没有返回语句，所以这是一个未定义的行为。实际上，您的代码甚至可以编译是一个很大的奇迹。整个多线程方法是不正确的，必须从头开始重新制作。