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

Question

这是cppcheck show waring的代码“[event.cpp：20] :(性能）函数参数'path'应该通过引用传递。”

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

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

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

为什么？

Answer 1

因为它应该！没有理由传递const副本，无论如何都无法修改它，所以为什么要复制它。在最坏的情况下，它必须为一个全新的字符串分配内存，然后一次一个字节地复制字符串。在最好的情况下，它可能会执行一些内部引用计数魔术，但如果您只是通过引用传递它，那么您最多只能复制一个指向堆栈中新点的指针。通过const std::string& path - 会更快。

init_watch中的path参数也应该由const引用传入，因为它也会无缘无故地复制。