关闭或取消对话框程序崩溃？？？

[yeyuanzhi2010]

我是在CConferenceView中调用CLogin登录对话框的，代码如下：
BOOL CConferenceView::PreCreateWindow(CREATESTRUCT& cs)
{
CLogin *log;
log = new CLogin;
if(log->DoModal() != IDOK)
{
delete m_pSocket;
m_pSocket = NULL;
return FALSE;
}
else
{
}
}
但是我在CLogin中关闭窗口或点击“取消”按钮，程序马上崩溃。怎么回事？谢谢！！

[bencharluo]

内存泄露或者变量已销毁，但是后来还是调用了？
调试跟踪一下啊

[zhouzhipen]

delete m_pSocket;
这句有问题,如果这是一个SOCKET句柄,那就不能用delete销毁.

[hit_spaceknight]

你在delete m_pSocket之前先判空一下试试
if( m_pSocket != NULL)
{
delete m_pSocket;
m_pSocket = NULL;
}

[Fly_InWind]

[Quote=引用 2 楼 zhouzhipen 的回复:]
delete m_pSocket;
这句有问题,如果这是一个SOCKET句柄,那就不能用delete销毁.
[/Quote]

这是最好的方法

[yeyuanzhi2010]

[Quote=引用 2 楼 zhouzhipen 的回复:]
delete m_pSocket;
这句有问题,如果这是一个SOCKET句柄,那就不能用delete销毁.
[/Quote]
那么怎么办？

[yeyuanzhi2010]

我去掉
delete m_pSocket;
m_pSocket = NULL;
运行后还是出现同样的错误

[sunm42000]

[Quote=引用 3 楼 hit_spaceknight 的回复:]
你在delete  m_pSocket之前先判空一下试试
if( m_pSocket != NULL)
{
    delete m_pSocket;
    m_pSocket = NULL;
}
[/Quote]
C++不会delete一个被赋为NULL的指针 所以这个if判断没有任何作用
lz 发现问题调试一下 看看每个变量是否都是你预期的值 这是最快的解决办法
实在想不明白看下c++primer