寻找更紧凑的语法（简单代码） - C++

Question

我正在重新学习 C++，我发现自己经常编写这样的代码：

vector<string> Text;
string Line;

while (getline(cin, Line))
{
    Text.push_back(Line);
}

我想知道是否有一种更紧凑的方法来编写仅使用基本功能（例如，没有用户编写的函数）的循环 - 或多或少，将所有内容置于条件中？

Answer 1

您可以使用 for 循环。我们可以在变量声明部分声明Line，并使用条件和增量部分来读取和放置行。这给了你

for(string Line; getline(cin, Line); Text.push_back(Line));

Answer 2

或多或少，把一切都放在条件？

你可以这样做

while (getline(cin, Line) && (Text.push_back(Line),true)) {}

之所以有效，是因为&& 被短路，并且因为逗号运算符计算第一个操作数，丢弃结果并返回第二个操作数的结果。

所以这是可能的，但你为什么要这样做呢？使代码尽可能密集对可读性几乎没有好处（实际上您的原始代码更具可读性并且使用更少的字符）。

Answer 3

嗯，以一些混淆为代价，

while (getline(cin, Line) && (Text.push_back(Line), 1));

会这样做：注意表达式分隔符运算符的使用，非正式地说，它会将push_back的void返回类型“转换”为int，因此可以将其与短路&&。

但根据经验，使用语言，而不是反对它。我的答案是后者。您在问题中呈现代码的方式就足够了。

Answer 4

冒着“但这正是 OP 的代码！”的风险，如果这是一个范围的整个主体（例如解析文本的函数），我个人会更喜欢这个版本：

vector<string> Text;
string Line;

while (getline(cin, Line))
  Text.push_back(Line);

或者，如果是更大范围的一部分，我可能会将所有四行组合在一起，并在前后添加空行以实现视觉连贯性（并可能在其前添加简短注释）：

// [lots of other code]

// Gather input from cin.
vector<string> Text;
string Line;
while (getline(cin, Line))
  Text.push_back(Line);

// [lots of other code]

我知道这并没有引入任何聪明的技巧，但这是给定代码的最紧凑和可读的形式，至少对我来说是这样。

如果你想要紧凑性高于一切，你可以选择垃圾变量，省略所有不必要的空格，甚至预先给类型加上别名（因为我们“经常编写”这种代码，这是一次性的），比如说， V<S> t;S l;while(getline(cin,l))t.push_back(l); 但没有人愿意阅读。

很明显，不仅仅是紧凑性在起作用。至于我，我希望在保持直观可读性的同时将噪音降到最低，我建议这是一个令人满意的目标。

我永远不会使用“将所有内容都放入循环条件”的建议，因为这极大地破坏了我期望的代码结构：循环的主要目的进入循环体。你可能不同意/有不同的期望，但在我看来，其他一切都只是为了炫耀你的缩小技能，它不会产生好的代码。

上面的代码实现了：对于这个简单的操作，大括号是噪音，重要的部分是循环体。 “但是getline不也很重要吗？” - 是的，老实说，我更喜欢它在循环体中的版本，例如假设的

vector<string> Text;
while (cin.hasLine())
  Text.push_back(readLine(cin));

这对我来说是一个理想的循环：条件 only 检查是否终止，循环体 only 是我们想要重复的操作。

标准算法会更好，但我不知道有什么在这里会有所帮助（范围或提升可能会提供，我不知道）。

---

在更抽象的层面上，如果 OP 经常写这个确切的代码，它显然应该是一个单独的函数。但即使不是，“大量其他代码”示例也将从该抽象中受益。

Answer 5

使用一条指令循环。一行可以写但是不推荐

while (getline(cin, Line)) Text.push_back(Line);