Text is not updated in qwidget.

[kumararajas]

This post is deleted!

[jsulm]

@kumararajas said in Text is not updated in qwidget.:

And I don't know why 'blocksignals' function is used here.

Wow, I didn't notice at all!

@sootoo23 Why do you block the timer signals? This way you cannot expect the signals to be emitted and the slots called!

Menu_Timer->blockSignals(true); // WHY?

[sootoo23]

@jsulm @kumararajas
The main problem is not a block signal.
The Timer is already blocked from use.

Timer has nothing to do with the problem.

The code that I uploaded is the code that removed the infinite loop (timer).

Mainwindow.cpp -> menuQwidget->show();
menuQwidget.cpp -> Dialog_Keypad->exec();

------------DialogKeypad.cpp------------
my problem... not updated >>>>> LB_SetVal->setText(SetValue);

[sootoo23]

@jsulm @kumararajas
------------mainwindow.cpp------------
MainWindow::MainWindow(QWidget *parent) :
QMainWindow(parent),
ui(new Ui::MainWindow)
{
...
...
#if (0)
//Removed endless loop timer
Hometimer = new QTimer(this);
connect(Hometimer, SIGNAL(timeout()),this,SLOT(HomeInfo_Update()));
Hometimer->start(1000);
#endif
}
void MainWindow::HomeInfo_Update()
{
...
...
ui->lb_HomeClk->setText(QString(TimeBuf));
...
}

------------menuQwidget.cpp------------
wnd_menu::wnd_menu(QWidget *parent) :
QWidget(parent),
ui(new Ui::wnd_menu)
{
...
...
#if (0)
//Removed endless loop timer
Menu_Timer = new QTimer;
connect(Menu_Timer, SIGNAL (timeout()), this, SLOT(MenuInfo_Update()));
Menu_Timer->start(1000);
Menu_Timer->blockSignals(true);
#endif
}

void wnd_menu::MenuInfo_Update()
{
...
...
ui->lb_MenuClk->setText(QString(TimeBuf));
...
...
}

------------DialogKeypad.cpp------------
Dialog_Keypad::Dialog_Keypad(QWidget *parent) :
QDialog(parent),
ui(new Ui::Dialog_Keypad)
{
ui->setupUi(this);

QString img_path = "/images/Keypad.jpg";
QImage img(img_path);    
QPixmap buf = QPixmap::fromImage(img);    
QGraphicsScene* MenuScene = new QGraphicsScene;

ui->KeypadFrame->setScene(MenuScene);
MenuScene->addPixmap(buf);

//text color setting
QPalette* palette = new QPalette();
palette->setColor(QPalette::WindowText,Qt::yellow);
ui->lb_SettingValue->setPalette(*palette);	


connect(ui->Keypad_Num0, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num1, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num2, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num3, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num4, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num5, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num6, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num7, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num8, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num9, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Dot,  SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));

KeyPadMapper.setMapping(ui->Keypad_Num0, 0);
KeyPadMapper.setMapping(ui->Keypad_Num1, 1);
KeyPadMapper.setMapping(ui->Keypad_Num2, 2);
KeyPadMapper.setMapping(ui->Keypad_Num3, 3);
KeyPadMapper.setMapping(ui->Keypad_Num4, 4);
KeyPadMapper.setMapping(ui->Keypad_Num5, 5);
KeyPadMapper.setMapping(ui->Keypad_Num6, 6);
KeyPadMapper.setMapping(ui->Keypad_Num7, 7);
KeyPadMapper.setMapping(ui->Keypad_Num8, 8);
KeyPadMapper.setMapping(ui->Keypad_Num9, 9);
KeyPadMapper.setMapping(ui->Keypad_Dot, 100);

connect(&KeyPadMapper, SIGNAL(mapped(int)), this, SLOT(SetKeypadData(int)));

}

void Dialog_Keypad::SetKeypadData(int val)
{
char keypadBuf[30] = {0,};
QLabel* LB_SetVal;

if(CurKeypadType == KEYPAD_NORMAL)
	LB_SetVal = ui->lb_SettingValue;
else if(CurKeypadType == KEYPAD_NETWORK)
	LB_SetVal = ui->lb_NetSetVal;
else if(CurKeypadType == KEYPAD_PASSWORD)
	LB_SetVal = ui->lb_PWSetVal;
else 
	LB_SetVal = ui->lb_SettingValue;

if(CurKeypadType == KEYPAD_PASSWORD)
{
	if(LB_SetVal->text().size() > 3)
	{
		printf("[%s] Password Length Over. :%s \n", __FUNCTION__, QByteArray(LB_SetVal->text().toLocal8Bit()).data());
		return;
	}
	
	SetValue = SetValue + QString::number(val);
	sprintf((char *)keypadBuf, "%s%s", QByteArray(LB_SetVal->text().toLocal8Bit()).data(), "*");
	LB_SetVal->setText(QString(keypadBuf));
	return;
}
else if(CurKeypadType == KEYPAD_NETWORK)
{
	if(LB_SetVal->text().size() > 14)
	{
		printf("[%s] Network Setting Str Length Over.\n", __FUNCTION__);
		return;
	}
	
	if(val == 100) //dot.
		sprintf((char *)keypadBuf, "%s%s", QByteArray(LB_SetVal->text().toLocal8Bit()).data(),".");
	else if((LB_SetVal->text().size() == 3)
			|| (LB_SetVal->text().size() == 7)
			|| (LB_SetVal->text().size() == 11))
		sprintf((char *)keypadBuf, "%s.%d", QByteArray(LB_SetVal->text().toLocal8Bit()).data(),val);
	else
		sprintf((char *)keypadBuf, "%s%d", QByteArray(LB_SetVal->text().toLocal8Bit()).data(),val);
}
else
{		
	if(val == 100) //dot.
		sprintf((char *)keypadBuf, "%s%s", QByteArray(LB_SetVal->text().toLocal8Bit()).data(),".");
	else
		sprintf((char *)keypadBuf, "%s%d", QByteArray(LB_SetVal->text().toLocal8Bit()).data(),val);
}

SetValue = QString(keypadBuf);
 My Problem >> LB_SetVal->setText(SetValue);

//qApp->processEvents();
//qApp->sendPostedEvents();
//qApp->processEvents(QEventLoop::ExcludeUserInput);
//LB_SetVal->repaint();
//LB_SetVal->parentWidget()->repaint();
//LB_SetVal->update();
//LB_SetVal->parentWidget()->update();

printf("[%s] %s\n", __FUNCTION__, keypadBuf);

}

[J.Hilk]

@sootoo23 Maybe you're doing the mapping wrong? I don't use it often, once in years I believe. So I'm not sure.

Since I'm assuming your running Qt 5.X try it with lambdas

replace

connect(ui->Keypad_Num0, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num1, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num2, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num3, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num4, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num5, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num6, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num7, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num8, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Num9, SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));
connect(ui->Keypad_Dot,  SIGNAL(clicked()), &KeyPadMapper, SLOT(map()));

KeyPadMapper.setMapping(ui->Keypad_Num0, 0);
KeyPadMapper.setMapping(ui->Keypad_Num1, 1);
KeyPadMapper.setMapping(ui->Keypad_Num2, 2);
KeyPadMapper.setMapping(ui->Keypad_Num3, 3);
KeyPadMapper.setMapping(ui->Keypad_Num4, 4);
KeyPadMapper.setMapping(ui->Keypad_Num5, 5);
KeyPadMapper.setMapping(ui->Keypad_Num6, 6);
KeyPadMapper.setMapping(ui->Keypad_Num7, 7);
KeyPadMapper.setMapping(ui->Keypad_Num8, 8);
KeyPadMapper.setMapping(ui->Keypad_Num9, 9);
KeyPadMapper.setMapping(ui->Keypad_Dot, 100);

connect(&KeyPadMapper, SIGNAL(mapped(int)), this, SLOT(SetKeypadData(int)));

with

connect(ui->Keypad_Num0, QPusButton::clicked, this, [=]{SetKeypadData(0);});
connect(ui->Keypad_Num1, QPusButton::clicked, this, [=]{SetKeypadData(1);});
connect(ui->Keypad_Num2, QPusButton::clicked, this, [=]{SetKeypadData(2);});
connect(ui->Keypad_Num3, QPusButton::clicked, this, [=]{SetKeypadData(3);});
connect(ui->Keypad_Num4, QPusButton::clicked, this, [=]{SetKeypadData(4);});
connect(ui->Keypad_Num5, QPusButton::clicked, this, [=]{SetKeypadData(5);});
connect(ui->Keypad_Num6, QPusButton::clicked, this, [=]{SetKeypadData(6);});
connect(ui->Keypad_Num7, QPusButton::clicked, this, [=]{SetKeypadData(7);});
connect(ui->Keypad_Num8, QPusButton::clicked, this, [=]{SetKeypadData(8);});
connect(ui->Keypad_Num9, QPusButton::clicked, this, [=]{SetKeypadData(9);});
connect(ui->Keypad_Dot,  QPusButton::clicked, this, [=]{SetKeypadData(100);});

[sootoo23]

@J.Hilk said in Text is not updated in qwidget.:

@sootoo23 Maybe you're doing the mapping wrong? I don't use it often, once in years I believe. So I'm not sure.
Since I'm assuming your running Qt 5.X try it with lambdas
replace

mapping is not wrong.
The function connected is called well and the output from the function is coming out well.

[J.Hilk]

@sootoo23 you're going through a loot of loops to create your QString,
are you sure that QString(keypadBuf) returns a valid QString? and not an empty one for example?

as a side not this one, for example could be reduced

{		
   if(val == 100) //dot.
   	sprintf((char *)keypadBuf, "%s%s", QByteArray(LB_SetVal->text().toLocal8Bit()).data(),".");
   else
   	sprintf((char *)keypadBuf, "%s%d", QByteArray(LB_SetVal->text().toLocal8Bit()).data(),val);
}
SetValue = QString(keypadBuf);
LB_SetVal->setText(SetValue);

//to
QString str = LB_SetVal->text() + val == 100 ? "." : QString::number(val);

LB_SetVal->setText(str);

[sootoo23]

@J.Hilk
The value is always good.
The problem is that settext can not be updated even if you put any value.

[J.Hilk]

@sootoo23 said in Text is not updated in qwidget.:

@J.Hilk
The value is always good.
The problem is that settext can not be updated even if you put any value.

I'm sorry, but the problem is with 99.9% certainty not the setText function I would say 1 of 4 things is the case:

• LB_SetVal is invalid or wrong. Doubtful because you say you calculate the correct value
• SetValue = QString(keypadBuf); returns an invalid or incorrect string. You can test this by setting a breakpoint and debuging, or printing SetValue to the console. e.g qDebug() << SetValue;
• You're blocking the event loop and your ui doesnt get updated
• Maybe you have set bool QObject::blockSignals(bool block) to true somewhere that is messing things up

also, what Qt-Version and compiler is this btw?

[sootoo23]

@J.Hilk

I found something strange.
 After changing the data to "Settext", execute hide () on the QWidget and show () again.
 I confirmed that it was updated.

1. LB_SetVal is always good.
   -> printf("LB_SetVal: %s, KeypadBuf: %s \n", QByteArray(LB_SetVal->text().toLocal8Bit()).data(),keypadBuf);
2. I removed all blockSignals.
   -> The QWidget in problem has no Timer and no BlockSignal.