C++链式栈实现代码运行后输出界面卡顿无响应，请求排查原因

看起来你的代码里有几个关键问题导致了死循环和未定义行为，咱们一步步拆解：

1. showstack函数的致命错误（直接导致卡顿） #

这个函数里的逻辑完全错误，直接造成了无限循环：

• 首先，全局的temp指针未初始化，你执行*temp = *top相当于给随机内存地址赋值，属于严重的未定义行为。
• 其次，循环里你一直输出top->s（永远是栈顶元素），而且temp = top->link——这意味着temp每次都被设置为栈顶节点的下一个节点，但top本身从未移动，只要栈里有超过1个节点，temp永远不会变成NULL，循环会无限执行，程序自然卡顿无响应。

正确的遍历逻辑应该是让temp从栈顶开始，逐个向后移动：

void showstack(){
    node* temp = top; // 用局部指针从栈顶开始遍历
    while(temp != NULL){
        cout << temp->s << " "; // 输出当前节点的值
        temp = temp->link; // 移动到下一个节点
    }
    cout << endl;
}

2. new_node函数的问题 #

• 函数声明为node* new_node(node a)，但没有任何return语句，这会导致未定义行为（编译器可能返回随机值）。其实这个函数的设计可以简化，直接专注于入栈逻辑即可：

void push_node(node a){
    node* p = new node;
    p->s = a.s;
    p->link = top; // 新节点指向当前栈顶
    top = p; // 更新栈顶为新节点
}

3. 全局指针的初始化问题 #

你声明了全局的temp、p指针但未初始化，这些指针会指向随机内存，直接解引用极易引发内存错误。建议尽量使用局部指针，避免全局变量带来的意外问题。

4. main函数里的小细节 #

• 循环条件while(i<=5)会让你入栈6个元素（i从0到5），如果这不是你的预期，可以改成while(i<5)。
• 入栈后直接输出top->s会重复打印栈顶元素，建议去掉或改成提示信息。

修正后的完整代码 #

#include <iostream>
#include <string>
using namespace std;

struct node {
    string s;
    node* link;
}*top = NULL; // 仅保留全局栈顶指针，其他用局部变量

// 入栈函数
void push_node(const string& val){
    node* p = new node;
    p->s = val;
    p->link = top;
    top = p;
}

// 遍历栈
void showstack(){
    node* temp = top;
    cout << "栈内元素（从栈顶到栈底）：";
    while(temp != NULL){
        cout << temp->s << " ";
        temp = temp->link;
    }
    cout << endl;
}

int main(){
    int i = 0;
    string input;
    do{
        cout << "请输入第" << i+1 << "个元素：";
        cin >> input;
        push_node(input);
        i++;
    }while(i < 5); // 入栈5个元素，可按需调整

    showstack();
    return 0;
}

这个版本修复了所有问题：去掉了危险的全局指针，修正了遍历逻辑，简化了入栈函数，程序可以正常运行，不会再出现卡顿。

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