обратная функция для полукокса * метания ошибки сегментации

Question

У меня есть следующие простые функции:

void reverse(char* str) { 
    if (str == NULL) 
     return; 
    char* end = str; 
    while(*end != NULL) { 
     end++; 
    } 
    end--; 
    while(str < end){ 
     char temp = *str; 
     *str++ = *end; 
     *end-- = temp; 
    } 
} 

int main(int argc, char* argv[]) { 
    char* second = "SOMETHING\0"; 
    cout << "Before Reverse String: " << second << '\n'; 
    reverse(second); 
    cout << "Reverse String: " << second << '\n'; 
} 

просто, правда? однако я получаю ошибку сегментации в строках

*str++ = *end 
*end-- = temp 

Что мне не хватает?

Спасибо!

Answer 1

Вы изменяете строковый литерал:

char* second = "SOMETHING\0"; 

что неопределенное поведение, одно решение было бы, чтобы присвоить литерал массив символов:

char second[] = "SOMETHING"; 

Кроме того, строковые литералы будет NULL прекращено , поэтому не нужно добавлять \0. Вы также должны изменить свой цикл while для сравнения с \0 вместо NULL. Хотя, он должен работать, так как они оба будут вычисляться 0, поскольку *end является char он читает лучше использовать \0:

while(*end != '\0') 

Answer 2

Изменить

char* second = "SOMETHING\0"; 

К

char second[] = "SOMETHING"; 

Примечание: строковые литералы не могут быть изменены, и вы не должны явно добавить \0 в строковый литерал.

while(*end != NULL) { 

должен быть

while(*end != '\0') { 

NULL используется только для нулевого указателя.