diff options
| -rw-r--r-- | .clang-tidy | 2 | ||||
| -rw-r--r-- | src/crepe/Component.cpp | 2 | ||||
| -rw-r--r-- | src/crepe/Component.h | 2 | ||||
| -rw-r--r-- | src/crepe/api/GameObject.cpp | 14 | ||||
| -rw-r--r-- | src/crepe/api/GameObject.h | 2 | ||||
| -rw-r--r-- | src/crepe/api/GameObject.hpp | 2 | ||||
| -rw-r--r-- | src/crepe/api/Metadata.cpp | 2 | ||||
| -rw-r--r-- | src/crepe/api/Metadata.h | 4 | ||||
| -rw-r--r-- | src/crepe/api/Script.hpp | 2 | ||||
| -rw-r--r-- | src/crepe/system/PhysicsSystem.cpp | 4 | ||||
| -rw-r--r-- | src/crepe/system/RenderSystem.cpp | 2 | ||||
| -rw-r--r-- | src/example/ecs.cpp | 6 | ||||
| -rw-r--r-- | src/example/scene_manager.cpp | 8 | 
13 files changed, 27 insertions, 25 deletions
diff --git a/.clang-tidy b/.clang-tidy index 3408f75..4d8170b 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -20,6 +20,8 @@ CheckOptions:      value: '_.*'    - key: 'readability-identifier-naming.ConstantParameterCase'      value: 'lower_case' +  - key: 'readability-identifier-naming.ConstantMemberCase' +    value: 'lower_case'    - key: 'readability-identifier-naming.VariableCase'      value: 'lower_case'    - key: 'readability-identifier-naming.VariableIgnoredRegexp' diff --git a/src/crepe/Component.cpp b/src/crepe/Component.cpp index af439ed..73c2d07 100644 --- a/src/crepe/Component.cpp +++ b/src/crepe/Component.cpp @@ -3,4 +3,4 @@  using namespace crepe; -Component::Component(game_object_id_t id) : GAME_OBJECT_ID(id) {} +Component::Component(game_object_id_t id) : game_object_id(id) {} diff --git a/src/crepe/Component.h b/src/crepe/Component.h index ee795f4..0fe60b2 100644 --- a/src/crepe/Component.h +++ b/src/crepe/Component.h @@ -38,7 +38,7 @@ public:  public:  	//! The id of the GameObject this component belongs to -	const game_object_id_t GAME_OBJECT_ID; +	const game_object_id_t game_object_id;  	//! Whether the component is active  	bool active = true;  }; diff --git a/src/crepe/api/GameObject.cpp b/src/crepe/api/GameObject.cpp index e009288..15330f3 100644 --- a/src/crepe/api/GameObject.cpp +++ b/src/crepe/api/GameObject.cpp @@ -9,11 +9,11 @@ using namespace std;  GameObject::GameObject(game_object_id_t id, const std::string & name,  					   const std::string & tag, const Point & position,  					   double rotation, double scale) -	: ID(id) { +	: id(id) {  	// Add Transform and Metadata components  	ComponentManager & mgr = ComponentManager::get_instance(); -	mgr.add_component<Transform>(this->ID, position, rotation, scale); -	mgr.add_component<Metadata>(this->ID, name, tag); +	mgr.add_component<Transform>(this->id, position, rotation, scale); +	mgr.add_component<Metadata>(this->id, name, tag);  }  void GameObject::set_parent(const GameObject & parent) { @@ -21,11 +21,11 @@ void GameObject::set_parent(const GameObject & parent) {  	// Set parent on own Metadata component  	vector<reference_wrapper<Metadata>> this_metadata -		= mgr.get_components_by_id<Metadata>(this->ID); -	this_metadata.at(0).get().parent = parent.ID; +		= mgr.get_components_by_id<Metadata>(this->id); +	this_metadata.at(0).get().parent = parent.id;  	// Add own id to children list of parent's Metadata component  	vector<reference_wrapper<Metadata>> parent_metadata -		= mgr.get_components_by_id<Metadata>(parent.ID); -	parent_metadata.at(0).get().children.push_back(this->ID); +		= mgr.get_components_by_id<Metadata>(parent.id); +	parent_metadata.at(0).get().children.push_back(this->id);  } diff --git a/src/crepe/api/GameObject.h b/src/crepe/api/GameObject.h index 223d72e..8389e6c 100644 --- a/src/crepe/api/GameObject.h +++ b/src/crepe/api/GameObject.h @@ -57,7 +57,7 @@ public:  public:  	//! The id of the GameObject -	const game_object_id_t ID; +	const game_object_id_t id;  };  } // namespace crepe diff --git a/src/crepe/api/GameObject.hpp b/src/crepe/api/GameObject.hpp index 7e6148c..bfba7fe 100644 --- a/src/crepe/api/GameObject.hpp +++ b/src/crepe/api/GameObject.hpp @@ -9,7 +9,7 @@ namespace crepe {  template <typename T, typename... Args>  T & GameObject::add_component(Args &&... args) {  	ComponentManager & mgr = ComponentManager::get_instance(); -	return mgr.add_component<T>(this->ID, std::forward<Args>(args)...); +	return mgr.add_component<T>(this->id, std::forward<Args>(args)...);  }  } // namespace crepe diff --git a/src/crepe/api/Metadata.cpp b/src/crepe/api/Metadata.cpp index ab9612b..76f11d7 100644 --- a/src/crepe/api/Metadata.cpp +++ b/src/crepe/api/Metadata.cpp @@ -4,4 +4,4 @@ using namespace crepe;  using namespace std;  Metadata::Metadata(game_object_id_t id, const string & name, const string & tag) -	: Component(id), NAME(name), TAG(tag) {} +	: Component(id), name(name), tag(tag) {} diff --git a/src/crepe/api/Metadata.h b/src/crepe/api/Metadata.h index f5e5c55..c61e006 100644 --- a/src/crepe/api/Metadata.h +++ b/src/crepe/api/Metadata.h @@ -31,9 +31,9 @@ public:  public:  	//! The name of the GameObject -	const std::string NAME; +	const std::string name;  	//! The tag of the GameObject -	const std::string TAG; +	const std::string tag;  	//! The id of the parent GameObject (-1 if no parent)  	game_object_id_t parent = -1;  	//! The ids of the children GameObjects diff --git a/src/crepe/api/Script.hpp b/src/crepe/api/Script.hpp index 6d111af..d96c0e8 100644 --- a/src/crepe/api/Script.hpp +++ b/src/crepe/api/Script.hpp @@ -19,7 +19,7 @@ T & Script::get_component() {  template <typename T>  std::vector<std::reference_wrapper<T>> Script::get_components() {  	ComponentManager & mgr = ComponentManager::get_instance(); -	return mgr.get_components_by_id<T>(this->parent->GAME_OBJECT_ID); +	return mgr.get_components_by_id<T>(this->parent->game_object_id);  }  } // namespace crepe diff --git a/src/crepe/system/PhysicsSystem.cpp b/src/crepe/system/PhysicsSystem.cpp index dd80312..cea8062 100644 --- a/src/crepe/system/PhysicsSystem.cpp +++ b/src/crepe/system/PhysicsSystem.cpp @@ -23,12 +23,12 @@ void PhysicsSystem::update() {  		switch (rigidbody.body_type) {  			case BodyType::DYNAMIC:  				for (Transform & transform : transforms) { -					if (transform.GAME_OBJECT_ID == rigidbody.GAME_OBJECT_ID) { +					if (transform.game_object_id == rigidbody.game_object_id) {  						rigidbody.velocity_x = 0;  						rigidbody.velocity_y = 0;  						std::vector<std::reference_wrapper<Force>> forces  							= mgr.get_components_by_id<Force>( -								rigidbody.GAME_OBJECT_ID); +								rigidbody.game_object_id);  						rigidbody.velocity_y  							+= rigidbody.gravity_scale * 1 * rigidbody.mass; diff --git a/src/crepe/system/RenderSystem.cpp b/src/crepe/system/RenderSystem.cpp index 2003eaf..5a07cc2 100644 --- a/src/crepe/system/RenderSystem.cpp +++ b/src/crepe/system/RenderSystem.cpp @@ -32,7 +32,7 @@ void RenderSystem::update() {  	for (const Sprite & sprite : sprites) {  		std::vector<std::reference_wrapper<Transform>> transforms -			= mgr.get_components_by_id<Transform>(sprite.GAME_OBJECT_ID); +			= mgr.get_components_by_id<Transform>(sprite.game_object_id);  		for (const Transform & transform : transforms) {  			render.draw(sprite, transform);  		} diff --git a/src/example/ecs.cpp b/src/example/ecs.cpp index 7593faf..0c64373 100644 --- a/src/example/ecs.cpp +++ b/src/example/ecs.cpp @@ -38,8 +38,8 @@ int main() {  	// Print the Metadata and Transform components  	for (auto & m : metadata) { -		cout << "Id: " << m.get().GAME_OBJECT_ID << " Name: " << m.get().NAME -			 << " Tag: " << m.get().TAG << " Parent: " << m.get().parent +		cout << "Id: " << m.get().game_object_id << " Name: " << m.get().name +			 << " Tag: " << m.get().tag << " Parent: " << m.get().parent  			 << " Children: ";  		for (auto & c : m.get().children) {  			cout << c << " "; @@ -47,7 +47,7 @@ int main() {  		cout << endl;  	}  	for (auto & t : transform) { -		cout << "Id: " << t.get().GAME_OBJECT_ID << " Position: [" +		cout << "Id: " << t.get().game_object_id << " Position: ["  			 << t.get().position.x << ", " << t.get().position.y << "]" << endl;  	} diff --git a/src/example/scene_manager.cpp b/src/example/scene_manager.cpp index bfa9479..efbf2c2 100644 --- a/src/example/scene_manager.cpp +++ b/src/example/scene_manager.cpp @@ -52,8 +52,8 @@ int main() {  	cout << "Metadata components of Scene1:" << endl;  	// Print the Metadata  	for (auto & m : metadata) { -		cout << "Id: " << m.get().GAME_OBJECT_ID << " Name: " << m.get().NAME -			 << " Tag: " << m.get().TAG << endl; +		cout << "Id: " << m.get().game_object_id << " Name: " << m.get().name +			 << " Tag: " << m.get().tag << endl;  	}  	// Set scene2 as the next scene @@ -67,8 +67,8 @@ int main() {  	cout << "Metadata components of Scene2:" << endl;  	// Print the Metadata  	for (auto & m : metadata) { -		cout << "Id: " << m.get().GAME_OBJECT_ID << " Name: " << m.get().NAME -			 << " Tag: " << m.get().TAG << endl; +		cout << "Id: " << m.get().game_object_id << " Name: " << m.get().name +			 << " Tag: " << m.get().tag << endl;  	}  	return 0;  |