Question

考虑使用以下代码将两个char arrays与delimiter：

连接起来

void addStrings(char* str1,char* str2,char del)
{
//str1=str1+str2
int len1=strlen(str1);
int len2=strlen(str2);
int i=0;
//char* temp=(char*) malloc((len1+1)*sizeof(char));
//strcpy(temp,str1);
str1=(char*) realloc(str1,(len1+len2+1)*sizeof(char));
printf("Here--%d\n",strlen(str1));
*(str1+len1)=del; //adding delimiter
for(i=0;i<=len2;i++)
    *(str1+len1+i+1)=*(str2+i);
printf("Concatenated String: %s\n",str1);
i=0;
    while( *(str1+i) != '\0')
    {
            printf("~~%d:%c\n",i,*(str1+i));
        i++;
    }

}

使用addStrings("A","test",'@');运行此功能时;代码崩溃为realloc下面的gdb output

Breakpoint 3, addStrings (str1=0x40212f <_data_start__+303> "A", str2=0x40212a <_data_start__+298> "test",
    del=64 '@') at string.c:34
34      int len1=strlen(str1);
(gdb) s
35      int len2=strlen(str2);
(gdb) s
36      int i=0;
(gdb) s
39      str1=(char*) realloc(str1,(len1+len2+1)*sizeof(char));
(gdb)

Program received signal SIGABRT, Aborted.
0x004012f2 in addStrings (str1=0xc0 <Address 0xc0 out of bounds>,
    str2=0xea60 <Address 0xea60 out of bounds>, del=0 '\000') at string.c:39
39      str1=(char*) realloc(str1,(len1+len2+1)*sizeof(char));

无法弄清楚它崩溃的原因？是因为我将str1作为auto变量传递而不是在heap上创建吗？

如果是这样的话？如何修改我的代码以接受auto以及heap变量？

Answer 1

您需要按地址传递目标字符串指针，并且必须保存先前分配的字符串的地址，或NULL（如果编码正确）。大小分配必须是长度+ 2（一个用于熟食分隔符，一个用于终止符）。结果可能如下所示：

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

void addStrings(char** str1, const char* str2,char del)
{
    size_t len1 = *str1 ? strlen(*str1) : 0;
    size_t len2 = str2 ? strlen(str2) : 0;
    char *res = realloc(*str1, len1 + len2 + 2);
    if (res)
    {
        res[len1] = del;
        memcpy(res + len1 + 1, str2, len2);
        res[len1 + 1 + len2] = 0;
        *str1 = res;
    }
}

int main()
{
    char *p = NULL;
    const char test[] = "test";
    int i=0;

    // prove it works with no input whatsoever
    addStrings(&p, NULL, 'X');
    printf("p = %p, %s\n", p, p);

    // loop on some input for awhile
    for (;i<10;++i)
    {
        addStrings(&p, test, '@');
        printf("p = %p, %s\n", p, p);
    }
    free(p);
    return 0;
}

<强>输出

p = 0x128610, X
p = 0x128610, X@test
p = 0x128610, X@test@test
p = 0x128620, X@test@test@test
p = 0x128620, X@test@test@test@test
p = 0x128620, X@test@test@test@test@test
p = 0x128620, X@test@test@test@test@test@test
p = 0x128640, X@test@test@test@test@test@test@test
p = 0x128640, X@test@test@test@test@test@test@test@test
p = 0x128640, X@test@test@test@test@test@test@test@test@test
p = 0x128670, X@test@test@test@test@test@test@test@test@test@test

<子> 编译： Apple LLVM版本5.1（clang-503.0.40）（基于LLVM 3.4svn）目标：i386-apple-darwin13.2.0 线程模型：posix

请注意某些传递的结果地址的更改。我将检查有效参数输入作为练习留给你。

连接两个字符串：C

1 个答案: