Question

我想对LoadString Win32函数的这个包装有你的意见。

int LoadWString( HINSTANCE hInstance_In, UINT uID_In, std::wstring &str_Out ){
    return LoadStringW( hInstance_In, uID_In, (LPWSTR)str_Out.c_str(), (int)str_Out.max_size());
}

因为它似乎按预期工作，问题更多是关于使用字符串max_size属性作为缓冲区大小，这是否有一些负面的缺点？

Answer 1

c_str()返回一个不可修改的指针。一定不能写。抛弃常量并写入受控序列会导致未定义的行为。

相反，只需查询指向资源部分的指针以及字符串长度，并在此数据上构造一个新的std::wstring对象：

std::wstring LoadStringW( unsigned int id )
{
    const wchar_t* p = nullptr;
    int len = ::LoadStringW( nullptr, id, reinterpret_cast<LPWSTR>( &p ), 0 );
    if ( len > 0 )
    {
        return std::wstring( p, static_cast<size_t>( len ) );
    }
    // Return empty string; optionally replace with throwing an exception.
    return std::wstring();
}

有几点值得注意：

该实现使用LoadString，将0传递给 nBufferMax 。这样做会返回一个指向资源部分的指针;没有执行额外的内存分配：

nBufferMax ：
如果此参数为0，则 lpBuffer 将接收指向资源本身的只读指针。
字符串资源是计数字符串，可以包含嵌入的NUL个字符。这要求使用std::string constructor采用明确的长度参数。 return std::wstring(p);可能会截断返回的字符串。
由于layout of string resources，通常无法判断任何给定ID的字符串资源是否为空，或者不存在。此答案中的实现如下：如果字符串资源为空或不存在，则返回空字符串。

c ++，Win32 LoadString包装器

1 个答案: