[Решено] sql запрос в делегате

[chu]

Есть две таблицы:
1) список элементов с описанием их свойств  items (id,  property1, property2, ...);
2) список операций с элементами  actions (id, item_id, info1, info2, ... ).
В actions.item_id записывается один из items.id.
Хотел написать делегат для второй таблицы, который бы изменял цвет ячейки в зависимости от одного из свойств соответствующего элемента. Для этого переопределил в делегате функцию paint и попытался из неё вызвать sql-запрос, но упустил что функция paint имеет квалификатор const.
Подскажите какой-нибудь другой подход

[GreatSnake]

Делать селект надо хотя бы при создании элемента, а ни как при его отрисовке.

[chu]

В голову пришло только такое решение:

Код:

    model = new QSqlQueryModel();
    model->setQuery("SELECT a.id, a.item_id, i.item_type AS item_type FROM actions a "
                    "INNER JOIN items i ON i.id = a.item_id ");

    MyDelegate *delegate = new MyDelegate(ui->actionsTV);
    ui->actionsTV->setModel(model);
    ui->actionsTV->setItemDelegateForColumn(2,delegate);
    ui->actionsTV->resizeColumnsToContents();

Сменил QSqlTableModel на QSqlQueryModel, во вью таким образом появился отдельный столбец для отображения цвета
В делегате:

Код:

void AnswerDelegate::paint(QPainter *painter, const QStyleOptionViewItem &option, const QModelIndex &index) const
{
    int id = index.data(Qt::DisplayRole).toInt();
    if (option.state & QStyle::State_Selected)
             painter->fillRect(option.rect, option.palette.highlight());
    if (option.state & QStyle::State_Selected)
             painter->setBrush(option.palette.highlightedText());
         else
             painter->setBrush(QBrush(colors.at(id)));
    painter->fillRect(option.rect, painter->brush());
}

Но мне этот метод не нравится: отдельная колонка для цвета, уход от QSqlTableModel. В данном случае для меня это не критично, но  на будущее хотелось бы знать, можно ли реализовать по другому...

[GreatSnake]

А кто мешает установить Qt::BackgroundRole и не городить огород?

[chu]

А кто мешает установить Qt::BackgroundRole и не городить огород?

Мешает недостаток знаний. Мне модель свою писать и там переопределять роль в методе QVariant MyModel::data(const QModelIndex &index, int role) const?  Но он тоже константный.

[chu]

Не смотря на то что метод data константный, запрос из него высылать можно, и это отлично
GreatSnake, спасибо!

[GreatSnake]

Мешает недостаток знаний. Мне модель свою писать и там переопределять роль в методе QVariant MyModel::data(const QModelIndex &index, int role) const?  Но он тоже константный.

Зачем нужна своя модель для изменения цвета ячейки
На этапе создания элементов рассчитывай цвет и выставляй его через QAbstractItemModel::setData() с Qt::BackgroundRole.

Не смотря на то что метод data константный, запрос из него высылать можно, и это отлично

Делать селект из data() тоже неразумно, т.к. этот метод вызывается при отрисовке и не только, т.е. очень часто.

[chu]

Если я правильно понял, нужно реализовать что-то такое:

Код:

    model = new QSqlTableModel();
    model->setTable("actions");
    model->setSort(model->fieldIndex("id"),Qt::AscendingOrder);
    model->select();
    int row(0);
    QSqlQuery q;
    q.exec(QString("SELECT i.item_type FROM items i "
                   "INNER JOIN actions a ON i.id = a.item_id ORDER BY a.id"));
    while(q.next()) {
        int item_type = q.value(0).toInt();
        qDebug() << model->setData(model->index(row++,1),QBrush(colors.at(item_type)), Qt::BackgroundRole);
        qDebug() << model->lastError() << row << colors.at(item_type);
    }

Но setData возвращает false, model->lastError() ошибки не выдает. Что не так?

[GreatSnake]

Но setData возвращает false, model->lastError() ошибки не выдает. Что не так?

Обычно "setData возвращает false", если её подсовывают левый index.
Сколько колонок в model? Случайно не 1?

[chu]

Но setData возвращает false, model->lastError() ошибки не выдает. Что не так?

Обычно "setData возвращает false", если её подсовывают левый index.
Сколько колонок в model? Случайно не 1?

нет. колонок 6

[GreatSnake]

см. что возвращает model->index(row++,1).
И непонятно почему у тебя row, который применяется к таблице "actions" рассчитывается от размерности "items".
Чего-то с логикой не то.

PS. хотя судя по "INNER JOIN actions" количество строк вроде как должно быть одинаковым.
Распечатай размерность модели после model->select():

Код

C++ (Qt)
qDebug() << model->rowCount() << model->columnCount()

[chu]

Код:

26 6 
QModelIndex(0,4,0x0,QSqlTableModel(0x11798d0) ) 
QModelIndex(1,4,0x0,QSqlTableModel(0x11798d0) ) 
QModelIndex(2,4,0x0,QSqlTableModel(0x11798d0) ) 
...

[chu]

С индексами все в порядке, потому что когда я применяю setData с Qt::EditRole все корректно срабатывает

Код:

    qDebug() << model->setData(model->index(0,1), 123, Qt::EditRole);

 а вот с Qt::BackgroundRole setData возвращает false

Код:

    qDebug() << model->setData(model->index(0,1),QBrush(Qt::green, Qt::SolidPattern), Qt::BackgroundRole);

Пробовал еще вызывать setData c FontRole и DisplayRole - так же возвращает false.
С чем это может быть связано?

[GreatSnake]

QModelIndex(2,4,0x0,QSqlTableModel(0x11798d0) )

model->index(row++,1)

Что-то странное... Как при запросе индекса для колонки 1 модель тебе отдаёт индекс с колонкой 4

[chu]

QModelIndex(2,4,0x0,QSqlTableModel(0x11798d0) )

model->index(row++,1)

Что-то странное... Как при запросе индекса для колонки 1 модель тебе отдаёт индекс с колонкой 4

запрос для 4ой колонки делал