Ver Mensaje Individual
  #1 (permalink)  
Antiguo 29/01/2014, 10:10
BKsiragon
 
Fecha de Ingreso: enero-2013
Ubicación: venezuela
Mensajes: 6
Antigüedad: 11 años, 10 meses
Puntos: 0
Pregunta Mejorar estructuras y procesos?

Hola saludos, escribo para que que mejoras u optimizaciones le pueden hacer a este programita para que tenga una mejor estructura y un mejor procesamiento de los datos.
aqui les dejo el codigo con el enunciado.


Código:
/*
	Desarrollar un programa que registre los datos 
	de un grupo de personas (cedula, nombre, apellido, teléfono). 
	El programa debe ser capaz de mostrar el registro completo de 
        una persona de acuerdo a la cedula ingresada.

*/


 #include <stdio.h>
 #include <conio.h> 
 #include <iostream>
 #include <string>
 using namespace std;
 
 int a=0;
 class Persona { 
   public:
   string nombre[50],apellido[50];
   int cedula[50],telefono[50];
   void registrar();
   void mostrar();
 }; 
 
  void Persona::registrar() { 
  cout<<"ingrese su nombre:"<<endl;
  cin>>nombre[a];
  cout<<"ingrese su apellido:"<<endl;
  cin>>apellido[a];
  cout<<"ingrese su cedula:"<<endl;
  cin>>cedula[a];
  cout<<"ingrese su telefono:"<<endl;
  cin>>telefono[a];
  cout<<"sus datos han sido registrados correctamente:"<<endl;
  system("pause");
  system("cls");
  
  a++;                    
 } 
 void Persona::mostrar(){
      int ci=0,i=0,encontrado=0;
      cout<<"\ningrese su cedula para mostrar sus datos:"<<endl;
      cin>>ci;
      for (i=0;i<51;i++){
          if (ci==cedula[i]){
              cout<<"nombre: "<<nombre[i]<<endl;
              cout<<"apellido: "<<apellido[i]<<endl;
              cout<<"cedula: "<<cedula[i]<<endl;
              cout<<"telefono: "<<telefono[i]<<endl;
              system("pause");
              system("cls");
              encontrado=1;
              break;
              }
          else
              encontrado=0;
      }
      if (encontrado==0) 
      {cout<<"No se encuentra nadie registrado con esta cedula"<<endl;}
      system("pause");
      system("cls");
         
      
}
 main() { 
   Persona p;
   int opc;    
    cout<<endl;
    cout<<"\n\nPor favor elija una opcion: \n1.-registrarse en el sistema \n2.-mostrar sus datos"<<endl;
    cin>>opc;
    while(opc){
       switch(opc){
          case 1:{
             p.registrar();
             break;}
          case 2:{
             p.mostrar();
             break;}
       }
    cout<<"\n\nPor favor elija una opcion: \n1.-registrarse en el sistema \n2.-mostrar sus datos"<<endl;
    cin>>opc;
    }
   cin.get();
   cin.get();
   return 0; 
 }