Должен ли я хранить целые объекты или указатели на объекты в контейнерах?

162

Проектирование новой системы с нуля. Я буду использовать STL для хранения списков и карт определенных долгоживущих объектов.

Вопрос: Должен ли я гарантировать, что мои объекты имеют конструкторы копирования и хранят копии объектов в моих контейнерах STL, или лучше вообще управлять жизнью и областью действия и просто хранить указатели на эти объекты в моих контейнерах STL?

Я понимаю, что это несколько не хватает деталей, но я ищу «теоретический» лучший ответ, если он существует, поскольку я знаю, что оба эти решения возможны.

Два очень очевидных недостатка в игре с указателями: 1) Я должен сам управлять распределением / освобождением этих объектов за пределами STL. 2) Я не могу создать временный объект в стеке и добавить его в мои контейнеры.

Есть что-то еще, что я пропускаю?

Stéphane
источник
36
Боже, я люблю этот сайт, это ТОЧНЫЙ вопрос, о котором я думал сегодня ... спасибо за то, что
задали
2
другая интересная вещь заключается в том, что мы должны проверить, действительно ли указатель был добавлен в коллекцию, а если нет, то, скорее всего, следует вызвать delete, чтобы избежать утечек памяти ... if ((set.insert (pointer)). second = false) {удалить указатель;}
javapowered

Ответы:

68

Так как люди вмешиваются в эффективность использования указателей.

Если вы планируете использовать std :: vector и если обновлений немного, и вы часто выполняете итерацию по своей коллекции, и это не полиморфный тип, то хранение «копий» объекта будет более эффективным, поскольку вы получите лучшую локальность ссылок.

Otoh, если обновления являются распространенными указателями хранения, это сэкономит затраты на копирование / перемещение.

Torbjörn Gyllebring
источник
7
С точки зрения локальности кэша, сохранение указателей на вектор может быть эффективным, если используется вместе с пользовательским распределителем для пуантеев. Пользовательский распределитель должен заботиться о местонахождении кэша, например, с использованием размещения new (см. En.wikipedia.org/wiki/Placement_syntax#Custom_allocators ).
Амит
47

Это действительно зависит от вашей ситуации.

Если ваши объекты маленькие, а копирование объекта является легковесным, то хранение данных в контейнере stl, на мой взгляд, является простым и более простым в управлении, поскольку вам не нужно беспокоиться об управлении временем жизни.

Если у вас большие объекты, и использование конструктора по умолчанию не имеет смысла, или копии объектов стоят дорого, то хранение с указателями, вероятно, является подходящим способом.

Если вы решили использовать указатели на объекты, взгляните на библиотеку Boost Pointer Container . Эта библиотека повышения оборачивает все контейнеры STL для использования с динамически размещаемыми объектами.

Каждый контейнер указателя (например, ptr_vector) становится владельцем объекта при его добавлении в контейнер и управляет временем жизни этих объектов для вас. Вы также получаете доступ ко всем элементам в контейнере ptr_ по ссылке. Это позволяет вам делать такие вещи, как

class BigExpensive { ... }

// create a pointer vector
ptr_vector<BigExpensive> bigVector;
bigVector.push_back( new BigExpensive( "Lexus", 57700 ) );
bigVector.push_back( new BigExpensive( "House", 15000000 );

// get a reference to the first element
MyClass& expensiveItem = bigList[0];
expensiveItem.sell();

Эти классы обертывают контейнеры STL и работают со всеми алгоритмами STL, что действительно удобно.

Существуют также средства для передачи владения указателем в контейнере вызывающей стороне (с помощью функции освобождения в большинстве контейнеров).

Ник Хаддад
источник
38

Если вы храните полиморфные объекты, вам всегда нужно использовать набор указателей базового класса.

То есть, если вы планируете хранить разные производные типы в своей коллекции, вы должны хранить указатели или быть съеденными даемоном.

Torbjörn Gyllebring
источник
1
Я любил нарезку Деймон!
idichekop
22

Извините, что прыгнул через 3 года после события, но предостережение здесь ...

В моем последнем большом проекте моей центральной структурой данных был набор довольно простых объектов. Примерно через год после разработки требований я понял, что объект должен быть полиморфным. Потребовалось несколько недель сложной и неприятной операции на головном мозге, чтобы исправить структуру данных как набор указателей базового класса и справиться со всеми сопутствующими повреждениями при хранении объектов, приведениях и т. Д. Мне потребовалось несколько месяцев, чтобы убедить себя, что новый код работает. Кстати, это заставило меня задуматься о том, насколько хорошо разработана объектная модель C ++.

В моем текущем большом проекте моя центральная структура данных представляет собой набор довольно простых объектов. Примерно через год в проекте (который происходит сегодня), я понял, что объект на самом деле должен быть полиморфным. Обратно в сеть, нашел эту ветку и нашел ссылку Ника на библиотеку контейнера указателя Boost. Это именно то, что я должен был написать в прошлый раз, чтобы все исправить, поэтому я попробую на этот раз.

Мораль, для меня, в любом случае: если ваша спецификация не на 100% отлита в камне, используйте указатели, и вы потенциально можете сэкономить много работы позже.

EML
источник
Спецификации никогда не бывают в камне. Я не думаю, что это означает, что вы должны использовать исключительно контейнеры-указатели, хотя контейнеры-указатели Boost, кажется, делают эту опцию гораздо более привлекательной. Я скептически отношусь к тому, что вам нужно полностью пересмотреть всю вашу программу, если вы решите, что контейнер объекта должен быть преобразован в контейнер указателя. Это может иметь место в некоторых проектах. В таком случае это хрупкий дизайн. В этом случае не вините свою проблему в «слабости» контейнеров объектов.
allyourcode
Вы могли бы оставить элемент в векторе с семантикой значения и сделать полиморфное поведение внутри.
Билли ONEAL
19

Почему бы не получить лучшее из обоих миров: сделайте контейнер умных указателей (таких как boost::shared_ptrили std::shared_ptr). Вам не нужно управлять памятью, и вам не нужно иметь дело с большими операциями копирования.

Бранан
источник
Чем этот подход отличается от того, что предложил Ник Хаддад при использовании библиотеки Boost Pointer Container Library?
Торстен Шенинг
10
@ ThorstenSchöning std :: shared_ptr не добавляет зависимости от повышения.
Джеймс Джонстон
Вы не можете использовать общие указатели для обработки вашего полиморфизма, поэтому вы в конечном итоге упустите эту функцию с этим подходом, если только вы явно не
приведете
11

Как правило, хранение объектов непосредственно в контейнере STL является наилучшим, поскольку оно является самым простым, наиболее эффективным и наиболее простым для использования объекта.

Если ваш объект имеет не копируемый синтаксис или является абстрактным базовым типом, вам нужно хранить указатели (проще всего использовать shared_ptr)

Грег Роджерс
источник
4
Это не очень эффективно, если ваши объекты большие, и вы часто перемещаете элементы.
allyourcode
3

Вы, кажется, хорошо понимаете разницу. Если объекты маленькие и их легко скопировать, то непременно сохраните их.

Если нет, я бы подумал о сохранении умных указателей (не auto_ptr, умный указатель подсчета ссылок) для тех, которые вы выделяете в куче. Очевидно, что если вы выбираете умные указатели, то вы не можете хранить выделенные объекты временного стека (как вы уже сказали).

@ Torbjörn хорошо подходит к нарезке.

Лу Франко
источник
1
Да, и никогда никогда никогда не создать коллекцию auto_ptr - х
Torbjörn Gyllebring
Да, auto_ptr не является умным указателем - он не считается.
Лу Франко
auto_ptr также не имеет неразрушающей семантики копирования. Акт присвоения auto_ptr из oneto anotherосвободит ссылку из oneи изменится one.
Энди Финкенштадт
3

Использование указателей будет более эффективным, поскольку контейнеры будут копировать только указатели, а не полные объекты.

Здесь есть некоторая полезная информация о контейнерах STL и умных указателях:

Почему неправильно использовать std :: auto_ptr <> со стандартными контейнерами?

17 из 26
источник
2

Если на объекты нужно ссылаться в другом месте кода, храните в векторе boost :: shared_ptr. Это гарантирует, что указатели на объект останутся действительными, если вы измените размер вектора.

То есть:

std::vector<boost::shared_ptr<protocol> > protocols;
...
connection c(protocols[0].get()); // pointer to protocol stays valid even if resized

Если никто не хранит указатели на объекты или список не увеличивается и не уменьшается, просто храните как обычные старые объекты:

std::vector<protocol> protocols;
connection c(protocols[0]); // value-semantics, takes a copy of the protocol

источник
1

Этот вопрос беспокоит меня некоторое время.

Я склонен хранить указатели, но у меня есть некоторые дополнительные требования (SWIG lua wrappers), которые могут не относиться к вам.

Самым важным моментом в этом посте является тестирование самостоятельно , используя ваши объекты

Я сделал это сегодня, чтобы проверить скорость вызова функции-члена в коллекции из 10 миллионов объектов 500 раз.

Функция обновляет x и y на основе xdir и ydir (все переменные-члены с плавающей точкой).

Я использовал std :: list для хранения обоих типов объектов и обнаружил, что сохранение объекта в списке немного быстрее, чем использование указателя. С другой стороны, производительность была очень близка, поэтому все сводится к тому, как они будут использоваться в вашем приложении.

Для справки, с -O3 на моем оборудовании указателям потребовалось 41 секунда, а необработанным объектам потребовалось 30 секунд.

Meleneth
источник