IscanderChe
IscanderChe26 июля 2019 г. 13:52

Проект Simple Tracker. Часть 5: сервер. Модель данных задач и представление

Рассмотрим подробно таблицу задач.

По требованиям, указанным вначале, таблица должна была выглядеть так.

На момент разработки стало понятно, что логика работы отдельно стоящего выпадающего списка становится сложной, и было принято решение реализовать его через делегата. Поэтому таблица теперь выглядит так.

Вначале реализуем делегата, потом займёмся заливкой цветом и запретом редактирования ячеек.


Для этого надо унаследовать QStyledItemDelegate и перезаписать несколько его методов.

// comboboxdelegate.h

#ifndef COMBOBOXDELEGATE_H
#define COMBOBOXDELEGATE_H

// Подключаем main.h, откуда нам понадобится и база данных, и список состояний задач
#include "main.h"
#include <QStyledItemDelegate>

class ComboBoxDelegate : public QStyledItemDelegate
{
    Q_OBJECT

public:
    ComboBoxDelegate();

    QWidget* createEditor(QWidget* parent, const QStyleOptionViewItem& option,
                          const QModelIndex& index) const override;

    void setEditorData(QWidget* editor, const QModelIndex& index) const override;

    void setModelData(QWidget* editor, QAbstractItemModel* model,
                      const QModelIndex& index) const override;

    void updateEditorGeometry(QWidget* editor,
        const QStyleOptionViewItem& option, const QModelIndex& index) const override;

    void paint(QPainter* painter,
        const QStyleOptionViewItem& option, const QModelIndex& index) const override;

private slots:
    void changedComboBox(int index);
};

#endif // COMBOBOXDELEGATE_H

Реализуем указанные методы.

// comboboxdelegate.cpp

#include "comboboxdelegate.h"
#include "customsortfilterproxymodel.h"
#include <QComboBox>
#include <QApplication>
#include <QMessageBox>

ComboBoxDelegate::ComboBoxDelegate()
{
}

// Создаём комбобокс
QWidget* ComboBoxDelegate::createEditor(QWidget* parent,
                                        const QStyleOptionViewItem& /*option*/,
                                        const QModelIndex& /*index*/) const
{
    QComboBox* comboBox = new QComboBox(parent);

    // Загружаем список состояний задач
    foreach(QString status, statusList)
        comboBox->addItem(status);

    comboBox->setCurrentIndex(0);

    // Создаём соединение между сигналом изменения индекса комбобокса и его обработчиком
    connect(comboBox, QOverload<int>::of(&QComboBox::currentIndexChanged),
            this, &ComboBoxDelegate::changedComboBox);

    return comboBox;
}

// Устанавливаем данные для комбобокса
void ComboBoxDelegate::setEditorData(QWidget* editor, const QModelIndex& index) const
{
    // Если поле 5 (колонка "Состояние")
    if(index.column() == 5)
    {
        // читаем данные из модели
        QString value = index.model()->data(index, Qt::EditRole).toString();
        QComboBox* comboBox = static_cast<QComboBox*>(editor);

        for(int i = 0; i < statusList.size(); ++i)
        {
            if(value == statusList.at(i))
                // и устанавливаем соответствующий индекс
                comboBox->setCurrentIndex(i);
        }
    }
}

// Записываем данные в модель (через базу данных)
void ComboBoxDelegate::setModelData(QWidget* editor, QAbstractItemModel* model,
                                    const QModelIndex& index) const
{
    // Если поле 5 (колонка "Состояние")
    if(index.column() == 5)
    {
        QComboBox* edit = static_cast<QComboBox*>(editor);
        QString status = edit->currentText();
        // Получаем исходную модель
        CustomSortFilterProxyModel* filterModel =
            static_cast<CustomSortFilterProxyModel*>(model);
        int row = index.row();
        // Получаем из модели поле с номером задачи
        QModelIndex idIndex = filterModel->index(row, 0);
        // Получаем номер задачи
        int id = filterModel->data(idIndex, Qt::DisplayRole).toInt();
        // Если состояние задачи переведено в "закрыта"
        if(status == "закрыта")
            // закрыть задачу
            database->closeTask(id, "manual");
        else
            // иначе обновить состояние задачи
            database->updateStatusTask(id, status);
    }
}

// Переустанавливаем геометрию комбобокса
void ComboBoxDelegate::updateEditorGeometry(QWidget* editor,
    const QStyleOptionViewItem& option, const QModelIndex& /*index*/) const
{
    editor->setGeometry(option.rect);
}

// Перерисовываем комбобокс
void ComboBoxDelegate::paint(QPainter* painter, const QStyleOptionViewItem& option,
                             const QModelIndex& index) const
{
    if(index.column() == 5)
    {
        QStyleOptionComboBox comboBoxStyleOption;
        comboBoxStyleOption.state = option.state;
        comboBoxStyleOption.rect = option.rect;
        comboBoxStyleOption.currentText = index.data(Qt::EditRole).toString();

        QApplication::style()->drawComplexControl(QStyle::CC_ComboBox, &comboBoxStyleOption, painter, 0);
        QApplication::style()->drawControl(QStyle::CE_ComboBoxLabel, &comboBoxStyleOption, painter, 0);

        return;
    }

    QStyledItemDelegate::paint(painter,option, index);
}

// Отправка сигнала модели, что данные должны быть записаны после изменения
// состояния комбобокса
void ComboBoxDelegate::changedComboBox(int /*index*/)
{
    emit commitData(qobject_cast<QComboBox*>(sender()));
}

Реализовалось не совсем гладко. Осталась беда с доступностью выпадающего списка: он начинает нормально работать только после двойного щелчка на нём. И всё бы ничего, но на двойной щелчок у меня подвешено редактирование задачи, диалоговое окно которого и появляется. После закрытия окна можно спокойно менять состояние задачи.

Теперь займёмся цветом и ячейками. Здесь необходимо унаследовать соответствующую модель данных.

// customsortfilterproxymodel.h

#ifndef CUSTOMSORTFILTERPROXYMODEL_H
#define CUSTOMSORTFILTERPROXYMODEL_H

// Подключаем main.h, из которого нам понадобятся список состояний и соответствующих им цветов
#include "main.h"
#include <QSortFilterProxyModel>

class CustomSortFilterProxyModel : public QSortFilterProxyModel
{
public:
    CustomSortFilterProxyModel();

    Qt::ItemFlags flags(const QModelIndex& index) const override;

    QVariant data(const QModelIndex& idx, int role) const override;
};

#endif // CUSTOMSORTFILTERPROXYMODEL_H
// customsortfilterproxymodel.cpp

#include "customsortfilterproxymodel.h"
#include <QBrush>
#include <QColor>

CustomSortFilterProxyModel::CustomSortFilterProxyModel()
{
}

// Устанавливаем для всех ячеек, кроме столбца "Состояние", свойства
// доступности ячеек и возможности их выбора, редактирование ячеек невозможно
Qt::ItemFlags CustomSortFilterProxyModel::flags(const QModelIndex& index) const
{
    if(index.isValid())
    {
        if(index.column() != 5)
        {
            Qt::ItemFlags flags = QSortFilterProxyModel::flags(index);
            flags = Qt::ItemIsEnabled | Qt::ItemIsSelectable;
            return flags;
        }
    }

    return QSortFilterProxyModel::flags(index);
}

// Устанавливаем для всех ячеек в строке цвет в зависимости от состояния задачи
QVariant CustomSortFilterProxyModel::data(const QModelIndex& idx, int role) const
{
    if(idx.isValid())
    {
        if(role == Qt::BackgroundRole)
        {
            for(int i = 0; i < statusList.size(); ++i)
            {
                if(QSortFilterProxyModel::data(this->index(idx.row(), 5)).toString() == statusList.at(i))
                    return QBrush(QColor(colorList.at(i)));
            }
        }
        else if(role == Qt::DisplayRole)
            return QSortFilterProxyModel::data(idx);
    }

    return QSortFilterProxyModel::data(idx, role);
}

Теперь вид таблицы соответствует обновлённым требованиям.

Рекомендуем хостинг TIMEWEB
Рекомендуем хостинг TIMEWEB
Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.

Вам это нравится? Поделитесь в социальных сетях!

Evgenii Legotckoi
  • 26 июля 2019 г. 15:37
  • (ред.)

По-моему, это всё-таки уже часть 5.

Вот здесь у вас такой код

            Qt::ItemFlags flags = QSortFilterProxyModel::flags(index);
            flags = Qt::ItemIsEnabled | Qt::ItemIsSelectable;
            return flags;

В данном случае вызывать метод базового класс не имеет смысла, поскольку вы всё равно полностью перезаписываете флаги, а не добавляете новые к тем, которые вернул метод базового класса. То есть можно просто так написать.

return Qt::ItemIsEnabled | Qt::ItemIsSelectable;

Вы придерживаетесь какого-то код стайла с ограничение количества колонок на строку?

        CustomSortFilterProxyModel* filterModel =
            static_cast<CustomSortFilterProxyModel*>(model);

У нас в проекте принято именовать через enum колонки в модели данных, чтобы не было вот таких магических чисел

if(index.column() != 5)

Такое именование может выглядеть следующим образом.

class CustomSortFilterProxyModel : public QSortFilterProxyModel
{
public:
    enum EColumn 
    {
        E_COLUMN__BEGIN = 0,
        E_COLUMN_NUMBER = E_COLUMN__BEGIN,
        E_COLUMN_DATE,
        E_COLUMN_TYPE,
        E_COLUMN_DESCRIPTION,
        E_COLUMN_STATE,
        E_COLUMN_DATE_OF_CLOSING,
        E_COLUMN_REVISION,
        E_COLUMN__END
    }
    CustomSortFilterProxyModel();

    Qt::ItemFlags flags(const QModelIndex& index) const override;

    QVariant data(const QModelIndex& idx, int role) const override;
};

Тогда такой код будет выглядеть так, это лучше, чем магическое число.

if(index.column() != E_COLUMN_DATE_OF_CLOSING)

Плюс я бы объединил проверки

 if (index.isValid() && index.column() != 5)

Вообще я думаю, что весь этот метод можно переписать тогда так

Qt::ItemFlags CustomSortFilterProxyModel::flags(const QModelIndex& index) const
{
    return (index.isValid() && index.column() != E_COLUMN_DATE_OF_CLOSING) ? (Qt::ItemIsEnabled | Qt::ItemIsSelectable) : QSortFilterProxyModel::flags(index);
}

Если не будет компилироваться, то тогда так

Qt::ItemFlags CustomSortFilterProxyModel::flags(const QModelIndex& index) const
{
    if (index.isValid() && index.column() != E_COLUMN_DATE_OF_CLOSING) 
    {
        return Qt::ItemIsEnabled | Qt::ItemIsSelectable;
    }
    return QSortFilterProxyModel::flags(index);
}

Вот этот метод стоит переписать так

QVariant CustomSortFilterProxyModel::data(const QModelIndex& idx, int role) const
{
    // Если индекс невалидный, то вы всё равно ничего хорошего дальше не получите даже из базового метода.
    if (!idx.isValid())
    {
        // Поэтому можно смело возвращать невалидный QVariant и по большей части не беспокоиться
        return QVariant();
    }

    if (role == Qt::BackgroundRole)
    {
        for (int i = 0; i < statusList.size(); ++i)
        {
            if (QSortFilterProxyModel::data(this->index(idx.row(), 5)).toString() == statusList.at(i))
            {
                return QBrush(QColor(colorList.at(i)));
            }
        }
    }

    // В вашем коде условие на Qt::DisplayRole бесполезное, поскольку role имеет аргумент по умолчанию Qt::DisplayRole,
    // то есть если вы даже не будете передавать ту роль в метод, то она всё равно туда попадёт, то есть просто лишняя проверка в коде
    return QSortFilterProxyModel::data(idx, role);
}
IscanderChe
  • 26 июля 2019 г. 16:38

Да, часть 5, поправил.

В данном случае вызывать метод базового класс не имеет смысла, поскольку вы всё равно полностью перезаписываете флаги, а не добавляете новые к тем, которые вернул метод базового класса. То есть можно просто так написать.

Согласен.

Вы придерживаетесь какого-то код стайла с ограничение количества колонок на строку?

Да, стараюсь вписаться в 80 символов. Если разделить окно просмотра кода на два экрана по вертикали, это актуально.

У нас в проекте принято именовать через enum колонки в модели данных, чтобы не было вот таких магических чисел

Согласен, что нужны enumы. Только я думаю, что будет лучше вынести их глобально куда-нибудь, т.к. у меня эти же магические числа используются и в теле основного класса.

Вообще я думаю, что весь этот метод можно переписать тогда так
return (index.isValid() && index.column() != E_COLUMN_DATE_OF_CLOSING) ? (Qt::ItemIsEnabled | Qt::ItemIsSelectable) : QSortFilterProxyModel::flags(index);

От такого однострочника у меня голова лопается.)) Лучше второй вариант.

Вот этот метод стоит переписать так

Согласен.

Спасибо за подробный разбор кода!

Evgenii Legotckoi
  • 26 июля 2019 г. 16:53
  • (ред.)

Да, стараюсь вписаться в 80 символов. Если разделить окно просмотра кода на два экрана по вертикали, это актуально.

Понимаю, к сожалению иногда названия методов или классов становятся достаточно большими, чтобы проект комфортно поддерживался, то есть сокращения неприемлемы, что актуально в крупном проекте. В тоже время код-стайл может не приветствовать подобный перенос того, что справа от оператора = на следующую строку.

Согласен, что нужны enumы. Только я думаю, что будет лучше вынести их глобально куда-нибудь, т.к. у меня эти же магические числа используются и в теле основного класса.

Это пока у вас одна модель данных на весь проект, а когда в проекте около 1000 моделей, то тогда лучше, чтобы каждая модель данных имела свой набор наименований колонок, которые бы подчинялись определённым правилам. Тем более, что наименования колонок имеют прямое отношение к модели данных, поэтому скорее другой программист ожидал бы видеть такой enum в области модели, чем где-то глобально.

Вы можете просто подключать заголовочный файл модели и использовать enum так

CustomSortFilterProxyModel::E_COLUMN_DATE_OF_CLOSING

Таким образом у вас будет информация, к какой области кода непосредственно относится колонка.

IscanderChe
  • 26 июля 2019 г. 17:16

Вы можете просто подключать заголовочный файл модели и использовать enum так

Да, так даже лучше. Спасибо.

Комментарии

Только авторизованные пользователи могут публиковать комментарии.
Пожалуйста, авторизуйтесь или зарегистрируйтесь
B

C++ - Тест 002. Константы

  • Результат:16баллов,
  • Очки рейтинга-10
B

C++ - Тест 001. Первая программа и типы данных

  • Результат:46баллов,
  • Очки рейтинга-6
FL

C++ - Тест 006. Перечисления

  • Результат:80баллов,
  • Очки рейтинга4
Последние комментарии
k
kmssr9 февраля 2024 г. 5:43
Qt Linux - Урок 001. Автозапуск Qt приложения под Linux как сделать автозапуск для флэтпака, который не даёт создавать файлы в ~/.config - вот это вопрос ))
АК
Анатолий Кононенко5 февраля 2024 г. 12:50
Qt WinAPI - Урок 007. Работаем с ICMP Ping в Qt Без строки #include <QRegularExpressionValidator> в заголовочном файле не работает валидатор.
EVA
EVA25 декабря 2023 г. 21:30
Boost - статическая линковка в CMake проекте под Windows Ошибка LNK1104 часто возникает, когда компоновщик не может найти или открыть файл библиотеки. В вашем случае, это файл libboost_locale-vc142-mt-gd-x64-1_74.lib из библиотеки Boost для C+…
J
JonnyJo25 декабря 2023 г. 19:38
Boost - статическая линковка в CMake проекте под Windows Сделал всё по-как у вас, но выдаёт ошибку [build] LINK : fatal error LNK1104: не удается открыть файл "libboost_locale-vc142-mt-gd-x64-1_74.lib" Хоть убей, не могу понять в чём дел…
G
Gvozdik19 декабря 2023 г. 8:01
Qt/C++ - Урок 056. Подключение библиотеки Boost в Qt для компиляторов MinGW и MSVC Для решения твой проблемы добавь в файл .pro строчку "LIBS += -lws2_32" она решит проблему , лично мне помогло.
Сейчас обсуждают на форуме
P
Pisych27 февраля 2023 г. 15:04
Как получить в массив значения из связанной модели? Спасибо, разобрался:))
AC
Alexandru Codreanu19 января 2024 г. 22:57
QML Обнулить значения SpinBox Доброго времени суток, не могу разобраться с обнулением значение SpinBox находящего в делегате. import QtQuickimport QtQuick.ControlsWindow { width: 640 height: 480 visible: tr…
BlinCT
BlinCT27 декабря 2023 г. 19:57
Растягивать Image на парент по высоте Ну и само собою дял включения scrollbar надо чтобы был Flickable. Так что выходит как то так Flickable{ id: root anchors.fill: parent clip: true property url linkFile p…
Дмитрий
Дмитрий10 января 2024 г. 15:18
Qt Creator загружает всю оперативную память Проблема решена. Удалось разобраться с помощью утилиты strace. Запустил ее: strace ./qtcreator Начал выводиться весь лог работы креатора. В один момент он начал считывать фай…
Evgenii Legotckoi
Evgenii Legotckoi12 декабря 2023 г. 17:48
Побуквенное сравнение двух строк Добрый день. Там случайно не высылается этот сигнал textChanged ещё и при форматировани текста? Если решиать в лоб, то можно просто отключать сигнал/слотовое соединение внутри слота и …

Следите за нами в социальных сетях