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 */ });

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

1 个答案: