If condition to stay for a fixed time

Reply to If condition to stay for a fixed time on Mon, 13 Jul 2020 06:58:59 GMT

SimonSchroeder — Mon, 13 Jul 2020 06:58:59 GMT

With the code that @sierdzio provided the radio button will stay on for at most 1 second. If a falls below 0.5 earlier than that it will turn the button off immediately. If you want something like exactly 1 second I would suggest introducing a boolean member variable which will be reset by the timer.

Furthermore, I suggest that you use a single stylesheet for the two states specifying the different style for different toggle states. The right one will be picked according to the checked state of the radio button.

Together, the code would look something like this:

if(!keep_on) // don't change anything if the timer is still running (neither extend the "on" period nor toggle it off)
{
    if(a > 0.5)
    {
        ui->rd_btn->show();
        ui->rd_btn->setChecked(true);

        keep_on = true;
        QTimer::singleShot(1000, this, [this]() { keep_on = false; ui->rdo_btn->setChecked(false); });
    }
    else
    {
        ui->rd_btn->setChecked(false);
    }
}

You would also need to set the stylesheet once in the constructor, for example.

Depending on what you really want, this might not be the right solution. If the on-state should be extended by another a > 0.5 while the timer is still running, you would need to adapt the code accordingly. If you want a <= 0.5 to immediatly switch to the off-state this needs changes as well.

Reply to If condition to stay for a fixed time on Fri, 10 Jul 2020 09:04:18 GMT

sierdzio — Fri, 10 Jul 2020 09:04:18 GMT

@russjohn834 said in If condition to stay for a fixed time:

Could you please comment what went wrong here?

The code here looks good. So the question is rather: how often is it called? Do you call this method many times per second, does a change often?

Reply to If condition to stay for a fixed time on Fri, 10 Jul 2020 09:02:29 GMT

russjohn834 — Fri, 10 Jul 2020 09:02:29 GMT

If I do this:

if(a > 0.5)
{
    ui->rdo_btn->show();
    ui->rdo_btn->setStyleSheet(StyleSheetOn);

    QTimer::singleShot(1000, this, [this, StyleSheetOff]() { ui->rdo_btn->setStyleSheet(StyleSheetOff); });
}
else
{
    ui->rdo_btn->setStyleSheet(StyleSheetOff);
}

Apparently it behaves as before (it momentarily shows the StyleSheetOn and shows StyleSheetOff ).

What I'm trying to do is , if the condition met (if(a > 0.5)) , radio button (rdo_btn) should show the StyleSheetOn for 1 sec and then StyleSheetOff.

Could you please comment what went wrong here?

Reply to If condition to stay for a fixed time on Fri, 10 Jul 2020 08:45:59 GMT

jsulm — Fri, 10 Jul 2020 08:45:59 GMT

@russjohn834

QTimer::singleShot(1000, this, [this, StyleSheetOff]() { ui->rdo_btn->setStyleSheet(StyleSheetOff); });

Reply to If condition to stay for a fixed time on Fri, 10 Jul 2020 08:44:22 GMT

russjohn834 — Fri, 10 Jul 2020 08:44:22 GMT

@sierdzio Hi, I tried as you suggested, I get an error saying:

mainwindow.cpp:348:87: error: variable 'StyleSheetOff' cannot be implicitly captured in a lambda with no capture-default specified
mainwindow.cpp:319:13: note: 'StyleSheetOff' declared here
mainwindow.cpp:348:40: note: lambda expression begins here

Reply to If condition to stay for a fixed time on Fri, 10 Jul 2020 08:19:17 GMT

sierdzio — Fri, 10 Jul 2020 08:19:17 GMT

Use a QTimer, so example like this:

if(a > 0.5)
{
    ui->rdo_btn->show();
    ui->rdo_btn->setStyleSheet(StyleSheetOn);

    QTimer::singleShot(1000, this, [this]() { ui->rdo_btn->setStyleSheet(StyleSheetOff); });
}
else
{
    ui->rdo_btn->setStyleSheet(StyleSheetOff);
}