From 085935ddd577b0c65b4330318b5ba20492d93126 Mon Sep 17 00:00:00 2001
From: Adrian Kummerländer
Date: Sat, 24 May 2014 13:59:42 +0200
Subject: Implemented ResultNodeFacade as a DOM node construction helper *
 wraps result node construction and appends it to root node on destruction *
 offers a simpler interface for common node construction patterns * simplifies
 result node construction in all external function implementations ** most
 noticeable in FunctionReadDirectory * expanded FunctionReadDirectory result
 nodes by name, extension, and full-path nodes

---
 src/function/read_file.cc | 29 ++++++-----------------------
 1 file changed, 6 insertions(+), 23 deletions(-)

(limited to 'src/function/read_file.cc')

diff --git a/src/function/read_file.cc b/src/function/read_file.cc
index 1a870a6..87423ab 100644
--- a/src/function/read_file.cc
+++ b/src/function/read_file.cc
@@ -3,11 +3,11 @@
 #include <xercesc/dom/DOMDocument.hpp>
 #include <xercesc/dom/DOMImplementation.hpp>
 #include <xercesc/dom/DOMElement.hpp>
-#include <xercesc/dom/DOMText.hpp>
 
 #include "boost/filesystem/fstream.hpp"
 
 #include "support/xerces_string_guard.h"
+#include "support/dom/result_node_facade.h"
 
 namespace {
 
@@ -51,29 +51,12 @@ xercesc::DOMDocument* FunctionReadFile::constructDocument(
 	);
 
 	if ( boost::filesystem::is_regular_file(filePath) ) {
-		xercesc::DOMElement* const resultNode(
-			domDocument->createElement(*XercesStringGuard<XMLCh>("result"))
-		);
-
-		resultNode->setAttribute(
-			*XercesStringGuard<XMLCh>("name"),
-			*XercesStringGuard<XMLCh>(filePath.filename().string())
-		);
-
-		xercesc::DOMText* const resultTextNode(
-			domDocument->createTextNode(
-				*XercesStringGuard<XMLCh>(readFile(filePath))
-			)
-		);
-
-		resultNode->appendChild(resultTextNode);
-		rootNode->appendChild(resultNode);
-	} else {
-		xercesc::DOMElement* const resultNode(
-			domDocument->createElement(*XercesStringGuard<XMLCh>("error"))
-		);
+		ResultNodeFacade result(domDocument, rootNode, "result");
 
-		rootNode->appendChild(resultNode);
+		result.setAttribute("name", filePath.filename().string());
+		result.setContent(readFile(filePath));
+	} else {
+		ResultNodeFacade result(domDocument, rootNode, "error");
 	}
 
 	return domDocument;
-- 
cgit v1.2.3