C语言中返回bool类型的栈入栈函数实现失败求助及修改方案咨询

你的代码出现异常的核心原因有两个：

1. 栈的初始化错误：你在主函数里定义了struct stos stack;——这是一个结构体变量而非指针，它的内存未被初始化，data和next都是随机的垃圾值，最后输出的69就是这种无意义的垃圾数据。
2. 参数传递的本质问题：add函数的参数是struct stos *stack，这属于传值传递——你在函数内部修改stack = new_element时，只是修改了函数内部的局部指针变量，完全不会影响主函数里原本的stack变量。

要让add函数真正修改主函数中的栈，你需要传递指针的指针（二级指针），这样才能在函数内部直接操控主函数里的栈指针。下面是修改后的完整代码：

#include <stdlib.h>
#include <stdbool.h>
#include <stdio.h> // 别忘了包含stdio.h以使用printf

struct stos {
    int data;
    struct stos *next;
};

bool add(struct stos **stack, int data) {
    struct stos *new_element = malloc(sizeof(struct stos));
    if (new_element != NULL) {
        new_element->data = data;
        new_element->next = *stack; // 新节点指向当前栈顶
        *stack = new_element; // 修改主函数中的栈指针，让它指向新节点
        return true;
    }
    return false;
};

int main() {
    struct stos *stack = NULL; // 初始化栈指针为NULL，表示空栈
    if (add(&stack, 2)) { // 传递栈指针的地址
        printf("Stack top data: %d\n", stack->data); // 现在可以正确访问栈顶数据
    } else {
        printf("Failed to push element\n");
    }
    // 记得后续释放栈内存，避免内存泄漏
    struct stos *temp;
    while (stack != NULL) {
        temp = stack;
        stack = stack->next;
        free(temp);
    }
    return 0;
}

• 用struct stos *stack = NULL初始化栈：空栈的栈顶指针设为NULL是链表实现栈的标准做法，能清晰标识栈的初始状态。
• 传递&stack给add函数：stack是一级指针，&stack就是二级指针struct stos **，这样add函数里的*stack直接对应主函数中的栈指针变量，修改*stack = new_element就能真正更新主函数里的栈顶位置。
• 返回bool的意义：用来明确标识入栈操作是否成功（比如内存分配失败时返回false），完全符合你想要的需求。

如果你坚持要在主函数中使用结构体变量而非指针（不推荐，因为链表栈的头节点用指针才是合理设计），那这种场景更适合用数组实现栈，链表结构本身依赖指针的灵活性来动态添加节点。

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