Форум на Исходниках.RU

Выполнение рекомендаций PVS-Studio о константности ссылок

tuchin — Tue, 28 Nov 2023 14:30:18 +0000

tuchin: Спасибо! Действительно, из описания все понятно.

Выполнение рекомендаций PVS-Studio о константности ссылок

sharky72 — Mon, 27 Nov 2023 20:10:12 +0000

sharky72: * это указатель. Это другое понятие.

DisplayMessage(string sMes, bool error)
DisplayMessage(const string& sMes, bool error)

void CopyFileToDir(vector vsFullname, vector vsFilename, vector vsFilenameNum, char* argv[])
void CopyFileToDir(const vector& vsFullname, const vector& vsFilename, const vector& vsFilenameNum, char* argv[])
А vsFilename внутри функции вообще не используется, этот параметр вообще лишний.

У PVS это предупреждение описано вполне понятным языком. Вместо копирования всего объекта в новый вы просто передаете ссылку на оригинальный объект с модификатором const, т.е. гововорите компилятору что объект внутри функции меняться не будет.
https://pvs-studio.ru/ru/docs/warnings/v813/

Выполнение рекомендаций PVS-Studio о константности ссылок

tuchin — Thu, 09 Nov 2023 14:14:59 +0000

tuchin: В программе от PVS-Studio получаю сообщение:

D:\MyProgramming\cppProcFiles\cppFun.cpp (25): error V813: Decreased performance. The 'sMes' argument should probably be rendered as a constant reference.

D:\MyProgramming\cppProcFiles\cppFun.cpp (186): error V813: Decreased performance. The 'vsFullname', 'vsFilenameNum' arguments should probably be rendered as constant references.

Как я понимаю, нужно сделать константные ссылки на аргументы фукнций. Я читал в книгах о константных ссылках, но так и не понял, как на практике действовать: куда ставить * (до или после описания типа переменной) и куда ставить &. Поясните, пожалуйста, на примере этой программы, как удовлетворить требованиям PVS-Studio. Архив с программой и тестовыми директориями прикрепил.
cppProcFilesTest.zip (, : 104)