C ++:分解代码而不传递一百万个参数

时间:2016-06-13 08:08:47

标签: c++ refactoring

我正在进行C / C ++编程(主要是C ++),我发现自己需要将代码分解为真正两倍的代码,除非每次出现" left"替换为"右"。一旦代码完成,我需要知道它是否是" left"或"对"正在执行的版本,但总而言之,两者都将返回一个我能理解的数字(一旦与左或右信息结合)。

在此设置中,每次更改都需要完成两次,这很烦人。

所以我可以简单地将左/右替换为"其他"并且两次调用分解函数,每次都知道我是否正在调用它" left"或者"对"。

现在,当我们到达那部分代码时,已经有一百万个变量(游标,ID,数组被填充等等)。因此,如果我想对左/右代码进行分解,我需要使用该函数来获得大量参数,并且看起来非常难看。

我也不想使用仅在这种情况下使用的属性来重载我的C ++类。

有什么建议可以在这里顺利进行分解吗?

    int arrayRight[many], arrayLeft[many], cursor;

    while(1)
    {
         rightThing = arrayRight[cursor];
         // Process with RightThing assigned
         // ...
         // ...
         // ...

        leftThing = arrayLeft[cursor]
         // Process with RightThing assigned
         // ...
         // ...
         // ...

        cursor++;
   }

2 个答案:

答案 0 :(得分:4)

试试这个? (它仅适用于C ++ 14,因为它使用自动lambda)

auto func = [&](auto& theThing){
    // blah, blah code
};

func(arrayRight[cursor]);
func(arrayLeft [cursor]);

[&]这里意味着将同一范围内的所有变量导入lambda函数。

对于较旧的C ++版本,我使用以下代码作为一种丑陋的方式(在C99的学校项目中)。

int* pData[2] = {arrayRight, arrayLeft};
for (int i=0; i<2; i++)
{
    int* theThing = pData[i];
    // blah, blah
}

我的一些朋友告诉我,宏可以容纳VA_ARGS,所以这里是一个宏观方式。它应该适用于GCC。但是对于MSVC 2003,它并没有起作用。 (__typeof需要替换为boost::typeof,旧版MSVC不支持匿名结构定义

#define in(...) __VA_ARGS__
#define PP_ARG_N( \
          _1,  _2,  _3,  _4,  _5,  _6,  _7,  _8,  _9, _10, \
         _11, _12, _13, _14, _15, _16, _17, _18, _19, _20, \
         _21, _22, _23, _24, _25, _26, _27, _28, _29, _30, \
         _31, _32, _33, _34, _35, _36, _37, _38, _39, _40, \
         _41, _42, _43, _44, _45, _46, _47, _48, _49, _50, \
         _51, _52, _53, _54, _55, _56, _57, _58, _59, _60, \
         _61, _62, _63, N, ...) N
#define PP_RSEQ_N()                                        \
         63, 62, 61, 60,                                   \
         59, 58, 57, 56, 55, 54, 53, 52, 51, 50,           \
         49, 48, 47, 46, 45, 44, 43, 42, 41, 40,           \
         39, 38, 37, 36, 35, 34, 33, 32, 31, 30,           \
         29, 28, 27, 26, 25, 24, 23, 22, 21, 20,           \
         19, 18, 17, 16, 15, 14, 13, 12, 11, 10,           \
          9,  8,  7,  6,  5,  4,  3,  2,  1,  0
#define PP_NARG_(...)    PP_ARG_N(__VA_ARGS__)
#define PP_NARG(...)     PP_NARG_(__VA_ARGS__, PP_RSEQ_N())

#define withInternal(dataType, desiredType, x, dataCnt, data...) for(struct {size_t __i; dataType __t[dataCnt];} __s = {0, data}; x = __s.__t[__s.__i], __s.__i < dataCnt; __s.__i++)
#define with(x, ...) withInternal(__typeof(__VA_ARGS__), __typeof(__VA_ARGS__), x, PP_NARG(__VA_ARGS__), __VA_ARGS__)
#define withConst(x, ...) withInternal(__typeof(__VA_ARGS__), __typeof((__VA_ARGS__) + 0), x, PP_NARG(__VA_ARGS__), __VA_ARGS__)
#define withType(tn, x, ...) withInternal(tn, tn, x, PP_NARG(__VA_ARGS__), __VA_ARGS__)

当你使用它时:

int main()
{
    int x;
    int s1=2, s2=3;
    with(x, in(s1, s2))
        cout<<x<<endl;
    withConst(x, in(45, 55))
        cout<<x<<endl;
    withType(int, x, in(45, s1, 55, s2))
        cout<<x<<endl;
    return 0;
}

我相信它更清楚。

  • with适用于变量(基本上),因为在将变量传递到with时,gcc __typeof会为x生成无法分配的自动常量类型。
  • withConst使用x+0技巧删除const类型的变量,但+运算符不适用于每种数据类型,因此它有局限性。
  • withType指定数据类型,适用于混合情况。

答案 1 :(得分:0)

我认为你可以尝试将这些东西拼凑在一起:

while(1)
{
    vector<int> things;

    int rightThing = arrayRight[cursor];
    things.push_back(rightThing);

    int leftThing = arrayLeft[cursor];
    things.push_back(leftThing);

    for(vector<int>::iterator thing = things.begin(); thing != things.end(); ++thing)
    {
        // Process Thing
        // ...
        // ...
        // ...
    }

    cursor++;
}