Podrían decirme en qué me equivoqué en este fragmento de código?

[jeff23]

Código (cpp) [Seleccionar] Expandir

void buscar(){
   string val;
   cout<<"digite el nombre a buscar";
   cin>>val;
   for(int i=0; i<miAgenda.size(); i++){
           if(val==miAgenda[i].nombres){
                   cout<<"nombre"<<miAgenda[i].nombres;
           }
           if(val==miAgenda[i].apellidos){
                   cout<<"apellido"<<miAgenda[i].apellidos;
           }
           if(val==miAgenda[i].email){
                   cout<<"correo"<<miAgenda[i].email;
           }
           if(val==miAgenda[i].fijo){
                   cout<<"tel.fijo"<<miAgenda[i].fijo;
           }
           if(val==miAgenda[i].celular){
                   cout<<"tel.cel"<<miAgenda[i].celular;
           }
   }
}

int main() {
   int np;
   char op = '0';

   while(op < '3'){
        system("CLS");
       menu();
       cin >> op;
       switch(op) {
           case '1':
                       buscar();
                       break;
           case '2':
                       break;
           default:
               cout << "\n\n\tERROR!";
               system("PAUSE");
               op = '0';
       }
   }
   cout << "\n\n\tGracias NOS VEMOS...\n\n";

 return 0;
}

La cosa es que tengo que hacer una función para buscar datos,ya la hice,pero no me funciona...



Mod: Los códigos deben ir en etiquetas GeSHi

[ivancea96]

La cosa es que tengo que hacer una función para buscar datos,ya la hice,pero no me funciona...

¿Qué no funciona? ¿Tira error? ¿No busca bien? Cual es el problema.

[jeff23]

El código se ejecuta,pero no funciona,osea,es como si ignorara ese fragmento

[jeff23]

Aquí está el código completo:

Código (cpp) [Seleccionar] Expandir

#include <cstdlib>
#include <iostream>
#include <vector>

using namespace std;


struct Persona{
    string nombres;
    string apellidos;
    char celular[10];
    char fijo[7];
    string email;

};
vector<Persona> miAgenda;

void menu(){
    cout<< "\n\tMENU PRINCIPAL";
    cout<< "\n\t1. Entrada de datos";
    cout<< "\n\t2. Mostrar datos";
    cout<< "\n\t3. Salir del programa";
    cout<< "\n\t4. Buscar datos";
    cout<< "\n\tEscoja opcion ";
}

void entrarDatos(){
    Persona p;

    cout<<"\n\tDigite su Nombres: ";
    cin>>p.nombres;
    cout<<"\n\tDigite su Apellidos: ";
    cin>>p.apellidos;
    cout<<"\n\tDigite su correo: ";
    cin>>p.email;
    cout<<"\n\tDigite su Telefono Fijo: ";
    cin>>p.fijo;
    cout<<"\n\tDigite su Telefono: celular ";
    cin>>p.celular;

    miAgenda.push_back(p);
}

void mostrarDatos(){

    cout<<"\n\t====== Agenda ======";
    for(int i=0; i<miAgenda.size();i++){
        cout<<"\n\tNombre : "<<miAgenda[i].nombres;
        cout<<"\n\tApellidos : "<<miAgenda[i].apellidos;
        cout<<"\n\tCorreo : "<<miAgenda[i].email;
        cout<<"\n\tTel.Fijo : "<<miAgenda[i].fijo;
        cout<<"\n\tTel.Cel : "<<miAgenda[i].celular;
        cout<<"\n\t------------------------------------";
    system("pause");
    }
}

void buscar(){
    string val;
    cout<<"digite el nombre a buscar";
    cin>>val;
    for(int i=0; i<miAgenda.size(); i++){
            if(val==miAgenda[i].nombres){
                    cout<<"nombre"<<miAgenda[i].nombres;
            }
            if(val==miAgenda[i].apellidos){
                    cout<<"apellido"<<miAgenda[i].apellidos;
            }
            if(val==miAgenda[i].email){
                    cout<<"correo"<<miAgenda[i].email;
            }
            if(val==miAgenda.fijo){
                    cout<<"tel.fijo"<<miAgenda[i].fijo;
            }
            if(val==miAgenda[i].celular){
                    cout<<"tel.cel"<<miAgenda[i].celular;
            }
    }
}

int main() {
    int np;
    char op = '0';

    while(op < '3'){
         system("CLS");
        menu();
        cin >> op;
        switch(op) {
            case '1':
                        entrarDatos();
                        break;
            case '2':
                        mostrarDatos();
                        break;
            case '3':
                        buscar();
                        break;
            case '4':
                        break;

            default:
                cout << "\n\n\tERROR!";
                system("PAUSE");
                op = '0';
        }
    }
    cout << "\n\n\tGracias NOS VEMOS...\n\n";

  return 0;
}


[ivancea96]

Debes aprender a encontrar los errores.
Te doy unas pautas:

• En caso de error en tiempo de ejecución:
  Localiza la línea o función que tira el error. Para ello, puedes colocar salidas por pantalla en varias partes del código. En el momento en que una salida no se represente, significa que el error está entre la última salida, y la que no se vió. Evidente, ¿no?
• En caso de código "que no hace nada":
  Este caso se suele dar generalmente, en 1 de estas 2 ocasiones: o código que realmente no hace nada, por algún error al escribirlo, o código que no se llega a ejecutar. En el segundo caso, que tal vez sea el tuyo, verifica que las condificones o bucles que haya son correctas (Es posible que una condición sea errónea). La técnica de las salidas por pantalla funciona bien ahí también.

Con esto te quiero decir, que te va a ser más fácil a ti buscar el error, que a los demás. No tires la toalla, y sigue buscando. Analiza el código bloque a bloque, hasta ver exactamente qué falla.