QAbstractItemView, или внутренности Qt Опции

[Litkevich Yuriy]

В классе QAbstractItemView есть метод setModel(QAbstractItemModel *model)

/*!
Sets the \a model for the view to present.

\bold{Note:} This function will also create and set a new selection model,
replacing any previously set with setSelectionModel(), but the old selection
model will not be deleted.

\sa selectionModel(), setSelectionModel()
*/

и подтверждение сказаному:

setSelectionModel(new QItemSelectionModel(d->model, this));

Для меня остается загадкой:
1) либо почему старая не удаляется?
2) либо почему старая не используется?

В примере %QTDIR%\demos\sqlbrowser при выполнении запроса устанавливается для табличного представления QSqlQueryModel, а при просмотре таблицы, для этого же представления, устанавливается QSqlTableModel. Человек реально много раз может переключаться с одной работы на другую. Тем самым модели выделения будут создаваться вновь и вновь. Как-то странно это все.

[SABROG]

Я мало что понял. Вопрос заключается в том почему тролли не сделали автоматическое удаление модели выделения и почему не используют старую модель выделения (старая это какая, которая была заменена новой, но осталась висеть в памяти и должна использоваться, если конструктор новой не сработал или что?).

[Litkevich Yuriy]

старая это какая, которая была заменена новой

да

[SABROG]

Кстати интересно. Написано, что setModel каждый раз устанавливает новую QItemSelectionModel, т.е. старая модель все-таки должна удаляться Qt иначе будет утечка памяти...

[Litkevich Yuriy]

т.е. старая модель все-таки должна удаляться Qt

этого нигде не написано, читай мой пост, там выделено жирным.


П.С. версия Qt в моей подписи

[SABROG]

В 4.4.3 ситуация таже. Все экземпляры QItemSelectionModel удаляются только в момент удаления компонентов на базе QAbstractItemView, т.к. именно он является предком. Т.е. только когда удаляются QColumnView, QHeaderView, QListView, QTableView, и QTreeView. Отсюда можно сделать вывод, что setModel в цикле на одну вьюху рано или поздно закончится плачевно. Я решил провести эксперимент таким образом:

        QTreeView *tree = new QTreeView(this);
        forever
        {
                tree->setModel(0);
        }

Судя по исходникам нулевой указатель вместо модели должен ставить статическую дефолтную модель при этом создавать каждый раз новую selectedModel. Поедание памяти снимал с диспетчера задач:



Затем я изменил исходник таким образом:

        forever
        {
        QTreeView *tree = new QTreeView(this);
                tree->setModel(0);
                delete tree;
        }

Поедание памяти продолжилось, но на этот раз не существенно, где-то на 10-50кб в секунду.

Сообщение отредактировал SABROG - 8.2.2009, 19:34

[Гость_Константин_*]

страшного ничего в этом нет. просто на момент установки новой модели "старая" селекшн-модель может ещё использоваться и, поскольку мы не хотим сегфолта, модель не удаляем (подобное часто встречается в сорцах кутэ) - на мой взгляд это лучше, чем пол-дюжины проверок в каждом методе (привет, гтк)!
если нужно часто "передёргивать" модель-источник и программист твёрдо уверен, что селекшн-модель в момент установки модели-источника нигде не используется, программист может смело делать так:

        QTreeView *tree = new QTreeView(this);
        forever
        {
                QItemSelectionModel* sm = tree->selectionModel();
                tree->setModel(0);
                sm->deleteLater;
        }

[SABROG]

Интересно почему Qt сама не делает deleteLater(). Или почему нет предупреждения о возможной утечке памяти. Скорее всего потому, что вьюха не рассчитана на такое использование модели. Видимо предполагается, что модель должна быть одна-две и вся работа по заполнению/очистки модели должна происходить только с одним экземпляром.

[Litkevich Yuriy]

Интересно почему Qt сама не делает deleteLater().

"старая" селекшн-модель может ещё использоваться

Или почему нет предупреждения о возможной утечке памяти.

\bold{Note:} This function will also create and set a new selection model,
replacing any previously set with setSelectionModel(), but the old selection
model will not be deleted.

тык

[SABROG]

"старая" селекшн-модель может ещё использоваться

ну с deleteLater она удалится после использования, когда управление вернется в режим "ожидания"

but the old selection
model will not be deleted.

Да, но тут как-то сказано вскользь, намеками. Тут не сказано, что об удалении должен заботится пользователь.

Вот какая еще штука есть QObjectCleanupHandler
Если ядро сама удалит какой-то QObject, то он автоматически удаляется из списка.