Запись в c-строку

Question

мой код segfaults, и я не знаю почему.

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, где я хочу, чтобы сохранить символ, в с-строку (если я использую именующее разыменования указателя, это та же самая проблема.) Это то, что он говорит:

(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 & RC книги и это упрощенный пример из главы 2.8 (функция удалить). Я понятия не имею, где проблема.

Answer 1

потому что char * s = "abcde"; создает строку в памяти readonly. попробуйте

char s[] = "abcde"; 

EDIT: объяснение: символ * является указателем, и "ABCDE" создается в постоянной памяти -> неизменны.

символ [] является массивом, который целиком хранится в стеке и инициализируется из памяти, так изменчиво

Answer 2

мое предположение является определение параметров, где вы определяете тип как массив символов. В то время как вы передаете указатель на гольца

Вы можете попробовать изменить первую строку следующим образом:

void overwrite(char *str, char x) { 

массив символов и указатель голец не семантически то же самое.

Answer 3

Если вы указали указатель на строковый литерал , объявите его как 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 4

Не согласен, но просто уточните: подумайте, что произойдет, если компилятор допустил это. Вы могли бы написать:

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

Если компилятор признает, что два литерала одинаковы и повторно использует их, но также позволяет линии 3, ваш выход будет:

xbcde 
xbcde 

Что, вероятно, не то, что вы хотели бы. Это было бы особенно загадочно, если бы два литерала были в широко разделенных частях программы.

Answer 5

Попробуйте:

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