C ++ 11基于范围的循环：通过值或引用const获取项目

读取基于范围环的一些例子他们建议两种主要方式^{1，2，3，4}

std::vector<MyClass> vec;

for (auto &x : vec)
{
  // x is a reference to an item of vec
  // We can change vec's items by changing x 
}

要么

for (auto x : vec)
{
  // Value of x is copied from an item of vec
  // We can not change vec's items by changing x
}

好。

当我们不需要更改vec项目时，IMO，示例建议使用第二个版本（按值）。为什么他们不建议const参考的内容（至少我没有发现任何直接建议）：

for (auto const &x : vec) // <-- see const keyword
{
  // x is a reference to an const item of vec
  // We can not change vec's items by changing x 
}

好不好 它不是在每次迭代中都避免了冗余副本const吗？

如果您不想更改的项目，以及要避免复印，然后auto const &是正确的选择：

for (auto const &x : vec)

任何建议您使用的人auto &都是错误的。别管他们。

总结：

• 选择auto x您想使用副本的时间。
• 选择auto &x何时要使用原始项目并可以对其进行修改。
• 选择auto const &x何时要使用原始项目，而不修改它们。

如果您有一个std::vector<int>或std::vector<double>，那么可以使用auto（带值副本）代替const auto&，因为复制一个int或一个double很便宜：

for (auto x : vec)
    ....

但是，如果您有一个std::vector<MyClass>，其中MyClass具有一些非平凡的复制语义（例如std::string，一些复杂的自定义类等），那么我建议使用const auto&来避免深层复制：

for (const auto & x : vec)
    ....

当我们不需要更改vec项目时，示例建议使用第一个版本。

然后他们提出了错误的建议。

为什么他们不建议const引用的内容

因为他们提出了错误的建议:-)您提到的是正确的。如果只想观察一个对象，则无需创建副本，也无需对其进行非const引用。

编辑：

我看到所有链接的参考都提供了在一系列int值或其他一些基本数据类型上进行迭代的示例。在那种情况下，由于复制an int并不昂贵，因此创建一个副本基本上等效于（如果效率更高的话）进行观察const &。

但是，用户定义类型通常不是这种情况。UDT的复制成本可能很高，并且如果您没有创建副本的理由（例如修改检索的对象而不更改原始对象），则最好使用const &。

我在这里会相反，说不需要auto const &基于范围的for循环。告诉我您是否认为以下功能很愚蠢（不是目的，而是编写方式）：

long long SafePop(std::vector<uint32_t>& v)
{
    auto const& cv = v;
    long long n = -1;
    if (!cv.empty())
    {
        n = cv.back();
        v.pop_back();
    }
    return n;
}

在这里，作者创建了一个const引用以v用于所有不修改v的操作。在我看来，这很愚蠢，并且可以为auto const &基于循环的范围中的变量使用相同的参数，而不仅仅是auto &。