Observer now used as intended

src/include/FightEnv.hpp

@@ -9,6 +9,11 @@
 #include "Animations/IncomingBanner.hpp"
 #include "Game.hpp"
 
+/*
+ * This class has the UI components and therefore provides these to the different fight phases
+ * Not an ideal solution
+ */
+
 class FightEnv {
   public:
     FightEnv();

src/include/ObserverPattern/Observer.hpp

@@ -58,17 +58,19 @@ class Subject {
             return &obs.get() == &observer;
         });
     }
+
+    virtual void notify(T entity) const {
+        for (const auto& obs: observers) {
+            obs.get().update(entity);
+        }
+    }
+
     bool valid = true;
 
   private:
     std::list<RefObserver> observers;
 
   protected:
-    void notify(T entity) const {
-        for (const auto& obs: observers) {
-            obs.get().update(entity);
-        }
-    }
 };
 
 #endif

src/main/UIObjects/UIEnemyOverview.cpp

@@ -57,7 +57,7 @@ void UIEnemyOverview::draw() {
 void UIEnemyOverview::changeHealth(int lostHealth) {
   int newHealth = std::max(this->enemy.health - lostHealth, 0);
   this->enemy.health = newHealth;
-  this->enemyStats.update(this->enemy);
+  this->enemy.notify(this->enemy);
 }
 
 void UIEnemyOverview::updatePickedColorText(std::string newText, sf::Color pickedColor) {

src/main/UIObjects/UIPlayerOverview.cpp

@@ -57,7 +57,7 @@ void UIPlayerOverview::init() {
 void UIPlayerOverview::changeHealth(int value) {
   int newHealth = std::max(this->player.health - value, 0);
   this->player.health = newHealth;
-  this->uiPlayerStats.update(this->player);
+  this->player.notify(this->player);
 }
 
 void UIPlayerOverview::draw() {