c++ - 误报警告 PVS Studio : V821 Decreased perfomance. 'rhs' 变量可以在较低级别的范围内构造

Question

PVS Studio 6.17(Windows 7、64 位、VS2017、C++-03)似乎在遵循简化代码时发出错误警告

#include <stack>
#include <string>
#include <vector>
bool fred(const std::string &x)
{
    return x == "ab";
}
std::vector<std::string> bar(std::stack<std::string> & s)
{
    std::vector<std::string> v;
    const std::string rhs(s.top()); // V821 Decreased perfomance. The 'rhs' variable can be constructed in a lower level scope.
    s.pop();
    const std::string lhs(s.top());
    s.pop();

    if (fred(lhs))
    {
        v.push_back(rhs);
    }
    return v;
}

PVS工作室的警告是

V821 性能下降。可以在较低级别的范围内构造“rhs”变量。

因为s是一个std::stack类型，对应的算法要求rhs元素从栈中弹出，看起来PVS-Studio是错误的。我错过了什么吗？

顺便说一句:

PVS Studio 消息中存在拼写错误:

   perfomance->performance

引用

• https://www.viva64.com/en/w/V821/

Answer 1

在评论中讨论了代码优化的方法。是的，它可以优化，但我认为它实际上没有意义。如果你非要用C++-03，那么，因为优化，代码会变得复杂，难以理解，这是不好的。嗯，当然，使用 std::move 是合适的。

现在说说PVS-Studio。分析器不对，在这里发出警告。不可能只获取并重新定位在 if 范围内创建的变量 rhs。分析器没有考虑到数据源会发生变化并且 s.top() 会返回另一个值。嗯，V821诊断是新的，也有缺点。我们将尝试消除此类误报。感谢您提供的示例，以及有关“性能”一词的拼写错误的信息。