我正在尝试编写代码来原地反转一个字符串（只是为了提高C编程和指针操作的能力），但我无法弄清楚为什么会出现段错误：
#include <string.h>
void reverse(char *s);
int main() {
char* s = "teststring";
reverse(s);
return 0;
}
void reverse(char *s) {
int i, j;
char temp;
for (i=0,j = (strlen(s)-1); i < j; i++, j--) {
temp = *(s+i); //line 1
*(s+i) = *(s+j); //line 2
*(s+j) = temp; //line 3
}
}
导致分割错误的是第2和第3行。我知道可能有更好的方法来解决这个问题，但我对于找出在我的代码中具体导致分割错误的原因很感兴趣。
更新：按要求包含了调用函数。
仅从代码上看无法确定问题。很可能，您正在传递指向无效内存、不可修改内存或其他类型的内存的指针，该内存不能以此处处理它的方式进行处理。
你是怎么调用你的函数的?
补充说明：您正在传递指向字符串字面值的指针。字符串字面值是不可修改的。您无法颠倒一个字符串字面量。请传递指向可修改字符串的指针。
char s[] = "teststring";
reverse(s);
这里已经解释得非常清楚了。"teststring"是一个字符串字面值。字符串字面值本身是一个不可修改的对象。实际上，编译器可能（并且会）将其放在只读内存中。当你像那样初始化一个指针时
char *s = "teststring";
指针直接指向字符串字面值的开头。一般情况下，任何试图修改指针所指向内容的尝试都会失败。你可以读取它，但不能写入它。因此，强烈建议仅使用指向const类型的指针来指向字符串字面值。
const char *s = "teststring";
但是当你声明你的s为：
char s[] = "teststring";
你得到了一个完全独立的数组s，它位于普通可修改的内存中，并使用字符串字面值进行初始化。这意味着那个独立的可修改数组s将从字符串字面值中被复制其初始值。之后，你的s数组和字符串字面值将继续作为完全独立的对象存在。该字面值仍然是不可修改的，而s数组可修改。
基本上，后者声明在功能上等同于：
char s[11];
strcpy(s, "teststring");
你的代码可能会因为多种原因而出现段错误。以下是我想到的一些可能性：
我认为第2种情况最有可能。您能否展示reverse的调用站点？
编辑
基于您的示例，答案肯定是#2。在C / C ++中，字符串文字是不可修改的。正确的类型实际上是const char *而不是char *。您需要将可修改的字符串传递到该缓冲区中。
快速示例：
char* pStr = strdup("foobar");
reverse(pStr);
free(pStr);
你是否正在测试类似这样的东西？
int main() {
char * str = "foobar";
reverse(str);
printf("%s\n", str);
}
char * str = strdup(foobar)，它应该可以正常工作（对我来说是这样的）。你的声明完全是错的：
char* s = "teststring";
“teststring”存储在代码段中，该段是只读的，就像代码一样。同时，s是指向“teststring”的指针，你试图更改只读内存范围的值，因此会出现分段错误。
但是，如果使用：
char s[] = "teststring";
s被初始化为“teststring”，当然这个字符串在代码段中，但是在这种情况下还有一个额外的复制操作，将其复制到堆栈中。
请参阅C FAQ列表中的问题1.32：
What is the difference between these initializations?
char a[] = "string literal"; char *p = "string literal";My program crashes if I try to assign a new value to
p[i].Answer:
A string literal (the formal term for a double-quoted string in C source) can be used in two slightly different ways:
As the initializer for an array of char, as in the declaration of
char a[], it specifies the initial values of the characters in that array (and, if necessary, its size).Anywhere else, it turns into an unnamed, static array of characters, and this unnamed array may be stored in read-only memory, and which therefore cannot necessarily be modified. In an expression context, the array is converted at once to a pointer, as usual (see section 6), so the second declaration initializes
pto point to the unnamed array's first element.Some compilers have a switch controlling whether string literals are writable or not (for compiling old code), and some may have options to cause string literals to be formally treated as arrays of
const char(for better error catching).(emphasis mine)
正如上面提供的一些答案，字符串内存是只读的。然而，一些编译器提供了一个选项来编译可写字符串。例如，使用gcc，3.x版本支持-fwritable-strings，但新版本不支持。
strlen()。 - Carl Norumstrlen并没有在循环条件中使用。 - AnT stands with Russia