форма диалога рисуется с задержкой

[boriss]

Добрый день!!
А не поможете вот с таким вопросом ... при вызове формы QDialog, форма рисуется с задержкой в несколько  секунд, прям заметно на экране. Что это может быть?

[Bepec]

Код в студию. описание работы краткое в студию. И чего ты хочешь добиться от программы в студию.

А там и подтянутся тролли и прочая формуная живность.

PS может тормозить изза:
- комп слабый
- просто тормозной
- что то ты неправильно делаешь*
- что то его тормозит
- мб он вообще не тормозит и тебе кажется?

[GreatSnake]

... в студию ... в студию ... в студию.

Прям Поле Чудес... )

[kambala]

наверное в конструкторе диалога разные страшные действия совершаются

[boriss]

приложение работает с БД. На форме 3 вьюхи с QsqlQueryModel. И вот когда вьюхи пустые все нормально работает, а когда с данными то немного с задержкой диалог открывается. Возможно это действительно нормально, но как-то непривычно

наверное в конструкторе диалога разные страшные действия совершаются

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

[boriss]

такое ощущение что сначала рисуется пустой диалог, безо всяких виджетов, а потом уже с виджетами все как-надо

[Bepec]

Проблема: задержка диалога

Я так представляю, у Вас в конструкторе происходит наполнение моделей данными из SQL.

Решение: конструктор очистить от лишнего, загрузку данных перенести в отдельную функцию (в идеале поток), поставить пользователю индикатор загрузки данных из баз/базы.

Проблема: слишком частое обновление баз, или же нечастое(раз в секунду) полное их обновление.

И мысль еще - вы обновление делаете насколько часто? Обновление раз в 20 мс тормозит программу не по детски, если производится полное обновление.

Решение: beginResetModel()/endResetModel() - обновление после "полного получения данных", а не пирамидально (пирамидально - это добавление 1 строки и автоматическое обновление предыдущих).

И код, код желательно. Мы ему плохо не сделаем, я его даже с ложечки покормлю.

[boriss]

Код

C++ (Qt)
#include <QMessageBox>
#include <QSqlError>
#include <QSqlQuery>
 
#include "sprmu.h"
#include "ui_sprmu.h"
 
sprMU::sprMU(QWidget *parent) :
    QDialog(parent),
    ui(new Ui::sprMU)
{
    ui->setupUi(this);
    ui->splitter_2 -> setStretchFactor(0,0);
    ui->splitter_2 -> setStretchFactor(1,1);
    ui->splitter_2 -> setStretchFactor(2,2);
    createButtons();
    //createActions();
 
    // ------- Отображаем Группы МУ
    GroupMUModel = new QSqlQueryModel(this);
    GroupMUModel->setQuery("SELECT MUGroupCode, MUGroupName , MUGroupId "
                           "FROM sprMUGroup "
                           "ORDER BY 1");
    if(GroupMUModel -> lastError().isValid()){
      QMessageBox::critical(this,tr("Ошибка SQL"),GroupMUModel -> lastError().text());
      return;
    }
    ui->GroupMUView -> setModel(GroupMUModel);
 
    // ------- Отображаем Подгруппы МУ
    UnGroupMUModel = new QSqlQueryModel(this);
    UnGroupMUModel -> setQuery("SELECT * "
                               "FROM sprMUUnGroup ");
    if(UnGroupMUModel -> lastError().isValid()){
        QMessageBox::critical(this,tr("Ошибка SQL"),UnGroupMUModel -> lastError().text());
        return;
    }
    ui->UnGroupMUView -> setModel(UnGroupMUModel);
 
    // ------- Отображаем мед. услуги
    MUModel = new QSqlQueryModel(this);
    MUModel -> setQuery("SELECT * FROM sprMU ");
    if(MUModel -> lastError().isValid()){
        QMessageBox::critical(this,tr("Ошибка SQL"),MUModel -> lastError().text());
        return;
    }
    ui->MUView -> setModel(MUModel);
 
}
 
sprMU::~sprMU()
{
    delete ui;
}
 
void sprMU::changeEvent(QEvent *e)
{
    QDialog::changeEvent(e);
    switch (e->type()) {
    case QEvent::LanguageChange:
        ui->retranslateUi(this);
        break;
    default:
        break;
    }
}
 
void sprMU::createButtons(){
    ui->GroupRefrButt -> setStyleSheet("QPushButton {border: none;padding: 4px;} QPushButton:hover {border: 2px outset #8f8f91;border-radius: 4px;} QPushButton:pressed {border: 2px inset #8f8f91;border-radius: 4px;}");
    ui->GroupAddButt -> setStyleSheet("QPushButton {border: none;padding: 4px;} QPushButton:hover {border: 2px outset #8f8f91;border-radius: 4px;} QPushButton:pressed {border: 2px inset #8f8f91;border-radius: 4px;}");
    ui->GroupEditButt -> setStyleSheet("QPushButton {border: none;padding: 4px;} QPushButton:hover {border: 2px outset #8f8f91;border-radius: 4px;} QPushButton:pressed {border: 2px inset #8f8f91;border-radius: 4px;}");
    ui->GroupDelButt -> setStyleSheet("QPushButton {border: none;padding: 4px;} QPushButton:hover {border: 2px outset #8f8f91;border-radius: 4px;} QPushButton:pressed {border: 2px inset #8f8f91;border-radius: 4px;}");
    connect(ui->GroupAddButt,SIGNAL(clicked()),this,SLOT(slotAddGroupMU()));
 
 
}
 
void sprMU::slotAddGroupMU(){
    QMessageBox::information(this,"","");
}
 

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

[Bepec]

Код:

#include <QMessageBox>
#include <QSqlError>
#include <QSqlQuery>

#include "sprmu.h"
#include "ui_sprmu.h"

sprMU::sprMU(QWidget *parent) :
    QDialog(parent),
    ui(new Ui::sprMU)
{
    ui->setupUi(this);
    ui->splitter_2 -> setStretchFactor(0,0);
    ui->splitter_2 -> setStretchFactor(1,1);
    ui->splitter_2 -> setStretchFactor(2,2);
    createButtons();
    //createActions();

// убиваем эти строки, выделенные красным (выносим в отдельную функцию)


    // ------- Отображаем Группы МУ
    GroupMUModel = new QSqlQueryModel(this);
    GroupMUModel->setQuery("SELECT MUGroupCode, MUGroupName , MUGroupId "
                           "FROM sprMUGroup "
                           "ORDER BY 1");
    if(GroupMUModel -> lastError().isValid()){
      QMessageBox::critical(this,tr("Ошибка SQL"),GroupMUModel -> lastError().text());
      return;
    }
    ui->GroupMUView -> setModel(GroupMUModel);

    // ------- Отображаем Подгруппы МУ
    UnGroupMUModel = new QSqlQueryModel(this);
    UnGroupMUModel -> setQuery("SELECT * "
                               "FROM sprMUUnGroup ");
    if(UnGroupMUModel -> lastError().isValid()){
        QMessageBox::critical(this,tr("Ошибка SQL"),UnGroupMUModel -> lastError().text());
        return;
    }
    ui->UnGroupMUView -> setModel(UnGroupMUModel);
  
    // ------- Отображаем мед. услуги
    MUModel = new QSqlQueryModel(this);
    MUModel -> setQuery("SELECT * FROM sprMU ");
    if(MUModel -> lastError().isValid()){
        QMessageBox::critical(this,tr("Ошибка SQL"),MUModel -> lastError().text());
        return;
    }
    ui->MUView -> setModel(MUModel);

Код:

}

sprMU::~sprMU()
{
    delete ui;
}

void sprMU::changeEvent(QEvent *e)
{
    QDialog::changeEvent(e);
    switch (e->type()) {
    case QEvent::LanguageChange:
        ui->retranslateUi(this);
        break;
    default:
        break;
    }
}

void sprMU::createButtons(){
    ui->GroupRefrButt -> setStyleSheet("QPushButton {border: none;padding: 4px;} QPushButton:hover {border: 2px outset #8f8f91;border-radius: 4px;} QPushButton:pressed {border: 2px inset #8f8f91;border-radius: 4px;}");
    ui->GroupAddButt -> setStyleSheet("QPushButton {border: none;padding: 4px;} QPushButton:hover {border: 2px outset #8f8f91;border-radius: 4px;} QPushButton:pressed {border: 2px inset #8f8f91;border-radius: 4px;}");
    ui->GroupEditButt -> setStyleSheet("QPushButton {border: none;padding: 4px;} QPushButton:hover {border: 2px outset #8f8f91;border-radius: 4px;} QPushButton:pressed {border: 2px inset #8f8f91;border-radius: 4px;}");
    ui->GroupDelButt -> setStyleSheet("QPushButton {border: none;padding: 4px;} QPushButton:hover {border: 2px outset #8f8f91;border-radius: 4px;} QPushButton:pressed {border: 2px inset #8f8f91;border-radius: 4px;}");
    connect(ui->GroupAddButt,SIGNAL(clicked()),this,SLOT(slotAddGroupMU()));


}

void sprMU::slotAddGroupMU(){
    QMessageBox::information(this,"","");
}

Убиваем выделенное красным или выносим в отдельную функцию, вызываемую после загрузки формы (или в отдельном потоке).

Соответственно пункт 1 моих и прочих предположений был верен.

PS извиняюсь перед господами, но в /code  цветом не выделяется

[boriss]

Верес, я может чего-то не понимаю ...

Убиваем выделенное красным или выносим в отдельную функцию, вызываемую после загрузки формы (или в отдельном потоке).

Мне надо чтобы данные на форме заполнялись сразу при открытии основной формы ... А доп. форму, или в данном случае сообщение, я вывожу по клику на кнопке уже основной формы. Вот эта доп. форма, а в данном случае сообщение, выводится с небольшой задержкой.

[Bepec]

Уважаемый Борис.
Давайте я вам объясню что вы делаете:

Форма загружается, прорисовывается и ... ждет когда окончатся функции, описанные в конструкторе. У вас в данном случае запрос к БД, который кстати может быть и более длительным(до 10-15 секунд), в зависимости от расположения базы, ее величины и величины канала связи.

Это плохо! Очень плохо.

Мой вариант (мб и не очень простой):

Код:

Ваш_Класс::Конструктор
{
    //  прописываем все ui методы (создание кнопок, формы, настройка их)
    // далее создаем поток загрузки (потомок от QThread в моем случае)
    LoadThread * thread = new QThread();
}

В самом же потоке вы вызываете все свои запросы, и передаете данные в основной поток (при этом интерфейс не виснет, пользователь видит программу с пустыми полями).

Получаете данные из потока, устанавливаете в View, показывая их пользователю. Пользователю так же можно вывести что-то типа  "Запрос к базам данных выполняется бззз...".

Дополнительная форма у вас я думаю работает аналогично ( в конструктор Вы все запихали). Потому и метод решения тот же.

PS по опыту знаю, лучше пользователь видит пустые поля с сообщением о загрузке сразу, чем видит тормоза при загрузке и думает, что тормозит ВАША программа, а не БД.

[boriss]

Спасибо большое за разъяснения. Будем пробовать