Question

我使用指针实现了堆栈。它正在编译和工作，但当堆栈为空时它不会下溢。它给了我一些垃圾价值。我认为问题出现在create_stack函数中。无论从堆栈中弹出多少数据是奇怪的，我都没有得到段错误。

有人可以帮忙吗？

这是我通过指针完成堆栈的完整实现。

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

enum action {PUSH = 1, POP, TOP, QUIT};

typedef struct node
{
    int data;
    struct node *lower;

}stack_node;

void clear_screen(void)
{
    system("cls");
}

static enum action get_user_action(void)
{
    int choice = 0;
    do
    {
        clear_screen();
        printf("%d Push data\n"
               "%d Pop Data\n"
               "%d See the top of the stack\n"
               "%d Exit\n\n"
               "Enter your choice -> ", PUSH, POP, TOP, QUIT);
        scanf("%d", &choice);
    } while (choice != PUSH && choice != POP && choice != TOP && choice != QUIT);
    return (enum action) choice;
}

void create_stack(stack_node **top, int *status)
{
    *top = malloc(sizeof(stack_node));

    *status = PUSH - 1;
    if (*top == NULL){
        *status = PUSH;
    }
}

void push(stack_node **top_stack, int *status, int data)
{
    *status = PUSH - 1;
    stack_node *node = malloc(sizeof(node));
    if (node == NULL)
    {
        *status = PUSH;
        return;
    }

    node -> data = data;
    if (*top_stack == NULL){
        node -> lower = NULL;
    }
    else{
        node -> lower = *top_stack;
    }
    *top_stack = node;
}

int pop(stack_node **top_stack, int *status)
{
    *status = PUSH - 1;
    if (*top_stack == NULL){
        *status = POP;
        return -1;
    }

    stack_node *node = *top_stack;
    int data = node -> data;
    *top_stack = node -> lower;
    free(node);

    return data;
}

int see_top(stack_node **top_stack, int *status)
{
    *status = PUSH - 1;
    if (*top_stack == NULL){
        *status = POP;
        return -1;
    }

    return (*top_stack) -> data;
}

int main(void)
{
    enum action choice;
    int status;

    stack_node *top = NULL;
    create_stack(&top, &status);

    if (status == PUSH)
    {
        printf("Not enough memory\n");
        return 1;
    }

    while ((choice = get_user_action()) != QUIT)
    {
        clear_screen();
        int data;
        switch (choice)
        {
        case PUSH:
            printf("Enter data to be pushed -> ");
            scanf("%d", &data);
            push(&top, &status, data);
            if (status == PUSH){
                printf("Not enough memory\n");
            }
            break;
        case POP:
            data = pop(&top, &status);
            if (status == POP){
                printf("Stack underflow\n");
            }
            else{
                printf("The data is %d\n", data);
            }
            break;
        case TOP:
            data = see_top(&top, &status);
            switch (status)
            {
            case POP:
                printf("Nothing in the stack\n");
                break;
            default:
                printf("The data at top is %d\n", data);
            }
            break;
        default:
            assert(!"You should not have reached this.");
        }
        getchar();
        getchar();
    }
}

Answer 1

创建堆栈时，为节点分配空间 - 不要用任何东西填充它。因此，在调用create_stack（）之后，您的堆栈上已经有一个空白节点。我想你不想那样做，只做

void create_stack(stack_node **top, int *status)
{
   *top = NULL;
   *status = PUSH -1;
}

工作得很好。无论如何，您在push（）调用期间分配内存，无论您在函数期间检查top_stack == NULL。或者你可以在你的堆栈节点中有一个标志来表明它没有被使用（然后在推送过程中你不会创建一个新的），但这对你想要的东西来说太复杂了。

Answer 2

在create_stack()函数中，您分配内存并且不会将其初始化为任何内容。它的data和lower部分仍然是垃圾。

当您弹出元素if (*top_stack == NULL)时，条件永远不会变为真（变为垃圾值不为空），因此在删除所有节点后它返回垃圾值。

除了堆栈下溢之外，在C工作中按指针堆栈

2 个答案: