使用宏替换重复的代码块,略有不同

Question

在我的源文件中,我有几个代码块,它们有很长的重复出现的部分.以此块为例:

void MemberFunctionBar([..], SomeClass param, [..]) {
    /* long common part */

    if constexpr (SafeCopy) {
        T copyOfParam(param);

        /* many lines of code */
        Foo(std::move(copyOfParam));
    } else {
        /* many lines of code (exactly the same as above) */
        Foo(param);
    }
}

现在我想将这些中间行只写一次,以获得更好的代码可维护性.此外,由于限制,不能进行进一步的功能调用.如果可能,代码应扩展为原始格式.

到目前为止,我正在考虑像这个宏(显然不起作用,因为宏中的预处理程序指令不被解释):

#define IF_SAFE_COPY_COPY_PARAM(block)                \
    if constexpr (SafeCopy) {                         \
        T paramCopy(param);                           \
        #define __the_param__ std::move(paramCopy)    \
        block                                         \
        #undef __the_param__                          \
    } else {                                          \
        #define __the_param__ param                   \
        block                                         \
        #undef __the_param__                          \
    }

void MemberFunctionBar([..], SomeClass param, [..]) {
    /* long common part */
    IF_SAFE_COPY_COPY_PARAM({
        /* many lines of code */
        Foo(__the_param__);
    })
}

即使有限制(没有额外的函数调用),我仍然愿意接受建议.

Answer 1

您可以使用更高阶的函数:

template <typename F>
void MemberFunctionBar( SomeClass param, F&& f) {
    if constexpr (SafeCopy) {
        T copyOfParam(param);

        std::forward<F>(f)(/*...*/);
        Foo(std::move(copyOfParam));
    } else {
        std::forward<F>(f)(/*...*/);
        Foo(param);
    }
}

用法:

MemberFunctionBar(my_param, [](/*...*/){ /* block of code */ });