В классе синглтоне есть метод который принимает объект класса SentPackage, олицетворяющего отправленный пакет. Внутри этого объекта находится довольно крупный вектор payload.
void PendingStorage::AddPackage(SentPackage& in_packet); // Внутри in_packet огромный вектор payload.
В методе AddPackage мне необходимо сохранить in_packet в std::map и тут я встал перед дилеммой. Я могу либо копировать объект либо переместить его (с помощью std::move). Во втором случае я довольно сильно выиграю по производительности т.к. в конструкторе копирования SentPackage внутренний вектор payload также будет перемещаться. Но получается, что in_packet во втором случае будет опустошён и далее не пригоден для использования, что конечно я буду учитывать в своём коде. Проблема может возникнуть кода мой синглтон будет использовать другой программист, который не чего не знает об изменениях передаваемого объекта внутри моего метода. По идее на эти изменения будут указывать отсутствие спецификатора const у in_packet и мои комментарии которые я напишу=) Но всё же мне кажется это очень небезопасным решением. Меня интересует ваше мнение, на сколько это вообще считается безопасным в нормальном сообществе и как принято разруливать такие ситуации в ваших проектах?
foo(const int& i)
. Если параметр передается по ссылке, то это означает, что функция МОЖЕТ изменить его. Как правило это оговаривается в документации