Возврат иконки из модели

[Swa]

Всем привет.
Есть набор данных, которые отображаются в модели, каждый элемент имеет свою иконку. Все иконки хранятся в ресурсах приложения, а объект хранит текстовый идентификатор иконки(имя).
Вопрос в том, как лучше возвращать эту иконку:
так

Код:

QHash<QString, QPixmap> icons;
...
QVariant Model::data(int role) const {
if (role == Qt::DecorationRole)
    return icons[iconName];
}

или так

Код:

QVariant Model::data(int role) const {
if (role == Qt::DecorationRole)
    return new Pixmap(iconName, "PNG");
}

[Old]

Вопрос в том, как лучше возвращать эту иконку:
или так

Код:

QVariant Model::data(int role) const {
if (role == Qt::DecorationRole)
    return new Pixmap(iconName, "PNG");
}

При каждой отрисовке строки загружать заново картинку?

[Bepec]

Всё хуже не загружать новую, а выделять под неё место

Истинно адский код

PS первый вариант нравится больше

[Old]

Всё хуже не загружать новую, а выделять под неё место

От утечки легко можно уйти. А вот загружать ее заново, даже из ресурсов, при каждой отрисовке строки очень накладно.

[Swa]

А вот загружать ее заново, даже из ресурсов, при каждой отрисовке строки очень накладно.

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

[Bepec]

Сделал то ты 

[Swa]

Что сделал я?

[GreatSnake]

Я думал, там как-то более экономно сделано.

Где "там"? Ведь это твой код

[Igors]

Используйте QPixmapCache

[GreatSnake]

Используйте QPixmapCache

Дык у него уже есть свой:

Код:

QHash<QString, QPixmap> icons;

[Igors]

Дык у него уже есть свой:

Код:

QHash<QString, QPixmap> icons;

Ну тоже можно, только ж надо его корректно пополнять

[Swa]

Я имею в виду, что view вызывает QAbstractItemModel::data (QModelIndex &, int role) каждый раз при перерисовке, а могла бы кешировать эти данные и запрашивать только когда они изменились. Но это я просто мечтаю.

[GreatSnake]

Я имею в виду, что view вызывает QAbstractItemModel::data (QModelIndex &, int role) каждый раз при перерисовке, а могла бы кешировать эти данные и запрашивать только когда они изменились. Но это я просто мечтаю.

Интересно на основании чего в твоём случае модель решит, что данные изменились
Ведь модель понятия не имеет про данные, ибо ты их в модель не помещаешь в данном случае вовсе.

[Swa]

Я имел в виду, что view мог бы кешировать эти данные, а обновлять при получении сигнала QAbstractItemModel::dataChanged.

[GreatSnake]

Я имел в виду, что view мог бы кешировать эти данные, а обновлять при получении сигнала QAbstractItemModel::dataChanged.

Мде... как всё запущено. Ещё раз и внимательно читай мой пост выше.