不要返回局部对象的指针,这个大家都知道,不要返回临时对象的指针,这个有时容易忽略。
最近用vc+ado做了一个activex,里面有这样一段代码:
BSTR fieldname = ptrRecord->Fields->GetItem(i)->Name;
pOutputRecord->SetFieldValue(0, fieldname, val);
SetFieldValue的声明是:
HRESULT CATLRecordSet::SetFieldValue(long line, BSTR fieldname, VARIANT val);
这个函数执行不成功,进入SetFieldValue函数后发现fieldname变了,具体地方是参数的val拷贝构造完毕后就变了,仔细想了一下才发现fieldname指向的是一个临时对象的指针,这个对象被释放了,所以内存可以被其它函数使用。
最后改成_bstr_t fieldname = ptrRecord->Fields->GetItem(i)->Name后一切OK!