[Qt 4.4.0, VS2005] Падение в QTreeView при отображении дерева из БД

[Sergeich]

Создаю SQLite базу из одной таблицы вида:
Id  | ParentId | Name
1   | 0 | Item1
2   | 1 | Item11
3   | 1 | Item12
4   | 0 | Item2

Код:

#include <QtGui>
#include <QtSql>

#include "sqltreemodel.h"

void doBranches( int id, const QString& name, int depth )
{
	int d = depth - 1;
	int n = qrand() % 10;
	QSqlQuery query;
	for ( int i = 0; i < n; i++ ) {
		QString str = name + QString('1' + i);
		query.exec( QString("INSERT INTO Object(ParentId, Name) VALUES(%1,'%2')").arg(id).arg(str) );
		query.exec( "SELECT last_insert_rowid()" );
		if ( query.first() && d ) {
			int ni = query.value(0).toInt();
			doBranches( ni, str, d );
		}
	}
}

int main(int argc, char *argv[])
{
    QApplication app(argc, argv);

    QSqlDatabase db = QSqlDatabase::addDatabase("QSQLITE");
	db.setDatabaseName(":memory:");
    if (!db.open()) {
		qFatal("Can't open database");
    }
	QSqlQuery query;
	query.exec("CREATE TABLE Object (Id integer primary key autoincrement, ParentId integer, Name varchar)");
	qsrand(QTime(0,0,0).secsTo(QTime::currentTime()));
	doBranches( 0, "Item", 3 );
	query.exec("CREATE INDEX ParentIdx ON Object(ParentId)");

	SqlTreeModel model(db);
    QTreeView view;
    view.setModel(&model);
    view.setWindowTitle(QObject::tr("SQL Tree Model"));
    view.show();
    return app.exec();
}

Создаю к ней read-only модель (за основу взят пример itemviews/simpletreemodel):

Код:

#include <QtGui>
#include <QtSql>

#include "sqltreemodel.h"

// Database access functions

QVariant SqlTreeModel::getData( int id, int column ) const 
{
	if ( column != 0 )
		return QVariant();
	QSqlQuery query(db);
	query.exec( QString("SELECT Name FROM Object WHERE Id = %1").arg(id) );
	if ( !query.next() ) 
		return QVariant();
	return query.value(0);	 
}

int SqlTreeModel::getParent( int id ) const 
{
	QSqlQuery query(db);
	query.exec( QString("SELECT ParentId FROM Object WHERE Id = %1").arg(id) );
	if ( !query.next() ) 
		return -1;
	return query.value(0).toInt();	 
}

int SqlTreeModel::getChildCount( int parent ) const 
{
	QSqlQuery query(db);
	query.exec( QString("SELECT count(*) FROM Object WHERE ParentId = %1").arg(parent) );
	if ( !query.next() )
		return 0;
	int cnt = query.value(0).toInt();
	return cnt;
}

int SqlTreeModel::getChild( int parent, int index ) const 
{
	QSqlQuery query(db);
	if ( !query.exec( QString("SELECT Id FROM Object WHERE ParentId = %1 ORDER BY Id").arg(parent) ) )
		return 0;
	int i = 0;
	while ( query.next() ) {
		if ( i == index ) {
			int id = query.value(0).toInt();
			return id;
		}
		++i;
	}
	return 0;
}

int SqlTreeModel::getChildIndex( int parent, int child ) const 
{
	QSqlQuery query(db);
	if ( !query.exec( QString("SELECT Id FROM Object WHERE ParentId = %1 ORDER BY Id").arg(parent) )  )
		return -1;
	int index = 0;
	while ( query.next() ) {
		int id = query.value(0).toInt();
		if ( id == child )
			return index;
		++index;
	}
	return -1;
}

//---------------------------------------------------

SqlTreeModel::SqlTreeModel(QSqlDatabase database, QObject *parent)
    : db(database), QAbstractItemModel(parent)
{
}


SqlTreeModel::~SqlTreeModel()
{
}


int SqlTreeModel::columnCount(const QModelIndex &parent) const
{
	return 1;	
}


QVariant SqlTreeModel::data(const QModelIndex &index, int role) const
{
    if (!index.isValid())
        return QVariant();

    if (role != Qt::DisplayRole)
        return QVariant();

	return getData( index.internalId(), index.column() );
}


Qt::ItemFlags SqlTreeModel::flags(const QModelIndex &index) const
{
    if (!index.isValid())
        return 0;

    return Qt::ItemIsEnabled | Qt::ItemIsSelectable;
}

QVariant SqlTreeModel::headerData(int section, Qt::Orientation orientation,
                               int role) const
{
    if (orientation == Qt::Horizontal && role == Qt::DisplayRole)
        return QString("Name");

    return QVariant();
}

QModelIndex SqlTreeModel::index(int row, int column, const QModelIndex &parent) const
{
    if (!hasIndex(row, column, parent))
        return QModelIndex();

    int parentId;

    if (!parent.isValid())
        parentId = 0;
    else
        parentId = parent.internalId();

    int childId = getChild(parentId, row);
    if (childId > 0)
        return createIndex(row, column, childId);
    else
        return QModelIndex();
}

QModelIndex SqlTreeModel::parent(const QModelIndex &index) const
{
    if (!index.isValid())
        return QModelIndex();

    int childId = index.internalId();
    int parentId = getParent(childId);

    if (parentId <= 0)
        return QModelIndex();

	int row = getChildIndex(parentId, childId);
    return createIndex(row, 0, parentId);
}

int SqlTreeModel::rowCount(const QModelIndex &parent) const
{
    if (parent.column() > 0)
        return 0;

	int parentId;
    if (!parent.isValid())
        parentId = 0;
    else
        parentId = parent.internalId();

    return getChildCount(parentId);
}

При раскрытии некоторых узлов дерева падает с ошибкой

Где я накосячил? Проект в аттаче.

[ритт]

SqlTreeModel::getChildIndex и SqlTreeModel::getParent могут вернуть -1
мне кажется, проблема из-за нехватки проверки возвращённого значения, например, в SqlTreeModel::parent

Код:

int row = getChildIndex(parentId, childId);
return createIndex(row, 0, parentId);

[Sergeich]

SqlTreeModel::getChildIndex и SqlTreeModel::getParent могут вернуть -1
мне кажется, проблема из-за нехватки проверки возвращённого значения, например, в SqlTreeModel::parent

Код:

int row = getChildIndex(parentId, childId);
return createIndex(row, 0, parentId);

Да пробовал уже, не помогает. В отладчике точки прерывания на все исключительные ситуации ставил - не попадает он туда!

[ритт]

у тебя валится в QTreeViewPrivate::layout (судя по ассерту), у меня в 4.4.0 - в QTreeView::sizeHintForColumn (и только при раскрытии дочернего списка )
собрал под 4.3.4 - полный порядок
могу посоветовать навестить трэкер

[Sergeich]

собрал под 4.3.4 - полный порядок

Неа... тоже валится, только намного реже

могу посоветовать навестить трэкер

В трекере вроде такого нет. Придется писать в техподдержку.

[Sergeich]

Письмо в техподдержку отменяется Разобрался, в чем косяк: в методе SqlTreeModel::parent неправильно вычислялся номер строки. Исправил на

Код:

QModelIndex SqlTreeModel::parent(const QModelIndex &index) const
{
	if (!index.isValid())
		return QModelIndex();

	int childId = index.internalId();
	int parentId = getParent(childId);

	if (!parentId)
		return QModelIndex();

	int row = getChildIndex(getParent(parentId), parentId);
	return createIndex(row, 0, parentId );
}

и все заработало как надо
Если кому надо, прилепил исправленный код.

[Sergeich]

Пока искал ошибку обнаружил два неприятных момента:
1. QSqlQuery::seek() не перемещается на последнюю запись запроса. Отписался троллям.
2. QTreeView все время обращается к модели, даже в paintEvent! Полный пиздец производительности!

[Alex03]

...
2. QTreeView все время обращается к модели, даже в paintEvent! Полный пиздец производительности!

Так в этом и весь смыст что во вьюхе данных нет, все данные в модели, поэтому при паинте идёт обращение за данными к модели, тут всё нормально, а вот если модель при этом, например, лезет в БД за данными, то это уже другой вопрос.

[Sergeich]

...
2. QTreeView все время обращается к модели, даже в paintEvent! Полный пиздец производительности!

Так в этом и весь смыст что во вьюхе данных нет, все данные в модели, поэтому при паинте идёт обращение за данными к модели, тут всё нормально, а вот если модель при этом, например, лезет в БД за данными, то это уже другой вопрос.

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

[Alex03]

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

Почему вьюха то? Она тока отображает, на то и вьюха.
А вот модель может и кэшировать. В том числе промежуточная порождённая от QxxxxxxProxyModel.

P.S. Кстати, а чёб тролям не написать какойнить QAbstractCacheProxyModel/QStandardItemCacheProxyModel?

[DarkPhoenix]

Подтверждаю написанное trdm, не получается скачать, 404 ошибка.

[pastor]

BW, trdm попросите Sergeich в личке выложить снова

[Admin]

после перезда ченить отвалилось - гляну

[Admin]

атачи грузятся теперь
как оказалось SMF форум не так просто в администрировании - фиг найдешь где есть что

[CroCIV]

Ням, Ням!!!