[SOLUCIONADO] Error a mostrar pila

[RaKKoS]

Buenas noches comunidad de Forosweb, espero estén bien tengo un problema en el momento de crear la pila, no se que estoy aplicando mal, llevo unas cuantas horas revisando y leyendo conceptos de apuntadores pero no consigo el error, he creado un programita para inicializar la pila, agregar elementos y remover, me inicia, agrega los elementos y los remueve sin embargo en el momento de mostrar los valores de la pila solo me muestra el ultimo valor ingresado seguido de cero, y cuando voy a eliminar elementos de la pila solo me muestra el ultimo valor agregado y lo demás en 0 ejemplo, si inicio la pila para 4 elementos 1 2 3 4, me muestra 4 0 0 0, y al querer eliminar la pila, elimina cada elemento pero me imprime en pantalla solamente el ultimo elemento ingresado en pantalla dejo el código para que me orienten, gracias de antemano

Código C:

#include <stdio.h>
#include <conio.h>
#include <stdlib.h>
 
void crearpila (void);
void agregar(void);
void removerpila(void);
void verpila(void);
void mayor(void);
void vaciar(void);
 
int *pila,pilaon=0,ele,arrepila[20],cont=0;
 
main()
{
int re;
char r;
do
{
pila=arrepila;
printf("Programa pila\n\n\n");
printf("\n1. Crear pila\n");
printf("2. Agregar elementos a la pila\n");
printf("3. Remover elementos de la pila\n");
printf("4. Visualizar elementos de la pila\n");
printf("5. Visualizar la cima de la pila\n");
printf("6. Vaciar la pila\n");
printf("7. Salir\n");
printf("Escoja una opcion por favor...\n");
scanf("%d",&re);
 
    switch(re)
     {
        case 1:
            crearpila();
            break;
        case 2:
            agregar();
            break;
        case 3:
            removerpila();
            break;
        case 4:
            verpila();
            break;
     }
}
while (re!=7);
getch();
}
 
void crearpila(void)
{
if (pilaon==NULL)
    {
     system("cls");
      puts("Cuantos elementos desea agregar a la pila\n");
     scanf("%d",&ele);
     printf("Se ha creado la pila de %d elementos",ele);
     pilaon=1;
     getch();
     system("cls");
 
    }
    else
    {
        puts("Ya ha creado la pila anteriormente. Vacie la pila para poder crear una nueva.");   
    }
 
}
 
void agregar(void)
{
if (pilaon==NULL)
{
puts("Debe crear una pila primero");
getch();
system("cls");
}
 
else
{
 
    
    if(cont==ele)
    {
    puts("La pila se encuentra llena"); 
    getch(); system("cls");
    }
    
    else
    {
    system("cls");
    puts("Ingrese elemento");
    scanf("%d",&*(pila));
    pila++;
    cont++;
    printf("Elemento ingresado correctamente");
    getch();
    system("cls");
    
    }       
    
}
}
 
void removerpila(void)
{
 
    if (pilaon==NULL)
    {
        printf("No hay elementos en la pila");
    }
    else
    {
        printf("El elemento %d ha sido removido",*(pila));
        pila--;
        cont--;
            if(cont==0)
            {
            pilaon=NULL;
            }
    }
    
 
 
}
 
void verpila(void)
{
for (int i=0;i<cont;i++)
{
printf("%d ",*(pila-i));    
}
 
 
}
 

[CalgaryCorpus]

do - while es un ciclo.
Por cada "vuelta" de ese ciclo haces pila=arrepila;
o sea, todas las veces llevas el puntero al mismo lugar. ¿Era esa la intencion?

[RaKKoS]

Hola Calgary gracias por tu observación y efectivamente ya he resuelto el problema por un descuido, estuve varias horas bueno ya he realizado la pila me ha quedado así

Código C:

#include <stdio.h>
#include <conio.h>
#include <stdlib.h>
 
void crearpila (void);
void agregar(void);
void removerpila(void);
void verpila(void);
void mayor(void);
void espacio(void);
 
int *pila,pilaon=0,ele,arrepila[20],cont=0;
 
main()
{
int re;
char r;
pila=arrepila;
do
{
 
printf("Programa pila\n\n\n");
printf("\n1. Crear pila\n");
printf("2. Agregar elementos a la pila\n");
printf("3. Remover elementos de la pila\n");
printf("4. Visualizar elementos de la pila\n");
printf("5. Visualizar la cima de la pila\n");
printf("6. Mostrar espacio de la pila\n");
printf("7. Salir\n");
printf("Escoja una opcion por favor...\n");
scanf("%d",&re);
 
    switch(re)
     {
        case 1:
            crearpila();
            break;
        case 2:
            agregar();
            break;
        case 3:
            removerpila();
            break;
        case 4:
            verpila();
            break;
        case 5:
            mayor();
               break;
        case 6:
            espacio();
                    break;
     }
}
while (re!=7);
getch();
}
 
void crearpila(void)
{
if (pilaon==NULL)
    {
     system("cls");
      puts("Cuantos elementos desea agregar a la pila\n");
     scanf("%d",&ele);
     printf("Se ha creado la pila de %d elementos",ele);
     pilaon=1;
     getch();
     system("cls");
 
    }
    else
    {
        puts("Ya ha creado la pila anteriormente. Vacie la pila para poder crear una nueva.");   
    }
 
}
 
void agregar(void)
{
if (pilaon==NULL)
{
puts("Debe crear una pila primero");
getch();
system("cls");
}
 
else
{
 
    
    if(cont==ele)
    {
    puts("La pila se encuentra llena"); 
    getch(); system("cls");
    }
    
    else
    {
    system("cls");
    puts("Ingrese elemento");
    scanf("%d",&*(pila));
    pila++;
    cont++;
    printf("Elemento ingresado correctamente");
    getch();
    system("cls");
    
    }       
    
}
}
 
void removerpila(void)
{
 
    if (pilaon==NULL)
    {
        printf("No hay elementos en la pila");
    }
    else
    {
        printf("El elemento %d ha sido removido",*(pila-1));
        pila--;
        cont--;
            if(cont==0)
            {
            pilaon=NULL;
            }
    }
    
 
 
}
 
void verpila(void)
{
    int a=1;
for (int i=0;i<cont;i++)
{
printf("%d ",*(pila-a));    
a++;
}
 
 
}
 
void mayor(void)
{
system("cls");
printf("%d ",*(pila-1));
}
 
void espacio(void)
{
    int espacio;
 if (cont==ele)
 {
    puts("Pila esta llena");
 }  
 else
 {
    espacio=ele-cont;
    printf("Quedan %d espacios en la pila",espacio);
 }
}
 

Mi pregunta es ¿esta bien hecha? como podría mejorarla mas, Gracias de antemano

[CalgaryCorpus]

Si pasaras a C++ tu codigo, podrias usar varias mejoras, sino en C hay algunas alternativas:

- Solo puedes tener 1 pila en tu programa. Mejor seria guardar todo al interior de una clase que permitiera tener mas de 1 y esconda las variables para su manejo al interior de ella. La alternativa hecha en C es definir una struct y que las funciones reciban como parametro a esa struct que modifican internamente. El programa principal se encargaria de pasar a las funciones cada vez.

- Solo puedes tener pilas de enteros. Mejor seria usar templates y permitir que quien use la pila defina que tipos de datos se guardan al interior.

- pilaon es un entero, mientras que NULL tipicamente es el puntero a nada. pilaon parece ser un tipo bool, el que ya existe en C++.

- La indentacion del codigo que presentas requiere mejoras. Deja la misma cantidad de espacios al abrir una llave, si usas tabs + espacios, se vera mal para quienes definan una cantidad de tabs distintos a los que te gustan, mejor usar espacios solamente.

y otras posibilidades:
- Incluye funciones que prueben la pila sin usar el menu, sino que directamente usen las funciones para manejar la cola y prueben que lo basico funciona.

[RaKKoS]

Gracias Calgary pos tus interesantes observaciones, practicare aplicando esas sugerencias que me has mencionado a si que me tocara investigar un poco sobre templates lo que no me quedo del todo claro fue tu ultima observación

y otras posibilidades:
- Incluye funciones que prueben la pila sin usar el menu, sino que directamente usen las funciones para manejar la cola y prueben que lo basico funciona.

Gracias de igual forma