这种使用 std::string 的方式安全吗？

Question

我正在编写一个字符串类（Utf8String），它包装（具有）一个 std::string（因为它很方便）。

出于某种原因，我必须访问包装的 std::string 的内部缓冲区才能直接写入其中。我知道这是肮脏的，违反了封装原则等等，但我必须这样做。

让下面的 Utf8String 类成员：

std::basic_string< char > m_sBuf;

inline char * Reserve( uint32_t nNewByteCap )
{
   m_sBuf.reserve( nNewByteCap );
   return const_cast< char * >( m_sBuf.c_str() );
}

inline void Resize( uint32_t nNewByteCount )
{
   // Some stuff ...
   m_sBuf.resize( nNewByteCount, ' ' );
}

为了做这样的事情：

Utf8String sMessage;
char * sBuffer = sMessage.Reserve( 1024 );
uint32_t nRealLen = some_c_function_write_to_buffer( sBuffer, sMessage.Capacity() );
sMessage.Resize( nRealLen );

我不确定 STL 在内部是如何工作的。据我所知，它接受在 std::string 中间丢失空字符，因此在字符串的实际结尾之前放置空字符应该不是问题。

由于 STL 实现在不同平台上可能不同，因此在我的平台上进行测试并不意味着它也可以在其他平台上运行。

您是否在这段代码中看到了可以破坏我正在使用的 std::string 对象的内容？感谢您的帮助。

PS：我不能使用 C++11。

Answer 1

来自http://en.cppreference.com/w/cpp/string/basic_string/c_str：

写入通过 c_str() 访问的字符数组是未定义的行为。 你不应该那样做。

更安全的代码：

Utf8String sMessage;
std::array<char, 1024> buffer;
uint32_t nRealLen = some_c_function_write_to_buffer( buffer.data(), buffer.size() );
sMessage.Assign(sBuffer.begin(), sBuffer.end());

最后一行假设你在你的字符串类中：

template<typename InIt>
void Assign(InIt begin, InIt end) {
    m_sBuf.assign(begin, end);
}

（或等价物）。

这样更好，因为它不使用对 c_str 返回的数据的访问，也不会尝试手动管理 std::string 中的内存。