将std::vector::size()绑定到const auto&是否为不良编码风格？

你给出的测试代码在语法层面是完全合法的——C规则下const左值引用会自动延长绑定的临时值生命周期，因此代码不会出现未定义行为，运行结果也完全正确。但这并不代表它是符合良好编码风格的写法，你导师的判断本质上源于C社区对“写法语义和实际行为匹配”的通用要求，具体原因可以拆解为三点：

• 第一，该写法没有任何实际收益，反而可能生成更冗余的机器码
  std::vector::size()的返回值是size_t（原生无符号整数），属于大小为4/8字节的标量类型，按值拷贝的开销极低。而const auto&本质在编译实现层面是一个常指针，在64位系统下同样占用8字节的存储空间，甚至可能因为多了一层间接寻址，比直接按值接收整数的效率略低。它和常见的“用const引用接收std::string、大结构体以避免高拷贝成本”的场景完全不同，不存在性能层面的价值。
• 第二，该写法会给代码读者造成预期错位
  当读到用const引用接收某个函数返回值的代码时，读者的第一预期是：这个返回值指向某对象内部的成员，后续如果原对象发生修改，这个引用指向的值也会同步变化。但实际上size()返回的是一个独立的整数拷贝，和原vec对象之间没有任何绑定关系——哪怕之后vec被销毁、被修改，这个cnt的值都不会发生变化。这种“写法传递的信号和实际行为不一致”的错位，是代码阅读时的额外认知负担，也是风格规范里明确不推荐的核心原因。
• 第三，该写法属于对const引用使用场景的误用
  C++社区的通用编码惯例里，const引用的适用边界非常清晰：
  • 接收需要避免高拷贝成本的大对象返回值（比如返回值是长字符串、大型自定义类）
  • 给函数传入不需要修改的大类型参数
    对于所有返回原生标量、小值类型的函数（比如获取容器元素个数、获取接口返回的整数状态），社区的通用写法都是直接按值接收，和测试代码对应的推荐写法为：

  int main(int argc, char **argv)
  {
      std::vector<int> vec;
      vec.push_back(1);
      vec.push_back(12);
      // 直接按const值接收size()的返回值
      const auto cnt = vec.size();
      cout << cnt << endl;
  }
  

需要补充说明的是，这种写法也不是完全没有合理的使用场景：如果是编写和容器类型无关的泛型代码，你无法确定不同容器的size()接口到底返回值还是返回引用，用const auto&做兜底绑定是非常常见的泛型技巧。但在具体的非泛型业务代码中，明明明确知道std::vector::size()的返回类型，还硬用const&接收，就属于没有必要的冗余写法了。

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