发个线程问题希望goodname来解答.

[appleshao]

不知道goodname大哥有没有看到,问题是这样的

#include <stdio.h> 
#include <windows.h> 
#include <process.h> 

int g_nCount1 = 0; 
int g_nCount2 = 0; 
BOOL g_bContinue = TRUE; 

HANDLE g_hEvent; 

UINT __stdcall ThreadFunc(LPVOID); 

int main(int argc, char* argv[]) 
{ 
    UINT uId; 
    HANDLE h[2]; 
    
    h[0] = (HANDLE)::_beginthreadex(NULL, 0, ThreadFunc, NULL, 0, &uId); 
    h[1] = (HANDLE)::_beginthreadex(NULL, 0, ThreadFunc, NULL, 0, &uId); 

g_hEvent=CreateEvent(NULL,FALSE,TRUE,NULL); 

//SetEvent(g_hEvent); 

    // 等待1秒后通知两个计数线程结束，关闭句柄 
    Sleep(1000); 
    g_bContinue = FALSE; 
    ::WaitForMultipleObjects(2, h, TRUE, INFINITE); 
    ::CloseHandle(h[0]); 
    ::CloseHandle(h[1]); 
    
CloseHandle(g_hEvent); 

    printf("g_nCount1 = %d \n", g_nCount1); 
    printf("g_nCount2 = %d \n", g_nCount2); 

    return 0; 
} 

UINT __stdcall ThreadFunc(LPVOID) 
{ 


    while(g_bContinue) 
    { 
WaitForSingleObject(g_hEvent,INFINITE); 

        g_nCount1++; 
        g_nCount2++; 

SetEvent(g_hEvent); 
    } 

    return 0; 
}
//这段代码是你在http://topic.csdn.net/u/20090424/07/746b011f-b6aa-4fb9-809b-22b6b7998b49.html给我的答案.
//可是我在http://topic.csdn.net/u/20090427/06/ec6fab36-d57f-447e-a637-f21d213b1bb3.html有了容哥给我的答案
//问这里的事件对象在子线程创建之前创建,那么很有可能子线程会抢了CPU,导致最后的输出不一样.可是结果却是输出一样,问一下为什么?

[wl_99888]

应该一样吧,你两个线程所做的都是将两个数加一,他们结束的路径都是通过 g_bContinue,他们退出前肯定都是把g_nCount1和g_nCount2都加1的.

[晨星]

这个程序看上去是有点问题。

[猫已经找不回了]

还是这个问题啊。

h[0] = (HANDLE)::_beginthreadex(NULL, 0, ThreadFunc, NULL, 0, &uId);
h[1] = (HANDLE)::_beginthreadex(NULL, 0, ThreadFunc, NULL, 0, &uId);

g_hEvent=CreateEvent(NULL,FALSE,TRUE,NULL); //为何要把CreateEvent放到后面，造成潜在的危险，然后又来问，为何我这里好正常呢

[晨星]

应该没什么复杂的原因，就是现代CPU太快了，你很难有机会观察到在g_hEvent被创建之前，原来的线程就被抢下台的情况。
你在CreateEvent那一行之前另一句Sleep(0)就可以看到效果了。

[晨星]

“加一句”，打错字了。。。- -b

[goodname]

下午工作了，现在才看到。

你那个帖子我只是给你修改你提出的问题，但是程序还是很不严密的。

其中有一个另外的帖子已经说过了，BOOL g_bContinue = TRUE; 前面需要加voilate
另一个正如楼上所说的，需要先创建event，后创建线程。

[mengde007]

UINT __stdcall ThreadFunc(LPVOID)
{


while(g_bContinue)
{
WaitForSingleObject(g_hEvent,INFINITE);

g_nCount1++;
g_nCount2++;

SetEvent(g_hEvent);
}

return 0;
}
这一个函数应该就能道出问题所在：第一次循环后事件为受信状态；那么导致另外一个线程不用等待g_hEvent;所以可能会出现抢占式；导致g_nCount1,g_nCount2读脏数据和写脏数据；

[appleshao]

不好意思昨天睡觉去了,但是还是没有人告诉我为什么这段有问题的代码最后的输出会一样呢?反复编译了2分钟也没发现子线程抢占CPU......

[appleshao]

[Quote=引用 3 楼 hairetz 的回复:]
还是这个问题啊。

h[0] = (HANDLE)::_beginthreadex(NULL, 0, ThreadFunc, NULL, 0, &uId);
h[1] = (HANDLE)::_beginthreadex(NULL, 0, ThreadFunc, NULL, 0, &uId);

g_hEvent=CreateEvent(NULL,FALSE,TRUE,NULL); //为何要把CreateEvent放到后面，造成潜在的危险，然后又来问，为何我这里好正常呢
[/Quote]

这个问题以前有人提问过吗?我还不知道为什么请老大给个链接让我看看啊

[「已注销」]

进来学习

[晨星]

[Quote=引用 8 楼 appleshao 的回复:]
不好意思昨天睡觉去了,但是还是没有人告诉我为什么这段有问题的代码最后的输出会一样呢?反复编译了2分钟也没发现子线程抢占CPU......
[/Quote]
没发现也是正常的。现代的CPU都很快，在子线程正式执行线程函数的代码之前，足以执行到CreateEvent了。。。

[晨星]

昨天不是告诉你了吗，你只要在CreateEvent那句之前加上一句：Sleep(0);或者SwitchToThread();迫使主线程放弃本次时间片所剩余的CPU时间，就完全可以观察到不等的效果了。

[cyldf]

[Quote=引用 8 楼 appleshao 的回复:]
不好意思昨天睡觉去了,但是还是没有人告诉我为什么这段有问题的代码最后的输出会一样呢?反复编译了2分钟也没发现子线程抢占CPU......
[/Quote]
赞时间~~

[晨星]

另外，所谓的“抢占”，是指高优先级线程的可以抢掉低优先级线程的时间，而对于平级的，那就没有抢占，只能是一个线程要么用完分配给它的时间片，要么因为某些原因主动挂起，才能再调度其它的平级或低优先级线程。

[liliangbao]

学习~

[测试菜鸟]

mark 学习