Symbian中窥探C++编程功底

MyString& StrUtil::newMyStringfromTDesC16(TDesC16& aDesC16)
{
HBufC8* hBuf8 = HBufC8::New(aDesC16.Length() * 3);
MyString ret;

if (hBuf8)
{
TPtr8 aPtr8(hBuf8->Des());
TInt cvtResult = CnvUtfConverter::ConvertFromUnicodeToUtf8(aPtr8, aDesC16);

ret.assign((const char *)(hBuf8->Ptr()), hBuf8->Size());
if (cvtResult != 0)
{
}

delete hBuf8;
hBuf8 = NULL;
}

return ret;
}

我们知道,C++中的内存分为堆和栈(忽略全局变量区ect),栈是存放临时变量的,栈内的资源在生命周期结束后会被释放掉。之所以这样做,是因为栈的资源一般来说是非常有限的。

那么上面那段程序中MyString ret;这句是在栈上建立一个临时对象

然后return ret;这句返回这个临时对象

但是不幸的是,ret遇到下面的}时生命周期结束,于是被释放掉!所以这段程序的返回值指向了一段已经被释放了的内存区域,于是出现不可知的bug。

改成下面即可:

MyString& StrUtil::newMyStringfromTDesC16(TDesC16& aDesC16)
{
HBufC8* hBuf8 = HBufC8::New(aDesC16.Length() * 3);
MyString *ret = new MyString();

if (hBuf8)
{
TPtr8 aPtr8(hBuf8->Des());
TInt cvtResult = CnvUtfConverter::ConvertFromUnicodeToUtf8(aPtr8, aDesC16);

ret->assign((const char *)(hBuf8->Ptr()), hBuf8->Size());
if (cvtResult != 0)
{
}

delete hBuf8;
hBuf8 = NULL;
}

return *ret;
}

MyString *ret = new MyString();是在堆空间里建立一个对象,而堆空间的资源必须由程序员显式的释放掉。