在类中创建对象的最佳实践

Question

我有一个通用类 Player，在该类中我声明了一个较小类的对象，例如 Weapon、Inventory 等。

当然，我使用指针 (shared_ptr) 来实例化我的 Player 对象，并使用相同的方法来实例化我的类中的任何对象。我应该继续这样做还是应该尝试通过不使用指针来使其更简单，这会带来一定的缺点吗？

class Player: public Human, public Control
    {
    public:
        Player();
       ~Player(){};

    private:
        Weapon           myWeapon;
        std::vector<std::tr1::shared_ptr<Weapon> > WeaponsList;

    };

对比

class Player: public Human, public Control
    {
    public:
        Player();
       ~Player(){};

    private:
        Weapon           myWeapon;
        std::vector<Weapon> WeaponsList;

    };

注意，我使用 shared_ptr 并希望保留一个选项，让 Weapon 在某个时候成为基类（但不要着急！）。

Answer 1

肯定是前者（即，是的，继续使用指针），只需确保定义了一个灵活的抽象类Weapon，其中包含播放器需要与之交互的所有方法/属性；您希望尽可能避免解析派生类型（这会使代码变得混乱且速度较慢）。

如果您不熟悉设计模式，我建议您阅读Factories 以及在这种情况下如何使用它们来处理分配。这将使您的代码易于维护/理解，如果有人想稍后在您的游戏中添加另一种武器。您将需要使用命名标签来阐明您的代码并选择要在您的工厂中构建的对象；这可以使用enums 来完成，但如果您不能使用强类型enums (c++11)，我倾向于使用命名空间或结构，例如：

struct WeaponID
{
    static const int DESERT_EAGLE = 0;
    static const int SHOTGUN = 1;
} 
// ... use in a switch/case e.g. WeaponID::SHOTGUN

您可以轻松查看如何在开关中使用此类标签并输出指向抽象类的共享指针（如果需要，您甚至可以定义别名标签）。

如果您怀疑您可能需要经常解析派生类型，那么考虑将前一个标记存储为抽象类的成员，这将比使用 dynamic_casts 更容易和更清晰。

如果您担心性能（我想您是 120 fps），请考虑使用 memory pools 的对象（武器、玩家等），尽管在游戏的上下文中，我预计数量游戏开始时对象（几乎）保持不变？

Answer 2

根据最初提供的信息，我认为唯一的答案是“视情况而定”。

• 如果你的WeaponsList 是一个指针向量，什么负责清除各种Weapon 对象？
• 如果WeaponsList是Weapon的向量，则需要保证拷贝构造函数是完整的。
• 什么是Weapon？它是一个基类，您将从中派生Sword、Dagger、Lance 等？或者武器的类型仅仅是物体的一个属性？
• 可以在没有Player 的情况下存在Weapon 对象吗？是否会有所有Player 对象指向的武器共享列表？

换句话说，我认为我们没有足够的细节来完全回答您的问题。

更新

使用评论中提供的其他信息：

如果你创建一个对象，它需要被清除（销毁）；当您使用 shared_ptr 时，应该自动处理。

由于Weapon 可以在没有Player 的情况下存在并且将来可能成为基类，因此使用指针向量可能是要走的路。