Проблемы редактирования DOM модели, Доработка модели к редактируемой Опции

[JohnZ]

В прицепе мин. проект, демонстрирующий проблемы редактирования.
ехе-шник там-же ... Проблемы в диалоге редактирования таблицы (через модель !).
Для того, чтобы к нему добраться, нужно в открытом конфиге (прилагается)
пройти по дереву от корня /->metadata->catalogues-> далее выбрать любой
справочник ( catalogue ) и в нём DblClick на таблице ( element ).
Появится диалог редактирования таблицы. Вопросы ...

1. В диалоге реализовано удаление поля, кнопка Del.
Сделано по подобию примера editabletreemodel.
После удаления, в bool DomItem::removeChildrens(int position, int count)
добавлена очистка хэш-а childItems.clear(), т.к. без этого
нет отоббражения действия в TreeView-e за диалогом, при развёрнутом
дереве полей таблицы.
В принципе к функции удаления претензий нет, работает отлично,
но правомочна-ли очистка хэш-а в плане утечек памяти ?
Ведь указатели на DomItem-ы хранятся только в хэше, а операция
QHash::clear() для QHash<int,DomItem*> childItems; не чистит хранимые
указатели на DomItem *, которые созданы операцией new !?
Если это действительно так, подскажите плз правильную реализацию ...

2. Реализация добавления поля аналогична удалению, в проект не ложил,
т.к. там понятно, через отдельный диалог.
Проблема в реализации bool swapChildrens(int pos1, int pos2);
Для добавления и удаления поля в модели есть соотв-щие ф-ции

    beginInsertRows(parent, position, position + rows - 1);   // !!!
    success = parentItem->insertChildrens(position, rows);
    endInsertRows();      // !!!

и

    beginRemoveRows(parent, position, position + rows - 1);   // !!!
    success = parentItem->removeChildrens(position, rows);
    endRemoveRows();   // !!!

Но в модели нет функций
beginSwapRows(parent, position, position + 1);
endSwapRows();
Как это можно реализовать (с отображением в TreeView-e), как в случае с удалением ?

3. Почему при открытии диалога, строка выбора в TreeView-e скачет хз куда ?
Странно это потому, что в оригинальном проекте, когда TreeView является
MdiChild-ом, такого не наблюдается ... Это так, из спортивного интереса :-)

Заранее благодарен за помощь.

Прикрепленные файлы

 simpledommodel.zip ( 195.06 килобайт ) Кол-во скачиваний: 218

 

[lanz]

1. Хэш никак с этим не связан. У меня все удаляется даже если закоментарить clear()
Память течет, именно потому что вы сказали.
Уберите хэш вообще, насовсем. Здесь он не нужен пока, только все запутывает.
Утекает вот отсюда:

    if (i >= 0 && i < domNode.childNodes().count()) {
        QDomNode childNode = domNode.childNodes().item(i);
        DomItem *childItem = new DomItem(childNode, i, this);
        childItems[i] = childItem;
        return childItem;
    }

Зачем вам параллельная иерархия DomItem которую вы пересоздаете каждый раз когда не найден элемент в хэше? Модель может работать и с кутешным DOM-деревом. Тоже сильно все упростит.

2. http://qt-project.org/doc/qt-4.8/qabstract...l#beginMoveRows
Или можете просто вызвать dataChanged для обоих строк после того как поменяете их внутри своей модели.

3. У меня все работает Не скачет.

[JohnZ]

1. Хэш никак с этим не связан. У меня все удаляется даже если закоментарить clear()

Если закоментарить clear(), то нет "динамики", изменения в дереве будут видны только
после закрытия диалога. Если он есть, то изменения видны даже ЗА диалогом !

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

Хм ... Я об этом даже не задумывался, надо обмозговать ... Может ты и прав ...

Утекает вот отсюда:

    if (i >= 0 && i < domNode.childNodes().count()) {
        QDomNode childNode = domNode.childNodes().item(i);
        DomItem *childItem = new DomItem(childNode, i, this);
        childItems[i] = childItem;
        return childItem;
    }

Всё правильно, я именно об этом "new" и спрашивал. Я уже пробовал вручную чистить элементы из хэш-а,
удаляя вначале все DomItem * и затем childItems.clear(), - вообще вылетает в ошибку доступа к "мусорной"
памяти при перестройке модели, откуда-то из глубин dll-ки ... Даже студия затыкается :-(

Зачем вам параллельная иерархия DomItem которую вы пересоздаете каждый раз когда не найден элемент в хэше?

В QT-шном примере так было сделано. Я взял за основу предполагая, что это наилучший способ работы с DOM-деревом.
Кроме того, оказалось просто расширить эту структуру (DomItem), добавляя элементы для своих нужд ( IDD, IDС и т.д. )

Модель может работать и с кутешным DOM-деревом. Тоже сильно все упростит.

А вот тут поподробнее можно ?
Начиная с какой версии QT есть такая фича ? (у меня 4.5.3)
Примерчик _редактируемой_ DOM-модели в TreeView есть ?
Хотя это и повлечёт солидную переделку проекта, но я за упрощение !
Подскажи где копать, т.е. что брать за основу ? (по аналогии с simpledommodel)
Совсем с нуля опять начинать не хоцца ...

2. http://qt-project.org/doc/qt-4.8/qabstract...l#beginMoveRows
Или можете просто вызвать dataChanged для обоих строк после того как поменяете их внутри своей модели.

beginMoveRows - Дык это начиная с 4.6.0. Видимо придётся переходить на 4.6.х Жаль ...
Линуксы 4-6 летней давности, работающие и поныне на "слабых" машинах, собраны на версии 4.3.х ... 4.5.х.

3. У меня все работает Не скачет.

Спишем на глюк QTreeView в SDI

lanz, Благодарю за помощь !

Сообщение отредактировал JohnZ - 27.11.2014, 18:20