将Realloc设置指针变为空

Question

void test(){ 
    char *c = malloc(strlen("I like coffe") + 1); 
    strcpy(c, "I like coffe"); 
    char **s = &c; 
    while(strlen(*s) < 25) 
     my_function(s); 
} 

void my_function(char **s){ 
    char *w = *s; 

    char *tmp = realloc(w, len + 2);//Error HERE. *s gets = "" 
    if(tmp != NULL) 
    w = tmp; 
    for(i= len; i>=p; i--){ 
     w[i+1] = w[i]; 
    } 
    w[p] = c;  
} 

该函数用于在char *中插入一个新字符。
另外，这个函数在while循环中。它工作正常，但在第三次循环运行时，它只设置*s = ""。
我以为通过使用char *tmp我可以保存数据，如果发生任何错误的事情。我不明白为什么P *s被设置为空字符串。

Answer 1

您忘记了在包含realloc()的函数中将新值分配给*s。

void test(void) // Unaltered; still broken! 
{ 
    char *c = malloc(strlen("I like coffe") + 1); 
    strcpy(c, "I like coffe"); 
    char **s = &c; 
    while (strlen(*s) < 25) 
     my_function(s); 
} 

void my_function(char **s) // Fixed one way 
{ 
    char *w = *s; 
    size_t len = strlen(w) + 1; // Define and initialize len 

    char *tmp = realloc(w, len + 2); 
    if (tmp != NULL) 
     w = tmp; 
    *s = w; // Reassign to `*s` 
} 

或者更简单地说：

void my_function(char **s) // Fixed another way 
{ 
    char *w = *s; 
    size_t len = strlen(w); // Define and initialize len 

    char *tmp = realloc(w, len + 2); 
    if (tmp != NULL) 
     *s = tmp; // Reassign to `*s` 
} 

分配到w只设置局部变量这是*s复印件;它不会重置调用代码中的指针。

请注意，即使使用此修复，test()中的循环将运行很长时间，因为没有任何更改c中字符串的长度。还有另一个问题：您没有将s的地址传递给my_function()，因此my_function()无法修改s。

void test(void) 
{ 
    char *c = malloc(strlen("I like coffe") + 1); 
    strcpy(c, "I like coffe"); 
    while (strlen(c) < 25) 
    { 
     my_function(&c); 
     strcat(c, "AZ"); // Grow string — not good in real code 
     printf("%2zu: <<%s>>\n", strlen(c), c); 
    } 
} 

void my_function(char **s) 
{ 
    char *w = *s; 
    size_t len = strlen(w) + 1; // Define and initialize len 

    char *tmp = realloc(w, len + 2); 
    if (tmp != NULL) 
     *s = tmp; // Reassign to `*s` 
} 

这会取消指向指向char的指针的指针test()。如果这是至关重要的，那么还有更多想法要做。

^{代码尚未正式测试过！}

现在测试的代码 - 你能说“猪耳朵”吗？复制错误材料的错误使我的测试代码失败。这是仪器化的工作版本 - valgrind为它提供了一个干净的健康法案。

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

static void my_function(char **s) 
{ 
    char *w = *s; 
    size_t len = strlen(w) + 1; // Define and initialize len 
    printf("M1: %p: %2zu: <<%s>>\n", (void *)w, len, w); 

    char *tmp = realloc(w, len + 2); 
    if (tmp != NULL) 
     *s = tmp; // Reassign to `*s` 
    printf("M2: %p: %2zu: <<%s>>\n", (void *)*s, strlen(*s), *s); 
} 

static void test(void) 
{ 
    char *c = malloc(strlen("I like coffe") + 1); 
    if (c == 0) 
    { 
     fprintf(stderr, "Out of memory\n"); 
     exit(EXIT_FAILURE); 
    } 
    strcpy(c, "I like coffe"); 
    printf("T1: %p: %2zu: <<%s>>\n", (void *)c, strlen(c), c); 
    while (strlen(c) < 25) 
    { 
     my_function(&c); 
     printf("T2: %p: %2zu: <<%s>>\n", (void *)c, strlen(c), c); 
     if (c == NULL) 
     { 
      fprintf(stderr, "Out of memory\n"); 
      exit(EXIT_FAILURE); 
     } 
     strcat(c, "AZ"); // Grow string — not good in real code 
     printf("T3: %p: %2zu: <<%s>>\n", (void *)c, strlen(c), c); 
    } 
    free(c); 
} 

int main(void) 
{ 
    test(); 
    return 0; 
}