Pilas C++ usando Estructuras (STRUCT)

[charmedever]

Hola q tal pueden ayudarme el programa debe de permitir insertar 4 elementos y despues visualizarlos de acuerdo al orden en que fueron insertados
Anexo el codigo para q me digan mis errores
//IMPLEMENTA UNA PILA
# include <iostream.h>
# include <conio.h>
# include <stdlib.h>

//prototipos de función
void llenar(char a);
char extraer();

const int tam=4;
char tope=-1;
char pila[tam];

int main()
{
  char a;

  cout<<"LLENAR PILA"<<endl;

  while(tope<tam)
  {
   cin>>a;
   llenar(a);
  }

  cout<<"Pulsar cualquier tecla para continuar";
  getche();
 
  system ("CLS");

  cout<<"Datos de la pila"<<endl;

  extraer();
  getche ();
  return -9;
}//fin main()

void llenar(char a)
{
  if(tope+1 >= tam)
      cout<<"Pila llena"<<endl;

  pila[++tope]=a;
}

char extraer()
{
  while(tope>=0)
  {
   cout<<pila[tope--]<<endl;
   getche ();

   if(tope==-1)
      cout<<"Pila vacia";

  }
}

[CeroX901]

y donde está el struct ??

-Usar getche(); para pausar la aplicacion?, mejor usa system("pause");

-La declaracion de la librería iostream, está mal hecha, es sin el .h

-En main retornas -9?

-La funcion extraer debe devolver un valor char, no devuelve nada.

Y por ultimo te recomiendo usar string en ves de char para esta ocasion.

PD: cuando vayas a insertar codigo utiliza las etiquetas correspondientes, donde aparece GeSHi en crear mensaje..

Salud2

[charmedever]

Modifique el codigo como me dijiste

Código (cpp) [Seleccionar] Expandir

//IMPLEMENTA UNA PILA
# include <iostream.h>
# include <conio.h>
# include <stdlib.h>
struct pilas
{
       int x;
}pi[3];
//prototipos de función
void llenar(pilas);
int extraer(pilas);
int tope=-1;

int main()
{

  cout<<"LLENAR PILA"<<endl;

  while(tope<3)
  {
                 for (int i=0; i<3; i++)
                 {
                     cin>>pi[i].x;
                     llenar (pi);
                     
                 }
  }

  cout<<"Pulsar cualquier tecla para continuar";
  getche();
 
  system ("CLS");

  cout<<"Datos de la pila"<<endl;

  extraer(pi); 
  getche ();
  return -9;
}//fin main()

void llenar(pilas)
{
  if(tope+1 >=3)
  cout<<"Pila llena"<<endl;
  pi[++tope]=pi[3];
}

int extraer(pilas)
{
  while(tope>=0)
  {
                for (int i=0; i<3; i++)
                {
                    cout <<pi[i].x;
                    cout<<pi[tope--]<<endl;
                }
                getche ();
                if(tope==-1)
                cout<<"Pila vacia";
  }
}


[Queta]

No hiciste nada de lo que te dijo...

¿Compilaste antes de enseñarnos el código?

Empezando por las librerías, no hace falta ni stdlib ni conio y es más, como bien te dijo CeroX901, iostream va sin la .h final. main sigue devolviendo -9 y no aplicas bien cout.

-Usar getche(); para pausar la aplicacion?, mejor usa system("pause");

Es mejor, en este caso, usar cin.get. Y tampoco usas bien la función extraer.

[charmedever]

las librerias las uso asi porq estoy trabajando con DEV C++ el iostream.h para cout y cin, conio.h para pausar pantalla getche() y stdlib.h para limpiar pantalla esas librerias siempre las he utilizado, la funcion main puede aceptar cualquier numero entero sea positivo o negativo

[Queta]

las librerias las uso asi porq estoy trabajando con DEV C++

Eso es irrelevante, a parte de que este IDE ya está medio obsoleto y su última actualización no es ni oficial.

el iostream.h para cout y cin

Repito, iostream va sin .h final y no usas cout correctamente.

conio.h para pausar pantalla getche()

Puedes perfectamente hacer servir una función estándar; ya te dí una solución...

stdlib.h para limpiar pantalla

Aunque no es recomendado el uso de system, quizás es el argumento de más peso .

la funcion main puede aceptar cualquier numero entero sea positivo o negativo

Exactamente, y la coherencia juega un papel importante en el desarrollo de un programa.

[rir3760]

Si se trata de compilar tu programa en Dev-C++ con las opciones predeterminadas se genera el mensaje:

Código [Seleccionar] Expandir

#warning This file includes at least one deprecated or antiquated header.
Please consider using one of the 32 headers found in section 17.4.1.2 of the C++
standard. Examples include substituting the <X> header for the <X.h> header for
C++ includes, or <iostream> instead of the deprecated header <iostream.h>. To
disable this warning use -Wno-deprecated.
Ello en relación a la inclusión de <iostream.h>, caso similar con <stdlib.h>. En cuanto a <conio.h> no es necesario utilizarla.

En cuanto a la función principal esta debe retornar cero o bien un valor distinto en caso de error (tu devuelves 9 de forma indiscriminada).

En mi opinión deberías revisar alguna pagina sobre pilas en la red (por ejemplo Wikipedia) y rescribir el programa desde cero ya que tiene varios errores.

Un saludo

[Queta]

En definitiva, lo que le intentamos explicar CeroX901 y yo y no entendió.

[CeroX901]

No sé exactamente lo que pide, recuerdo mi primer programa en la U que trataba sobre Listas doblemente enlazadas con estructuras y punteros, me quedó hermoso  ese programa , no sé si quiere hacer lo mismo..

[charmedever]

Hola modifique el codigo lo compile con DEV C++ y me corre, me deja ingresar 4 elementos de tipo entero pero no me deja visualizarlos en el orden en q los ingrese
Anexo el codigo

Código (cpp) [Seleccionar] Expandir

//IMPLEMENTA UNA PILA
# include <iostream.h>
# include <conio.h>
# include <stdlib.h>
struct pilas
{
       int x;
}pi[3];
//prototipos de función
void llenar(pilas);
int extraer(pilas);
int tope=-1;

int main()
{

  cout<<"LLENAR PILA"<<endl;

  while(tope<3)
  {
                 for (int i=0; i<3; i++)
                 {
                     cin>>pi[i].x;
                     llenar (pi[i]);
                 }
  }
  cout<<"Pulsar cualquier tecla para continuar";
  getche();
  system ("CLS");
  cout<<"Datos de la pila"<<endl;
  for (int i=0; i<3; i++)
  {
      cout <<pi[i].x;
      extraer(pi[i]);
  }
  getche ();
  return 0;
} //fin de main

void llenar(pilas)
{
  if(tope+1 >=3)
  cout<<"Pila llena"<<endl;
  pi[++tope]=pi[3];
}

int extraer(pilas)
{
  while(tope>=0)
  {
                for (int i=0; i<3; i++)
                {
                    pi[--tope]=pi[3];
                    getche ();
                }
                if(tope==-1)
                cout<<"Pila vacia";
  }
  return 0;
}