qosd: создание узлов переведено на фабричный метод
Решил обойтись простым ~~советским~~ статическим методом:
template <typename T>
class FabricMixin {
public:
template <typename... Args>
static std::shared_ptr<T> create(Args&&... args) {
return std::shared_ptr<T>(new T(std::forward<Args>(args)...));
}
};
Ну ладно, он не так просто выглядит на первый взгляд. Но, по сути, всё,
что он делает: параметризует метод типом возвращаемого указателя, и
передаёт все аргументы как есть в конструктор заданного типа. Решил
сделать так, чтобы не копипастить тело конструктора. Ну, вдруг я,
например, трассировку туда добавить захочу. Правда, есть недостаток у
такого решения:
class SimpleNode : ...,
public FabricMixin<SimpleNode> {
friend class FabricMixin<SimpleNode>;
Не очень удобное подключение: а) нужно внести по крайней мере две
строчки; б) автоматически самого себя параметром шаблона передавать
нельзя.
Вносить этот метод в `BaseNode` (или около) не хотел, чтобы не
пробрасывать оконечный тип по всей иерархии. Да и от указанных выше
проблем он не избавляет. Зато можно будет относительно безболезненно
выпилить этот класс, если ему подвернётся достойная замена.
This commit is contained in:
@@ -0,0 +1,11 @@
|
||||
#pragma once
|
||||
#include <memory>
|
||||
|
||||
template <typename T>
|
||||
class FabricMixin {
|
||||
public:
|
||||
template <typename... Args>
|
||||
static std::shared_ptr<T> create(Args&&... args) {
|
||||
return std::shared_ptr<T>(new T(std::forward<Args>(args)...));
|
||||
}
|
||||
};
|
||||
@@ -2,16 +2,20 @@
|
||||
|
||||
#include "nodes/BaseNode.h"
|
||||
#include "mixins/HierarchicalLinkMixin.h"
|
||||
#include "mixins/FabricMixin.h"
|
||||
#include "Logger.h"
|
||||
|
||||
/// \brief Класс сложного (составного) узла дерева.
|
||||
/// Может содержать несколько дочерних ComplexNode и один SimpleNode.
|
||||
class ComplexNode : public BaseNode,
|
||||
virtual public HierarchicalLinkMixin<INode> {
|
||||
virtual public HierarchicalLinkMixin<INode>,
|
||||
public FabricMixin<ComplexNode> {
|
||||
public:
|
||||
~ComplexNode() {
|
||||
Logger::get("ConDes").dbg(std::string("--- Complex destructor called for: ") + name_);
|
||||
}
|
||||
private:
|
||||
friend class FabricMixin<ComplexNode>;
|
||||
ComplexNode(std::string name) : BaseNode(std::move(name)) {
|
||||
Logger::get("ConDes").dbg(std::string("--- Complex constructor called for: ") + name_);
|
||||
}
|
||||
|
||||
@@ -2,17 +2,21 @@
|
||||
|
||||
#include "nodes/BaseNode.h"
|
||||
#include "mixins/LazyLinkMixin.h"
|
||||
#include "mixins/FabricMixin.h"
|
||||
#include "links/OneToOneLink.h"
|
||||
#include "Logger.h"
|
||||
|
||||
/// \brief Класс простого (листового) узла дерева.
|
||||
/// Может содержать только одного дочернего ComplexNode.
|
||||
class SimpleNode : public BaseNode,
|
||||
virtual public LazyLinkMixin<OneToOneLink<INode>> {
|
||||
virtual public LazyLinkMixin<OneToOneLink<INode>>,
|
||||
public FabricMixin<SimpleNode> {
|
||||
public:
|
||||
~SimpleNode() {
|
||||
Logger::get("ConDes").dbg(std::string("--- Simple destructor called for: ") + name_);
|
||||
}
|
||||
private:
|
||||
friend class FabricMixin<SimpleNode>;
|
||||
SimpleNode(std::string name) : BaseNode(std::move(name)) {
|
||||
Logger::get("ConDes").dbg(std::string("--- Simple constructor called for: ") + name_);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user