求这段代码错在哪里?

[mz02005]

#include "stdafx.h"
#include <windows.h>
#include <comdef.h>
#include <list>
#include "e:\sdbg\interfaces\interfaces.h"

class CDev
{
public:
ISerialDevice* m_pdev;

CDev (ISerialDevice* pdev) : m_pdev (pdev) {}

CDev (const CDev& dev)
{
m_pdev = dev.m_pdev;
}

~CDev () {
if (m_pdev)
{
m_pdev->Release ();
m_pdev = NULL;
}
}
};

int main(int argc, char* argv[])
{
::CoInitialize (0);
ISerialDevice * psd = 0;
CLSID clsid;
HRESULT hr = ::CLSIDFromProgID (_bstr_t ("YMIDevices.QDSeries"), &clsid);
if (FAILED(hr))
return 1;
hr = ::CoCreateInstance (clsid, 0, 5, IID_ISerialDevice, (void**) &psd);
if (FAILED(hr))
return 1;

std::list<CDev> devlist;
psd->AddRef ();
devlist.push_back (CDev (psd));

::CoUninitialize ();
return 0;
}

[快乐鹦鹉]

你得先说有什么样的错误啊。

[legend-never-die]

编译出什么错？

[mz02005]

结束前的一次析构Release时，那个~CDev中的m_pdev指针非法。我试着将psd->AddRef()这句去掉然后在拷贝构造函数中增加m_pdev->AddRef()也不行。

[mz02005]

up

[sb339]

将
~CDev () {
if (m_pdev)
{
m_pdev->Release ();
m_pdev = NULL;
}
}
这段代码改为：

~CDev () {
if (m_pdev！=NULL)
{
m_pdev->Release ();
m_pdev = NULL;
}
}

[mz02005]

这样写我看不出有什么区别啊？大家是否还有什么办法？

[mz02005]

up

[Earthdog]

把这句话去掉吧
::CoUninitialize ();


你的问题出在，你将你的接口包装到一个类中去了，这相当于一个智能指针，你的指针的Release是在类的析构中调用的，而你的类的析构调用是在main函数结束之后才会调用，这时你已经调用了CoUninitialize，在这之后Release指针是会报错的

[mz02005]

真是太感谢了！

[dongfa]

是自动析构的问题了.

int main(int argc, char* argv[])
{
::CoInitialize (0);
{ //增加
ISerialDevice * psd = 0;
CLSID clsid;
HRESULT hr = ::CLSIDFromProgID (_bstr_t ("YMIDevices.QDSeries"), &clsid);
if (FAILED(hr))
return 1;
hr = ::CoCreateInstance (clsid, 0, 5, IID_ISerialDevice, (void**) &psd);
if (FAILED(hr))
return 1;

std::list<CDev> devlist;
psd->AddRef ();
devlist.push_back (CDev (psd));
}//增加

::CoUninitialize ();
return 0;
}