Почему эта программа дает ошибку сегментации?

Question

Это вопрос начинающих: почему это нарушение/сообщение об ошибке?

#include <stdio.h>  
#include <stdlib.h> 
#include <string.h> 

char *strtrim_right(char *p) 
{ 
    char *end; 
    int len; 
    len = strlen(p); 
    while (*p && len) 
    { 
    end = p + len-1; 
    if(isalpha(*end)) 
    *end =0; 
    else 
    break;  
    } 
    return(p); 
} 


int main() 
{ 
    char *x="PM123BFD"; 
    strtrim_right(x); 
    printf("%s", x); 
    return 0; 
} 

Answer 1

Изменить

char *x="PM123BFD"; 

в

char x[]="PM123BFD"; 

Вы не можете изменить строковый литерал, поэтому вместо того, чтобы передать функции в char массив, который может изменить.

Answer 2

Я не понимаю, почему он должен сломать - я предпочел бы ожидать бесконечный цикл: условие while всегда будет истинным, и цикл никогда не останется.

Завершите работу в петле, она оборвана. Затем посмотрите на переменные, которые у вас есть: вы никогда не изменяете значения ни p, ни len. Это неправильно. Кроме того, код внутри цикла намного сложнее, чем нужно. Подумайте, действительно ли вам нужны три переменные здесь.

Answer 3

Ok благодаря 2 ответов выше здесь является то, что, кажется, хорошо в настоящее время:

#include <stdio.h>  
#include <stdlib.h> 
#include <string.h> 

char *strtrim_right(char *p) 
{ 
    char *end; 
    int len; 
    len = strlen(p); 
    while (*p && len) 
    { 
    end = p + len-1; 
    if(isalpha(*end)) 
    *end =0; 
    else 
    break; 
len = strlen(p); 
    } 
    return(p); 
} 


int main() 
{ 
    char x[]="PM123BFD"; 
    strtrim_right(x); 
    printf("%s", x); 
    return 0; 
}