-replace getInterface with dynamic_cast

Author: kamchatka-volcano

src/attribute_node.h

@@ -1,15 +1,14 @@
 #pragma once
 #include "iattribute.h"
 #include "idocumentnode.h"
+#include <string>
 
 namespace htcpp {
 
 class StreamReader;
 
 class AttributeNode : public IDocumentNode,
                       public IAttribute {
-    DOCUMENT_NODE_INTERFACE_ACCESS(IAttribute)
-
 public:
     explicit AttributeNode(std::string name, StreamReader& stream);
     std::string_view name() const override;

src/codenode.cpp

@@ -1,7 +1,8 @@
 #include "codenode.h"
-#include "streamreader.h"
 #include "errors.h"
+#include "streamreader.h"
 #include "utils.h"
+#include <gsl/assert>
 #include <utility>
 
 namespace htcpp{

src/codenode.h

@@ -1,5 +1,4 @@
 #pragma once
-#include "document_node_interface_access.h"
 #include "idocumentnode.h"
 #include "idocumentnoderenderer.h"
 #include "nodeextension.h"
@@ -27,7 +26,6 @@ class CodeNode {
 
 class ExpressionNode : public IDocumentNode,
                        public IDocumentNodeRenderer {
-    DOCUMENT_NODE_INTERFACE_ACCESS(IDocumentNodeRenderer)
 public:
     explicit ExpressionNode(StreamReader& stream);
     std::string renderingCode() const override;
@@ -38,8 +36,6 @@ class ExpressionNode : public IDocumentNode,
 
 class StatementNode : public IDocumentNode,
                       public IDocumentNodeRenderer {
-    DOCUMENT_NODE_INTERFACE_ACCESS(IDocumentNodeRenderer)
-
 public:
     explicit StatementNode(StreamReader& stream);
     std::string renderingCode() const override;
@@ -50,7 +46,6 @@ class StatementNode : public IDocumentNode,
 
 class GlobalStatementNode : public IDocumentNode,
                             public IDocumentNodeRenderer {
-    DOCUMENT_NODE_INTERFACE_ACCESS(IDocumentNodeRenderer)
 public:
     explicit GlobalStatementNode(StreamReader& stream);
     std::string renderingCode() const override;

src/control_flow_statement_node.h

@@ -1,10 +1,7 @@
 #pragma once
-#include "document_node_interface_access.h"
 #include "idocumentnode.h"
 #include "idocumentnoderenderer.h"
 #include "nodeextension.h"
-#include <sfun/optional_ref.h>
-#include <gsl/assert>
 
 namespace htcpp {
 
@@ -15,7 +12,6 @@ enum class ControlFlowStatementNodeType {
 
 class ControlFlowStatementNode : public IDocumentNode,
                                  public IDocumentNodeRenderer {
-    DOCUMENT_NODE_INTERFACE_ACCESS(IDocumentNodeRenderer)
 
 public:
     ControlFlowStatementNode(ControlFlowStatementNodeType, NodeExtension nodeExtension);

src/document_node_interface_access.h

@@ -1,12 +1,13 @@
 #pragma once
 #include <sfun/interface.h>
+#include <optional>
 #include <string_view>
 
 namespace htcpp{
 
 class IConvertibleToProcedure : private sfun::interface<IConvertibleToProcedure> {
 public:
-    virtual std::string_view procedureName() const = 0;
+    virtual std::optional<std::string_view> procedureName() const = 0;
 };
 
 }

src/iconvertible_to_procedure.h

@@ -1,60 +1,18 @@
 #pragma once
-#include "document_node_interface_access.h"
 #include <sfun/interface.h>
-#include <sfun/optional_ref.h>
-#include <gsl/assert>
-#include <string>
 #include <typeinfo>
-#include <memory>
-#include <vector>
 
 namespace htcpp {
-class INodeCollection;
-class IDocumentNodeRenderer;
-class IRenderedAsStringPart;
-class IConvertibleToProcedure;
-class IAttribute;
-
-template<typename TInterface>
-struct InterfaceGetterMapping;
 
 class IDocumentNode : private sfun::interface<IDocumentNode>
 {
 public:
-    template<typename TInterface>
-    auto getInterface() const
-    {
-        return (this->*InterfaceGetterMapping<TInterface>::getterPtr())();
-    }
-
-    template<typename TInterface>
-    auto getInterface()
-    {
-        return (this->*InterfaceGetterMapping<TInterface>::getterPtr())();
-    }
-
     template<typename T>
     bool is()
     {
         return typeid(*this) == typeid(T);
     }
-
-private:
-    DOCUMENT_NODE_ADD_INTERFACE_GETTER(INodeCollection);
-    DOCUMENT_NODE_ADD_INTERFACE_GETTER(IDocumentNodeRenderer);
-    DOCUMENT_NODE_ADD_INTERFACE_GETTER(IRenderedAsStringPart);
-    DOCUMENT_NODE_ADD_INTERFACE_GETTER(IConvertibleToProcedure);
-    DOCUMENT_NODE_ADD_INTERFACE_GETTER(IAttribute);
-
-    template<typename TInterface>
-    friend struct InterfaceGetterMapping;
 };
 
-DOCUMENT_NODE_REGISTER_INTERFACE(INodeCollection);
-DOCUMENT_NODE_REGISTER_INTERFACE(IDocumentNodeRenderer);
-DOCUMENT_NODE_REGISTER_INTERFACE(IRenderedAsStringPart);
-DOCUMENT_NODE_REGISTER_INTERFACE(IConvertibleToProcedure);
-DOCUMENT_NODE_REGISTER_INTERFACE(IAttribute);
-
 
 } // namespace htcpp

src/idocumentnode.h

@@ -8,7 +8,7 @@ std::vector<std::unique_ptr<IDocumentNode>> flattenNodes(std::vector<std::unique
 {
     auto flattenNodes = std::vector<std::unique_ptr<IDocumentNode>>{};
     for (auto& node : nodes) {
-        if (const auto nodeCollection = node->template getInterface<INodeCollection>())
+        if (const auto nodeCollection = dynamic_cast<INodeCollection*>(node.get()))
             std::ranges::move(nodeCollection->flatten(), std::back_inserter(flattenNodes));
         else
             flattenNodes.emplace_back(std::move(node));
@@ -20,9 +20,10 @@ std::vector<std::unique_ptr<IDocumentNode>> optimizeNodes(std::vector<std::uniqu
 {
     auto processedNodes = std::vector<std::unique_ptr<IDocumentNode>>{};
     for (auto& node : nodes) {
-        if (auto text = node->getInterface<IRenderedAsStringPart>()) {
-            if (!processedNodes.empty() && processedNodes.back()->getInterface<IRenderedAsStringPart>()) {
-                auto prevText = processedNodes.back()->getInterface<IRenderedAsStringPart>()->content();
+        if (auto text = dynamic_cast<IRenderedAsStringPart*>(node.get())) {
+            if (!processedNodes.empty() &&
+                dynamic_cast<IRenderedAsStringPart*>(processedNodes.back().get()) != nullptr) {
+                const auto prevText = dynamic_cast<IRenderedAsStringPart*>(processedNodes.back().get())->content();
                 processedNodes.back() = std::make_unique<TextNode>(prevText + text->content());
             }
             else

src/node_utils.cpp

@@ -1,10 +1,11 @@
 #include "procedurenode.h"
 #include "errors.h"
 #include "idocumentnoderenderer.h"
+#include "node_utils.h"
 #include "nodereader.h"
 #include "streamreader.h"
 #include "utils.h"
-#include "node_utils.h"
+#include <gsl/assert>
 
 namespace htcpp{
 
@@ -58,7 +59,7 @@ std::string ProcedureNode::renderingCode() const
 {
     auto result = std::string{};
     for (auto& node : contentNodes_)
-        result += node->getInterface<IDocumentNodeRenderer>()->renderingCode();
+        result += dynamic_cast<IDocumentNodeRenderer*>(node.get())->renderingCode();
     return result;
 }
 

src/procedurenode.cpp

@@ -1,5 +1,4 @@
 #pragma once
-#include "document_node_interface_access.h"
 #include "idocumentnode.h"
 #include "idocumentnoderenderer.h"
 #include <memory>
@@ -11,7 +10,6 @@ class StreamReader;
 
 class ProcedureNode : public IDocumentNode,
                       public IDocumentNodeRenderer {
-    DOCUMENT_NODE_INTERFACE_ACCESS(IDocumentNodeRenderer)
 
 public:
     ProcedureNode(std::string procedureName, StreamReader& stream);