为什么这个程序段错误

Question

在编译并运行这个小程序以反转字符串时，在任何输出发生之前我都会遇到分段错误。如果这是一个明显的问题，请原谅我，我对 C 还是很陌生。

#include <stdio.h>

int reverse(char string[], int length); 

int main() {
char string[] = "reversed";

  printf("String at start of main = %s", string);
  reverse(string, sizeof(string));
  printf("%s\n", string);

return 0;

}

// Reverse string 
int reverse(char string[], int length) {
 int i;
 char reversed[] = {};
 int temp;

 for(i = 0; i < length; ++i) {
 temp = string[i];
 reversed[length - i] = temp;

 }
 return 0; 
}

Answer 1

因为这个：

首先你创建一个零元素的数组：

char reversed[] = {};

稍后您尝试写入该数组超出其边界：

reversed[length - i] = temp;

更新：

上面的意思是你需要分配大小只有在运行时才知道的内存（它是length）。通常的 C 风格的做法是……将内存分配的负担推给调用者：

int reverse(const char* string, char* destination, int length);

此函数将写入调用者提供的缓冲区，现在调用者还必须确保：

1. 缓冲区足够大
2. 缓冲区的内存在应该被释放时被释放

Answer 2

虽然它在这种情况下有效，但通常 sizeof(string) 应该是 `strlen(string)。通常，当使用 char 指针时，sizeof 运算符将只返回单个指针的大小 - 而不是整个数组。 . 在reverse()中，你的reverse数组没有分配，你可以这样分配：

char* reversed = (char*) malloc( length+1 );

我们将长度加一以说明字符串末尾的空字符。

Answer 3

您的代码不能编译为 C。声明

char reversed[] = {};

无效。 C 语言中没有空的 {} 初始化器（它只存在于 C++ 中）。此外，空初始化器在未指定大小的数组声明中没有任何意义（这使得该代码也不能编译为 C++），因为在 C 和 C++ 中都没有零大小数组这样的东西。

要么发布真实代码，要么重新标记您的问题（如果这应该是 C++）。

Answer 4

在 C 语言中，您必须仔细考虑变量和数组占用的内存。

当您编写char reversed[] = {} 时，您正在创建一个全新的零大小数组。 （这显然不是严格正确的 C，但它是提问者的 gcc 编译器正在发生的事情。毕竟，报告是运行时的分段错误而不是编译时的语法错误。）

语句reversed[length - i] 然后尝试将数据写入您没有空间的数组元素，因为reversed 没有大小。

你有两个选择：

• 创建一个大小合适的reversed 数组（也许像@Bob Kaufman 所说的那样使用malloc）然后从reverse() 函数中返回它。
• 通过在string 自身内移动字符来“就地”反转字符串。

在适当的位置反转字符串可能更可取——如果您动态分配内存，那么您必须担心再次释放它，这可能会很痛苦。