C++读取文件行填充字符串vector的高效元素追加方案问询

现有实现的合理性与问题修正 #

你最初使用 std::move 配合 push_back 追加元素的逻辑本身是合理的：std::string 的 move 操作仅交换内部指针、大小、容量字段，无堆内存拷贝，开销远低于复制操作。但现有代码存在两个可优化/修正的点：

• while (!file.eof()) 是经典错误写法：eofbit 只有在读取操作尝试失败后才会置位，会导致循环多执行一次，最终往 vector 中多插入一个空字符串。正确的循环判断应该将 std::getline 本身作为循环条件：while (std::getline(file, s))，仅当读取到有效行时才进入循环。
• 可以将临时字符串 s 定义在循环外部，复用它已经分配的堆内存，避免每次循环构造、析构字符串的额外开销：

std::string s;
while (std::getline(file, s))
{
    strings.push_back(std::move(s));
}

你期望的「直接追加元素并获取引用」接口，标准库已原生支持 #

你设想的 create_and_push_back 并非不存在，完全可以通过标准库的 emplace_back 实现，分两个版本场景：

• C++17 及以上版本：std::vector::emplace_back 的返回值就是新插入元素的引用，完全符合你的预期，和你写的理想代码逻辑完全一致：

std::vector<std::string> strings;
// 默认构造空字符串插入vector末尾，直接返回它的引用
std::string& s = strings.emplace_back();
// 直接向该引用写入内容，无中间拷贝、move开销
std::getline(file, s);

循环读取的场景下还可以进一步简化为：

while (std::getline(file, strings.emplace_back())) {}

• C++11/14 版本：该版本下 emplace_back 仅返回 void，你之前想到的「先 push_back 空字符串再通过 back() 取引用」就是当时的通用标准实现，本身没有效率问题：只要在拿到 back() 引用后、使用它之前没有插入其他元素触发 vector 扩容，引用就不会失效：

strings.push_back(std::string{});
std::string& s = strings.back();
std::getline(file, s);

选型效率参考 #

• 如果文件各行长度差异不大，优先选择「复用临时字符串 + move push_back」的写法：临时字符串的堆内存可被反复复用，避免每次构造新字符串时的堆分配开销，整体效率最高。
• 如果文件各行长度差异极大，或者追求代码简洁性，可以使用 C++17 下 getline 直接写入 emplace_back 返回引用的写法，省去临时字符串的声明。
• 如果提前能预估文件大致行数，可以调用 strings.reserve(预估行数) 预先分配 vector 内存，避免多次扩容的开销，进一步提升读取性能。

内容的提问来源于stack exchange，提问作者Lou