如何设计一个简单的 std::string-to-boost::posix_time::ptime 解析库

How to design a simple std::string-to-boost::posix_time::ptime parsing library

我最近发现需要将 std::string 解析为 boost::posix_time::ptime。如何完成这个任务的问题在这里得到了很好的回答:How to parse date/time from string?.

我知道它只需要几行代码(基本上是 istringstream 声明、locale/time_input_facet imbuance 和提取到 ptime),但我想将该功能包装在一个方便的函数中并使其可用一个 C++ 库,适用于我使用 ptimes 的各种程序。

如果可能的话,我想从 C++ 专家那里得到一些关于我的设计的反馈,这样我就可以知道我是否做错了什么,或者是否有更好的方法。特别是,我不太熟悉语言环境和 time_input_facets 的工作原理,所以我希望我没有内存泄漏或任何问题。

这里是:

namespace mydt {

    void imbueDTFormat(std::istringstream& iss, const std::string& format );
    void parse(const std::string& input, const std::string& format, boost::posix_time::ptime& out ); // (1)
    void parse(const std::string& input, std::istringstream& iss, boost::posix_time::ptime& out ); // (2)
    void parse(std::istringstream& iss, boost::posix_time::ptime& out ); // (3)

    void imbueDTFormat(std::istringstream& iss, const std::string& format ) {
        iss.imbue(std::locale(std::locale::classic(), new boost::posix_time::time_input_facet(format) )); // see <http://en.cppreference.com/w/cpp/locale/locale/locale>: "Overload 7 is typically called with its second argument, f, obtained directly from a new-expression: the locale is responsible for calling the matching delete from its own destructor."
    } // end imbueDTFormat()

    void parse(const std::string& input, const std::string& format, boost::posix_time::ptime& out ) { // (1)
        static std::istringstream iss;
        imbueDTFormat(iss, format );
        parse(input, iss, out );
    } // end parse()

    void parse(const std::string& input, std::istringstream& iss, boost::posix_time::ptime& out ) { // (2)
        // assumes iss has already been imbued with the desired time_input_facet
        iss.str(input);
        parse(iss, out );
    } // end parse()

    void parse(std::istringstream& iss, boost::posix_time::ptime& out ) { // (3)
        // assumes iss has already been imbued with the desired time_input_facet AND has been initialized with the input str
        iss >> out;
    } // end parse()

} // end namespace mydt

我写的第一个函数是parse(1)。它提供了最简单的界面;只需传递输入字符串、格式字符串和 ptime OUT var,解析就完成了。它使用静态 istringstream 来完成解析,因此不需要分配。但是当我在写完函数后查看函数时,我意识到如果你有一个重复想要解析的单一格式,那么重复从中分配一个新的 time_input_facet 并注入相同的 istringstream 是一种浪费它。

所以我想你可以做得更好,方法是让调用代码创建自己的(可能是静态的)istringstream,注入格式一次,然后重复使用该 istringstream 进行解析。因此,我为此编写了 parse (2)。因此调用代码可以为每种格式提供一个专用函数,如下所示:

void parseServerDT(const std::string& input, boost::posix_time::ptime& out );
void parseHostDT(const std::string& input, boost::posix_time::ptime& out );

// in main, or some other code context
boost::posix_time::ptime serverDT; parseServerDT(getServerDTStr(), serverDT );
boost::posix_time::ptime hostDT; parseHostDT(getHostDTStr(), hostDT );

void parseServerDT(const std::string& input, boost::posix_time::ptime& out ) {
    static bool first = true;
    static std::istringstream iss;
    if (first) {
        first = false;
        mydt::imbueDTFormat(iss, SERVERDT_FORMAT );
    } // end if
    mydt::parse(input, iss, out );
} // end parseServerDT()

void parseHostDT(const std::string& input, boost::posix_time::ptime& out ) {
    static bool first = true;
    static std::istringstream iss;
    if (first) {
        first = false;
        mydt::imbueDTFormat(iss, HOSTDT_FORMAT );
    } // end if
    mydt::parse(input, iss, out );
} // end parseHostDT()

我认为这种设计应该为调用代码提供最大的便利,并且应该最小化对内存和性能的影响。您可以定义任意数量的 parseXDT() 函数(甚至可以创建一个宏来减少这些函数中的重复代码。)

如有任何反馈,我们将不胜感激。谢谢!

至少

  1. 使静态线程局部化。由于缺少编译器,这涉及动态分配它们(这不是问题,因为这是一次性成本)

  2. 最重要的是,在重新使用之前清除流错误状态和内容

    void parseHostDT(const std::string& input, boost::posix_time::ptime& out ) {
        thread_local std::istringstream* iss = [] {
            first = false;
            mydt::imbueDTFormat(iss, HOSTDT_FORMAT);
        }();
    
        iss->clear();
        iss->str("");
        mydt::parse(input, iss, out);
    }