tylen/ESP-Ventilation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

state-handler: add proper way of non-blinking lcd

Browse Source 
This commit adds a save() method for comparison of
values communicating via main and stateHandler to avoid
printing every milliseconds
This commit is contained in:
Vasily Davydov
2022-10-16 14:29:22 +03:00
parent 338bcea5ce
commit 15bf2150ca
2 changed files with 24 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
StateHandler/src/StateHandler.cpp
View File

@@ -13,8 +13,6 @@ StateHandler::StateHandler (LiquidCrystal *lcd)
current = &StateHandler::stateInit;
(this->*current) (Event (Event::eEnter));
current_mode = MANUAL;
saved_set_value = 0;
saved_curr_value = 0;
}
StateHandler::~StateHandler ()
@@ -112,13 +110,7 @@ StateHandler::stateManual (const Event &event)
handleControlButtons (event.value);
break;
case Event::eTick:
if (saved_curr_value != event.value
|| saved_set_value != value[MANUAL].getCurrent ())
{
saved_curr_value = event.value;
saved_set_value = value[MANUAL].getCurrent ();
displaySet (value[MANUAL].getCurrent (), event.value);
}
save (event.value, value[MANUAL].getCurrent (), MANUAL);
break;
}
}
@@ -137,13 +129,7 @@ StateHandler::stateAuto (const Event &event)
handleControlButtons (event.value);
break;
case Event::eTick:
if (saved_curr_value != event.value
|| saved_set_value != value[AUTO].getCurrent ())
{
saved_curr_value = event.value;
saved_set_value = value[AUTO].getCurrent ();
displaySet (value[AUTO].getCurrent (), event.value);
}
save (event.value, value[AUTO].getCurrent (), AUTO);
break;
}
}
@@ -167,3 +153,15 @@ StateHandler::handleControlButtons (uint8_t button)
break;
}
}
void
StateHandler::save (int eventValue, int counterValue, size_t mode)
{
if (saved_curr_value[mode] != eventValue
|| saved_set_value[mode] != counterValue)
{
saved_curr_value[mode] = eventValue;
saved_set_value[mode] = counterValue;
displaySet (counterValue, eventValue);
}
}