Оглавление
V818. It is more efficient to use an initialization list rather than an assignment operator.
Анализатор обнаружил, что конструктор реализован не оптимально и можно оптимизировать инициализацию членов класса.
Рассмотрим пример:
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Была ли полезна эта страница документации?
