Почему является это строковое реверсирование C кодом, вызывающим отказ сегментации? [дубликат]

Ваш код может быть нарушен по ряду причин. Вот те, которые приходят на ум

1. s is NULL
2. s указывает на константную строку, которая хранится в памяти только для чтения
3. s не завершается NULL

Я думаю, что № 2 является наиболее вероятным. Не могли бы вы показать нам место вызова обратного?

РЕДАКТИРОВАТЬ

Исходя из вашего примера № 2, безусловно, ответ. Строковый литерал в C / C ++ не подлежит изменению. Правильный тип на самом деле const char * , а не char * . Что вам нужно сделать, так это передать в этот буфер изменяемую строку.

Краткий пример:

char* pStr = strdup("foobar");
reverse(pStr);
free(pStr);

Ваше объявление полностью неверно:

char* s = "teststring";

"teststring" хранится в сегменте кода, который доступен только для чтения, как и код. И, s - указатель на «teststring», в то же время вы пытаетесь изменить значение диапазона памяти только для чтения. Таким образом, ошибка сегментации.

Но с:

char s[] = "teststring";

s инициализируется "teststring", которая, конечно, находится в сегменте кода, но в этом случае происходит дополнительная операция копирования в стек.

Вы тестируете что-то вроде этого?

int main() {
    char * str = "foobar";
    reverse(str);
    printf("%s\n", str);
}

Это делает str строковым литералом, и вы, вероятно, не сможете его редактировать (segfaults для меня). Если вы определите char * str = strdup (foobar) , он должен работать нормально (подходит для меня).

Какой компилятор и отладчик вы используете? Используя gcc и gdb, я бы скомпилировал код с флагом -g, а затем запустил его в gdb. Когда он выходит из строя, я просто выполняю обратную трассировку (команда bt в gdb) и смотрю, какая строка вызывает проблему. Кроме того, я бы просто запустил код шаг за шагом, «наблюдая» за значениями указателя в gdb и точно знаю, в чем проблема.

Удачи.

См. Вопрос 1.32 в списке часто задаваемых вопросов по C:

В чем разница между этими инициализациями?

 char a [] = "строковый литерал";
char * p = "строковый литерал";

Моя программа вылетает, если я пытаюсь присвоить новое значение p [i] .

Ответ:

строковый литерал (формальный термин для строки в двойных кавычках в исходном коде C ) можно использовать двумя немного разными способами:

В качестве инициализатора для массива символов, как и в объявлении char a [] , он указывает начальные значения символов в этом массиве ( и, если необходимо, его размер).

В любом другом месте он превращается в безымянный статический массив символов, и этот безымянный массив может храниться в постоянной памяти и, следовательно, не может быть обязательно изменен . В контексте выражения массив, как обычно, сразу же преобразуется в указатель (см. Раздел 6), поэтому второе объявление инициализирует p , чтобы указать на первый элемент безымянного массива.

Я думаю, strlen не может работать, поскольку s не завершается NULL. Таким образом, поведение вашего for итерации отличается от ожидаемого. Так как результат strlen будет больше длины s, вы запишете в память там, где вам не должно быть.

Вдобавок s указывает на постоянные строки, хранящиеся в постоянной памяти. Вы не можете его изменить. Попробуйте запустить инициализацию с помощью функции gets, как это сделано в примере strlen