C语言指针操作中结构体变量值无法按预期打印的问题

你的代码核心问题出在调用cleanup(p)后的指针访问逻辑上——此时p已经是野指针，访问它会触发未定义行为，导致打印结果完全不符合预期。下面我来拆解原因并给出具体修复方案：

问题根源分析 #

1. 指针值传递的误区：如果你的cleanup函数只是简单调用free(pointer)，这里的pointer是值传递——函数内部拿到的是全局变量p的副本，修改这个副本不会影响外部的p本身。所以free内存后，全局p仍然指向已经被操作系统回收的内存地址，这就是典型的野指针。
2. 释放后未置空指针：即使内存被成功释放，你没有把p置为NULL，后续的if(p != NULL)判断会误以为指针仍然有效，进而去访问已经失效的内存，最终打印出乱码、错误值甚至直接崩溃。

具体修复方案 #

方案1：针对全局指针修正cleanup #

因为你的p是全局变量，cleanup可以直接操作它，释放后务必将其置空：

void cleanup() {
    if(p != NULL) {
        free(p);
        p = NULL; // 关键操作：释放后把指针置空，彻底避免野指针
    }
}

调用时直接写cleanup()即可，无需传参。

方案2：用指针的指针传递（更推荐，摆脱全局变量依赖） #

全局变量容易引发耦合问题，更规范的做法是使用局部指针，通过指针的指针让cleanup能修改外部的指针值：

void cleanup(MyType** pointer) {
    // 先检查外层指针和目标指针的有效性
    if(pointer != NULL && *pointer != NULL) {
        free(*pointer);
        *pointer = NULL; // 直接修改外部的指针为NULL
    }
}
// 调用时传入指针的地址
cleanup(&p);

完整修正代码示例 #

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

typedef struct type {
    int a;
    char b[10];
}MyType;

void appenddata(MyType* p) {
    if(p != NULL) {
        // 这里可以添加你的业务逻辑，比如更新结构体内容
        p->a = 20;
        strcpy(p->b, "updated");
        printf("After appenddata: p->a = %d, p->b = %s\n", p->a, p->b);
    }
}

void cleanup(MyType** pointer) {
    if(pointer != NULL && *pointer != NULL) {
        free(*pointer);
        *pointer = NULL;
    }
}

void mycode() {
    MyType* p = (MyType *)malloc(sizeof(MyType)); // 使用局部指针，更安全
    if(p != NULL) {
        p->a = 10;
        strcpy(p->b, "sample");
        printf("Initial: p->a = %d, p->b = %s\n", p->a, p->b);
    }

    appenddata(p);
    cleanup(&p);

    if(p != NULL) {
        printf("p->a - %d\n",p->a);
        printf("p->b - %s\n",p->b);
    } else {
        printf("Pointer has been freed and set to NULL, cannot access members.\n");
    }
}

int main() {
    mycode();
    return 0;
}

关键注意事项 #

• 内存释放后必须置空指针：这是防止野指针访问的核心习惯，能让后续的空指针检查真正生效。
• 尽量避免全局指针：全局变量会增加代码耦合度，也容易引发这类指针管理失误，优先使用局部指针配合指针的指针传递。
• 访问指针前先判空：任何时候解引用指针，都要先执行if(p != NULL)检查，避免空指针崩溃。

内容的提问来源于stack exchange，提问作者ssg