c - C字符串指针函数strdel

有人能解释一下为什么我会得到“分段错误…”以及如何在这段代码上修复它吗？

#include<stdio.h>

int str_length(char *s) {
    int length = 0, i;
    for(i = 0; *s; i++) {
        s++;
    }
    return i;
}

char *strdel(char *s, int pos, int n) {
    int i;
    char *p, str[] = "";
    p = str;
    for(i = 0; i < str_length(s) - n + 1; i++)  {
        if(i >= pos) {
            *(p + i) = *(s + i + n);
        }
        else {
            *(p + i) = *(s + i);
        }
    }
    s = str;
    return s;
}

int main() {
    char *str = "abcdef";
    printf("str_lengh: %d\n", str_length(str));
    printf("strdel: %s\n", strdel(str, 1, 2));
    return 0;
}

我得到这个输出：

str_lengh: 6
strdel: adef
Segmentation fault (core dumped)

此外，是否有更好的方法来创建函数：
char*strdel（char*s，int pos，int n）；
从pos位置删除的n个字符比我删除的要多？

最佳答案

我想你写得太多了。。。

char *strdel(char *s, int pos, int n) {
    int i;
    char *p, str[] = "";
    p = str; // p points to str which is "" and is on the stack with length 0.
    for(i = 0; i < str_length(s) - n + 1; i++)  {
        if(i >= pos) {
            *(p + i) = *(s + i + n); // now you are writing onto the stack past p
        }
        else {
            *(p + i) = *(s + i);// now you are writing onto the stack past p
        }
    }
    s = str; // now s points to space on stack
    return s; // now you return a pointer to the stack which is about to disapear
}

每当你写过p，这是经常的，你会遇到未定义的行为。乌兰巴托
您正在写入尚未在堆或堆栈上分配的空间。
你可以写一个只在s上工作的strdel版本。如果我理解strdel的话，可以这样做：（粗略地说，没有经过测试！，需要对pos和n）进行边界检查

char *strdel(char *s, int pos, int n) {
    char *dst = s + pos, *src = s + pos + n;
    while(*src) {
        *dst++ = *src++;
    }
    *dst = 0;
    return s;
}