Rastreo y muestreo de dato en un arreglo

[baker1562]

Hola, tengo un problemita y desearia que me ayudaran, en el case 4 del programa, tengo que ingresar el ID de una persona(previamente se tuvo que haber guardado,en la opcion 1, la de los socios), si se indentifica muestra a quien el nombre de la persona, y si no muestra que no existe un ID asi en memoria, y no lo he podido hacer, y ademas me sale un error en una variable al final del programa y no he podido comprobar si funciona, alguien que me AYUDEEE!!

Código C++:

#include <iostream>
#include <iomanip>
 
using namespace std;
 
 typedef struct{
    int ISBN;
    char nombre_libro[100];
    char editorialfadeIn[100];
    int numero_edicionfadeIn;
    char autorfadeIn[100];
    int identificadorfadeIn;
    char nombrefadeIn[100];
    char apellidosfadeIn[100];
    int fecha_registrofadeIn;
    int estadofadeIn;
 } datos_libros;
 
 
 int buscar(datos_libros vector[],int longitud, int valor){
 
   for(int i=0;i<longitud;i++){
      if (vector[i].ISBN == valor)
          return i;
   }
   return -1;
 }
 
 
int main ()
{
    
    datos_libros libros[4];
    int opcion;
    int i = 0;
    int j = 0;
    int z;
    int y;
    int ISBN=0;
    int socio;
    int k;
    int x;
    int ID;
    string respuesta;
    
    
    while(respuesta!="N" && respuesta!="n")
    {
       system("color 0f");
       cout<<"******Biblioteca Publica^El Raton Feliz^******"<<endl;
       cout<<endl;
       cout<<"Digite #1, para ingresar nuevo socio."<<endl;
       cout<<"Digite #2, para cambiar estado del socio."<<endl;
       cout<<"Digite #3, para ingresar nuevo libro."<<endl;
       cout<<"Digite #4, para prestar libro."<<endl;
       cout<<"Digite #5, pàra recibir libro."<<endl;
       cout<<"Digite #6, para desplegar lista de libros."<<endl;
       cout<<"Digite #7, para desplegar lista de socios."<<endl;
       cout<<"Digite #8, para desplegar lista de libros prestados actualmente"<<endl;
       cout<<endl;
       cout<<"Que opcion desea realizar?"<<endl;
       cin>>opcion;
       system("cls");
       
       
        switch(opcion){
             
         case 1:
              if(i<4)
              {
              cout<<"******Ingreso de nuevo socio******"<<endl;
              cout<<endl;
              cout<<"Ingrese nombre del socio # "<<i<<":"<<endl;
              cin>>libros[i].nombrefadeIn;
              cout<<endl;
              cout<<"Ingrese apellidos del socio:"<<endl;
              cin>>libros[i].apellidosfadeIn;
              cout<<endl;
              cout<<"Ingrese cedula:"<<endl;
              cin>>libros[i].identificadorfadeIn;
              cout<<endl;
              cout<<"Ingrese fecha de registro:(formato dd/mm/aaaa)"<<endl;
              cin>>libros[i].fecha_registrofadeIn;
              cout<<endl;
              cout<<"Ingrese #1 para usuario activo o ingreso #2 para usuario inactivo:"<<endl;
              cin>>libros[i].estadofadeIn;
              cout<<endl;
              system("cls");
              if(libros[i].estadofadeIn==1)
              {
                system("color 0e");
                cout<<"El socio de nombre "<<libros[i].nombrefadeIn<<" esta activo!!!"<<endl;
                system("pause");
                system("cls");
                }
                else
                {
                    system("color 0e");
                    cout<<"El socio de nombre "<<libros[i].nombrefadeIn<<" esta inactivo!!!"<<endl;
                    system("pause");
                    system("cls");
                    }
                    cout<<endl;
              i=i+1;
              }
              else
              {   
                  system("color 4e");
                  cout<<"ERROR/No se pueden ingresar mas datos/Memoria de socios llena"<<endl;
              system("pause");
               }
               break;
               
               case 2:
                    cout<<"*******Cambio de estado de los socios******"<<endl;
                    cout<<endl;
                    cout<<"Ingrese numero de socio(valido de 0 a 3)"<<endl;
                    cin>>socio;
                    cout<<endl;
                    k=socio;
                    if(k>3)
                    {
                           system("cls");
                           system("color 0b");
                           cout<<"ERROR//Numero de socio inexistente//Ingrese un numero de socio valido!!!"<<endl;
                           cout<<endl;
                           system("pause");
                           system("cls");
                           
                           break;
                           }
                    if(libros[k].estadofadeIn==1)
                    {
                       cout<<"El socio #"<<k<<" de nombre: "<<libros[k].nombrefadeIn<<" se encuentra activo"<<endl;
                       cout<<endl;
                       cout<<"Para cambia el estado del socio a inactivo, presione #2!!"<<endl;
                       cin>>libros[k].estadofadeIn;
                       cout<<endl;
                       cout<<"Cambio de estado del socio a inactivo, fue hecho satisfactoriamente!!"<<endl;
                       system("pause");
                       system("cls");
                       }
                       else
                       {
                           cout<<"El socio #"<<k<<" de nombre: "<<libros[k].nombrefadeIn<<" se encuentra inactivo"<<endl;
                           cout<<endl;
                           cout<<"Para cambiar el estado del socio a activo, presione#1!!"<<endl;
                           cin>>libros[k].estadofadeIn;
                           cout<<endl;
                           cout<<"Cambio de socio a activo, fue hecho satisfactoriamente!!"<<endl;
                           system("pause");
                           system("cls");
                           }
                           cout<<endl;
                           
                           
                           
                    break;
                    
           case 3:
 
               if(j<4)
                       {
                       cout<<"******Ingreso de Nuevo Libro******"<<endl;
                       cout<<endl;
                       cout<<"Ingrese el nombre del libro nuevo # "<<j<<":"<<endl;
                       cin>>libros[j].nombre_libro;
                       cout<<endl;
                       cout<<"Ingrese numero de edicion:"<<endl;
                       cin>>libros[j].numero_edicionfadeIn;
                       cout<<endl;
                       cout<<"Ingrese editorial:"<<endl;
                       cin>>libros[j].editorialfadeIn;
                       cout<<endl;
                       cout<<"Ingrese el ISBN del libro:"<<endl;
                       cin>>ISBN;
                       cout<<endl;
                       while (buscar(libros,j,ISBN)!=-1){
                         cout << "ERROR//Ya has introducido un libro con ese ISBN" << endl;   
                         cout<<endl;         
                         cout<<"Ingrese el ISBN correcto del libro:"<<endl;
                         cin>>ISBN;
                         system("cls");
                       }        
                       
                       libros[j].ISBN = ISBN;
                       j=j+1;
                       }
                       else
                       {   system("color 0c");
                           cout<<"ERROR//Memoria de libros llena"<<endl;
                           system("pause");
                           }
                           break;
                           
                   case 4:
                        cout<<"******Prestamo de libro******"<<endl;
                        cout<<endl;
                        cout<<"Ingrese la ID de la persona en cuestion:"<<endl;
                        cin>>ID;
                        x=i;
                        for(int i=0;i<x;i++)
                        if(libros[i].identificadorfadeIn==ID)
                        {
                          cout<<"El socio de con la cedula "<<ID<<" es de nombre: "<<libros[i].nombrefadeIn<<endl;
                          cout<<endl;
                          
                          }
                          else
                          {
                              cout<<"No existe una persona con ese ID"<<endl;
                              cout<<endl;
                              }
                              
                           break;        
                   case 6:
                            cout<<"Nombre del Libro"<<setw(20)<<"ISBN"<<endl;
                            z=j;
                            for(int j=0;j<z;j++)
                            cout<<libros[j].nombre_libro<<setw(31)<<libros[j].ISBN<<endl;
                            break;
                            
                     case 7:
                              cout<<"Socios"<<setw(20)<<"Cedula"<<endl;
                              y=i;
                              for(int i=0;i<y;i++)
                              cout<<libros[i].nombrefadeIn<<setw(8)<<libros[i].apellidosfadeIn<<setw(13)<<libros[i].identificadorfadeIn<<endl;
                              }
                              }
                       
       system("color 0f");
       cout<<"******Biblioteca Publica-El Raton Feliz******"<<endl;                
       cout<<"Desea realizar otra accion. s/n?"<<endl;
       cin>>respuesta;
       system("cls");
       }
       return 0;
       }
 

De antemano muchas gracias

[sam90]

Solo tienes un problema de llaves, fijate que al final del case 7 pusiste dos llaves cuandp deberia ser una. Ese tipo de error deberias ir aprendiendo a leerlos desde el compilador y probar de solucionarlo por tu cuenta. Ayuda al aprendizaje.

Otra cosas para remarca que no me gusta mucho, y que cuesta ala interpretacion, vos llevas en la variable i el numero de socios registrados. ya de por si no es un nombre muy descriptivo. Segun fijate que cuando queres hacer un for desde 0 hasta i te pasa que queres usar a i como un contador. Entonces declaras una nueva varaible para poder saber el valor y definis en ese ambito una nueva i. Eso mezcla mucho las cosas. Si tan solo cambiaras el nombre de i por numerosocios (por ejemplo) y dejarias a i como un simple contador no deberias hacer esas vueltas para iterar. Son solo sugerencia que ayudan a vos y a lo demas a solucionar tus problemas.
Espero que te sirva.
La llave demas esta en la linea 229. Para ver que llave esta de mas en esto codigo muy largos te puede ayudar una buena identacion, comentarios cuando cerras las llaves, o un buen editor que te vaya marcando con colores las llaves.

[baker1562]

si gracias por el comentario, ya lo arregle, y tenes razon, es que uso i, para que la salida del case 7 y el 4 vayan de la mano con los socios, porque digamos que en el 7 muestro los socios que hay, y tan solo he ingresado 1, me sale el primer socio, y un monton de caracteres raros, donde no hay nada, pero igual creare una variable cantidadsocios o cantidadlibros, segun como se necesiten, graxx por tu ayuda!!

[sam90]

lo unico que tienes uqe hacer es renombrear a la variable i por un nombre mas discriptivo. La i uqe esta en lo for se tiene que seguir llamando i. Y asi te ahorras intrucciones tales como:
y=i;

Realmente inacesarias.

[baker1562]

para evitar hacer un nuevo tema necesito que me ayudes con el mismo case 4: con un while que tengo ahi, si meto un ID que no existe me tira que ingrese un el ID correcto, hasta ahi todo bien, pero si meto un ID correcto me muestra a quien pertenece y me hace un bucle infinito

Código C++:

#include <iostream>
#include <iomanip>
 
using namespace std;
 
 typedef struct{
    int ISBN;
    char nombre_libro[100];
    char editorialfadeIn[100];
    int numero_edicionfadeIn;
    char autorfadeIn[100];
    int identificadorfadeIn;
    char nombrefadeIn[100];
    char apellidosfadeIn[100];
    int fecha_registrofadeIn;
    int estadofadeIn;
 } datos_libros;
 
 
 int buscar(datos_libros vector[],int longitud, int valor){
 
   for(int i=0;i<longitud;i++){
      if (vector[i].ISBN == valor)
          return i;
   }
   return -1;
 }
 
 
int main ()
{
    
    datos_libros libros[4];
    int opcion;
    int i = 0;
    int j = 0;
    int ISBN=0;
    int socio;
    int k;
    int ID;
    int cantidad_libros;
    int cantidad_socios;
    string respuesta;
    
    
    while(respuesta!="N" && respuesta!="n")
    {
       system("color 0f");
       cout<<"******Biblioteca Publica^El Raton Feliz^******"<<endl;
       cout<<endl;
       cout<<"Digite #1, para ingresar nuevo socio."<<endl;
       cout<<"Digite #2, para cambiar estado del socio."<<endl;
       cout<<"Digite #3, para ingresar nuevo libro."<<endl;
       cout<<"Digite #4, para prestar libro."<<endl;
       cout<<"Digite #5, pàra recibir libro."<<endl;
       cout<<"Digite #6, para desplegar lista de libros."<<endl;
       cout<<"Digite #7, para desplegar lista de socios."<<endl;
       cout<<"Digite #8, para desplegar lista de libros prestados actualmente"<<endl;
       cout<<endl;
       cout<<"Que opcion desea realizar?"<<endl;
       cin>>opcion;
       system("cls");
       
       
        switch(opcion){
             
         case 1:
              if(i<4)
              {
              cout<<"******Ingreso de nuevo socio******"<<endl;
              cout<<endl;
              cout<<"Ingrese nombre del socio # "<<i<<":"<<endl;
              cin>>libros[i].nombrefadeIn;
              cout<<endl;
              cout<<"Ingrese apellidos del socio:"<<endl;
              cin>>libros[i].apellidosfadeIn;
              cout<<endl;
              cout<<"Ingrese cedula:"<<endl;
              cin>>libros[i].identificadorfadeIn;
              cout<<endl;
              cout<<"Ingrese fecha de registro:(formato dd/mm/aaaa)"<<endl;
              cin>>libros[i].fecha_registrofadeIn;
              cout<<endl;
              cout<<"Ingrese #1 para usuario activo o ingreso #2 para usuario inactivo:"<<endl;
              cin>>libros[i].estadofadeIn;
              cout<<endl;
              system("cls");
              if(libros[i].estadofadeIn==1)
              {
                system("color 0e");
                cout<<"El socio de nombre "<<libros[i].nombrefadeIn<<" esta activo!!!"<<endl;
                system("pause");
                system("cls");
                }
                else
                {
                    system("color 0e");
                    cout<<"El socio de nombre "<<libros[i].nombrefadeIn<<" esta inactivo!!!"<<endl;
                    system("pause");
                    system("cls");
                    }
                    cout<<endl;
              i=i+1;
              cantidad_socios=i;
              }
              else
              {   
                  system("color 4e");
                  cout<<"ERROR/No se pueden ingresar mas datos/Memoria de socios llena"<<endl;
              system("pause");
               }
               break;
               
               case 2:
                    cout<<"*******Cambio de estado de los socios******"<<endl;
                    cout<<endl;
                    cout<<"Ingrese numero de socio(valido de 0 a 3)"<<endl;
                    cin>>socio;
                    cout<<endl;
                    k=socio;
                    if(k>3)
                    {
                           system("cls");
                           system("color 0b");
                           cout<<"ERROR//Numero de socio inexistente//Ingrese un numero de socio valido!!!"<<endl;
                           cout<<endl;
                           system("pause");
                           system("cls");
                           
                           break;
                           }
                    if(libros[k].estadofadeIn==1)
                    {
                       cout<<"El socio #"<<k<<" de nombre: "<<libros[k].nombrefadeIn<<" se encuentra activo"<<endl;
                       cout<<endl;
                       cout<<"Para cambia el estado del socio a inactivo, presione #2!!"<<endl;
                       cin>>libros[k].estadofadeIn;
                       cout<<endl;
                       cout<<"Cambio de estado del socio a inactivo, fue hecho satisfactoriamente!!"<<endl;
                       system("pause");
                       system("cls");
                       }
                       else
                       {
                           cout<<"El socio #"<<k<<" de nombre: "<<libros[k].nombrefadeIn<<" se encuentra inactivo"<<endl;
                           cout<<endl;
                           cout<<"Para cambiar el estado del socio a activo, presione#1!!"<<endl;
                           cin>>libros[k].estadofadeIn;
                           cout<<endl;
                           cout<<"Cambio de socio a activo, fue hecho satisfactoriamente!!"<<endl;
                           system("pause");
                           system("cls");
                           }
                           cout<<endl;
                           
                           
                           
                    break;
                    
           case 3:
 
               if(j<4)
                       {
                       cout<<"******Ingreso de Nuevo Libro******"<<endl;
                       cout<<endl;
                       cout<<"Ingrese el nombre del libro nuevo # "<<j<<":"<<endl;
                       cin>>libros[j].nombre_libro;
                       cout<<endl;
                       cout<<"Ingrese numero de edicion:"<<endl;
                       cin>>libros[j].numero_edicionfadeIn;
                       cout<<endl;
                       cout<<"Ingrese editorial:"<<endl;
                       cin>>libros[j].editorialfadeIn;
                       cout<<endl;
                       cout<<"Ingrese el ISBN del libro:"<<endl;
                       cin>>ISBN;
                       cout<<endl;
                       while (buscar(libros,j,ISBN)!=-1){
                         cout << "ERROR//Ya has introducido un libro con ese ISBN" << endl;   
                         cout<<endl;         
                         cout<<"Ingrese el ISBN correcto del libro:"<<endl;
                         cin>>ISBN;
                         system("cls");
                       }        
                       
                       libros[j].ISBN = ISBN;
                       j=j+1;
                       cantidad_libros=j;
                       }
                       else
                       {   system("color 0c");
                           cout<<"ERROR//Memoria de libros llena"<<endl;
                           system("pause");
                           }
                           break;
                           
                   case 4:
                        cout<<"******Prestamo de libro******"<<endl;
                        cout<<endl;
                        cout<<"Ingrese la ID de la persona en cuestion:"<<endl;
                        cin>>ID;
                        
                        while(libros[i].identificadorfadeIn!=ID)
                        {
                        for(int i=0;i<cantidad_socios;i++)
                        if(libros[i].identificadorfadeIn==ID)
                        {
                          cout<<"El socio de con la cedula "<<ID<<" es de nombre: "<<libros[i].nombrefadeIn<<endl;
                          cout<<endl;
                          
                          }
                          else
                          {
                              system("cls");
                              cout<<"ERROR//No existe una persona con ese ID"<<endl;
                              cout<<endl;
                              system("pause");
                              system("cls");
                              cout<<"Ingrese la ID correcto:"<<endl;
                              cin>>ID;
                              
                              }
                              }
                              
                           break;        
                   case 6:
                            cout<<"Nombre del Libro"<<setw(20)<<"ISBN"<<endl;
                            for(int j=0;j<cantidad_libros;j++)
                            cout<<libros[j].nombre_libro<<setw(31)<<libros[j].ISBN<<endl;
                            break;
                            
                     case 7:
                              cout<<"Socios"<<setw(20)<<"Cedula"<<endl;
                              for(int i=0;i<cantidad_socios;i++)
                              cout<<libros[i].nombrefadeIn<<setw(8)<<libros[i].apellidosfadeIn<<setw(13)<<libros[i].identificadorfadeIn<<endl;
                              break;
                              }
                       
       system("color 0f");
       cout<<"******Biblioteca Publica-El Raton Feliz******"<<endl;                
       cout<<"Desea realizar otra accion. s/n?"<<endl;
       cin>>respuesta;
       system("cls");
       }
       return 0;
       }
 

no se que estoy haciendo mal en while del case 4

[sam90]

La variable i del while es la que lleva el conteo. en tu caso siempre va a estar uno mas arriba del indice correcto. Es como si fuese el tope. Por eso al fuscar el identificador de ese indice siempre va aver basura y siempre a ser distinto del while.
Yo sacaria el while. si la id es correcta, que vuelva ingresar en el case 4 para buscar.