Передача массива данных в paintEvent

Сохрани данные в поле класса.

А тебе для чего? Какая стоит задача?

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

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

Если делать более правильно, то у тебя должен быть отдельно класс, отображающий квадрат правильного цвета (типа, класс отображения датчика), отдельно класс, раскидывающий множество классов отображения датчика по какой-то сетке. В первом классе делаешь сеттер исправен/неисправен, принимать должен тип bool. Во втором принимаешь данные от датчиков и дёргаешь вышеупомянутый сеттер. По идее, тебе даже paintEvent() переопределять так не придётся, поскольку в качестве первого класса можно использовать какой-нибудь QLabel. Реализация первого класса будет выглядеть как-то так:

class SensorView: public QLabel
  {
	Q_OBJECT
	
	private:
		QColor m_brokenColor;
		QColor m_normalColor;
		QSize m_viewSize;
		bool m_isBroken;
		
	public:
		SensorView():
			m_brokenColor(Qt::GlobalColor::red),
			m_normalColor(Qt::GlobalColor::green),
			m_viewSize(32, 32);
		  { setBroken(false); }
		  
		void setBroken(bool broken);
		void setViewSize(const QSize &size);
  };
  
  
  
void SensorView::setBroken(bool broken)
  {
	m_isBroken = broken;
	
	QPixmap pm(m_viewSize);
	pm.fill(broken? m_brokenColor: m_normalColor);
	
	setPixmap(pm);
  }
  
  
  
void SensorView::setViewSize(const QSize &size)
  {
	m_viewSize = size;
	setBroken(m_isBroken);
  }

походу ты его неслабо грузанул щас.

Разве сложно описано? Я спать хочу, мог и не так что-то выразить

Да нет не особа сложно. Но можно было и по рабоче крестьянски написать.

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

По рабоче-крестьянски в сонном состоянии не разумею, сорян

Очень простое решение. Создаешь глобальный массив int. Старший бит - признак датчик исправен/нет, остальные 31 - его адрес. Когда нужно перерисовать, проходишь циклом по массиву.

Не догнал, а где сетка по которой будут расположены квадратики?

а сетка с квадратиками во вьюхе, которую дергает та, принимающая сигналы с обновлениями статусов, модель.

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

глобальный массив int

Синглтон штоле? РЦ никто не отменял.

Когда нужно перерисовать

Да, когда?

циклом по массиву

А чтоб быстрее было — отправь на обработку об пень в желЕ.

Синглтон - что это ? А чем не устраивает вот такой код это образец

[code] void MainWindow::paintEvent(QPaintEvent *event) { Q_UNUSED(event); QPainter painter(this);

QFont font=painter.font() ;
font.setPointSize(18);
painter.setFont(QFont("monospace",12));

painter.drawText((352 + 32*0 + 5),(32 + 32*0 - 10),"1");
painter.drawText((352 + 32*1 + 5),(32 + 32*0 - 10),"2");
painter.drawText((352 + 32*2 + 5),(32 + 32*0 - 10),"3");
painter.drawText((352 + 32*3 + 5),(32 + 32*0 - 10),"4");
painter.drawText((352 + 32*4 + 5),(32 + 32*0 - 10),"5");
painter.drawText((352 + 32*5 + 5),(32 + 32*0 - 10),"6");
painter.drawText((352 + 32*6 + 5),(32 + 32*0 - 10),"7");
painter.drawText((352 + 32*7 + 5),(32 + 32*0 - 10),"8");
painter.drawText((352 + 32*8 + 5),(32 + 32*0 - 10),"9");


painter.drawText((352 + 32*0 - 35),(32 + 32*1 - 10),"1");
painter.drawText((352 + 32*0 - 35),(32 + 32*2 - 10),"2");
painter.drawText((352 + 32*0 - 35),(32 + 32*3 - 10),"3");
painter.drawText((352 + 32*0 - 35),(32 + 32*4 - 10),"4");
painter.drawText((352 + 32*0 - 35),(32 + 32*5 - 10),"5");
painter.drawText((352 + 32*0 - 35),(32 + 32*6 - 10),"6");
painter.drawText((352 + 32*0 - 35),(32 + 32*7 - 10),"7");
painter.drawText((352 + 32*0 - 35),(32 + 32*8 - 10),"8");
painter.drawText((352 + 32*0 - 35),(32 + 32*9 - 10),"9");
painter.drawText((352 + 32*0 - 35),(32 + 32*10 - 10),"10");
painter.drawText((352 + 32*0 - 35),(32 + 32*11 - 10),"11");
painter.drawText((352 + 32*0 - 35),(32 + 32*12 - 10),"12");
painter.drawText((352 + 32*0 - 35),(32 + 32*13 - 10),"13");
painter.drawText((352 + 32*0 - 35),(32 + 32*14 - 10),"14");


for(unsigned char y = 0; y < 14; y ++){

 for(unsigned char x = 0; x < 9; x++){

      painter.setBrush(QBrush(Qt::gray, Qt::SolidPattern));
      painter.drawRect((352 + 32*x), (32 + 32*y), 20, 20);
      painter.setBrush(QBrush(Qt::gray, Qt::SolidPattern));
      painter.drawRect((352 + 32*x + 5), (32 + 32*y + 5), 10, 10);
 }

}

} [/code]

Вызывается и рисует квадратики.

Ты смотрел как часто он вызывается?

раз 5 секунд А что есть варианты сделать быстрее ?

пейнт эвент раз в 5 сек?

Да раз в 5 секунд

как проверял?

У меня таймер настроен на 5 секунд. Каждые 5 секунд происходит вызов paintEvent. Вот и все. Или вы что-то другое имеете в виду?

Синглтон - что это ?

https://en.wikipedia.org/wiki/Singleton_pattern

А чем не устраивает вот такой код это образец

Вы не знаете, что такое циклы?

painter.drawText((352 + 32*0 + 5),(32 + 32*0 - 10),"1");
painter.drawText((352 + 32*1 + 5),(32 + 32*0 - 10),"2");
painter.drawText((352 + 32*2 + 5),(32 + 32*0 - 10),"3");
painter.drawText((352 + 32*3 + 5),(32 + 32*0 - 10),"4");
painter.drawText((352 + 32*4 + 5),(32 + 32*0 - 10),"5");
painter.drawText((352 + 32*5 + 5),(32 + 32*0 - 10),"6");
painter.drawText((352 + 32*6 + 5),(32 + 32*0 - 10),"7");
painter.drawText((352 + 32*7 + 5),(32 + 32*0 - 10),"8");
painter.drawText((352 + 32*8 + 5),(32 + 32*0 - 10),"9");


painter.drawText((352 + 32*0 - 35),(32 + 32*1 - 10),"1");
painter.drawText((352 + 32*0 - 35),(32 + 32*2 - 10),"2");
painter.drawText((352 + 32*0 - 35),(32 + 32*3 - 10),"3");
painter.drawText((352 + 32*0 - 35),(32 + 32*4 - 10),"4");
painter.drawText((352 + 32*0 - 35),(32 + 32*5 - 10),"5");
painter.drawText((352 + 32*0 - 35),(32 + 32*6 - 10),"6");
painter.drawText((352 + 32*0 - 35),(32 + 32*7 - 10),"7");
painter.drawText((352 + 32*0 - 35),(32 + 32*8 - 10),"8");
painter.drawText((352 + 32*0 - 35),(32 + 32*9 - 10),"9");
painter.drawText((352 + 32*0 - 35),(32 + 32*10 - 10),"10");
painter.drawText((352 + 32*0 - 35),(32 + 32*11 - 10),"11");
painter.drawText((352 + 32*0 - 35),(32 + 32*12 - 10),"12");
painter.drawText((352 + 32*0 - 35),(32 + 32*13 - 10),"13");
painter.drawText((352 + 32*0 - 35),(32 + 32*14 - 10),"14");

Отлично сворачивается в:

for (int i = 0; i < 9; ++i)
    painter.drawText((352 + 32*i + 5),(32 + 32*0 - 10), QString("%1").arg(i+1));

for (int i = 1; i <= 14; ++i)
    painter.drawText((352 + 32*0 - 35),(32 + 32*i - 10),QString("%1").arg(i));

Проще не извращаться со всякими там paintEvent, таблицами и QLabel, а заюзать Graphics View Framework.

Две минуты в Paint’e Qt Creator’e:

http://esxi.z-lab.me:666/~exl_lab/movies/qt-gvf.gif
http://esxi.z-lab.me:666/~exl_lab/others/QtGVF.zip

А чем плох мой метод?

а ты впиши в том методе: qDebug() << «ой»; и посмотри, раз в 5 сек или нет.

Тем, что ты хочешь передать какие-то данные в метод глобальной отрисовки, который в нормально спроектированной программе должен лишь рисовать.

завтра он придет с вопросами про кнопки и менюшки, ты их тоже на графикс вью ему рисовать будешь?

Каждые 5 секунд происходит вызов paintEvent.

Как именно? Напрямую что ли? paintEvent() вызывается методом update() или repaint(), насколько я помню. Не нужно явно вызывать paint() или paintEvent(), поскольку эти методы являются перегруженными и фреймворк вызывает их автоматически, когда ты, например, сворачиваешь/разворачиваешь окно, меняешь фокус или просто проводишь курсором над канвасом.

Причём тут кнопки и менюшки? У него вполне отчётливый вопрос:

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

Лучшей задачи для Graphics View Framework и не придумаешь. Особенно если датчиков много. Завтра ему потребуется прикрепить какую-нибудь дополнительную информацию, которая показывается при наведении курсором на квадратик датчика и он будет два дня ковыряться в своей paintEvent-лапше.

допинфа при наведении - тултип, сам же знаешь.

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

Да вы правы примерно раз в 1 секунду вызывается paintEvent()

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

допинфа при наведении - тултип, сам же знаешь.

И как ты в глобальный paintEvent где он эти квадратики на канвасе рисует будешь его засовывать?

и где я (или ктото выше) предлагал использовать пейнтивент?

См. стартовое сообщение этой темы.

Не могу так сделать пишу не для десктоп а для промышленного пк со своими заморочками.

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

А что вызывает paintEvent ?

QWidget::repaint()

Причем тут QWidget::repaint() ? Я не запускаю repaint().

Почему происходит циклически вызов paintEvent ?

Это хорошая идея выводить информацию при наведении курсора на квадратик. Но у меня проблема квадратиков нужно 128 штук а дисплей и меня 640x480. И размер квадратика такой что пальцем не попадешь.Нужна экранная лупа или что-то типа того. Не знаю в общем.

Я тебе скинул код выше, смотри, разбирайся. Я бы сделал через Graphics View Framework.

Тебя не поймешь. Спросил что вызывает paintEvent? Я ответила функция repaint. Она вызывает paintEvent сразу. Другие вызовы скорей приходят из очереди event loop. Кто их туда кладет, фиг его знает но ты их должен полюбому обрабатывать иначе виджет исказится.

А ты учишься или уже на работе так тупишь?

Кстати, да, у вьюхи можно цвет на DisplayRole повесить, вообще удобно будет. Но тут уже вопрос, насколько вьюха подходит эстетически

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

Потому что ты не понимаешь, как устроено рисование в Qt. Почитай документацию:

Each widget performs all painting operations from within its paintEvent() function. This is called whenever the widget needs to be redrawn, either as a result of some external change or when requested by the application.

И городить своё рисование в такой задаче, особенно после той простыни, на раз сворачиваемой в цикл — OCHE плохая затея. Воспользуйся готовыми виджетами или, как EXL посоветовал, используй Graphics Items

EXL Как можно добавить «в ваш пример», второй вложенный квадрат в первый? У квадратов будут различные цвета.

Получается квадрат в квадрате.

Эм. Если этот вложенный квадрат это просто нарисованный квадрат без сущности элемента, которым можно управлять, то, например, так:

QColor getColor(const bool state)
{
    return (state) ? QColor("#29AC3B") : QColor("#EB3838");
}

void Sensor::paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget)
{
    QColor fillColor = getColor(ok), innerColor = getColor(!ok);
    painter->fillRect(boundingRect(), fillColor);
    painter->fillRect(boundingRect().intersected(QRectF(10, 10, 10, 10)), innerColor);

    QFont font("Monospace", 8);
    painter->setFont(font);
    painter->setPen(Qt::GlobalColor::white);
    painter->drawText(1, 10, QString("%1").arg(number + 1));
}

Виноват. Управлять им надо. Получается если вписать квадрат в квадрат то получится внешняя рамка и внутри область в форме квадрата. Вот у рамки будет свой цвет а у внутреннего квадрата свой цвет. Управление цветами будет независимое.

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

Если да, то в чём, собственно, проблема?

И что ты имеешь в виду под утверждением «управлять им надо»? Если только менять цвет – то зачем тебе этот квадратик как отдельный элемент? Я подразумевал под управлением, например, отдельное движение этого квадратика по полю или в пределах рамки и пр. сложные случаи.