无论如何都要读取已知的字节数,直接读入std :: string,而不创建临时缓冲区吗?
例如,目前我可以通过
来完成boost::uint16_t len;
is.read((char*)&len, 2);
char *tmpStr = new char[len];
is.read(tmpStr, len);
std::string str(tmpStr, len);
delete[] tmpStr;
答案 0 :(得分:11)
std::string
有一个你可以使用的resize
函数,或者一个会做同样的构造函数:
boost::uint16_t len;
is.read((char*)&len, 2);
std::string str(len, '\0');
is.read(&str[0], len);
这是未经测试的,我不知道是否要求字符串具有连续存储。
答案 1 :(得分:6)
您可以使用copy_n和insert_iterator的组合
void test_1816319()
{
static char const* fname = "test_1816319.bin";
std::ofstream ofs(fname, std::ios::binary);
ofs.write("\x2\x0", 2);
ofs.write("ab", 2);
ofs.close();
std::ifstream ifs(fname, std::ios::binary);
std::string s;
size_t n = 0;
ifs.read((char*)&n, 2);
std::istream_iterator<char> isi(ifs), isiend;
std::copy_n(isi, n, std::insert_iterator<std::string>(s, s.begin()));
ifs.close();
_unlink(fname);
std::cout << s << std::endl;
}
没有复制,没有黑客,没有超支的可能性,没有未定义的行为。
答案 2 :(得分:2)
你可以使用像getline这样的东西:
#include <iostream>
#include <string>
using namespace std;
int main () {
string str;
getline (cin,str,' ');
}
答案 3 :(得分:2)
我会使用矢量作为缓冲区。
boost::uint16_t len;
is.read((char*)&len, 2); // Note if this file was saved from a different architecture
// then endianness of these two bytes may be reversed.
std::vector buffer(len); // uninitialized.
is.read(&buffer[0], len);
std::string str(buffer.begin(),buffer.end());
虽然您可能会使用字符串作为缓冲区(如GMan所述)。字符串成员处于连续位置的标准不能保证(因此请检查当前的实现并在移植到另一个编译器/平台时进行需要检查的重要注释)。
答案 4 :(得分:0)
您只是优化代码长度或尝试在此处保存自己的副本吗?临时缓冲区出了什么问题?
我认为你实际上正在规避字符串的保护,试图直接写这样做。如果您担心复制到std :: string的性能,因为您已经发现它会以某种方式影响应用程序的性能,我会直接使用char *。
编辑:做更多看...... initializing std::string from char* without copy
在第二个答案中,它非常明确表示你无法实现你想要实现的目标(即填充std :: string而不需要迭代char *来复制。)
看看你的加载例程(可能在这里发布吗?)并最小化分配:new和delete当然不是免费的,所以你至少可以节省一些时间,如果你不必经常重新创建缓冲区。我总是觉得通过memset将缓冲区擦除为0或者每次迭代终止数组的第一个索引都会有用,但是一旦你对算法有信心,就可以快速消除代码以保证性能。
答案 5 :(得分:0)
一种简单的方法是:
std::istream& data
const size_t dataSize(static_cast<size_t>(data.rdbuf()->in_avail()));
std::string content;
content.reserve( dataSize);
data.read(&content[0], dataSize);