返回值为&时的一点注意事项

std::string ConfigParse::GetStateTriggerTopicNameFromJson()

{

std::string topicName ;

if (!initialized)

{

std::cout << "Please run LoadCfgFile() for initialization first!" << std::endl;

return topicName;

}

if (!(j.contains("state_trigger")))

{

std::cout << "state_trigger not found in json file!" << std::endl;

return topicName;

}

return j["state_trigger"]["topicName"];

}

return topicName;

在这里，topicName 是一个局部变量，它的生命周期仅限于该函数的作用域内。因此，当函数返回 topicName 的引用时，这将导致返回一个已经销毁的对象的引用，这是一种未定义的行为。

return j["state_trigger"]["topicName"];

在这里，j["state_trigger"]["topicName"] 返回一个 std::string 对象，但它并没有返回一个引用，而是返回了一个临时的对象。因此，这个函数将会返回一个指向临时对象的引用，这也是一种未定义的行为。

要修复这个问题，您可以将函数的返回类型更改为 std::string，并在函数末尾使用 std::move() 将 topicName 转移为返回值：

这样就可以避免返回一个对已经销毁的对象的引用，同时也可以避免返回一个对临时对象的引用。