c++c++20variadic-templatesfold-expressioncompile-time-type-checking

Fold expression for a parameter pack with comma operator: How to add additional parameters when expanding the pack?


I want to design a compile-time-string class CTString that can e.g. be constructed from a parameter pack of string literals. This works using a comma-fold-expression (for this toy example, I tried to avoid the use of any system headers to make it self-contained):

template<unsigned N>
struct CTString
{
    char m_chars[N + 1U];

    template<unsigned... Ns>
    constexpr CTString(const char (&...s)[Ns])
    {
        auto* p{ m_chars };
        ((p = CopyN_(s, Ns - 1U, p)), ...);
        *p = '\0';
    }

    // copy size characters and return one past last copy:
    constexpr char* CopyN_(const char* pFrom, unsigned size, char* pTo)
    {
        for (auto i{ 0U }; i < size; ++i)
            *(pTo++) = *(pFrom++);
        return pTo;
    }
};

template<unsigned... Ns>
constexpr auto concat(const char(&...s)[Ns])
{
    return CTString<(0U + ... + (Ns - 1U))>{s...};
}

constexpr auto cHelloWorld{ concat("Hello", "World") };
static_assert(cHelloWorld.m_chars[9] == 'd');
static_assert(cHelloWorld.m_chars[10] == '\0');

Now I have an additional use case to insert a separator after each literal. How can I expand/fold the parameter pack to insert e.g. the literal "|" after each element of the pack? This is my feeble attempt that fails because the expression (s, "|")... does not work: The comma here just leads to the left operand being discarded:

template<unsigned... Ns>
constexpr auto concatWithSeparator(const char(&...s)[Ns])
{
    return CTString<(0U + ... + Ns)>{(s, "|")...};
}
// Compilation error:
constexpr auto cHelloCommaSeparated{ concatWithSeparator("Hello", "World") };

I can work around this problem by introducing a helper class and having the compile-time-string also accept packs of the helper class in its constructors. But I was wondering whether there is neat idiom that I am missing.(I did read and re-read this great article, but to no avail: C++20 idioms for parameter packs

The code that compiles is here: Godbolt Un-comment the last line to see how it fails.


Solution

  • A simple and pragmatic solution would be to re-use your concat function:

    template<unsigned... Ns>
    constexpr auto concatWithSeparator(const char(&...s)[Ns])
    {
        return CTString<(0U + ... + Ns)>{concat(s, "|").m_chars...};
    }
    

    https://godbolt.org/z/hzv5qv6no.


    If you want to know how to interleave the parameter pack with the seperator |, I don't think there is a simple solution. One option is to first create a tuple, whose elements are the parameter pack interleaved with the seperator:

    auto tup = std::tuple_cat(
        std::tuple<const char(&)[Ns], const char(&)[2] >(s, "|")...
    );
    

    As a next step, you would need to translate the tuple back to a parameter pack, which in turn can be passed as argument(s) for the CTString constructor.

    You could do this by forwarding the tuple to a templated helper function using std::index_sequence. With C++20, you have templated lambdas so that the helper function could be a lambda defined in the body of concatWithSeparator that you immediately evaluate:

    template<unsigned... Ns>
    constexpr auto concatWithSeparator(const char(&...s)[Ns])
    {
        auto tup = std::tuple_cat(
            std::tuple<const char(&)[Ns], const char(&)[2] >(s, "|")...
        );
        return [&]<std::size_t ... Is>(std::index_sequence<Is...>)
        {
            return CTString<(0U + ... + Ns)>{std::get<Is>(tup)...};
        }
        (
            std::make_index_sequence<2*sizeof...(Ns)>{}
        );
    }
    

    https://godbolt.org/z/W1jhcrGc5


    A slightly more readible version of the second solution would be to use std::apply:

    template<unsigned... Ns>
    constexpr auto concatWithSeparator(const char(&...s)[Ns])
    {
        return std::apply(
            [&](auto const&... args){ return CTString<(0U + ... + Ns)>(args...); },
            std::tuple_cat(std::tuple<const char(&)[Ns], const char(&)[2] >{s, "|"}...)
        );
    }
    

    https://godbolt.org/z/q3GshYx5a