Проблема с QBrush, реагирует только на синюю компоненту. Опции Подписка на тему Сообщить другу Версия для печати Скачать тему Подписка на этот форум Режимы отображения Переключить на: Древовидный Стандартный Переключить на: Линейный

[Andrewshkovskii]

Есть paintEvent :

void Test1Widget::paintEvent(QPaintEvent * event)
{
    PaintForTest1 = new QPainter(this);
    PaintForTest1->setRenderHint(QPainter::Antialiasing, true);
    QRgb rgb;
    rgb = (this->RedSlider->value(), this->GreenSlider->value(), this->BlueSlider->value());
    QColor mycolor(rgb);
    PaintForTest1->setBrush(QBrush(mycolor, Qt::SolidPattern));
    PaintForTest1->drawRect(1, 1, 100, 100);
}

3 слайдера(их значения от 0 до 255) соединены с виджетом вот так :

void Test1Widget::SetConnections()
{
connect(RedSlider,SIGNAL(valueChanged(int)),this,SLOT(update()));
connect(GreenSlider,SIGNAL(valueChanged(int)),this,SLOT(update()));
connect(BlueSlider,SIGNAL(valueChanged(int)),this,SLOT(update()));
}

Проблема : QBrush реагирует только на синюю компоненту цвета. остальные игнорирует.почему?

[SABROG]

rgb = (this->RedSlider->value(), this->GreenSlider->value(), this->BlueSlider->value());

Похоже у меня брешь в знаниях C++. Что сия запись означает? Есть догадка, что это что-то типа объявления 3х безымянных переменных, а скобки играют ту же роль что и в математических операциях, может быть компилер и берет только последнюю. Особенно странно становится, когда я узнаю, что QRgb не класс и не структура, а алиас типа для unsigned int, в котором кстати 4 байта. Альфаканал еще.

На warning'и компилятора вообще внимания не обращаешь?

warning: left-hand operand of comma has no effect
warning: right-hand operand of comma has no effect

Сообщение отредактировал SABROG - 1.3.2009, 16:19

[kwisp]

Andrewshkovskii,

PaintForTest1 = new QPainter(this);

подозреваю что PaintForTest1 член класса.
если так то под него каджый раз память выделяться будет при каждом новом вызове paintEvent.
а где память освобождается?

QRgb rgb;
rgb = (this->RedSlider->value(), this->GreenSlider->value(), this->BlueSlider->value());
QColor mycolor(rgb);
PaintForTest1->setBrush(QBrush(mycolor, Qt::SolidPattern));

может легче написать

 PaintForTest1->setBrush(QBrush(QColor(this->RedSlider->value(), this->GreenSlider->value(), this->BlueSlider->value()));

[Litkevich Yuriy]

rgb = (this->RedSlider->value(), this->GreenSlider->value(), this->BlueSlider->value());

надо заменить на:

rgb = qRgb(this->RedSlider->value(), this->GreenSlider->value(), this->BlueSlider->value());

и вроде всё

подозреваю что PaintForTest1 член класса.
если так то под него каджый раз память выделяться будет при каждом новом вызове paintEvent.
а где память освобождается?

хорошее замечание.

[kwisp]

извините великодушно.
зачем вообще этот rgb в данном случае????
нужен цвет отрисовки.
это не суть важно конечно...

[SABROG]

извините великодушно.
зачем вообще этот rgb в данном случае????
нужен цвет отрисовки.
это не суть важно конечно...

Не нужен конечно, т.к. конструктор QBrush принимает QColor, а тот в свою очередь умеет r,g,b параметры брать. К тому Qt::SolidPattern стоит по-умолчанию, его не обязательно прописывать.

Сообщение отредактировал SABROG - 1.3.2009, 17:25

[kwisp]

Не нужен конечно

+1
на что я и намекаю.
ведь это обработчик события отрисовки тут необходимо минимум лишней нагрузки дать. зачем вызывать лишние конструкторы.
вот если PaintForTest1 член класса .
зачем при каждой отрисовке делать ему PaintForTest1->setRenderHint(QPainter::Antialiasing, true);
можно в конструкторе класса задать это свойство и все эффект тот же или я ошибаюсь в чем то?

потом стрнно как вообще рисует что то.... если PaintForTest1 не разрушается по выходу из функции и ему не делают end()
.....

A painter is activated by the begin() function and the constructor that takes a QPaintDevice argument. The end() function, and the destructor, deactivates it.

Сообщение отредактировал kwisp - 1.3.2009, 17:41

[Andrewshkovskii]

Ох, раскритиковали)
Насчет создания/уничтожения - это же локальные переменные метода, они создаются при вызове и уничтожаются после выполнения метода, насколько мне известно.

ЛОКАЛЬНЫЕ переменные подразделяются на АВТОМАТИЧЕСКИЕ и СТАТИСТИЧЕСКИЕ.

Автоматические переменные уничтожаются сразу после завершения того блока, в котором они описаны.

Автоматические переменные описываются со словом auto.

auto float с;

По умолчанию все переменные являются автоматическими, поэтому префикс auto можно не указывать.

Потом,если не создавать каждый раз в методое QPainter, то никакой отрисовки вообще не происходит(по крайней мере у меня, только что проверял, может я и вправду такой криворукий)
Насчет выставления цветов в одном выражении спасибо!:)

[Константин]

void Test1Widget::paintEvent(QPaintEvent * event)
{
    QPainter p(this);
    p.setRenderHint(QPainter::Antialiasing, true);
/* весь этот блок в контексте paintEvent - моразм!
    QRgb rgb;
    rgb = (this->RedSlider->value(), this->GreenSlider->value(), this->BlueSlider->value());
    QColor mycolor(rgb);
*/
    p.setBrush(QBrush(mycolor, Qt::SolidPattern));
    p.drawRect(1, 1, 100, 100);
}

вынеси mycolor в члены класса и разгранич доступ к нему. для удобства можно добавить слоты ака setRed/setGreen/setBlue, в которых будет предварительно высчитываться нужный цвет, - их и связывать уже с соответствующими сигналами.
а так ты не виджет лепишь, а какое-то монструзное незнамо-что...

[Litkevich Yuriy]

Насчет создания/уничтожения - это же локальные переменные метода,

в том-то и дело, что у тебя это не локальная переменная метода, твой код:

PaintForTest1 = new QPainter(this);

т.е. она объявлена где-то раньше.
А вот у Константина да.