在 C 代码中释放的位置

Question

我有一个可用的 C 代码。它连接 2 个字符数组并打印出解决方案。
但是我这里有一个动态内存管理（malloc），现在必须释放它。
谁能告诉我在哪里免费？为什么？

char * stringcat(const char * str1, const char * str2) {
    int length_1 = strlength(str1);
    int length_2 = strlength(str2);
    int length = length_1 + length_2 + 1;

    char * newstr = malloc(sizeof(char) * length);

    for(int i=0; i < length; i++)
    {
        char charToCopy = '\0';
        if(i < length_1)
        {
            charToCopy = *(str1 + i);
        } else if(i < (length_1 + length_2)) {
            charToCopy = *(str2 + i - length_1);
        }

        *(newstr + i) = charToCopy;
    }

    return newstr;
}


int strlength(char *c) {
    int count = 0;
    while(*(c + count) != '\0') {
        count++;
    }
    return count;
}


int main() {

    char surname[] = "Simon";
    char lastname[] = "Arndt";

    printf("%s\n", stringcat(surname, lastname));

    return 0;
}

Answer 1

通常你应该在不再需要它的时候释放内存。为此，您应该将指针存储在某处。

int main() {
    char surname[] = "Simon";
    char lastname[] = "Arndt";
    char *res;

    res = stringcat(surname, lastname); // first store it
    if(res == NULL)
    {
        fprintf(stderr, "error");
        return 1;
    }

    // then print it
    puts(res); // equivalent to printf("%s\n", res);


    free(res);

    return 0;
}

不要忘记 C 字符串是 \0 终止的，这意味着当你 为字符串动态分配内存，你应该至少分配 strlen(string) + 1:

char * newstr = malloc(length + 1);

if(newstr == NULL)
    return NULL;

如果是字符串，则不需要sizeof(char)，因为它总是一个。

对于其他类型，避免使用sizeof(<type>)，容易出错。 而是这样做：

int *arr = malloc(10 * sizeof *arr);

sizeof *arr 将始终返回正确的字节数。

编辑

您应该始终检查malloc 不会返回NULL。当然，对于一个 像这样的简单程序，你可以跳过它。但在一个更大的项目中，你 绝对应该检查一下。

Answer 2

您泄漏内存是因为您没有将stringcat 的结果分配给char *。这样做。然后free 在你之后print。或者（在一个简单的程序中，比如这里介绍的）不要担心，让mainexiting 为您解决问题。

char *name = stringcat(surname, lastname);
printf("%s\n", name);
free(name);