为什么 cppcheck 说“函数参数应该通过引用传递”？

Question

这是cppcheck show warning "[event.cpp:20]: (performance) Function parameter 'path' should be pass by reference."的代码

void
event::set_path(const std::string path)
{
    this->_path = path;
}

但包括字符串参数在内的其他代码不显示此警告，例如：

int
watcher::init_watch(const struct stat *sb, std::string path, bool linked)
{
    int wd;
        ....
}

为什么？

Answer 1

因为它应该！没有理由传递一个 const 副本，反正你也不能修改它，那为什么要复制它。在最坏的情况下，它必须为一个全新的字符串分配内存，然后一次复制一个字节的字符串。在最好的情况下，它可能会做一些内部引用计数魔法，但如果你只是通过引用传递它，那么你最多将一个指针复制到堆栈中的新点。路过const std::string& path - 会快很多。

init_watch 中的路径参数也应该通过 const 引用传入，因为那样也会无缘无故地复制。