Qt: QAbstractItemModel работает медленно, если я бросаю сигнал layoutChanged(). В чем проблема?

Question

[Robotex]

Я унаследовался от QAbstractItemModel и использую эту модель в TableView в QML.
Я пытаюсь удалить элемент:

bool DownloadListModel::removeDownloadById(const QString &id)
{
    DownloadFile* download = getDownloadById(id);
    int index = getDownloadIndex(download);
    if (download) {
        beginRemoveRows(QModelIndex(),index,index);

        _downloadsList.removeOne(download);
        _downloadsMap.remove(id);
        download->deleteLater();

        _db->deleteDownload(id);

        endRemoveRows();

        return true;
    }
    return false;
}

Это работает, но есть одно но: когда я выбираю в таблице элемент, подсвечивается другой элемент (если удален один, то будет подсвечен элемент непосредственно над выбранным). Поэтому я пытаюсь послать сигнал layoutChanged():

bool DownloadListModel::removeDownloadById(const QString &id)
{
    DownloadFile* download = getDownloadById(id);
    int index = getDownloadIndex(download);
    if (download) {
        emit layoutAboutToBeChanged();
        beginRemoveRows(QModelIndex(),index,index);

        _downloadsList.removeOne(download);
        _downloadsMap.remove(id);
        download->deleteLater();

        _db->deleteDownload(id);

        endRemoveRows();
        emit layoutChanged();

        return true;
    }
    return false;
}

Теперь оно работает правильно, но медленно. В чем проблема? Как это делать правильно?

Answer 1

[xseven]

Сложно сказать, что работает медленно не видя всего кода.
Для начала я бы посоветовал пройтись профилировщиком + дать ссылку на код если возможно.

Вообще несколько странный подход на мой взгляд.
Почему не используется removeRows/removeColumns и их антагонисты insert?
Если так хочется removeById почему не вывести это в виде адаптора?
Несколько проще на мой взгляд по максимуму использовать возможности интерфейса стандартной модели.
Плюс как вычисляется индекс и где сигналы dataChanged? Вы видимо поэтому используете посылку сигналов для layout. Т.к. без посылки dataChanged не стоит надеяться на правильное отображение или выделение.

Comments

[Robotex]

А можно поподробнее про вычисление индекса и что после этого отправлять в dataChanged? layout кидаю, т.к. у меня после удаления во вьюхе индексы ползут, видимо из-за этого.

Код мне достался от предыдущего разработчика по наследству, вот сижу допиливаю.

removeRows/removeColumns - т.е. надо переопределить эти методы и вызывать уже их?

[xseven]

@Robotex
Лучше использовать индекс по умолчанию, т.е. не переопределять, до тех пор пока это действительно не надо.
Индекс добавляется/удаляется при beginInsertRows/beginRemoveRows.
Накидал небольшой пример как переопределить удалять/добавлять
Можно использовать выделение по ctrl (разных элементов) и shift(элементов идущих подряд). Плюс прокомментировал некоторые места
https://github.com/xseven/qa-toaster.git
Проект abstractlist

[Robotex]

Хорошо, спасибо, попробую. А если я меняю запись, и меняется всего одно поле, то лучше добавлять в dataChanged третий аргумент с именем роли или слать также как при добавлении/удалении?
И может подскажете еще с такой проблемой: данные из этой модели подаются в QSortFilterProxyModel, а она пересортировывает полностью весь список, при любом изменении данных (пришлось отключить сортировку). Может быть можно как-то заставить ее следить только за определенной ролью и сортировать только при ее изменении?

[xseven]

@Robotex
Посмотрите пример для начала.
Попробуйте внести небольшие изменения чтобы смоделировать нужное вам поведение.
dataChanged имеет вроде два параметра только индекс начала диапазона данных которые изменились и индекс конца.
По поводу ролей: я по максимуму использую DisplayRole. Иногда checkStateRole и др.
С proxySort обычно удается обойтись переопределением acceptRows, но не очень понял вопрос.

Понимаю, что времени всегда не хватает, но если не читали почитайте qt-project.org/doc/qt-4.8/model-view-programming.html

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

В общем случае ваша стратегия должна быть таковой что:
-Разработать структуру данных которая будет максимально простой в добавлении удалении т.е. например QList внутри QList стоит избегать и т.д.
-Отнаследоваться от необходимой абстрактной модели. Переопределить необходимые функции НЕ ДОБАВЛЯЯ новых не принадлежащих базовому классу (т.е. если в вашей модели есть ф-ция MySuperFunc то у вас что-то идет не так)
-Отобразить (Переопределить вид если нужны сложные манипуляции drag/drop и т.д.)
-Переопределить делегат если нужно сложное отображение
-Добавить модель прослойку если нужно менять порядок(фильтровать и т.д.)

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

[Robotex]

Прошу прощения, оказалось, что в проекте используется не QAbstractItemModel, а QAbstractListModel. И в принципе оно особых нареканий не вызывает, проблемы похоже идут из QSortFilterProxyModel.

А проблема такая:
в QAbstractListModel у нас классы, которые имеют разные поля.
QSortFilterProxyModel сортирует их по определенному полю, а также выполняет фильтрации по определенному полю
Проблема в следующем: если мне нужно, чтобы классы сортировались, скажем по дате добавления, а мы при этом изменим имя, то все записи будут пересортированы. А нужно, чтобы любые изменения игнорились, кроме изменений того поля, по которому ведется сортировка.

И сейчас в проекте используется два QSortFilterProxyModel - результат одного передается на вход другому: один фильтрует по тегам, другой по имени и сортирует по разным полям. Возможно ли сделать две фильтрации одним классом (они могут быть применены одновременно: например, нам надо показать только записи с определенными тегам с определенной маской имени, при этом отсортировать по определенному полю)?

[xseven]

@Robotex
- Да я так и понял, что QAbstractListModel
- QSortFilterProxyModel должен быть один, вернее можно сколько угодно, только проиводительность будет падать резко
- Да, можно:
+ Переопределите filterAcceptRow
+ Для задания выражения фильтрационного используйте setFilterRegExp()
+ Колонку по которой сортировать задайте в функции sort()

[Robotex]

А для автоматической сортировки как колонку задать?

Я так полагаю функцию lesserThan тоже надо переопределить.

[xseven]

@Robotex Cудя по документации, подхода два: явно вызвать sort или переопределить lessThan

[Robotex]

@xseven а с этим вопросом помочь не сможете? QML: как обновить внешний вид TableView после отображения контекстного меню?