Skip to content

Commit ed8fef8

Browse files
Copilotdorkmo
andcommitted
Consolidate duplicate switch cases for cleaner code
Co-authored-by: dorkmo <[email protected]>
1 parent 2bbbd6f commit ed8fef8

File tree

1 file changed

+6
-18
lines changed

1 file changed

+6
-18
lines changed

TankAlarm-112025-Client-BluesOpta/TankAlarm-112025-Client-BluesOpta.ino

Lines changed: 6 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1923,21 +1923,15 @@ static float readTankSensor(uint8_t idx) {
19231923
// Determine edge detection based on hall effect sensor type
19241924
switch (cfg.hallEffectType) {
19251925
case HALL_EFFECT_UNIPOLAR:
1926-
// Unipolar: triggers on one pole (active low), detect falling edge
1926+
case HALL_EFFECT_ANALOG:
1927+
// Unipolar and Analog: triggers on one pole (active low), detect falling edge
19271928
edgeDetected = (lastState == HIGH && currentState == LOW);
19281929
break;
19291930
case HALL_EFFECT_BIPOLAR:
1930-
// Bipolar/Latching: alternates between states, detect both edges
1931-
edgeDetected = (lastState != currentState);
1932-
break;
19331931
case HALL_EFFECT_OMNIPOLAR:
1934-
// Omnipolar: responds to either pole, detect both edges
1932+
// Bipolar/Latching and Omnipolar: detect both edges (state changes)
19351933
edgeDetected = (lastState != currentState);
19361934
break;
1937-
case HALL_EFFECT_ANALOG:
1938-
// For analog sensors in digital mode, detect falling edge (threshold crossing)
1939-
edgeDetected = (lastState == HIGH && currentState == LOW);
1940-
break;
19411935
}
19421936

19431937
if (edgeDetected) {
@@ -2015,21 +2009,15 @@ static float readTankSensor(uint8_t idx) {
20152009
// Determine edge detection based on hall effect sensor type
20162010
switch (cfg.hallEffectType) {
20172011
case HALL_EFFECT_UNIPOLAR:
2018-
// Unipolar: triggers on one pole, detect falling edge (active low)
2012+
case HALL_EFFECT_ANALOG:
2013+
// Unipolar and Analog: triggers on one pole, detect falling edge (active low)
20192014
edgeDetected = (lastState == HIGH && currentState == LOW);
20202015
break;
20212016
case HALL_EFFECT_BIPOLAR:
2022-
// Bipolar/Latching: alternates between states, count both edges
2023-
edgeDetected = (lastState != currentState);
2024-
break;
20252017
case HALL_EFFECT_OMNIPOLAR:
2026-
// Omnipolar: responds to either pole, count both edges
2018+
// Bipolar/Latching and Omnipolar: count both edges (state changes)
20272019
edgeDetected = (lastState != currentState);
20282020
break;
2029-
case HALL_EFFECT_ANALOG:
2030-
// For analog sensors in digital mode, detect falling edge
2031-
edgeDetected = (lastState == HIGH && currentState == LOW);
2032-
break;
20332021
}
20342022

20352023
if (edgeDetected) {

0 commit comments

Comments
 (0)