[SOLUCIONADO] Busqueda de comparacion de cadenas con arreglo de estructuras

[dato000]

Buenas, hacia tiempo no venia por aqui, cosas de la vida

Gente, por favor, necesito su ayuda, estoy atorado con este punto, pensaba que iba a funcionar, y me tiene extrañado el porque no funciona, si en mi prueba de escritorio deberia servir bien, es más tal vez un problema de abstracción lógica que de programación en si.

Verán, tengo que realizar una busqueda de un nombre de una persona, en un arreglo de 10 estructuras he guardado esa información de cada persona (nombre, edad), de forma tal que se tiene una estructura rapida de cada persona para buscar ya sea por nombre o por edad de persona.

Ahora, cuando estoy intentando buscar por el nombre de la persona, funciona, encuentra a la persona en ese arreglo, todo va bien, hasta el momento de imprimir en pantalla, pues solo me imprime siempre la primera posición del arreglo.

Esta es la parte en cuestión:

Código (c) [Seleccionar] Expandir


typedef struct persona
{
   char nombre[50];
   int edad;
} PERSONA;

int main()
{
   int i;
   int posicionBusqueda=0;
   char opcion;
   char nombreBusqueda[50];
   PERSONA arregloPersonas[10] = {  { "manu chao", 50},
                                    { "maradona", 64},
                                    { "pele", 79},
                                    { "Tyrion", 34},
                                    { "Lady", 26},
                                    { "Lorena", 20},
                                    { "Manuela", 15},
                                    { "Jacobo", 7},
                                    { "Salome", 3},
                                    { "Camilo", 27}  };

   printf("\nNombre:  %s", arregloPersonas[9].nombre);
   printf("\nEdad:  %i", arregloPersonas[9].edad);

   printf("HISTORIAL DE PERSONAS\n\n");
   printf("a. Buscar por nombre\n");
   printf("b. Buscar por edad\n");
   printf("c. Salir\n\n");

   printf("Digite una opcion:  "); scanf("%c", &opcion);

   if(opcion == 'a' || opcion == 'A')
   {
       printf("Escriba el nombre de la persona que busca:  "),
       scanf("%s", nombreBusqueda);
       printf("Buscando...");
       //sleep(2);
       for(i=0; i<10; i++)
       {
           //if(nombreBusqueda == arregloPersonas[i].nombre)
           if(strcmp(nombreBusqueda, arregloPersonas[i].nombre))
           {
               posicionBusqueda=i;
               printf("\n\n\nNOMBRE ENCONTRADO!!\n");
               printf("\n--Informacion de la Persona--");
               printf("\nNombre de la Persona Buscada:  %s", arregloPersonas[i].nombre);
               printf("\nEdad de la Persona Buscada:  %i", arregloPersonas[i].edad);
               break;
           }
           else
           {
               printf("\n\nNOMBRE NO SE ENCUENTRA EN LA BASE DE DATOS.");
               printf("\nIntente nuevamente.");
               break;
           }

           //posicionBusqueda++;
       }

   }

...


Particularmente esta área, no se porque siempre me imprime la primera posición del arreglo de estructuras de las personas

Código (c) [Seleccionar] Expandir


for(i=0; i<10; i++)
       {
           //if(nombreBusqueda == arregloPersonas[i].nombre)
           if(strcmp(nombreBusqueda, arregloPersonas[i].nombre))
           {
               posicionBusqueda=i;
               printf("\n\n\nNOMBRE ENCONTRADO!!\n");
               printf("\n--Informacion de la Persona--");
               printf("\nNombre de la Persona Buscada:  %s", arregloPersonas[i].nombre);
               printf("\nEdad de la Persona Buscada:  %i", arregloPersonas[i].edad);
               break;
           }
           else
           {
               printf("\n\nNOMBRE NO SE ENCUENTRA EN LA BASE DE DATOS.");
               printf("\nIntente nuevamente.");
               break;
           }

           //posicionBusqueda++;
       }



En verdad apreciaria cualquier aporte, me quede atorado y sigo investigando como remediar eso, pero realmente me dejo atonito que simplemente no funcionara siendo algo tan simple pero como  

[do-while]

¡Buenas!

El fallo está aquí:

Código (c) [Seleccionar] Expandir


if(strcmp(nombreBusqueda, arregloPersonas[i].nombre))


strcmp te devuelve el "orden alfabético" del primer argumento con respecto al segundo. Si el primero es alfabéticamente menor te devolverá un valor negativo, si es mayor positivo y si son iguales te devolverá cero. Tienes que poner un operador de negación ! antes de strcmp para que la condición sea cierta solo si la función te devuelve una condición falsa (0).

¡Saludos!

[dato000]

mmm no no es eso, ya lo habia intentado, verás, necesito que se cumpla la condición, si cambio la condición también tendria que cambiar el orden del if-else, y quedaria exactamente igual, porque si utilizo el operador ! lo que me indica para este caso es que lo encuentra pero no lo reporta.

Hasta hace unos minutos me di cuenta que si quito el break de la condición principal, imprime todo los datos Hasta la posición en la que encuentra el valor, que efectivamente, deberia imprimir.

Código (c) [Seleccionar] Expandir


if(strcmp(arregloPersonas[i].nombre, nombreBusqueda))
           {

               printf("\n\n\nNOMBRE ENCONTRADO!!\n");
               printf("\n--Informacion de la Persona--");
               printf("\nNombre de la Persona Buscada:  %s", arregloPersonas[i].nombre);
               printf("\nEdad de la Persona Buscada:  %i", arregloPersonas[i].edad);
               //break;
           }
           else
           {
               printf("\n\nNOMBRE NO SE ENCUENTRA EN LA BASE DE DATOS.");
               printf("\nIntente nuevamente.");
               break;
           }


Por ese lado funciona, pero no me imprime la posición que busco (y es solo esa posición del arreglo de struct que necesito) mmmmmmm me quede corto con algo tan básico pero que pues, uds entienden, estanca.

Esto es lo que obtengo, por poner un ejemplo:




--------------------------------------------------------EDITADO--------------------------------------------

He notado una cosa, al editar la siguiente linea:

Código (c) [Seleccionar] Expandir

if(strcmp(arregloPersonas[i].nombre, nombreBusqueda) == 0)

Ahora todo da falso, claro, yo pensaba que podria utilizarse como un true/false, pero no habia visto que podia tomar 3 valores, en efecto, no se comportaba como esperaba, pero ahora menos, será por la naturaleza de las lineas de texto?? porque no son recibidas de la misma manera en las variables al ejecutar la comparación??

[rir3760]

no se porque siempre me imprime la primera posición del arreglo de estructuras de las personas

Porque:

* La función strcmp retorna un valor diferente de cero si las cadenas son distintas, solo retornara cero si son iguales.

+

* El bucle donde se realiza la búsqueda solo se ejecutara una vez ya que en ambas ramas de la sentencia condicional if terminas el bucle con la sentencia "break":

Código (c) [Seleccionar] Expandir

for (i = 0; i < 10; i++) {
   if (strcmp (nombreBusqueda, arregloPersonas[i].nombre)) {
      /* ... */
     
      break; /* Terminar el bucle si se cumple la condicion del if ... */
   }else {
      /* ... */
     
      break; /* ... y si no se cumple ... tambien */
   }
   
   //posicionBusqueda++;
}

+

* En la primera iteración del bucle se compara la palabra introducida por el usuario y leída con:

Código (c) [Seleccionar] Expandir

printf ("Escriba el nombre de la persona que busca:  "), /* mejor ';' */
scanf ("%s", nombreBusqueda);
Con el campo "nombre" del primer elemento del array el cual consiste de dos palabras:

Código (c) [Seleccionar] Expandir

PERSONA arregloPersonas[10] = {  { "manu chao", 50}, /* ... */
En buen cristiano no hay forma de que la palabra introducida por el usuario sea igual a esas dos palabras.

Para solucionarlo debes leer la entrada del usuario con fgets (asegurate de eliminar el resto de la linea anterior) y utilizar un bucle sin cuerpo mas o menos así:

Código (c) [Seleccionar] Expandir

for (i = 0; i < 10 && strcmp(nombreBusqueda, arregloPersonas[i].nombre); i++)
   ;
if (i != 10 /* Hay que evitar magic numbers, mejor una macro*/)
   puts("Nombre encontrado!");
else
   puts("El nombre no se encuentra en la base de datos");

Un saludo

[dato000]

Pues francamente no ha funcionado, ya habia pensado lo del string con espacio, y lo habia arreglado, pero sigue igual, el break debo usarlo, porque sino me imprime todas las posiciones del arreglo, por lo que por ende, solo deberia romperse el ciclo for cuando realiza una comparacion exitosa, pero no es así, esto esta raro, pero siempre pasa con estos strings y caracteres en particular con C.

Ahora, lo más extraño es que fgets no quiere funcionar, se que es absurdo usar gets, pero fgets no quiere realizar lo que debe, no trabaja ningún caso, y ya probe diferentes cosas. También probe fflush(stdin) para limpiar el buffer de memoria al suponer que se produce un desbordamiento por el salto de linea en la lectura de la primera opción. Pero sigue igual  .



rir3760, te agradezco lo que mencionas de verdad que si, pero necesito imprimir en pantalla los datos del struct (nombre, edad) dependiendo de la busqueda, no es suficiente indicar con que se encuentra dentro del arreglo, porque no es lo que solicitaron.

Meh al final yo creo que lo voy a hacer en c++ para ver si ya no hay inconvenientes.

[Blaster]

rir3760, te agradezco lo que mencionas de verdad que si, pero necesito imprimir en pantalla los datos del struct (nombre, edad) dependiendo de la busqueda, no es suficiente indicar con que se encuentra dentro del arreglo, porque no es lo que solicitaron.

Solo es cuestión de agregarle algunos printf y ya

Código (cpp) [Seleccionar] Expandir

for (i = 0; i < 10 && strcmp(nombreBusqueda, arregloPersonas[i].nombre); i++);
        if (i != 10)
        {
            printf("\n\n\nNOMBRE ENCONTRADO!!\n");
            printf("\n--Informacion de la Persona--");
            printf("\nNombre de la Persona Buscada:  %s", arregloPersonas[i].nombre);
            printf("\nEdad de la Persona Buscada:  %i", arregloPersonas[i].edad);
        }
        else
            puts("El nombre no se encuentra en la base de datos");


Y del porque no funciona tu programa, ya te lo explicaron los compañeros mas arriba

Saludos

[dato000]

No funciona así, como ya dije arriba también, pero te agradezco la intención.

realmente prefiero usar printf y scanf a gets y puts, cuestión de gustos.

Igual gente ya tengo la solución, al menos funciona para este simple ejercicio:

Código (cpp) [Seleccionar] Expandir


if(opcion == 'a' || opcion == 'A')
    {
        cout << "Escriba el nombre de la persona que busca:  "; cin >> nombreBusqueda;
        cout << nombreBusqueda << endl;
        cout << "Buscando..." << endl;
        sleep(2);
        for (i = 0; i < 10; i++)
        {

            while(strcmp(nombreBusqueda, arregloPersonas[i].nombre)==0 && i<10)
            {
                cout << "\n\n\nNOMBRE ENCONTRADO!!\n";
                cout << "\n--Informacion de la Persona--";
                cout << "\nNombre de la Persona Buscada:  " << arregloPersonas[i].nombre;
                cout << "\nEdad de la Persona Buscada:  " << arregloPersonas[i].edad;
                i++;
            }
        }
    }


Pues lo termine haciendo en C++, pero lo importante es utilizar un ciclo repetitivo dentro de otro ciclo, porque? bueno, como decirlo, fue como un presentimiento, el comportamiento de las cadenas se me hacia sospechoso, algo que pasa como cuando se usan matrices, así que se me ocurrio usarlo, ya sirvio, opte también por quitar lo de NO ENCONTRADO, para no complicarme tanto la vida, igual es un ejercicio, eso es todo.

Vale les agradezco su aporte, su tiempo y esfuerzo, SLDS

[Blaster]

 En primer lugar es innecesario el uso de dos bucles, el uso que das del while puede se fácilmente reemplazado por un if

Ahora voy a explicarte mas claramente porque no funciona tu código, supongamos que tenemos como entrada pele , luego ingresamos al bucle for de inmediato nos encontramos con la siguiente condición if

Código (cpp) [Seleccionar] Expandir

if(strcmp(nombreBusqueda, arregloPersonas[i].nombre))

Esto es como decir "si nombreBusqueda es diferente a arregloPersonas.nombre entonces ejecutar la instrucción contenida en el if" en este caso ya que manu chao es diferente a pele pasara a ingresar en el if ejecutando todos los printf y finalmente el break terminara el bucle.

En el caso de que la entrada sea manu chao se hará la comprobación y como obviamente son iguales retornara false por ende pasara a ejecutarse el else mostrando que el nombre no se encuentra en la base de datos y de nuevo el break terminara el bucle

Como seguro te has dado cuenta el bucle solo realiza una iteración siempre por eso te muestra la información del primer elemento únicamente.

Saludos