冗餘代碼

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; 
}