pump

lib/Pump/pump.cpp

@@ -36,6 +36,9 @@ void Pump::loop() {
         case State::Error :
             this->stateError();
             break;
+
+        default :
+            break;
     }
 }
 
@@ -74,37 +77,13 @@ uint16_t Pump::getFlow() {
     return 0;
 }
 
-uint16_t Pump::getStatus() {
+uint8_t Pump::getStatus() {
-    switch (this->state) {
+    return (uint8_t) this->state;
-        case State::Off :
-            return 0;
-            break;
-
-        case State::On :
-            return 1;
-            break;
-
-        case State::Timer :
-            return 2;
-            break;
-
-        case State::Standby :
-            return 3;
-            break;
-
-        case State::Error :
-            return -1;
-            break;
-        
-        default:
-            return -2;
-            break;
-    }
 }
 
 bool Pump::checkSensors() {
     if (millis() - this->lastMillisCheckSensor < Pump::delayCheckSensor)
-        return;
+        return true;
     this->lastMillisCheckSensor = millis();
     
     //TODO: read pressure
@@ -171,6 +150,10 @@ void Pump::changeState(State state) {
             this->handlePins(false);
             break;
 
+        case State::OnStupid :
+            this->handlePins(true);
+            break;
+
         default:
             break;
     }

lib/Pump/pump.h

@@ -4,12 +4,13 @@
 
 class Pump {
     public:
-        enum State {
+        enum State : uint8_t {
-            Off,
+            Off = 0,
-            On,
+            On = 1,
-            Timer,
+            Timer = 2,
-            Standby,
+            Standby = 3,
-            Error
+            Error = 100,
+            OnStupid = 200
         };
 
         Pump(uint8_t pumpPin, uint8_t pressurePin, uint8_t flowPin, uint8_t ledPin);
@@ -23,7 +24,7 @@ class Pump {
         uint16_t getPressure();
         uint32_t getRunningTime();
         uint16_t getFlow();
-        uint16_t getStatus();
+        uint8_t getStatus();
 
         State getState() { return this->state; }
 
@@ -32,7 +33,6 @@ class Pump {
         static const uint16_t blinkTime = 1000;
         static const uint16_t maxPressure = 1000;
         static const uint16_t maxMaxPressureTime = 30000;
-        static const uint16_t delayTimer = 1000;
 
         bool checkSensors();
         void handlePins(bool state);