使用_bstr_t在函数中传递BSTR*类型的参数

Question

执行此操作的正确方法是什么：

_bstr_t description;
errorInfo->GetDescription( &description.GetBSTR() );

或：

_bstr_t description;
errorInfo->GetDescription( description.GetAddress() );

其中 IError:GetDescription 定义为：

HRESULT GetDescription (BSTR *pbstrDescription);

我知道我可以轻松做到这一点：

BSTR description= SysAllocString (L"Whateva"));
errorInfo->GetDescription (&description);
SysFreeString (description);

谢谢

Answer 1

BSTR 是引用计数的，我严重怀疑如果您使用 GetAddress()，它是否能正常工作。遗憾的是，无法使用源代码来仔细检查这一点。我一直都是这样做的：

BSTR temp = 0;
HRESULT hr = p->GetDescription(&temp);
if (SUCCEEDED(hr)) {
    _bstr_t wrap(temp, FALSE);
    // etc..
}

Answer 2

跟进 @Hans 的答案 - 构造 _bstr_t 的适当方法取决于 GetDescription 是否返回您拥有的 BSTR，或者一个引用您不必释放的内存。

这里的目标是最大限度地减少副本数量，同时避免对返回数据进行任何手动调用 SysFreeString。我将修改所示的代码以澄清这一点：

BSTR temp = 0;
HRESULT hr = p->GetDescription(&temp);
if (SUCCEEDED(hr)) {
    _bstr_t wrap(temp, false);    // do not copy returned BSTR, which
                                  // will be freed when wrap goes out of scope.
                                  // Use true if you want a copy.
    // etc..
}

Answer 3

较晚的答案可能不适用于早期（或更高）版本的 Visual Studio；然而，
VS 12.0 具有内联 _bstr_t 实现，并且显然在调用 GetBSTR() 时会创建一个 Data_t 实例，其 m_RefCount 为 1 处女_bstr_t。因此，第一个示例中的 _bstr_t 生命周期看起来没问题：

_bstr_t description;
errorInfo->GetDescription( &description.GetBSTR() );

但是如果 _bstr_t 脏了，现有的内部 m_wstr 指针将被覆盖，泄漏它引用的前一个内存。

通过使用以下operator&，可以使用脏_bstr_t，因为它首先是通过Assign(nullptr)清除的。重载还提供了使用地址运算符代替 GetBSTR() 的便利；

BSTR *operator&(_bstr_t &b) {
    b.Assign(nullptr);
    return &b.GetBSTR();
}

因此，您的第一个示例可能如下所示：

_bstr_t description(L"naughty");
errorInfo->GetDescription(&description);

此评估基于 VS 12.0 中的 comutil.h。

Answer 4

_bstr_t（及其 ATL 兄弟 CComBSTR）是 BSTR 的资源所有者。从代码中观察，“GetAddress”似乎是专门为使用 BSTR 输出参数的用例而设计的，其中客户端获得所有权并期望释放 BSTR。 MSDN 指出：“释放任何现有字符串并返回新分配的字符串的地址。”。

_bstr_t bstrTemp;
HRESULT hr = p->GetDescription(bstrTemp.GetAddress());

请注意，如果 _bstr_t 已经拥有 BSTR，则使用“GetAddress()”并不等同于使用“&GetBSTR()”。 “GetBSTR”仅返回商店位置，而不释放现有位置。

注意：文档中没有说明“GetAddress”的具体用例；这是我通过查看源代码和使用 ATL 对应部分 CComBSTR 的经验得出的结论。

由于用户“caoanan”对此解决方案提出质疑，我将 Microsoft 的源代码粘贴到此处：

inline BSTR* _bstr_t::GetAddress()
{
    Attach(0);
    return &m_Data->GetWString();
}

inline wchar_t*& _bstr_t::Data_t::GetWString() throw()
{
    return m_wstr;
}

inline void _bstr_t::Attach(BSTR s)
{
    _Free();

    m_Data = new Data_t(s, FALSE);
    if (m_Data == NULL) {
        _com_issue_error(E_OUTOFMEMORY);
    }
}

inline _bstr_t::Data_t::Data_t(BSTR bstr, bool fCopy)
    : m_str(NULL), m_RefCount(1)
{
    if (fCopy && bstr != NULL) {
        m_wstr = ::SysAllocStringByteLen(reinterpret_cast<char*>(bstr),
                                         ::SysStringByteLen(bstr));

        if (m_wstr == NULL) {
            _com_issue_error(E_OUTOFMEMORY);
        }
    }
    else {
        m_wstr = bstr;
    }
}

Answer 5

    int GetDataStr(_bstr_t & str) override {
    BSTR data = str.Detach();
    int res = m_connection->GetDataStr( &data );
    str.Attach(data);
    return res;
}

Answer 6

感谢 bvj 和 Gast128 的回答，很明显，您可以通过 GetAddress() 将空的 bstr_t 传递给具有签名 GetDescription(BSTR* BS)，这将创建一个新的 BSTR，但您绝不能以这种方式将包含值的 bstr_t 传递给函数旨在使用包装的BSTR，因为GetAddress()将释放包装的BSTR。我给出一个完整的例子。

#include <iostream>
#include <comdef.h>

void CreateBSTR(BSTR* bstr)
{
  *bstr = SysAllocString(L"Read the manual!");
}

void UseBSTR(BSTR* bstr)
{
  std::wcout << L"Content: " << *bstr << std::endl;
}

int main()
{
  bstr_t bs_t;
  CreateBSTR(bs_t.GetAddress());
  std::wcout << L"Description: " << bs_t << std::endl;

  BSTR BS = bs_t.Detach();
  UseBSTR(&BS);
  bs_t.Attach(BS);
  //Don't: UseBSTR(bs_t.GetAddress());
}