From cff284cedde9f0cc133ff2855557299ce1d8083c Mon Sep 17 00:00:00 2001 From: Loek Le Blansch Date: Wed, 18 Dec 2024 14:20:59 +0100 Subject: add fixed/frame update functions to script --- src/crepe/system/ScriptSystem.cpp | 18 +++++++++++++----- src/crepe/system/ScriptSystem.h | 16 +++++++++++++++- 2 files changed, 28 insertions(+), 6 deletions(-) (limited to 'src/crepe/system') diff --git a/src/crepe/system/ScriptSystem.cpp b/src/crepe/system/ScriptSystem.cpp index 746285f..9977396 100644 --- a/src/crepe/system/ScriptSystem.cpp +++ b/src/crepe/system/ScriptSystem.cpp @@ -1,7 +1,6 @@ #include "../api/BehaviorScript.h" #include "../api/Script.h" #include "../manager/ComponentManager.h" -#include "../util/dbg.h" #include "ScriptSystem.h" @@ -9,10 +8,19 @@ using namespace std; using namespace crepe; void ScriptSystem::fixed_update() { - dbg_trace(); + LoopTimerManager & timer = this->mediator.loop_timer; + duration_t delta_time = timer.get_scaled_fixed_delta_time(); + this->update(&Script::fixed_update, delta_time); +} - ComponentManager & mgr = this->mediator.component_manager; +void ScriptSystem::frame_update() { LoopTimerManager & timer = this->mediator.loop_timer; + duration_t delta_time = timer.get_delta_time(); + this->update(&Script::frame_update, delta_time); +} + +void ScriptSystem::update(void (Script::* update_function)(duration_t), const duration_t & delta_time) { + ComponentManager & mgr = this->mediator.component_manager; RefVector behavior_scripts = mgr.get_components_by_type(); for (BehaviorScript & behavior_script : behavior_scripts) { @@ -26,7 +34,7 @@ void ScriptSystem::fixed_update() { script->initialized = true; } - duration_t delta_time = timer.get_scaled_fixed_delta_time(); - script->update(delta_time); + (*script.*update_function)(delta_time); } } + diff --git a/src/crepe/system/ScriptSystem.h b/src/crepe/system/ScriptSystem.h index 612c2ae..ca4534a 100644 --- a/src/crepe/system/ScriptSystem.h +++ b/src/crepe/system/ScriptSystem.h @@ -2,6 +2,8 @@ #include "System.h" +#include "../manager/LoopTimerManager.h" + namespace crepe { class Script; @@ -16,13 +18,25 @@ class ScriptSystem : public System { public: using System::System; /** - * \brief Call Script::update() on all active \c BehaviorScript instances + * \brief Call Script::fixed_update() on all active \c BehaviorScript instances * * This routine updates all scripts sequentially using the Script::update() * method. It also calls Script::init() if this has not been done before on * the \c BehaviorScript instance. */ void fixed_update() override; + + /** + * \brief Call Script::frame_update() on all active \c BehaviorScript instances + * + * This routine updates all scripts sequentially using the Script::update() + * method. It also calls Script::init() if this has not been done before on + * the \c BehaviorScript instance. + */ + void frame_update() override; + +private: + void update(void (Script::* update_function)(duration_t), const duration_t & delta_time); }; } // namespace crepe -- cgit v1.2.3 From 5605d438a242a6252f7a27ce772eb092b62d78ae Mon Sep 17 00:00:00 2001 From: Loek Le Blansch Date: Wed, 18 Dec 2024 14:21:09 +0100 Subject: `make format` --- src/crepe/api/Script.cpp | 1 - src/crepe/facade/Texture.cpp | 5 ++--- src/crepe/system/ScriptSystem.cpp | 4 ++-- src/crepe/system/ScriptSystem.h | 2 +- 4 files changed, 5 insertions(+), 7 deletions(-) (limited to 'src/crepe/system') diff --git a/src/crepe/api/Script.cpp b/src/crepe/api/Script.cpp index 6de6830..b147252 100644 --- a/src/crepe/api/Script.cpp +++ b/src/crepe/api/Script.cpp @@ -61,4 +61,3 @@ bool Script::get_key_state(Keycode key) const noexcept { return false; } } - diff --git a/src/crepe/facade/Texture.cpp b/src/crepe/facade/Texture.cpp index cd06439..6fb22e1 100644 --- a/src/crepe/facade/Texture.cpp +++ b/src/crepe/facade/Texture.cpp @@ -1,8 +1,8 @@ -#include "../util/dbg.h" +#include "../Resource.h" #include "../facade/SDLContext.h" #include "../manager/Mediator.h" #include "../types.h" -#include "../Resource.h" +#include "../util/dbg.h" #include "Texture.h" @@ -27,4 +27,3 @@ const ivec2 & Texture::get_size() const noexcept { return this->size; } const float & Texture::get_ratio() const noexcept { return this->aspect_ratio; } SDL_Texture * Texture::get_img() const noexcept { return this->texture.get(); } - diff --git a/src/crepe/system/ScriptSystem.cpp b/src/crepe/system/ScriptSystem.cpp index 9977396..93b4853 100644 --- a/src/crepe/system/ScriptSystem.cpp +++ b/src/crepe/system/ScriptSystem.cpp @@ -19,7 +19,8 @@ void ScriptSystem::frame_update() { this->update(&Script::frame_update, delta_time); } -void ScriptSystem::update(void (Script::* update_function)(duration_t), const duration_t & delta_time) { +void ScriptSystem::update(void (Script::*update_function)(duration_t), + const duration_t & delta_time) { ComponentManager & mgr = this->mediator.component_manager; RefVector behavior_scripts = mgr.get_components_by_type(); @@ -37,4 +38,3 @@ void ScriptSystem::update(void (Script::* update_function)(duration_t), const du (*script.*update_function)(delta_time); } } - diff --git a/src/crepe/system/ScriptSystem.h b/src/crepe/system/ScriptSystem.h index ca4534a..095515f 100644 --- a/src/crepe/system/ScriptSystem.h +++ b/src/crepe/system/ScriptSystem.h @@ -36,7 +36,7 @@ public: void frame_update() override; private: - void update(void (Script::* update_function)(duration_t), const duration_t & delta_time); + void update(void (Script::*update_function)(duration_t), const duration_t & delta_time); }; } // namespace crepe -- cgit v1.2.3 From b01252d280b13de9732a0095f7cbf85ed9068aa1 Mon Sep 17 00:00:00 2001 From: Loek Le Blansch Date: Wed, 18 Dec 2024 14:23:34 +0100 Subject: update ScriptSystem doxygen --- src/crepe/system/ScriptSystem.h | 27 +++++++++++---------------- 1 file changed, 11 insertions(+), 16 deletions(-) (limited to 'src/crepe/system') diff --git a/src/crepe/system/ScriptSystem.h b/src/crepe/system/ScriptSystem.h index 095515f..cb214bd 100644 --- a/src/crepe/system/ScriptSystem.h +++ b/src/crepe/system/ScriptSystem.h @@ -11,31 +11,26 @@ class Script; /** * \brief Script system * - * The script system is responsible for all \c BehaviorScript components, and - * calls the methods on classes derived from \c Script. + * The script system is responsible for all \c BehaviorScript components, and calls the methods + * on classes derived from \c Script. */ class ScriptSystem : public System { public: using System::System; - /** - * \brief Call Script::fixed_update() on all active \c BehaviorScript instances - * - * This routine updates all scripts sequentially using the Script::update() - * method. It also calls Script::init() if this has not been done before on - * the \c BehaviorScript instance. - */ + +public: + //! Call Script::fixed_update() on all active \c BehaviorScript instances void fixed_update() override; + //! Call Script::frame_update() on all active \c BehaviorScript instances + void frame_update() override; +private: /** - * \brief Call Script::frame_update() on all active \c BehaviorScript instances + * \brief Call Script `*_update` member function on all active \c BehaviorScript instances * - * This routine updates all scripts sequentially using the Script::update() - * method. It also calls Script::init() if this has not been done before on - * the \c BehaviorScript instance. + * \note This routine also calls Script::init() if this has not been done before on the \c + * BehaviorScript instance. */ - void frame_update() override; - -private: void update(void (Script::*update_function)(duration_t), const duration_t & delta_time); }; -- cgit v1.2.3 From 01e77b907cb02ea8a161f620664feb3b16ccb697 Mon Sep 17 00:00:00 2001 From: Loek Le Blansch Date: Wed, 18 Dec 2024 17:58:42 +0100 Subject: feedback --- src/crepe/system/ScriptSystem.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/crepe/system') diff --git a/src/crepe/system/ScriptSystem.h b/src/crepe/system/ScriptSystem.h index cb214bd..257b615 100644 --- a/src/crepe/system/ScriptSystem.h +++ b/src/crepe/system/ScriptSystem.h @@ -1,9 +1,9 @@ #pragma once -#include "System.h" - #include "../manager/LoopTimerManager.h" +#include "System.h" + namespace crepe { class Script; -- cgit v1.2.3