Загрузка процессора 50% при обработке события QEvent::Paint

[radislav72]

Старался сделать форму ввода с подсвечиваемыми полями:
http://www.prog.org.ru/topic_13273_0.html

Все работает хорошо, но при постоянном перерисовывании окна загрузка процессора возрастает до 50%. Как-то не хорошо получается. Форма ввода достаточно большая- до 30 полей.

Как правильней сделать, чтобы и поля выделялись, и процессор зря не нагружался?

Код такой:

Код:

bool mem_new_phpDialog::eventFilter(QObject *o, QEvent *e)
{
    QDate limDate;
    limDate.setDate(1900, 1, 1);

    if(e->type() == QEvent::Paint)
    {
        if(o->property("requiredField").toBool())
        {
            QWidget * w = dynamic_cast<QWidget *>(o);
            if(w && w->parentWidget()) {
                w->parentWidget()->update(QRect(w->pos(), w->size()).adjusted(-2, -2, 2, 2));
            }
        } else {
            QPainter p(qobject_cast<QWidget *>(o));
            QPen pen_;
            pen_.setWidth(1);
            pen_.setColor(QColor(255, 183, 183)); // Set border color
            p.setPen(pen_);
            p.setRenderHint(QPainter::Antialiasing);

            bool ready_ = true;

            foreach(QObject * obj, o->children())
            {
                if(obj->property("requiredField").toBool()) // Check widget
                {

                    if(
                            (obj->inherits("QLineEdit") && obj->property("text").toString().isEmpty()) ||
                            (obj->inherits("QComboBox") && obj->property("currentText").toString().isEmpty()) ||
                            (obj->inherits("QComboBox") && obj->property("currentText").toString().toLower() == tr("- не вказано -")) ||
                            (obj->inherits("QDateEdit") && obj->property("date").toDate() <= limDate) ||
                            (obj->inherits("QGroupBox") && obj->property("objectName").toString() == "gbSex" && ui->rbUnknown->isChecked())
                            )
                    {
                        ready_ &= false;
                        QWidget * w = dynamic_cast<QWidget *>(obj);
                        if(w)
                        {
                            QRect r(w->pos(), w->size());
                            p.drawRect(r.adjusted(-1, -1, 1, 1));
                        }
                    }
                }
            }
        }
    }
    return QDialog::eventFilter(o, e);
}

С Qt только начинаю, так что извиняюсь, если вопрос примитивный...

[GreatSnake]

Это фильтр будет отрабатывать для _всех_ событий _всех_ объектов созданных на диалоге.
Поэтому обработка QEvent::Paint будет также отрабатывать для всех объектов.
Поэтому нужно убрать ваш foreach и вставить проверку для каждого типа в отдельности.
И держать limDate, который используется только для QDateEdit как минимим неразумно.
Вызывать update() из paintEvent тоже неразумно, т.к. порождается новый QEvent::Paint.
Вообщем у вас получился hack причём _очень_ тяжелый для Qt.

[fuCtor]

Старался сделать форму ввода с подсвечиваемыми полями:
http://www.prog.org.ru/topic_13273_0.html

Все работает хорошо, но при постоянном перерисовывании окна загрузка процессора возрастает до 50%. Как-то не хорошо получается. Форма ввода достаточно большая- до 30 полей.

Как правильней сделать, чтобы и поля выделялись, и процессор зря не нагружался?

Код такой:

Код:

bool mem_new_phpDialog::eventFilter(QObject *o, QEvent *e)
{
    QDate limDate;
    limDate.setDate(1900, 1, 1);

    if(e->type() == QEvent::Paint)
    {
        if(o->property("requiredField").toBool())
        {
            QWidget * w = dynamic_cast<QWidget *>(o);
            if(w && w->parentWidget()) {
                w->parentWidget()->update(QRect(w->pos(), w->size()).adjusted(-2, -2, 2, 2));
            }
        } else {
            QPainter p(qobject_cast<QWidget *>(o));
            QPen pen_;
            pen_.setWidth(1);
            pen_.setColor(QColor(255, 183, 183)); // Set border color
            p.setPen(pen_);
            p.setRenderHint(QPainter::Antialiasing);

            bool ready_ = true;

            foreach(QObject * obj, o->children())
            {
                if(obj->property("requiredField").toBool()) // Check widget
                {

                    if(
                            (obj->inherits("QLineEdit") && obj->property("text").toString().isEmpty()) ||
                            (obj->inherits("QComboBox") && obj->property("currentText").toString().isEmpty()) ||
                            (obj->inherits("QComboBox") && obj->property("currentText").toString().toLower() == tr("- не вказано -")) ||
                            (obj->inherits("QDateEdit") && obj->property("date").toDate() <= limDate) ||
                            (obj->inherits("QGroupBox") && obj->property("objectName").toString() == "gbSex" && ui->rbUnknown->isChecked())
                            )
                    {
                        ready_ &= false;
                        QWidget * w = dynamic_cast<QWidget *>(obj);
                        if(w)
                        {
                            QRect r(w->pos(), w->size());
                            p.drawRect(r.adjusted(-1, -1, 1, 1));
                        }
                    }
                }
            }
        }
    }
    return QDialog::eventFilter(o, e);
}

С Qt только начинаю, так что извиняюсь, если вопрос примитивный...

Код мой, но не обратил внимание на нагрузку сначала.

Убери кусок

Код

C++ (Qt)
            QWidget * w = dynamic_cast<QWidget *>(o);
            if(w && w->parentWidget()) {
                w->parentWidget()->update(QRect(w->pos(), w->size()).adjusted(-2, -2, 2, 2));
            }
 

и вместо установки фильтра на контейнер и на поля, конекть сигнал об изменении содержимого поля со слотом Update() контейнера и будет все нормально тогда. Но фильтр на контейнера должен оставаться. У себя уже исправил.

[radislav72]

Большое спасибо за подсказки, все работает хорошо... Все-таки без опыта работы в Qt самому разбираться со всем трудоемко, а так, на форуме- намного быстрее получается.