make SolarCharger methods const

Author: AndreasBoehm

include/SolarCharger.h

@@ -14,28 +14,28 @@ class SolarChargerClass {
 
     // TODO(andreasboehm): below methods are taken from VictronMppt to start abstracting
     // solar chargers without breaking everything.
-    size_t controllerAmount();
-    uint32_t getDataAgeMillis();
-    uint32_t getDataAgeMillis(size_t idx);
+    size_t controllerAmount() const;
+    uint32_t getDataAgeMillis() const;
+    uint32_t getDataAgeMillis(size_t idx) const;
 
     // total output of all MPPT charge controllers in Watts
-    int32_t getOutputPowerWatts();
+    int32_t getOutputPowerWatts() const;
 
     // total panel input power of all MPPT charge controllers in Watts
-    int32_t getPanelPowerWatts();
+    int32_t getPanelPowerWatts() const;
 
     // sum of total yield of all MPPT charge controllers in kWh
-    float getYieldTotal();
+    float getYieldTotal() const;
 
     // sum of today's yield of all MPPT charge controllers in kWh
-    float getYieldDay();
+    float getYieldDay() const;
 
     // minimum of all MPPT charge controllers' output voltages in V
-    float getOutputVoltage();
+    float getOutputVoltage() const;
 
-    std::optional<VeDirectMpptController::data_t> getData(size_t idx = 0);
+    std::optional<VeDirectMpptController::data_t> getData(size_t idx = 0) const;
 
-    bool isDataValid();
+    bool isDataValid() const;
 
 private:
     void loop();

src/SolarCharger.cpp

@@ -51,7 +51,7 @@ void SolarChargerClass::loop()
     }
 }
 
-size_t SolarChargerClass::controllerAmount()
+size_t SolarChargerClass::controllerAmount() const
 {
     std::lock_guard<std::mutex> lock(_mutex);
 
@@ -62,7 +62,7 @@ size_t SolarChargerClass::controllerAmount()
     return 0;
 }
 
-bool SolarChargerClass::isDataValid()
+bool SolarChargerClass::isDataValid() const
 {
     std::lock_guard<std::mutex> lock(_mutex);
 
@@ -73,7 +73,7 @@ bool SolarChargerClass::isDataValid()
     return false;
 }
 
-uint32_t SolarChargerClass::getDataAgeMillis()
+uint32_t SolarChargerClass::getDataAgeMillis() const
 {
     std::lock_guard<std::mutex> lock(_mutex);
 
@@ -84,7 +84,7 @@ uint32_t SolarChargerClass::getDataAgeMillis()
     return 0;
 }
 
-uint32_t SolarChargerClass::getDataAgeMillis(size_t idx)
+uint32_t SolarChargerClass::getDataAgeMillis(size_t idx) const
 {
     std::lock_guard<std::mutex> lock(_mutex);
 
@@ -97,7 +97,7 @@ uint32_t SolarChargerClass::getDataAgeMillis(size_t idx)
 
 
 // total output of all MPPT charge controllers in Watts
-int32_t SolarChargerClass::getOutputPowerWatts()
+int32_t SolarChargerClass::getOutputPowerWatts() const
 {
     std::lock_guard<std::mutex> lock(_mutex);
 
@@ -109,7 +109,7 @@ int32_t SolarChargerClass::getOutputPowerWatts()
 }
 
 // total panel input power of all MPPT charge controllers in Watts
-int32_t SolarChargerClass::getPanelPowerWatts()
+int32_t SolarChargerClass::getPanelPowerWatts() const
 {
     std::lock_guard<std::mutex> lock(_mutex);
 
@@ -121,7 +121,7 @@ int32_t SolarChargerClass::getPanelPowerWatts()
 }
 
 // sum of total yield of all MPPT charge controllers in kWh
-float SolarChargerClass::getYieldTotal()
+float SolarChargerClass::getYieldTotal() const
 {
     std::lock_guard<std::mutex> lock(_mutex);
 
@@ -133,7 +133,7 @@ float SolarChargerClass::getYieldTotal()
 }
 
 // sum of today's yield of all MPPT charge controllers in kWh
-float SolarChargerClass::getYieldDay()
+float SolarChargerClass::getYieldDay() const
 {
     std::lock_guard<std::mutex> lock(_mutex);
 
@@ -145,7 +145,7 @@ float SolarChargerClass::getYieldDay()
 }
 
 // minimum of all MPPT charge controllers' output voltages in V
-float SolarChargerClass::getOutputVoltage()
+float SolarChargerClass::getOutputVoltage() const
 {
     std::lock_guard<std::mutex> lock(_mutex);
 
@@ -156,7 +156,7 @@ float SolarChargerClass::getOutputVoltage()
     return 0;
 }
 
-std::optional<VeDirectMpptController::data_t> SolarChargerClass::getData(size_t idx)
+std::optional<VeDirectMpptController::data_t> SolarChargerClass::getData(size_t idx) const
 {
     std::lock_guard<std::mutex> lock(_mutex);