Анализатор обнаружил, что конструктор реализован не оптимально и можно оптимизировать инициализацию членов класса.
Рассмотрим пример:
class UserInfo { std::string m_name; public: UserInfo(const std::string& name) { m_name = name; } };
Сначала член 'm_name' инициализируется как пустая строка, а затем уже в этот член копируется строка из переменной 'name'. В C++03 это приведет к лишней аллокации для пустой строки. Как минимум, этот код можно улучшить, сразу вызвав конструктор копирования через список инициализации.
UserInfo(const std::string& name) : m_name(name) { }
Если вы используете C++11, то можно пойти дальше. Посмотрим, как можно сконструировать объект UserInfo:
std::string name = "name"; UserInfo u1(name); // 1 copy UserInfo u2("name"); // 1 ctor, dtor + 1 copy UserInfo u3(GetSomeName()); // 1 copy
Если строки достаточно длинные и не попадают под Small String Optimization, то здесь произойдут лишние аллокации памяти и лишние копирования. Чтобы этого избежать, сделаем передачу аргумента по значению:
UserInfo(std::string name) : m_name(std::move(name)) { }
Теперь все временные значения не будут порождать лишних копий, так как сработает move конструктор.
std::string name = "name"; UserInfo u1(name); // 1 copy + 1 move UserInfo u2("name"); // 1 ctor, dtor + 1 move UserInfo u3(GetSomeName()); // 2 move UserInfo u4(std::move(name)); // 2 move