From 1400ade977e13e3b535d3c2fddb6e15de3c9b5a5 Mon Sep 17 00:00:00 2001
From: Kelly Rauchenberger <fefferburbia@gmail.com>
Date: Thu, 8 Feb 2018 13:43:10 -0500
Subject: Moved sprite rendering into AnimatingSystem

Refactored how systems work slightly. Now, rendering can be done by a number of systems working together. Since the AnimatingSystem handles the animation of sprites, it should also handle the rendering of them. Because of this, the RenderingSystem has been removed.
---
 src/systems/animating.cpp | 30 ++++++++++++++++++++++++++++++
 src/systems/animating.h   |  3 +++
 src/systems/rendering.cpp | 37 -------------------------------------
 src/systems/rendering.h   | 22 ----------------------
 4 files changed, 33 insertions(+), 59 deletions(-)
 delete mode 100644 src/systems/rendering.cpp
 delete mode 100644 src/systems/rendering.h

(limited to 'src/systems')

diff --git a/src/systems/animating.cpp b/src/systems/animating.cpp
index fcbfca5..91fe925 100644
--- a/src/systems/animating.cpp
+++ b/src/systems/animating.cpp
@@ -1,6 +1,7 @@
 #include "animating.h"
 #include "game.h"
 #include "components/animatable.h"
+#include "components/transformable.h"
 
 void AnimatingSystem::tick(double)
 {
@@ -28,6 +29,35 @@ void AnimatingSystem::tick(double)
   }
 }
 
+void AnimatingSystem::render(Texture& texture)
+{
+  std::set<id_type> spriteEntities =
+    game_.getEntityManager().getEntitiesWithComponents<
+      AnimatableComponent,
+      TransformableComponent>();
+
+  for (id_type entity : spriteEntities)
+  {
+    auto& sprite = game_.getEntityManager().
+      getComponent<AnimatableComponent>(entity);
+
+    auto& transform = game_.getEntityManager().
+      getComponent<TransformableComponent>(entity);
+
+    Rectangle dstrect {
+      static_cast<int>(transform.getX()),
+      static_cast<int>(transform.getY()),
+      transform.getW(),
+      transform.getH()};
+
+    const AnimationSet& aset = sprite.getAnimationSet();
+    texture.blit(
+      aset.getTexture(),
+      aset.getFrameRect(sprite.getFrame()),
+      dstrect);
+  }
+}
+
 void AnimatingSystem::startAnimation(id_type entity, std::string animation)
 {
   auto& sprite = game_.getEntityManager().
diff --git a/src/systems/animating.h b/src/systems/animating.h
index 150a74a..d6a89a5 100644
--- a/src/systems/animating.h
+++ b/src/systems/animating.h
@@ -3,6 +3,7 @@
 
 #include "system.h"
 #include <string>
+#include "renderer.h"
 
 class AnimatingSystem : public System {
 public:
@@ -13,6 +14,8 @@ public:
 
   void tick(double dt);
 
+  void render(Texture& texture);
+
   void startAnimation(id_type entity, std::string animation);
 
 };
diff --git a/src/systems/rendering.cpp b/src/systems/rendering.cpp
deleted file mode 100644
index 8219732..0000000
--- a/src/systems/rendering.cpp
+++ /dev/null
@@ -1,37 +0,0 @@
-#include "rendering.h"
-#include "game.h"
-#include "components/animatable.h"
-#include "components/transformable.h"
-
-void RenderingSystem::tick(double)
-{
-  texture_.fill(texture_.entirety(), 0, 0, 0);
-
-  std::set<id_type> spriteEntities =
-    game_.getEntityManager().getEntitiesWithComponents<
-      AnimatableComponent,
-      TransformableComponent>();
-
-  for (id_type entity : spriteEntities)
-  {
-    auto& sprite = game_.getEntityManager().
-      getComponent<AnimatableComponent>(entity);
-
-    auto& transform = game_.getEntityManager().
-      getComponent<TransformableComponent>(entity);
-
-    Rectangle dstrect {
-      static_cast<int>(transform.getX()),
-      static_cast<int>(transform.getY()),
-      transform.getW(),
-      transform.getH()};
-
-    const AnimationSet& aset = sprite.getAnimationSet();
-    texture_.blit(
-      aset.getTexture(),
-      aset.getFrameRect(sprite.getFrame()),
-      dstrect);
-  }
-
-  texture_.renderScreen();
-}
diff --git a/src/systems/rendering.h b/src/systems/rendering.h
deleted file mode 100644
index a53ee64..0000000
--- a/src/systems/rendering.h
+++ /dev/null
@@ -1,22 +0,0 @@
-#ifndef RENDERING_H_76ABC02A
-#define RENDERING_H_76ABC02A
-
-#include "system.h"
-#include "renderer.h"
-#include "consts.h"
-
-class RenderingSystem : public System {
-public:
-
-  RenderingSystem(Game& game) : System(game)
-  {
-  }
-
-  void tick(double dt);
-
-private:
-
-  Texture texture_ {GAME_WIDTH, GAME_HEIGHT};
-};
-
-#endif /* end of include guard: RENDERING_H_76ABC02A */
-- 
cgit 1.4.1