C++组合最佳实践

Question

我有麻烦选择我的问题的最佳解决方案。目前，我正在开发我的第一个C++项目，我正在使用Qt BTW，但是因为我是.NET开发人员，所以有些东西让我对C++有点了解:)。大家知道，在一个收集世界的Java或.NET中，最好的做法是使用依赖注入来实现松耦合类，使其更具可测试性和可维护性，但我不知道在C++中该做什么。 C++没有GC，所以我们应该关心每个内存分配，并且会导致太多问题。

例如，我有一个类“规则”，其中有一个成员字段指向“应用程序”类：

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; 
}; 

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

Answer 1

主观意见;

第一：就像克里斯说的，智能指针是内存泄漏的一个很好的选择。尽量使用它们。

第二：一个好的设计方法是只删除创建者中的指针。我的意思是，一个班级创建并破坏他们的指针。不要删除其他类中的指针，这将允许您拥有更清晰，可读和可维护的代码。

在您的示例中，Rule'must'不会删除应用程序指针。想想你的设计：我不明白为什么规则是应用程序的父项。我想应用程序可以处理很多规则。

Answer 2

正如其他人所述，使用智能指针来表示所有权。在使用Qt时，你应该阅读Memory management in Qt?