中国开发网: 论坛: 程序员情感CBD: 贴子 286244
Fish
今天又测试了一下,昨天的小结有错误
昨天代码改动的多,贴错了一些,实际上,最初的代码是这样的:

XMLDOC->createElement(CComBSTR(L"RoadNo"),&pElement);
pElement->put_text(CComBSTR(array[i].RoadNo));
pElement->setAttribute(L"sn", CComVariant(StringToOleStr((IntToStr(n++))))) ;
TwoNode->appendChild(pElement,&TempNode);

经测试,这个 pElement->put_text( CComBSTR(....)....) 和接下来的
setAttribute 中的 StringToOleStr 函数有问题,这两个地方都会导致内存泄漏。

======================================================

昨晚的最终结果是:

XMLDOC->createElement(CComBSTR(L"RoadNo"),&pElement);
bstr = ::SysAllocString( WideString( IntToStr( array[i].RoadNo ) ) );
pElement->put_text( bstr );
::SysFreeString( bstr );
pElement->setAttribute(L"sn", CComVariant(WideString((IntToStr(n++))))) ;
TwoNode->appendChild(pElement,&TempNode);

改动了两个地方,问题解决

===========================================================

今天早上,又测试了一下,发现实际上使用 WideString 的时候,不用手工分配和释放
也不会导致内存泄漏。代码如下:

XMLDOC->createElement(CComBSTR(L"RoadNo"),&pElement);
pElement->put_text( WideString( IntToStr( array[i].RoadNo ) ) );
pElement->setAttribute(L"sn", CComVariant(WideString((IntToStr(n++))))) ;
TwoNode->appendChild(pElement,&TempNode);

===============================================================

综上所述,问题是出在那个 CComBSTR(...) 和 StringToOleStr( ... ) 这两个地方,
分析 atlbase.h 的 CComBSTR 定义部分,看到:

/*explicit*/ CComBSTR(const CComBSTR& src)
{
m_str = src.Copy();
}

如果传递进来的参数是 CComBSTR 类型的话,那么其构造函数是复制了参数内容,
而 CComBSTR 的析构函数只是释放了 m_str,并未释放参数变量里的内存资源,
从而导致了内存泄漏。

而WideString 则没有问题,是因为 WideString 本身被封装称能释放所占资源,
在隐式的类型转换过程中可以被释放掉。

StringToOleStr 估计也是类似的原因,这个函数没有源码可以分析。
大家都是出来卖的,何苦自己人为难自己人
那些活好的,或者活新的,或者花样多的,
或者老板拉皮条功夫好能拉到肯多花钱的客的,
拜托不要老是打击年老色衰的同行了

老鱼记事 老鱼侃棋 老鱼围脖


相关信息:


欢迎光临本社区,您还没有登录,不能发贴子。请在 这里登录