Получение ошибки в стеке (Push & Pop)

Вы не должны использовать циклы 'for' для извлечения и нажатия.

#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
# define MAX 5
int stack1[MAX];
int top=-1;
void push(int);
int pop();
void display();
main()
{
int choice,num;
while(1)
{
    printf("\nEnter Your Choice: \n");
    printf("Enter 1. to Push \n");
    printf("Enter 2. to pop \n");
    printf("Enter 3. to Display \n");
    printf("Enter 4. To Exit\n");
    scanf("%d",&choice);
switch(choice)
{

case 1:
    printf("Enter Elements to be pushed\n");
    int z;
    for(z=1;z<=5;z++)
    {
    printf("Enter a the code In Room %d: ",z);
    scanf("%d",&num);
    push(num);//Function call, Calling push
    }
    break;
case 2:
    printf("Numbers to be poped\n");
    int b;
    for(b=1;b<=5;b++)
    {
    num=pop();
    }//Return a integer Value;
    break;

case 3:
    display();
    break;
case 4:
    exit(1);
default:
    printf("Invalid Choice\n");
}
}
} 
void push(int element)//Push Function
{
int x;

if(top== MAX-1)//Check if Stack is Full
{
    printf("Stack Overflow \n");
    return;//Terminate the function
}
top=top+1;//start from -1 and gets incremented
stack1[top]=element;//insert elements at each step

}
int pop()
{
int a;


int element;
if(top==-1)
{
    printf("Stack EMPTY can't delete anything");
     return;
}
element=stack1[top];//In stack Elements are Always delete from TOP
top=top-1;//To shift the pointer
printf("%d has been deleted \n", element);
return element;

}
void display()
{
int i;
if(top==-1)
{
    printf("Stack Is empty can't display");
    return;//Terminates Display function
}
printf("\n\n");
for(i=top;i>=0;i--)//Displays top elements and decrements on each step
    printf("%d\n",stack1[i]);
}

У вас есть 2 for петли. В основной функции есть цикл внутри переключателя, а также есть цикл внутри функций push и pop.

Функции push и pop будут push или pop только одним элементом и не потребуют внутри него цикла.

Это было бы более очевидно, если бы вы использовали отступы при написании кода. Это одна из причин правильного отступа вашего кода.

Давайте сделаем это шаг за шагом.

1). Вы запустили цикл for, чтобы пользователь мог ввести 5 элементов для вашего стека. Как только он вводит номер, вызывается ваша функция push. В push снова у вас есть цикл, который выполняется 5 раз. Таким образом, один и тот же элемент помещается в стек 5 раз. Итак, при следующем выборе стек уже заполнен. Таким образом, сообщение о переполнении стека.

2). Теперь ваш стек выглядит примерно так: [5,5,5,5,5], предполагая, что 5 было введенным числом 5, в вашей функции pop вы снова запускаете цикл. В идеале вы должны извлекать только один элемент за раз для пользователя. Итак, поскольку вы запускаете цикл, все 5 элементов извлекаются, и поскольку в стеке находится только первый элемент, это то, что вы видите на выходе.