Moved DOM document construction logic into local functions

* this change makes the underlying similarity between all currently implemented external functions obvious
* they more or less only differ in their document construction logic
** i.e. there is a possibility for a second layer of abstraction between the classes derived from xalan::Function and the actual functionality
* added boost::filesystem::path taking overload of the _iterate_ member method to FilesystemGuard
** this overload functions as the _master_ overload that is called by all other overloads which merely perform type conversion and path resolution
** this overload doesn't perform path resolution

1 files changed, 49 insertions, 41 deletions

diff --git a/src/function/read_file.cc b/src/function/read_file.cc
index 8fcc04b..ba94909 100644
--- a/src/function/read_file.cc
+++ b/src/function/read_file.cc
@@ -12,6 +12,8 @@
 
 namespace {
 
+using InputXSLT::XercesStringGuard;
+
 inline std::string readFile(const boost::filesystem::path& filePath) {
 	boost::filesystem::ifstream file(filePath);
 
@@ -21,6 +23,50 @@ inline std::string readFile(const boost::filesystem::path& filePath) {
 	);
 }
 
+xercesc::DOMDocument* constructDocument(
+	const boost::filesystem::path& filePath
+) {
+	xercesc::DOMDocument* const domDocument(
+		xercesc::DOMImplementation::getImplementation()->createDocument(
+			nullptr,
+			*XercesStringGuard<XMLCh>("content"),
+			nullptr
+		)
+	);
+
+	xercesc::DOMNode* const rootNode(
+		domDocument->getDocumentElement()
+	);
+
+	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"))
+		);
+
+		rootNode->appendChild(resultNode);
+	}
+
+	return domDocument;
+}
+
 }
 
 namespace InputXSLT {
@@ -34,10 +80,10 @@ xalan::XObjectPtr FunctionReadFile::execute(
 	const xalan::XObjectPtr argument,
 	const xalan::Locator* locator
 ) const {
-	const FilesystemContext fs_context(locator);
+	const FilesystemContext fsContext(locator);
 
 	const boost::filesystem::path filePath(
-		fs_context.resolve(argument->str())
+		fsContext.resolve(argument->str())
 	);
 
 	DomDocumentCache::optional_item optionalCachedDocument(
@@ -45,47 +91,9 @@ xalan::XObjectPtr FunctionReadFile::execute(
 	);
 
 	if ( !optionalCachedDocument.first ) {
-		xercesc::DOMDocument* const domDocument(
-			xercesc::DOMImplementation::getImplementation()->createDocument(
-				nullptr,
-				*XercesStringGuard<XMLCh>("content"),
-				nullptr
-			)
-		);
-
-		xercesc::DOMNode* const rootNode(
-			domDocument->getDocumentElement()
-		);
-
-		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"))
-			);
-
-			rootNode->appendChild(resultNode);
-		}
-
 		optionalCachedDocument = this->document_cache_->create(
 			filePath.string(),
-			domDocument
+			constructDocument(filePath)
 		);
 	}