0

我通过使用指针实现了堆栈。它正在编译和工作，但当堆栈为空时它不会下溢。它给了我一些垃圾价值。我认为问题出在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();
    }
}
4

2 回答 2

2

当你创建堆栈时，你为一个节点分配空间——并且不要用任何东西填充它。因此，在调用 create_stack() 之后，您的堆栈上已经有了一个空白节点。我猜你不想那样做

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

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

于 2013-07-31T06:40:33.410 回答
1

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

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

于 2013-07-31T06:44:01.960 回答