Question

我的代码段错误，我不知道为什么。

 1  #include <stdio.h>
 2
 3  void overwrite(char str[], char x) {
 4    int i;
 5    for (i = 0; str[i] != '\0'; i++)
 6      str[i] = x;
 7  }
 8
 9  int main(void) {
10    char *s = "abcde";
11    char x = 'X';
12    overwrite(s, x);
13    printf("%s\n", s);
14    return 0;
15  }

gdb调试器告诉我，问题出在第6行，我想将一个char存储到c-string中（如果我使用左值指针解除引用，那就是同样的问题。）这就是他所说的： / p>

(gdb) run
Starting program: /tmp/x/x 

Breakpoint 1, overwrite (str=0x8048500 "abcde", x=88 'X') at x.c:5
5         for (i = 0; str[i] != '\0'; i++)
(gdb) s
6           str[i] = x;
(gdb) 

Program received signal SIGSEGV, Segmentation fault.
0x080483e3 in overwrite (str=0x8048500 "abcde", x=88 'X') at x.c:6
6           str[i] = x;
(gdb) q

我正在学习K＆amp; R-C书籍，这是第2.8章（删除功能）的简化示例。我不知道问题出在哪里。

Answer 1

因为char * s =“abcde”;在只读内存中创建字符串。试试

char s[] = "abcde";

编辑：解释：char *是指针，“abcde”是在只读内存中创建的 - ＆gt;不可变的。

char []是数组，它完全存储在堆栈中并从内存中初始化，因此是可变的

Answer 2

定义指向字符串文字的指针时，请将其声明为const char *。

const char *s = "abcde";

这样，当您尝试将该字符串发送到overwrite（）函数时，您的编译器会抱怨。

const char *s = "abcde";
char t[] = "fghij";
char x = 'X';

overwrite(s, x); /* oops */
overwrite(t, x); /* ok */

Answer 3

不同意，但只是详细说明：考虑一下如果编译器允许这样做会发生什么。你可以写：

char *s1="abcde";
char *s2="abcde";
s1[0]='x';
puts(s1);
puts(s2);

如果编译器识别出两个文字是相同的并重新使用它们，但是也允许第3行，那么你的输出将是：

xbcde
xbcde

这可能不是你想要的。如果两个文字在程序的广泛分离部分中，这将特别神秘。

Answer 4

试试：

#include <iostream>
#include <cstring>

using namespace std;

void overwrite(char[], char);

int main(void)
{
        char *s = strdup("abcde");
        char X = 'X';
        overwrite(s, X);
        cout << s << endl;

        if(s!=NULL)
                delete [] s;

        return 0;
}

void overwrite(char str[], char x)
{
        for(int i=0; str[i]!='\0'; i++)
                str[i] = x;
}

Answer 5

我的猜测是参数定义，您可以在其中将类型定义为字符数组。当您传递指向char

的指针时

您可以尝试将第一行更改为：

 void overwrite(char *str, char x) {

char数组和char指针在语义上不相同。

写入c-string

5 个答案: