From eb854e72efa780fd6ceb6e7457b6ac6429a54dad Mon Sep 17 00:00:00 2001 From: Adrian Kummerländer Date: Sat, 10 May 2014 15:51:34 +0200 Subject: Implemented StylesheetParameterGuard class * removes responsibility for clearing parameters from the generate member methods * abstracts parameter escaping and setting and handles map conversion * marked actual generate member method as private and added StylesheetParameterGuard reference argument ** "frontend" generate member methods instantiate a StylesheetParameterGuard instance and pass it to the actual generate method ** this enables central default parameter definition while offering the possibility for custom parameters --- src/support/stylesheet_parameter_guard.cc | 36 +++++++++++++++++++++++ src/support/stylesheet_parameter_guard.h | 35 ++++++++++++++++++++++ src/transformation_facade.cc | 48 +++++++++++++++---------------- src/transformation_facade.h | 13 +++++---- 4 files changed, 102 insertions(+), 30 deletions(-) create mode 100644 src/support/stylesheet_parameter_guard.cc create mode 100644 src/support/stylesheet_parameter_guard.h (limited to 'src') diff --git a/src/support/stylesheet_parameter_guard.cc b/src/support/stylesheet_parameter_guard.cc new file mode 100644 index 0000000..f82c2d5 --- /dev/null +++ b/src/support/stylesheet_parameter_guard.cc @@ -0,0 +1,36 @@ +#include "stylesheet_parameter_guard.h" + +namespace InputXSLT { + +StylesheetParameterGuard::StylesheetParameterGuard( + xalan::XalanTransformer& transformer +): transformer_(transformer) { } + +StylesheetParameterGuard::StylesheetParameterGuard( + xalan::XalanTransformer& transformer, + const map& parameters +): transformer_(transformer) { + this->set(parameters); +} + +StylesheetParameterGuard::~StylesheetParameterGuard() { + this->transformer_.clearStylesheetParams(); +} + +void StylesheetParameterGuard::set(const map& parameters) { + for ( auto&& parameter : parameters ) { + this->set(parameter.first, parameter.second); + } +} + +void StylesheetParameterGuard::set( + const std::string& key, + const std::string& value +) { + this->transformer_.setStylesheetParam( + key.data(), + std::string("'" + value + "'").data() + ); +} + +} diff --git a/src/support/stylesheet_parameter_guard.h b/src/support/stylesheet_parameter_guard.h new file mode 100644 index 0000000..f0622ad --- /dev/null +++ b/src/support/stylesheet_parameter_guard.h @@ -0,0 +1,35 @@ +#ifndef INPUTXSLT_SRC_SUPPORT_STYLESHEET_PARAMETER_GUARD_H_ +#define INPUTXSLT_SRC_SUPPORT_STYLESHEET_PARAMETER_GUARD_H_ + +#include + +#include +#include + +#include "common.h" + +namespace InputXSLT { + +class StylesheetParameterGuard { + public: + typedef std::unordered_map map; + + explicit StylesheetParameterGuard(xalan::XalanTransformer&); + StylesheetParameterGuard( + xalan::XalanTransformer&, + const map& + ); + + ~StylesheetParameterGuard(); + + void set(const map&); + void set(const std::string&, const std::string&); + + private: + xalan::XalanTransformer& transformer_; + +}; + +} + +#endif // INPUTXSLT_SRC_SUPPORT_STYLESHEET_PARAMETER_GUARD_H_ diff --git a/src/transformation_facade.cc b/src/transformation_facade.cc index b411b38..698cdb1 100644 --- a/src/transformation_facade.cc +++ b/src/transformation_facade.cc @@ -26,14 +26,34 @@ TransformationFacade::~TransformationFacade() { } int TransformationFacade::generate(const std::string& target) { + StylesheetParameterGuard guard(this->transformer_); + + return this->generate(target, guard); +} + +int TransformationFacade::generate( + const std::string& target, + const StylesheetParameterGuard::map& parameters +) { + StylesheetParameterGuard guard(this->transformer_, parameters); + + return this->generate(target, guard); +} + +int TransformationFacade::generate( + const std::string& target, + StylesheetParameterGuard& parameters +) { const boost::filesystem::path targetFilePath( boost::filesystem::absolute(target) ); - this->setParameters({ - { "target-file", targetFilePath.filename().string() }, - { "parent-directory", targetFilePath.parent_path().filename().string() } - }); + parameters.set( + "target-file", targetFilePath.filename().string() + ); + parameters.set( + "parent-directory", targetFilePath.parent_path().filename().string() + ); std::stringstream emptyStream(""); xalan::XSLTInputSource inputSource(emptyStream); @@ -49,27 +69,7 @@ int TransformationFacade::generate(const std::string& target) { std::cerr << this->transformer_.getLastError() << std::endl; } - this->transformer_.clearStylesheetParams(); - return resultCode; } -int TransformationFacade::generate( - const std::string& target, - const parameter_map& parameters -) { - this->setParameters(parameters); - - return this->generate(target); -} - -void TransformationFacade::setParameters(const parameter_map& parameters) { - for ( auto&& parameter : parameters ) { - this->transformer_.setStylesheetParam( - parameter.first.data(), - std::string("'" + parameter.second + "'").data() - ); - } -} - } diff --git a/src/transformation_facade.h b/src/transformation_facade.h index 4dc087b..3172daa 100644 --- a/src/transformation_facade.h +++ b/src/transformation_facade.h @@ -4,28 +4,29 @@ #include #include -#include #include "common.h" +#include "support/stylesheet_parameter_guard.h" namespace InputXSLT { class TransformationFacade { public: - typedef std::unordered_map parameter_map; - - TransformationFacade(const std::string&); + explicit TransformationFacade(const std::string&); ~TransformationFacade(); int generate(const std::string&); - int generate(const std::string&, const parameter_map&); + int generate( + const std::string&, + const StylesheetParameterGuard::map& + ); private: const xalan::XalanCompiledStylesheet* transformation_; xalan::XalanTransformer transformer_; - void setParameters(const parameter_map&); + int generate(const std::string&, StylesheetParameterGuard&); }; -- cgit v1.2.3