右值引用参数是否必须始终被移动？clang-tidy告警场景咨询

你的问题出在：右值引用参数buffer是一个命名的右值引用，在函数内部它会被当作左值处理。直接调用process(buffer)会匹配到const std::vector<uint8_t>&的重载，完全没有利用右值的可移动特性，这也是clang-tidy触发cppcoreguidelines-rvalue-reference-param-not-moved告警的原因。

针对这个场景，有两种符合C++核心准则的最佳实践：

1. 采用值传递惯用法（Pass-by-Value Idiom） #

把核心逻辑放在接受值传递的函数中，通过左值/右值重载分别触发拷贝或移动，既简化代码又充分利用移动语义：

// 核心处理逻辑：接受值传递，内部直接使用
void process(std::vector<uint8_t> buffer)
{
    // ... process the message ...
}

// 左值引用重载：拷贝构造传递给核心函数
void process(const std::vector<uint8_t>& buffer)
{
    process(std::vector<uint8_t>(buffer));
}

// 右值引用重载：移动构造传递给核心函数
void process(std::vector<uint8_t>&& buffer)
{
    process(std::move(buffer));
}

这种写法的好处是：

• 左值传入时会触发一次拷贝，右值传入时只会触发一次移动（成本远低于拷贝）
• 核心逻辑只需要维护一份，避免代码重复
• 完全符合clang-tidy的检查规则，不会触发告警

2. 完美转发（Perfect Forwarding） #

如果你的目的是转发参数到其他函数，而非直接处理，可以使用模板转发引用（Forwarding Reference）来保留参数的值类别：

// 核心处理函数
void process(const std::vector<uint8_t>& buffer)
{
    // ... process the message ...
}

// 转发模板：完美转发参数到核心函数
template<typename T>
void process(T&& buffer)
{
    // 保留参数的原始值类别转发
    process(std::forward<T>(buffer));
}

这种方式适合需要将参数转发给其他重载函数的场景，确保左值还是左值，右值还是右值，不会浪费移动机会。

临时消告警的修正（不推荐长期使用） #

如果暂时不想修改核心的const&版本，至少要在右值重载中用std::move将参数转为右值——不过这种写法只能让你通过clang-tidy检查，本质上还是调用const&版本（因为const&可以绑定右值），并没有真正利用移动语义：

void process(std::vector<uint8_t>&& buffer)
{
    // 将命名右值引用转为右值，触发告警消失
    process(std::move(buffer));
}

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