我正在开发一个学校项目的游戏。我们使用一个由其中一个团队制作的引擎。引擎的构建对我来说不清楚,似乎是一种反模式。然而,似乎没有人能够向我清晰地解释设计选择。该引擎应该使用“组件化”设计,但我看不到它在使用。下面是组件、复合和实体类的代码。我的问题简而言之就是:这段代码是否使用了有效的设计模式,还是过于复杂只是为了“实现设计模式”,从而导致反模式?
下面这段代码是Player.cpp,它使用了组合和实体作为一种混合类型的对象(我真的不理解其逻辑)。
我不相信这是基于组件的设计。应该删除实体(Entity),只使用组合(Composite)和组件(Component)。组件基类应该为空,永远不直接使用。例如一个名为'Rigidbody'的组件,保存刚体数据结构和一些函数,覆盖完全虚拟的组件基类。
Component.cpp:
#include "Engine\Component.h"
#include "Engine\Composite.h"
Component::Component(Composite* parent)
{
this->parent = parent;
}
Component::~Component()
{
}
Entity.cpp
#include "Engine\Entity.h"
#include "Engine\Game.h"
Entity::Entity(Composite* parent):Composite(parent)
{
this->mass = 1;
node = NULL;
}
void Entity::update()
{
Composite::update();
this->angularVelocity += this->angularAccelaration;
this->orientation += this->angularVelocity;
this->accelaration = (1 / this->mass) * this->force;
this->velocity += this->accelaration;
this->position += this->velocity;
if (node != NULL)
{
this->node->setPosition(this->position);
this->node->setRotation(this->orientation);
}
}
void Entity::draw()
{
Composite::draw();
if (node == NULL) return;
if (!this->visible)
{
this->node->setVisible(false);
return;
}
this->node->setVisible(true);
this->node->render();
}
void Entity::createNode(std::string modelPath)
{
// Get the mesh
irr::scene::IAnimatedMesh* mesh = Game::getSceneManager()->getMesh(modelPath.c_str());
// Create model entity
this->node = Game::getSceneManager()->addMeshSceneNode( mesh );
this->node->setMaterialFlag(EMF_FOG_ENABLE, true);
}
Entity::~Entity()
{
Composite::~Composite();
if (node != NULL)
{
node->drop();
}
}
Composite.cpp
#include "Engine\Composite.h"
Composite::Composite(Composite* parent):Component(parent)
{
}
Composite::~Composite()
{
for (std::list<Component*>::iterator i = components.begin(); i != components.end(); ++i)
{
delete (*i);
}
components.clear();
}
void Composite::handleMessage(unsigned int message, void* data)
{
for (std::list<Component*>::iterator i = components.begin(); i != components.end(); ++i)
{
(*i)->handleMessage(message, data);
}
}
void Composite::update()
{
for (std::list<Component*>::iterator i = components.begin(); i != components.end(); ++i)
{
(*i)->update();
}
}
void Composite::draw()
{
for (std::list<Component*>::iterator i = components.begin(); i != components.end(); ++i)
{
(*i)->draw();
}
}
void Composite::addComponent(Component* component)
{
components.push_back(component);
}
void Composite::removeComponent(Component* component)
{
components.remove(component);
delete component;
}
下面这段代码是Player.cpp,它使用了组合和实体作为一种混合类型的对象(我真的不理解其逻辑)。
Player.cpp
#include "Player.h"
#include "Messages.h"
#include <iostream>
Player::Player(Composite* parent) : Entity(parent)
{
createNode("../assets/sydney.md2");
//i = 0;
//v3f = vector3df(0,0,0);
/*networker = new NetworkComponent();
addComponent(networker);
networker->registerVar(&i);
networker->registerVar(&v3f);*/
}
void Player::update() {
Composite::update();
//std::cout<<i<<std::endl;
//std::cout<<"vectorx="<<v3f.X<<"\n";
}
void Player::handleMessage(unsigned int message, void* data) {
switch(message) {
case DAMAGE: /* Do something */;
}
delete data;
}
Player::~Player()
{
Entity::~Entity();
}
我不相信这是基于组件的设计。应该删除实体(Entity),只使用组合(Composite)和组件(Component)。组件基类应该为空,永远不直接使用。例如一个名为'Rigidbody'的组件,保存刚体数据结构和一些函数,覆盖完全虚拟的组件基类。
Player
也应该是一个组件吗? - yngccc