0

我对 C 中的指针仍然很陌生，并且一直在尝试制作字符串函数。这个应该连接两个字符串，但是当我尝试运行它时，它崩溃了。

void concatenate (char *a, char *b, int size_a, int size_b) {
    char *c, *d, *e;
    c = a;
    d = b;
    int i, j, k;
    int l = 1;

    for(i = 0; i<size_a; i++) {
        *(e+i) = *(c+i);
    }
    for (j = size_a; j<size_a+size_b; j++) {
        *(e+j) = *(d+(j-(j-l)));
        l++;
    }
    for (k = 0; k< size_a+size_b; k++){
        printf("%c",*(e+k));
    }
}

我究竟做错了什么？

4

3 回答 3

2

您正在尝试访问未初始化指针e指向的内存，这会导致未定义的行为。

您的函数应如下所示：

char* concatenate (const char *a, const char *b, int size_a, int size_b) {
    // create new string:
    char* newStr = malloc (size_a + size_b + 1);
    int i, j;

    // copy first string into it:
    for (i = 0; i<size_a; i++)
        newStr[i] = a[i];

    // copy second string into it:
    for (j = 0; j < size_b; j++)
        newStr[i + j] = b[j];

    newStr[i + j] = '\0';
    return newStr;
}

请注意，函数不会更改传递给它的字符串，因此它们可以作为const. size_aandsize_b应该等于strlen(a)and ，这strlen(b)也意味着如果你总是将 null 终止的字符串传递给这个函数，你可以在它的主体内自己计算它们的长度（这样你就可以去掉最后两个参数）。

于 2013-02-19T19:24:49.503 回答
0

e 是一个未初始化的指针，这意味着它指向内存中的某个随机位置。当你开始写它，就好像它是一个指向字符数组的指针一样，你的程序会崩溃（或者更糟的是，改变一些状态神秘地导致稍后未指定的行为）。

于 2013-02-19T19:23:24.847 回答
0

您的代码似乎无缘无故地被混淆了。问题是您分配给e未初始化的 。以下几乎是您的代码以更易读的形式和已e分配。不过，我还没有测试过。

void concatenate (char *c, char *d) {    
    char *e; 
    e = calloc(strlen(c) + strlen(d) + 1, sizeof(char));
    int i, j;
    int l = 1;

    for(i = 0; c[i]; i++) {
        e[i] = c[i];
    }
    l = i;
    for (j = 0; d[j]; j++) {
        e[l + j] = d[j];
    }
    e[l + j] = '\0';

    printf("%s", e);

    free(e);
}
于 2013-02-19T19:30:44.407 回答