Если оставлять в стороне вопрос системы родителей Qt, то использовать умные указатели следует всегда.
Разберем чуть подробнее ваш пример:
std::vector <Foo> foos;
for (int i = 0; i < 10; i++)
{
Foo foo;
// do stuff with foo
foos.push_back(foo);
}
После этого ваш код свалится с ошибкой. Добрый компилятор сам сгенерировал вам конструктор копирования, только он не в курсе, что указатель чем-то владеет, поэтому на выходе из скоупа объект foo
будет удален, а указатель, который лежал внутри и который все еще используется в объекте, помещенном в вектор, теперь смотрит на удаленные данные.
Допустим мы напишем ручками конструктор копирования, это не так и сложно для нашего класса. Теперь по правилам компилятор не сможет сгенерировать для класса конструктор перемещения, а нам возможно хотелось бы сделать foos.push_back(std::move(foo));
чтобы не дергать лишних копирований. Придется написать еще и конструктор перемещения, не забыв аккуратно занулить неиспользуемые указатели в тех местах, откуда перемещены ресурсы. И про noexcept
желательно не забыть, хотя современные IDE обычно подскажут.
Теперь представим себе что нужно расширить класс еще одним полем. Надо пройтись по всем конструкторам и ассайнментам и добавить везде корректное поведение, нигде ничего не забыв.
А всего-то можно было сделать ваш bar
или std::unique_ptr
или std::shared_ptr
в зависимости от нужного поведения и все остальное компилятор сделает за вас.
Самый лучший код - код, который не написан, в нем нет багов. Пусть компилятор делает грязную работу за вас.
new
иdelete
. Если вам нужно выделить один объект, берите умный указатель, если несколько, берите вектор. Ваш пример, кстати, портит память после копирования объекта. Когда вы исправите этот и другие недостатки, получится плохо написанная половина умного указателя.