From ccbaf0573f2c3562e0f830e1448a2e0fa222f499 Mon Sep 17 00:00:00 2001
From: Nomango <nomango@qq.com>
Date: Tue, 10 Sep 2019 13:17:47 +0800
Subject: [PATCH] Add RenderComponent & UpdateComponent & EventComponent

---
 projects/kiwano.vcxproj             |  1 +
 projects/kiwano.vcxproj.filters     |  3 ++
 src/kiwano-audio/src/audio.h        | 11 +++---
 src/kiwano-imgui/src/ImGuiModule.h  |  4 +-
 src/kiwano-network/src/HttpClient.h |  2 +-
 src/kiwano/base/Component.cpp       | 51 ++++++++++++++++++++++++
 src/kiwano/base/Component.h         | 60 ++++++++++++++++++++++++++---
 src/kiwano/base/Director.h          |  4 +-
 src/kiwano/base/Input.h             |  3 +-
 src/kiwano/platform/Application.cpp | 51 +++++++++++-------------
 src/kiwano/platform/Application.h   | 12 +++---
 src/kiwano/renderer/Renderer.h      |  3 +-
 12 files changed, 154 insertions(+), 51 deletions(-)
 create mode 100644 src/kiwano/base/Component.cpp

diff --git a/projects/kiwano.vcxproj b/projects/kiwano.vcxproj
index 38c631e2..487c448c 100644
--- a/projects/kiwano.vcxproj
+++ b/projects/kiwano.vcxproj
@@ -115,6 +115,7 @@
     <ClCompile Include="..\src\kiwano\2d\Transform.cpp" />
     <ClCompile Include="..\src\kiwano\2d\Transition.cpp" />
     <ClCompile Include="..\src\kiwano\base\AsyncTask.cpp" />
+    <ClCompile Include="..\src\kiwano\base\Component.cpp" />
     <ClCompile Include="..\src\kiwano\base\EventDispatcher.cpp" />
     <ClCompile Include="..\src\kiwano\base\EventListener.cpp" />
     <ClCompile Include="..\src\kiwano\base\Input.cpp" />
diff --git a/projects/kiwano.vcxproj.filters b/projects/kiwano.vcxproj.filters
index 00c3e074..2c0c057d 100644
--- a/projects/kiwano.vcxproj.filters
+++ b/projects/kiwano.vcxproj.filters
@@ -491,5 +491,8 @@
     <ClCompile Include="..\src\kiwano\renderer\Brush.cpp">
       <Filter>renderer</Filter>
     </ClCompile>
+    <ClCompile Include="..\src\kiwano\base\Component.cpp">
+      <Filter>base</Filter>
+    </ClCompile>
   </ItemGroup>
 </Project>
\ No newline at end of file
diff --git a/src/kiwano-audio/src/audio.h b/src/kiwano-audio/src/audio.h
index b1303a5f..a28c03f4 100644
--- a/src/kiwano-audio/src/audio.h
+++ b/src/kiwano-audio/src/audio.h
@@ -30,15 +30,11 @@ namespace kiwano
 	{
 		class KGE_API Audio
 			: public Singleton<Audio>
-			, public Component
+			, public ComponentBase
 		{
 			KGE_DECLARE_SINGLETON(Audio);
 
 		public:
-			void SetupComponent() override;
-
-			void DestroyComponent() override;
-
 			// 开启设备
 			void Open();
 
@@ -50,6 +46,11 @@ namespace kiwano
 				const Transcoder::Buffer& buffer
 			);
 
+		public:
+			void SetupComponent() override;
+
+			void DestroyComponent() override;
+
 		protected:
 			Audio();
 
diff --git a/src/kiwano-imgui/src/ImGuiModule.h b/src/kiwano-imgui/src/ImGuiModule.h
index 6883edcc..48c1ba48 100644
--- a/src/kiwano-imgui/src/ImGuiModule.h
+++ b/src/kiwano-imgui/src/ImGuiModule.h
@@ -28,7 +28,9 @@ namespace kiwano
 	{
 		class ImGuiModule
 			: public Singleton<ImGuiModule>
-			, public Component
+			, public RenderComponent
+			, public UpdateComponent
+			, public EventComponent
 		{
 			KGE_DECLARE_SINGLETON(ImGuiModule);
 
diff --git a/src/kiwano-network/src/HttpClient.h b/src/kiwano-network/src/HttpClient.h
index a026705d..73b52f7a 100644
--- a/src/kiwano-network/src/HttpClient.h
+++ b/src/kiwano-network/src/HttpClient.h
@@ -31,7 +31,7 @@ namespace kiwano
 	{
 		class KGE_API HttpClient
 			: public Singleton<HttpClient>
-			, public Component
+			, public ComponentBase
 		{
 			KGE_DECLARE_SINGLETON(HttpClient);
 
diff --git a/src/kiwano/base/Component.cpp b/src/kiwano/base/Component.cpp
new file mode 100644
index 00000000..df6924eb
--- /dev/null
+++ b/src/kiwano/base/Component.cpp
@@ -0,0 +1,51 @@
+// Copyright (c) 2016-2018 Kiwano - Nomango
+// 
+// Permission is hereby granted, free of charge, to any person obtaining a copy
+// of this software and associated documentation files (the "Software"), to deal
+// in the Software without restriction, including without limitation the rights
+// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+// copies of the Software, and to permit persons to whom the Software is
+// furnished to do so, subject to the following conditions:
+// 
+// The above copyright notice and this permission notice shall be included in
+// all copies or substantial portions of the Software.
+// 
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+// THE SOFTWARE.
+
+#include "Component.h"
+
+namespace kiwano
+{
+
+	ComponentBase::ComponentBase()
+		: flag_(0)
+	{
+	}
+
+	bool ComponentBase::Check(const Int32 flag)
+	{
+		return !!(flag_ & flag);
+	}
+
+	RenderComponent::RenderComponent()
+	{
+		flag_ |= flag;
+	}
+
+	UpdateComponent::UpdateComponent()
+	{
+		flag_ |= flag;
+	}
+
+	EventComponent::EventComponent()
+	{
+		flag_ |= flag;
+	}
+
+}
diff --git a/src/kiwano/base/Component.h b/src/kiwano/base/Component.h
index 1198284b..29e4b5b8 100644
--- a/src/kiwano/base/Component.h
+++ b/src/kiwano/base/Component.h
@@ -25,23 +25,73 @@
 
 namespace kiwano
 {
-	class RenderTarget;
 
-	class KGE_API Component
+#define KGE_DEFINE_COMPONENT_FLAG ( 0x01 << (__COUNTER__ % 32) )
+
+	class KGE_API ComponentBase
 	{
 	public:
 		virtual void SetupComponent() = 0;
 		virtual void DestroyComponent() = 0;
 
-		virtual void BeforeUpdate() {}
-		virtual void OnUpdate(Duration) {}
-		virtual void AfterUpdate() {}
+		bool Check(const Int32 flag);
+
+	protected:
+		ComponentBase();
+
+	protected:
+		Int32 flag_;
+	};
+
+
+	class RenderTarget;
+
+	class KGE_API RenderComponent
+		: public virtual ComponentBase
+	{
+	public:
+		static const Int32 flag = KGE_DEFINE_COMPONENT_FLAG;
+
+		RenderComponent();
 
 		virtual void BeforeRender() {}
+
 		virtual void OnRender(RenderTarget*) {}
+
 		virtual void AfterRender() {}
+	};
+
+
+	class KGE_API UpdateComponent
+		: public virtual ComponentBase
+	{
+	public:
+		static const Int32 flag = KGE_DEFINE_COMPONENT_FLAG;
+
+		UpdateComponent();
+
+		virtual void BeforeUpdate() {}
+
+		virtual void OnUpdate(Duration) {}
+
+		virtual void AfterUpdate() {}
+	};
+
+
+	class KGE_API EventComponent
+		: public virtual ComponentBase
+	{
+	public:
+		static const Int32 flag = KGE_DEFINE_COMPONENT_FLAG;
+
+		EventComponent();
 
 		virtual void HandleEvent(Event&) {}
+
 		virtual void HandleMessage(HWND, UInt32, WPARAM, LPARAM) {}
 	};
+
+
+#undef KGE_DEFINE_COMPONENT_FLAG
+
 }
diff --git a/src/kiwano/base/Director.h b/src/kiwano/base/Director.h
index 4799fc3f..04554526 100644
--- a/src/kiwano/base/Director.h
+++ b/src/kiwano/base/Director.h
@@ -28,7 +28,9 @@ namespace kiwano
 	// 导演
 	class KGE_API Director
 		: public Singleton<Director>
-		, public Component
+		, public UpdateComponent
+		, public RenderComponent
+		, public EventComponent
 	{
 		KGE_DECLARE_SINGLETON(Director);
 
diff --git a/src/kiwano/base/Input.h b/src/kiwano/base/Input.h
index 3ae61dbf..23f4e720 100644
--- a/src/kiwano/base/Input.h
+++ b/src/kiwano/base/Input.h
@@ -29,7 +29,8 @@ namespace kiwano
 {
 	class KGE_API Input
 		: public Singleton<Input>
-		, public Component
+		, public UpdateComponent
+		, public EventComponent
 	{
 		KGE_DECLARE_SINGLETON(Input);
 
diff --git a/src/kiwano/platform/Application.cpp b/src/kiwano/platform/Application.cpp
index eef496e1..6dee2216 100644
--- a/src/kiwano/platform/Application.cpp
+++ b/src/kiwano/platform/Application.cpp
@@ -87,7 +87,7 @@ namespace kiwano
 		Renderer::GetInstance()->Init(config.render);
 
 		// Setup all components
-		for (Component* c : components_)
+		for (auto c : comps_)
 		{
 			c->SetupComponent();
 		}
@@ -148,11 +148,11 @@ namespace kiwano
 		{
 			inited_ = false;
 
-			for (auto iter = components_.rbegin(); iter != components_.rend(); ++iter)
+			for (auto iter = comps_.rbegin(); iter != comps_.rend(); ++iter)
 			{
 				(*iter)->DestroyComponent();
 			}
-			components_.clear();
+			comps_.clear();
 		}
 
 		// Destroy all instances
@@ -167,35 +167,28 @@ namespace kiwano
 		// Logger::DestroyInstance();
 	}
 
-	void Application::Use(Component* component)
+	void Application::Use(ComponentBase* component)
 	{
 		if (component)
 		{
 
 #if defined(KGE_DEBUG)
-			if (components_.contains(component))
+			if (comps_.contains(component))
 			{
 				KGE_ASSERT(false && "Component already exists!");
 			}
 #endif
 
-			components_.push_back(component);
-		}
-	}
+			comps_.push_back(component);
 
-	void Application::Remove(Component* component)
-	{
-		if (component)
-		{
-			for (auto iter = components_.begin(); iter != components_.end(); ++iter)
-			{
-				if ((*iter) == component)
-				{
-					(*iter)->DestroyComponent();
-					components_.erase(iter);
-					break;
-				}
-			}
+			if (component->Check(RenderComponent::flag))
+				render_comps_.push_back(dynamic_cast<RenderComponent*>(component));
+
+			if (component->Check(UpdateComponent::flag))
+				update_comps_.push_back(dynamic_cast<UpdateComponent*>(component));
+
+			if (component->Check(EventComponent::flag))
+				event_comps_.push_back(dynamic_cast<EventComponent*>(component));
 		}
 	}
 
@@ -207,7 +200,7 @@ namespace kiwano
 	void Application::Update()
 	{
 		// Before update
-		for (Component* c : components_)
+		for (auto c : update_comps_)
 		{
 			c->BeforeUpdate();
 		}
@@ -240,14 +233,14 @@ namespace kiwano
 			const auto dt = (now - last) * time_scale_;
 			last = now;
 
-			for (Component* c : components_)
+			for (auto c : update_comps_)
 			{
 				c->OnUpdate(dt);
 			}
 		}
 
 		// After update
-		for (auto rit = components_.rbegin(); rit != components_.rend(); ++rit)
+		for (auto rit = update_comps_.rbegin(); rit != update_comps_.rend(); ++rit)
 		{
 			(*rit)->AfterUpdate();
 		}
@@ -256,20 +249,20 @@ namespace kiwano
 	void Application::Render()
 	{
 		// Before render
-		for (Component* c : components_)
+		for (auto c : render_comps_)
 		{
 			c->BeforeRender();
 		}
 
 		// Rendering
 		Renderer* renderer = Renderer::GetInstance();
-		for (Component* c : components_)
+		for (auto c : render_comps_)
 		{
 			c->OnRender(renderer);
 		}
 
 		// After render
-		for (auto rit = components_.rbegin(); rit != components_.rend(); ++rit)
+		for (auto rit = render_comps_.rbegin(); rit != render_comps_.rend(); ++rit)
 		{
 			(*rit)->AfterRender();
 		}
@@ -277,7 +270,7 @@ namespace kiwano
 
 	void Application::DispatchEvent(Event& evt)
 	{
-		for (Component* c : components_)
+		for (auto c : event_comps_)
 		{
 			c->HandleEvent(evt);
 		}
@@ -298,7 +291,7 @@ namespace kiwano
 		}
 
 		// Handle Message
-		for (Component* c : app->components_)
+		for (auto c : app->event_comps_)
 		{
 			c->HandleMessage(hwnd, msg, wparam, lparam);
 		}
diff --git a/src/kiwano/platform/Application.h b/src/kiwano/platform/Application.h
index 2b24c3dd..148199de 100644
--- a/src/kiwano/platform/Application.h
+++ b/src/kiwano/platform/Application.h
@@ -82,12 +82,7 @@ namespace kiwano
 
 		// 添加组件
 		void Use(
-			Component* component
-		);
-
-		// 卸载组件
-		void Remove(
-			Component* component
+			ComponentBase* component
 		);
 
 		// 设置时间缩放因子
@@ -116,6 +111,9 @@ namespace kiwano
 		bool	inited_;
 		Float32	time_scale_;
 
-		Vector<Component*>	components_;
+		Vector<ComponentBase*>		comps_;
+		Vector<RenderComponent*>	render_comps_;
+		Vector<UpdateComponent*>	update_comps_;
+		Vector<EventComponent*>		event_comps_;
 	};
 }
diff --git a/src/kiwano/renderer/Renderer.h b/src/kiwano/renderer/Renderer.h
index 86861fbb..d422352a 100644
--- a/src/kiwano/renderer/Renderer.h
+++ b/src/kiwano/renderer/Renderer.h
@@ -55,7 +55,8 @@ namespace kiwano
 	// 渲染器
 	class KGE_API Renderer
 		: public Singleton<Renderer>
-		, public Component
+		, public RenderComponent
+		, public EventComponent
 		, public RenderTarget
 	{
 		KGE_DECLARE_SINGLETON(Renderer);