c++组合最佳实践
C++ composition best practices
我很难选择解决问题的最佳方案。目前,我正在做我的第一个c++项目,我使用的是Qt,但是因为我是。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。如果我使用智能指针,如QSharedPointer而不是应用程序*会更好吗?你的经验是什么,缺点是什么,最好的方法是什么。我很乐意听你的建议。这里还有另一个棘手的部分。如果我把这个类交给团队中的另一个开发人员,或者我发布了这个库。开发人员可以轻松地编写如下内容:
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;
};
谢谢。非常感谢你的帮助。
主观看法;
首先:就像Chris说的,智能指针是解决内存泄漏的一个很好的选择。尽量使用它们。
第二:一个好的设计方法是只在创建者中删除指针。我的意思是,一个类创建和销毁它们的指针。不要删除其他类中的指针,这将使你的代码更清晰、可读和可维护。
在你的例子中,规则'must' not delete Application pointer。想想你的设计:我不明白为什么规则是应用程序的父类。我想一个应用程序可能有很多规则。
正如其他人提到的,使用智能指针来表示所有权。当使用Qt时,你应该阅读在Qt内存管理?
相关文章:
- 在C#中处理C++指针而不使用unsafe的最佳方法
- 在C++中,将大的无符号浮点数四舍五入为整数的最佳方法是什么
- 如何在OMNET++中指定与命令行参数组合的输出文件名
- 实现无开销push_back的最佳方法是什么
- 可组合的lambda/std::函数与std::可选
- 如何将两个不同矢量的同一位置的两个元素组合在一起
- 在c代码之间共享数据的最佳方式
- 使用std::source_location报告错误的最佳实践
- 派生类销毁的最佳实践是什么
- 混合组合和继承的C++问题
- 我需要将多个函数组合为一个函数
- 构建可组合有向图(扫描仪生成器的汤普森构造算法)
- 将寄存器设计成可由C和C++访问的外设的最佳实践
- 在两台机器之间进行时间戳的最佳c++chrono函数是什么
- 通过组合不同的类型来创建唯一的id
- 设计生成所有 n 位数字组合的递归函数的最佳方法是什么?
- 找到组合对之间共享元素的最佳方式
- 在C++中定义许多组合对象的最佳方法
- c++组合最佳实践
- 最佳实践是使用枚举使用按位参数组合