Безопасен ли такой код QList<QPushButton*>?

[usr00210]

Код:

int main( int argc, char *argv[] )
{
    QApplication AppInstance( argc, argv );
    QList<QPushButton*> qPButtonList;

    ...

    AppInstance.exec();

    foreach( QPushButton* pButton, qPButtonList ) // Безопасен ли и вообще правилен ли такой код?
        delete pButton;

    return 0;
}

[Пантер]

Сам контейнер не очищается. Используй лучше.

Код

C++ (Qt)
qDeleteAll (qPButtonList);
qPButtonList.clear ();
 

[Alex_cs_gsp]

КуТ делает копию контейнера при входе в фоич. Следовательно список, который в фоич, будет содержать те же указатели что и список выше. А затем происходит удаление памяти по этим указателям, выходит, что список уже будет содержать не валидные указатели, что уже плохо, даже если отсутствует другой код.

[usr00210]

Спасибо!

[Igors]

КуТ делает копию контейнера при входе в фоич. Следовательно список, который в фоич, будет содержать те же указатели что и список выше.

Будет, но контейнер не копируется, просто применяется константный итератор

[Alex_cs_gsp]

Что-то Шлее заврался.

[BRE]

Что-то Шлее заврался.

Не заврался. С точки зрения C++ контейнер будет копироваться во временный объект, но с точки зрения Qt, самого копирования данных не произойдет, временному контейнеру будет присвоен указатель на управляющую структуру контейнера и увеличен счетчик использования. При разрушении временного объекта этот счетчик просто уменьшиться.

[usr00210]

Что-то Шлее заврался.

Не заврался. С точки зрения C++ контейнер будет копироваться во временный объект, но с точки зрения Qt, самого копирования данных не произойдет, временному контейнеру будет присвоен указатель на управляющую структуру контейнера и увеличен счетчик использования. При разрушении временного объекта этот счетчик просто уменьшиться.

в документации Qt для этого есть специальная терминология deep copy - когда фактически создается дубликать и еще какое-то копирование (термин забыл), когда возвращается указатель и увеличивается счетчик объектов.

[BRE]

в документации Qt для этого есть специальная терминология deep copy - когда фактически создается дубликать и еще какое-то копирование (термин забыл), когда возвращается указатель и увеличивается счетчик объектов.

Implicit Sharing

[usr00210]

в документации Qt для этого есть специальная терминология deep copy - когда фактически создается дубликать и еще какое-то копирование (термин забыл), когда возвращается указатель и увеличивается счетчик объектов.

Implicit Sharing

нет, этот термин они используют для спец классов оберток, а тут речь идет именно о механизме копирования shallow copy (нашел!)

[Igors]

Не мешало бы добавить в документацию что foreach для "не Qt" контейнера (напр std::vector) может окончиться печально

[BRE]

нет, этот термин они используют для спец классов оберток, а тут речь идет именно о механизме копирования shallow copy (нашел!)

Implicit Sharing - это название самой технологии.
В assistant есть отдельная глава, посвященная ей.

[usr00210]

Не мешало бы добавить в документацию что foreach для "не Qt" контейнера (напр std::vector) может окончиться печально

а там вроде бы написано, что безопасно можно использовать только Qt контейнеры. вроде бы 

[Alex_cs_gsp]

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

[BRE]

Мало верится, что константный итератор реализуется копированием контейнера...

Эээ. Как понять "константный итератор реализуется копированием контейнера"?