是vc6库的问题么?高分特请高人解答.

[mengjj]

一个很简单的程序,在VC6下编译运行报错(内存错误,看了下是delete p;的问题),但不知道为什么,拷贝其到VS2005下编译运行,好像不会报错了,请问高人,是vc6库的问题,还是我的代码的问题,高分相送!!!
全部代码如下:

#include <windows.h>
#include <queue>

using namespace std;

struct MYMSG
{
long m;
};

HANDLE g_hLockEvent = NULL;
std::queue<MYMSG *> g_queue;

void Lock()
{
WaitForSingleObject(g_hLockEvent, INFINITE);
}

void Unlock()
{
SetEvent(g_hLockEvent);
}

MYMSG *PopMsg()
{
MYMSG *p = NULL;
Lock();
if (!g_queue.empty())
{
p = g_queue.front();
g_queue.pop();
}
Unlock();
return p;
}

void PushMsg(MYMSG *p)
{
if (p == NULL)
return;
Lock();
g_queue.push(p);
Unlock();
}

DWORD WINAPI Thread1(LPVOID param)
{
while (true)
{
MYMSG *p = new MYMSG;
PushMsg(p);
}
return 0;
}

DWORD WINAPI Thread2(LPVOID param)
{
while (true)
{
MYMSG *p = PopMsg();
if (p != NULL)
delete p;
}
return 0;
}


void main()
{
g_hLockEvent = CreateEvent(NULL, FALSE, TRUE, NULL);
CreateThread(NULL, 0, Thread1, NULL, 0, NULL);
CreateThread(NULL, 0, Thread2, NULL, 0, NULL);

HANDLE hEvent = CreateEvent(NULL, FALSE, FALSE, NULL);
WaitForSingleObject(hEvent, INFINITE);
}

[bragi523]

jf吧

[skynfly]

楼主把分给分了吧

[野男孩]

恭喜lz!

[webing520]

可以结帖，对答案不满意就可以了吧。我觉得楼主应该把解决方法贴出来呢，大家学习一下。

[chqiang0_0]

俺也来凑个热闹。。。呵呵

[sunlin7]

if (!g_queue.empty())
{
p = g_queue.front();
g_queue.pop(); // pop_font();
}
应该是这里不对。

而且楼主的这个设计很糟糕，应该重新设计一下。

[贪玩的老鼠]

你的事件根本就没有LOCK作用,线程访问肯定有冲突.
只有SetEvent,没有ResetEvent怎么能同步呢.

[NTooL]

恭喜楼主，接分。

[lint2008]

学习。

[saifutang]

重构一下吧。多线程的要小心。

[triesagain]

学习学习

[bao20080513shengbin]

顶

[jasonM2008]

结贴给分！

[MoXiaoRab]

接分

[fandh]

变成散分贴了！

[RichyMong]

恭喜lz啊

[mengjj]

哎,自己找到原因了.

[mengjj]

怎么才能把贴子删了?

[山的那边还是山~]

fen o!!!

[mengjj]

这个问题是我在做个东西的时候遇到的,找了半天发现是我写的消息队列的问题,所以把它单独提出来测试.

程序其实是没有问题的,问题出在工程设置方面,没有选择多线程库(因为创建的工程是空的控制台程序,默认是单线程的), 所以就出现问题了.有兴趣的朋友可以自己试下.