C语言栈实现报错求助：结构体与指针操作问题

我来帮你拆解代码里的编译错误和逻辑问题，一步步解决：

1. 结构体命名的大小写不匹配 #

你定义了typedef struct Node{ int data; struct node *link; }node;，这里struct node *link里的小写node是未定义的——你定义的结构体类型是struct Node（大写N），而不是struct node。正确的写法应该是让内部的指针指向struct Node，或者统一typedef的命名：

// 统一命名，更清晰
typedef struct Node {
    int data;
    struct Node *link; // 这里要和结构体名一致，用大写Node
} Node;

typedef struct Stack {
    Node *topnode; // 直接用上面typedef后的Node类型
    int count;
} Stack;

2. Push函数里的局部变量陷阱 #

你在push里定义了node newnode;，这是一个栈上的局部变量，当push函数执行完毕后，这个变量的内存会被系统回收，你把它的地址&newnode赋值给栈顶指针，会导致后续访问这个指针时出现野指针问题，完全不可用。必须用malloc动态分配堆内存：

void push(int data, Stack *ourstack) {
    // 动态分配新节点内存
    Node *newnode = (Node*)malloc(sizeof(Node));
    if (newnode == NULL) { // 一定要检查malloc是否成功
        printf("内存分配失败！\n");
        return;
    }
    newnode->data = data;
    newnode->link = ourstack->topnode; // 新节点指向原来的栈顶
    ourstack->topnode = newnode; // 更新栈顶为新节点
    ourstack->count++; // 别忘了栈的计数加1
}

3. 指针访问结构体成员的语法错误 #

• 当你有一个指向结构体的指针（比如Stack *ourstack），访问它的成员要用->运算符，而不是.。所以ourstack.topnode是错的，应该写成ourstack->topnode。
• 你之前尝试的(*ourstack).(*topnode)是完全错误的语法，正确的写法要么是(*ourstack).topnode（等价于ourstack->topnode），访问节点的data就是(*ourstack).topnode->data或者ourstack->topnode->data。

4. Main函数里的调用和输出错误 #

• 调用push的时候，你传的是push(1,mystack)，但push需要的是Stack*类型的参数，所以应该传地址push(1, &mystack)。
• printf里的(mystack.(*topnode).data)语法完全错误，正确的写法是mystack.topnode->data（因为mystack是结构体变量，用.访问topnode，topnode是指针，用->访问data）。另外还要注意，栈为空的时候topnode是NULL，直接访问会崩溃，所以最好先判断。

修正后的完整代码 #

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

typedef struct Node {
    int data;
    struct Node *link;
} Node;

typedef struct Stack {
    Node *topnode;
    int count;
} Stack;

void push(int data, Stack *ourstack) {
    Node *newnode = (Node*)malloc(sizeof(Node));
    if (!newnode) {
        perror("malloc failed");
        return;
    }
    newnode->data = data;
    newnode->link = ourstack->topnode;
    ourstack->topnode = newnode;
    ourstack->count++;
}

// 新增pop函数完善栈功能
int pop(Stack *ourstack) {
    if (ourstack->topnode == NULL) {
        printf("栈为空，无法弹出！\n");
        return -1; // 用-1表示错误，实际场景可以用更严谨的错误处理
    }
    Node *temp = ourstack->topnode;
    int data = temp->data;
    ourstack->topnode = temp->link;
    free(temp);
    ourstack->count--;
    return data;
}

int main() {
    Stack mystack = {NULL, 0}; // 初始化栈：topnode为NULL，count为0
    push(1, &mystack);
    push(2, &mystack);
    
    if (mystack.topnode != NULL) {
        printf("栈顶元素：%d\n", mystack.topnode->data);
    }
    
    printf("弹出元素：%d\n", pop(&mystack));
    printf("现在栈顶元素：%d\n", mystack.topnode->data);
    
    // 释放剩余内存，避免内存泄漏
    while (mystack.topnode != NULL) {
        pop(&mystack);
    }
    return 0;
}

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