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 循环内。它工作正常，但在循环运行第 3 次时，它只是设置 *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`
}

这消除了指向 test() 中 char 的指针。如果那很重要，还有更多的思考要做。

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

代码现已测试 - 你能说 "pig's ear" 吗？复制粘贴错误的 material 使我的测试代码失败。这是经过检测的工作版本 — 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;
}

Realloc 设置指针为空

Realloc setting pointer to empty

c

realloc