交换指针数组中的元素

Question

我在 C 中有以下数组（使用随机名称）

char * inputs[6] = {
  "Kangaroo my shoe", "Fly high dragonfly",
  "Philosophical Monkey", "Jumping Ape",
  "Fearful lemurs", "Tall Giraffes"
};

目标是用下一个元素交换第一个元素。就像这样做

char * inputs[6] = {
  "Fly high dragonfly", "Kangaroo my shoe",
  "Jumping Ape", "Philosophical Monkey",
  "Tall Giraffes", "Fearful lemurs"
};

我已经尝试了以下以及更多。

ArrSwap(char *Arr[]){
  int i;
  for(i=0;i<6;i++){
    void*temp = Arr[i];
    Arr[i]=Arr[i+1];
    Arr[i+1] = temp;
  }
} 

提前致谢！我花了几个小时试图弄清楚这一点，每次我这样做时都会以混乱的顺序结束，或者没有交换第一个和最后一个预期的任何元素。

Answer 1

你的代码有两个问题-

1. 在函数ArrSwap() 中，您正在循环索引0 到5。因此，当i 变为5 时，Arr[i+1] 指向Arr[6]，这会导致从超出数组边界的内存中读取。这是未定义的行为。
2. 其次，如果我正确地理解了您的要求，您希望交换交换的第一个和第二个元素，然后是第三个和第四个，依此类推。因此，您的代码存在逻辑错误。在这种情况下，ArrSwap() 中的循环需要增加 2 而不是 1。

所以在解决了这两个问题后，代码可能看起来像这样 -

void ArrSwap(char *Arr[]){
    int i;
    for(i=0; i<5; i+=2) {
        char *temp = Arr[i];
        Arr[i]=Arr[i+1];
        Arr[i+1] = temp;
    }
}

请注意，ArrSwap() 函数中进行了以下更改 -

• 循环测试条件从i < 6更改为i < 5（原因在上面第1点中提到）。正如 ikegami 所提到的，最佳做法是将数组大小作为参数传递（例如 n）并在循环条件中使用 n - 1。
• 逻辑是固定的（如上面第 2 点所述）
• 对于交换，使用char * 而不是void *（Arr 中的元素类型为char *）
• ArrSwap() 的返回类型被称为void。 void 清楚地表明ArrSwap() 不会返回任何内容。

修复后inputs 将是：

{"Fly high dragonfly", "Kangaroo my shoe", "Jumping Ape", "Philosophical Monkey", "Tall Giraffes", "Fearful lemurs"}