diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/crepe/api/Script.h | 17 | ||||
| -rw-r--r-- | src/crepe/system/ScriptSystem.cpp | 18 | ||||
| -rw-r--r-- | src/crepe/system/ScriptSystem.h | 16 | ||||
| -rw-r--r-- | src/test/ScriptTest.cpp | 18 | ||||
| -rw-r--r-- | src/test/ScriptTest.h | 3 | 
5 files changed, 56 insertions, 16 deletions
diff --git a/src/crepe/api/Script.h b/src/crepe/api/Script.h index 2422cdc..5f68928 100644 --- a/src/crepe/api/Script.h +++ b/src/crepe/api/Script.h @@ -48,14 +48,23 @@ protected:  	 */  	virtual void init() {}  	/** -	 * \brief Script update function (empty by default) +	 * \brief Script fixed update function (empty by default)  	 *  	 * \param delta_time Time since last fixed update  	 * -	 * This function is called during the ScriptSystem::update() routine if the \c BehaviorScript -	 * component holding this script instance is active. +	 * \note This function is called during the ScriptSystem::update() routine if the \c +	 * BehaviorScript component holding this script instance is active.  	 */ -	virtual void update(duration_t delta_time) {} +	virtual void fixed_update(duration_t delta_time) {} +	/** +	 * \brief Script frame update function (empty by default) +	 * +	 * \param delta_time Time since last frame update +	 * +	 * \note This function is called during the ScriptSystem::update() routine if the \c +	 * BehaviorScript component holding this script instance is active. +	 */ +	virtual void frame_update(duration_t delta_time) {}  	//! \}  	//! ScriptSystem calls \c init() and \c update() 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<BehaviorScript> behavior_scripts = mgr.get_components_by_type<BehaviorScript>();  	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 diff --git a/src/test/ScriptTest.cpp b/src/test/ScriptTest.cpp index 66b8193..40aa25c 100644 --- a/src/test/ScriptTest.cpp +++ b/src/test/ScriptTest.cpp @@ -28,7 +28,8 @@ void ScriptTest::SetUp() {  TEST_F(ScriptTest, Default) {  	MyScript & script = this->script;  	EXPECT_CALL(script, init()).Times(0); -	EXPECT_CALL(script, update(_)).Times(0); +	EXPECT_CALL(script, fixed_update(_)).Times(0); +	EXPECT_CALL(script, frame_update(_)).Times(0);  }  TEST_F(ScriptTest, UpdateOnce) { @@ -38,7 +39,7 @@ TEST_F(ScriptTest, UpdateOnce) {  		InSequence seq;  		EXPECT_CALL(script, init()).Times(1); -		EXPECT_CALL(script, update(_)).Times(1); +		EXPECT_CALL(script, fixed_update(_)).Times(1);  		system.fixed_update();  	} @@ -46,9 +47,16 @@ TEST_F(ScriptTest, UpdateOnce) {  		InSequence seq;  		EXPECT_CALL(script, init()).Times(0); -		EXPECT_CALL(script, update(_)).Times(1); +		EXPECT_CALL(script, fixed_update(_)).Times(1);  		system.fixed_update();  	} + +	{ +		InSequence seq; + +		EXPECT_CALL(script, frame_update(_)).Times(1); +		system.frame_update(); +	}  }  TEST_F(ScriptTest, UpdateInactive) { @@ -59,7 +67,7 @@ TEST_F(ScriptTest, UpdateInactive) {  		InSequence seq;  		EXPECT_CALL(script, init()).Times(0); -		EXPECT_CALL(script, update(_)).Times(0); +		EXPECT_CALL(script, fixed_update(_)).Times(0);  		behaviorscript.active = false;  		system.fixed_update();  	} @@ -68,7 +76,7 @@ TEST_F(ScriptTest, UpdateInactive) {  		InSequence seq;  		EXPECT_CALL(script, init()).Times(1); -		EXPECT_CALL(script, update(_)).Times(1); +		EXPECT_CALL(script, fixed_update(_)).Times(1);  		behaviorscript.active = true;  		system.fixed_update();  	} diff --git a/src/test/ScriptTest.h b/src/test/ScriptTest.h index f3dbda4..8637df0 100644 --- a/src/test/ScriptTest.h +++ b/src/test/ScriptTest.h @@ -29,7 +29,8 @@ public:  	public:  		MOCK_METHOD(void, init, (), (override)); -		MOCK_METHOD(void, update, (crepe::duration_t), (override)); +		MOCK_METHOD(void, fixed_update, (crepe::duration_t), (override)); +		MOCK_METHOD(void, frame_update, (crepe::duration_t), (override));  	};  	crepe::OptionalRef<crepe::BehaviorScript> behaviorscript;  |