冗余代码

Question

我有一个列表：

id num1 ... 
----------- 
1 123 ... 
1 456 ... 
2 789 ... 
2 666 ... 

，并希望基于它创建一个对象数组：

{ 1, [123, 456], [...] }, 
{ 2, [789, 666], [...] } 

这里是我的伪代码：

int previous_id = -1; 
array a1 = null; // B 
array a2 = null; // B 
array a3 = null; // B 
array a4 = null; // B 
while (++c) { // c is a pointer pointing to the list 
    if (c.id != previous_id && previous_id != -1) { 
    j[i++].id = previous_id; // A 
    j[i++].data1 = a1;   // A 
    j[i++].data2 = a2;   // A 
    j[i++].data3 = a3;   // A 
    j[i++].data4 = a4;   // A 
    a1 = null; // B 
    a2 = null; // B 
    a3 = null; // B 
    a4 = null; // B 
    } 
    a1.add(c.num1); 
    a2.add(c.num2); 
    a3.add(c.num3); 
    a4.add(c.num4); 
    previous_id = c.id; 
} 
j[i++].id = previous_id; // A 
j[i++].data1 = a1;   // A 
j[i++].data2 = a2;   // A 
j[i++].data3 = a3;   // A 
j[i++].data4 = a4;   // A 

它正在工作，但有一些冗余代码，即A和B

是否可以合并它们使其更加简洁明了？

Answer 1

有一对夫妇的事情可以做，以提高你的代码的清晰度，如果是这样的目标。

而不是使用不同的名称，使用索引。

“数组”没有定义，所以我想这是一个表的东西。你可能有这样的事情：

#define NB_ARRAYS 4 
    array myArrays[NB_ARRAYS]; 

这样一来，你就可以遍历数组的，并最终随时更改，今后阵列的数量。

好的想法是，你不再需要单独列出你的数组，单个循环就足够了。因此：

a1 = null; // B 
    a2 = null; // B 
    a3 = null; // B 
    a4 = null; // B 

成为

{ int i; for (i=0; i<NB_ARRAYS; i++) myArrays[i] = NULL; } 

你可能会抱怨，这并不比第一个版本更好，但实际上你可以隐藏这个宏背后的复杂性：

#define INIT_ARRAYS(a) { int i; for (i=0; i<NB_ARRAYS; i++) a[i] = NULL; } 

甚至内联函数背后更好：

static inline void initArrays(array* a) { int i; for (i=0; i<NB_ARRAYS; i++) a[i] = NULL; } 

所以它变成：

initArrays(myArrays); 

这是更清楚。

重复使用相同的原则，这会导致你的代码是这样的：

int previous_id = -1; 
    initArrays(myArrays); 

    while (++c) 
    { // c is a pointer pointing to the list 
     addToArrays(myArrays, c); 
     if (c.id != previous_id && previous_id != -1 || c.islast()) 
     { 
      j[i++].id = previous_id; // A 
      setArrays(j, i, myArrays); i+=NB_ARRAYS; 
      if (!c.islast()) { initArrays(myArrays); } 
     } 
    } 
    previous_id = c.id; 

这应该是更容易阅读，因此，更容易维护。

另外：尽量使用更容易阅读变量。在这个例子中，我不知道c，i或j代表什么，也不知道它们在哪里定义。使用5-6个字符来正确命名它们并没有多少成本，并且确实有助于代码维护。

Answer 2

你的代码有点太伪代码 - 我的喜好 - 建议非常依赖于你的实际实现。不过，我会提出一些想法。

为A部分：

struct的用C将允许你这样做：

someStruct temp = {previous_id, a1, a2, a3, a4}; 
j[i++] = temp; 

对于B部分：

我认为null实际指的是一个空的数组，而不是一个数组，你可能有一个固定长度的数组，其长度为单独的长度指示符。

您可以始终有一个长度数组，而不是一个2D主数组，而整个长度数组为memset而不是逐个设置它们。

Answer 3

只是找到了一种方法来合并A，避免一定复位B：

int previous_id = -1; 
array a1 = null; // B 
array a2 = null; // B 
array a3 = null; // B 
array a4 = null; // B 
while (++c) { // c is a pointer pointing to the list 
    a1.add(c.num1); 
    a2.add(c.num2); 
    a3.add(c.num3); 
    a4.add(c.num4); 
    if (c.id != previous_id && previous_id != -1 
     || c.islast()) { 
    j[i++].id = previous_id; // A 
    j[i++].data1 = a1;   // A 
    j[i++].data2 = a2;   // A 
    j[i++].data3 = a3;   // A 
    j[i++].data4 = a4;   // A 
    if (!c.islast()) { 
     a1 = null; // B 
     a2 = null; // B 
     a3 = null; // B 
     a4 = null; // B 
    } 
    } 
    previous_id = c.id; 
}