From dcfb475ca19adf145d4fdd5c1d977ada6e62cef3 Mon Sep 17 00:00:00 2001 From: Vasily Davydov Date: Fri, 14 Oct 2022 16:22:30 +0300 Subject: [PATCH] state-hadler: change switch to if in displaySet This commit resolves an issue of having a boolean in switch case --- StateHandler/src/StateHandler.cpp | 40 +++++++++++++++---------------- 1 file changed, 19 insertions(+), 21 deletions(-) diff --git a/StateHandler/src/StateHandler.cpp b/StateHandler/src/StateHandler.cpp index 9186236..15db4c4 100644 --- a/StateHandler/src/StateHandler.cpp +++ b/StateHandler/src/StateHandler.cpp @@ -26,33 +26,31 @@ StateHandler::displaySet (unsigned int value1, unsigned int value2) char line_up[16] = { 0 }; char line_down[16] = { 0 }; - switch (current_mode) + if (current_mode == MANUAL) { - /* - * MANUAL MODE: - * ---------------- - * SPEED: 20% - * PRESSURE: XXPa - * ---------------- - */ - case MANUAL: + /* + * MANUAL MODE: + * ---------------- + * SPEED: 20% + * PRESSURE: XXPa + * ---------------- + */ snprintf (line_up, 16, "SPEED: %02d%", value1); snprintf (line_down, 16, "PRESSURE: %02dPa", value2); - break; - /* - * AUTO MODE: - * ---------------- - * P. SET: 35Pa - * P. CURR: XXPa - * ---------------- - */ - case AUTO: + } + else + { + /* + * AUTO MODE: + * ---------------- + * P. SET: 35Pa + * P. CURR: XXPa + * ---------------- + */ snprintf (line_up, 16, "P. SET: %02dPa", value1); snprintf (line_down, 16, "P. CURR: %02dPa", value2); - break; - default: - break; } + _lcd->clear (); _lcd->setCursor (0, 0); _lcd->print (line_up);