C++ 组合最佳实践答案

【问题标题】：C++ composition best practicesC++ 组合最佳实践
【发布时间】：2011-12-23 04:55:02
【问题描述】：

我在为我的问题选择最佳解决方案时遇到了麻烦。目前，我正在开发我的第一个 C++ 项目，我正在使用 Qt BTW，但是因为我是一名 .NET 开发人员，所以有些事情让我对 C++ 感到不满 :)。众所周知，在 Java 或 .NET 等垃圾收集的世界中，最佳实践是对松散耦合类使用依赖注入，使其更具可测试性和可维护性，但我不知道在 C++ 中该怎么做。 C++ 没有 GC，所以我们应该注意每个内存分配，这会引起很多问题。

例如，我有一个“Rule”类，它有一个指向“Application”类的成员字段：

class Rule : public QObject
{
public:
    explicit Rule(QObject *parent = 0);

    unsigned int id();
    void setId(unsigned int id);

    Application* application();
    void setApplication(Application* application)
    {
        m_Application = application
        m_Application->setParent(this);
    }

};

在构造函数中，我将 NULL 分配给应用程序实例变量。在 setApplication 中，我将其指定为应用程序的父级。多亏了 Qt，当 Rule（父级）被销毁时，应用程序实例将被自动删除。这是一个好方法吗？如果我使用诸如 QSharedPointer 之类的智能指针而不是 Application* 会更好吗？你的经验是什么，有什么缺点，最好的方法是什么。我很乐意听取您的建议。这里还有另一个棘手的部分。如果我将该课程提供给团队中的另一个开发人员，或者我发布了该库会怎样。开发人员可以轻松编写如下内容：

Application app;
app.setId(1);
Rule rule;
rule.setApplication(&app); //When rule will be destroyed, the program would crash because app is allocated on the stack.

或

Application *app = new Application();
app->setId(20);
Rule *rule = new Rule();
rule->setApplication(app);
Application *appToAnotherLocation = new Application();
rule->setApplication(appToAnotherLocation); // This wouldn't result in memory leak, because app is already child of rule, but if I didn't used Qt this would be a problem... probably :)

现在智能指针呢？是否有一些规则，什么时候应该使用它们，什么时候不应该使用它们？我有一个 IRepository 接口，它总是返回 QSharedObject 而不是指针。这是一个好方法还是我过度使用它？

class IRepository
{
public:
    virtual bool save(Application & application) = 0;
    virtual bool save(Rule & rule) = 0;
    virtual bool save(History & history) = 0;

    virtual bool remove(Application & application) = 0;
    virtual bool remove(Rule & rule) = 0;
    virtual bool remove(History & history) = 0;

    virtual QSharedPointer<Application> getApplication(unsigned int id) = 0;

    virtual QSharedPointer<Rule> getRule(unsigned int id) = 0;
    virtual QList< QSharedPointer<Rule> > getRules(unsigned int applicationId) = 0;

    virtual QSharedPointer<History> getHistory(unsigned int id) = 0;
    virtual QList< QSharedPointer<History> > getHistories(unsigned int applicationId) = 0;
    virtual QList< QSharedPointer<History> > getHistories(unsigned int applicationId, QDateTime dateFrom, QDateTime dateTo) = 0;
};

谢谢。非常感谢您的帮助。

【问题讨论】：

您的示例缺少指针（声明中的* 用于指针）。顺便说一句，qt 应用程序有它自己的内存管理
一般来说，智能指针是好的，看这个关于RAII的描述。
使用智能指针来表达所有权。原始指针意味着所有权在其他地方进行管理。您几乎不应该使用 delete 关键字。

标签： c++ oop memory-management

【解决方案1】：

主观意见；

首先：就像 Chris 所说，智能指针是内存泄漏的一个很好的替代方案。尽量使用它们。

第二：一个好的设计方法是只在创建者中删除指针。我的意思是，一个类创建和销毁它们的指针。不要删除其他类中的指针，这将使您拥有更清晰、可读和可维护的代码。

在您的示例中，规则“不得”删除应用程序指针。想想你的设计：我不明白为什么规则是应用程序的父级。我想一个应用程序可能会使用很多规则。

【讨论】：

【解决方案2】：

正如其他人所提到的，使用智能指针来表达所有权。使用 Qt 时，您应该阅读 Memory management in Qt?

【讨论】：