qosd: исправлен комментарий

Примеров итератора с доступом на чтение я много оставил, а пример
модификации дерева только один. И тот я сразу забыл перевести на
итераторы (range-based for loop).

Т.к. проход теперь не по вектору (он давал экземпляр умного указателя),
а по обычному указателю (собственному прокси, если точнее), то узел
разрушался уже в процессе его отключения от дерева. Добавил удержание
ссылки в сам `unlinkParent()` (ну, нам действительно может быть нужно
только безвозвратное удаление поддерева) и пример в `main.cpp` расширил
и прокомментировал.

include/links/LeafLink.h

@@ -3,14 +3,10 @@
 #include <stdexcept>
 
 /// \brief Связь для листового узла, не допускающая дочерних элементов.
-template <class TElem>
+class LeafLink : public BaseLink {
-class LeafLink : public BaseLink<TElem> {
 public:
-	using ElemPtr = std::shared_ptr<TElem>;
+	using BaseLink::BaseLink;
-
+	void addChild(const NodePtr&) override {
-	LeafLink(std::shared_ptr<TElem> e) : BaseLink<TElem>(e) {}
+		throw std::logic_error("LeafLink cannot have children");
-
-	void addChild(const ElemPtr&) override {
-		throw std::logic_error("Leaf cannot have children");
 	}
 };

include/mixins/HierarchicalLinkMixin.h

@@ -1,21 +1,18 @@
 #pragma once
 #include <iostream>
-#include <memory>
-
-#include "ifaces/INode.h"
 #include "mixins/LazyLinkMixin.h"
+#include <memory>
 #include "links/OneToManyLink.h"
 #include "links/OneToOneLink.h"
 #include "Logger.h"
 
 /// \brief Миксин для иерархических связей между элементами.
-/// \tparam INode Тип дочернего элемента.
+/// Автоматически выбирает тип связи (один-ко-многим или один-к-одному) в зависимости от типа дочернего элемента.
-/// Автоматически выбирает тип связи (один-ко-многим или один-к-одному) в зависимости от типа
+/// \tparam TElem Тип дочернего элемента.
-/// дочернего узла. Если тип дочернего узла совпадает с родителем, то используется связь
+template <class TElem>
-/// один-ко-многим. При попытке подключить узел отличного типа выбирается связь один-к-одному.
+class HierarchicalLinkMixin : public LazyLinkMixin<OneToOneLink<TElem>> {
-class HierarchicalLinkMixin : public LazyLinkMixin<OneToOneLink<INode>> {
+	using LinkPtr = std::shared_ptr<ILink<TElem>>;
-	using LinkPtr = std::shared_ptr<ILink<INode>>;
+	using ElemPtr = std::shared_ptr<TElem>;
-	using ElemPtr = std::shared_ptr<INode>;
 
 public:
 	~HierarchicalLinkMixin() override {
@@ -24,7 +21,7 @@ public:
 
 	void linkChild(const ElemPtr& child) override {
 		hierarchicalInit(child);
-		LazyLinkMixin<OneToOneLink<INode>>::linkChild(child);
+		LazyLinkMixin<OneToOneLink<TElem>>::linkChild(child);
 	}
 
 protected:
@@ -37,10 +34,10 @@ protected:
 
 		if (typeid(*child) == typeid(*this)) {
 			Logger::get("Mixin").dbg("---  Mutate to OneToMany");
-			newLink = std::make_shared<OneToManyLink<INode>>(*this);
+			newLink = std::make_shared<OneToManyLink<TElem>>(*this);
 		} else {
 			Logger::get("Mixin").dbg("---  Mutate to OneToOne");
-			newLink = std::make_shared<OneToOneLink<INode>>(*this);
+			newLink = std::make_shared<OneToOneLink<TElem>>(*this);
 		}
 
 		if (newLink && this->link_)

include/nodes/ComplexNode.h

@@ -8,7 +8,7 @@
 /// \brief Класс сложного (составного) узла дерева.
 /// Может содержать несколько дочерних ComplexNode или один SimpleNode.
 class ComplexNode : public BaseNode,
-		    virtual public HierarchicalLinkMixin,
+		    virtual public HierarchicalLinkMixin<INode>,
 		    public FabricMixin<ComplexNode> {
 public:
 	~ComplexNode() {

include/nodes/LeafNode.h

@@ -1,23 +0,0 @@
-#pragma once
-
-#include "nodes/BaseNode.h"
-#include "mixins/LazyLinkMixin.h"
-#include "mixins/FabricMixin.h"
-#include "links/LeafLink.h"
-#include "Logger.h"
-
-/// \brief Класс простого (листового) узла дерева.
-/// Может содержать только одного дочернего ComplexNode.
-class LeafNode : public BaseNode,
-		 virtual public LazyLinkMixin<LeafLink<INode>>,
-		 public FabricMixin<LeafNode> {
-public:
-	~LeafNode() {
-		Logger::get("ConDes").dbg(std::string("--- Leaf destructor called for: ") + name_);
-	}
-private:
-	friend class FabricMixin<LeafNode>;
-	LeafNode(std::string name) : BaseNode(std::move(name)) {
-		Logger::get("ConDes").dbg(std::string("--- Leaf constructor called for: ") + name_);
-	}
-};

src/main.cpp

@@ -4,7 +4,6 @@
 
 #include "nodes/SimpleNode.h"
 #include "nodes/ComplexNode.h"
-#include "nodes/LeafNode.h"
 #include "iterators/Traversal.h"
 
 
@@ -59,18 +58,9 @@ int main() {
 			auto child3 = ComplexNode::create("ComplexChild3");
 			root->linkChild(child3);
 
-			auto leaf3 = LeafNode::create("LeafSubChild3");
+			child3->linkChild(SimpleNode::create("SimpleSubChild3"));
-			child3->linkChild(leaf3);
 
 			{
-				// Негативный сценарий 0: попытка добавить SimpleNode к LeafNode
-				try {
-					leaf3->linkChild(SimpleNode::create("ShouldFail"));
-					logger.err("[ERROR] Не должно было получиться добавить SimpleNode к LeafNode!");
-				} catch (const std::logic_error& e) {
-					logger.warn(std::string("[Ожидаемое исключение] ") + e.what());
-				}
-
 				// Негативный сценарий 1: попытка добавить второй SimpleNode к ComplexNode
 				try {
 					child2->linkChild(SimpleNode::create("ShouldFail"));