Funcion de busqueda?

[Bladsmickey]

Código C:

#include <stdio.h>
#include <windows.h>
#define p printf
#define s scanf
 
struct agenda{
char nombre[25][100];
char telefono[25][100];
int edad[100];
int n;
}a;
 
 
void reordenar(char x[]){
int i2,i, temp1;
char temp2,temp3;
 
for(i=0 ; x[i] ; i++)
for(i2=0 ; x[i2] ; i++){
if(x[i2]<x[i]){
temp1=x[i2];
x[i2]=x[i];
x[i]=temp1;
}}}
 
void buscarcosas(char buscarl[]){
int i;
for(i=0 ; i<a.n ; i++){
if(buscarl==a.nombre[i]){
p("Encontrado!\n");
p("Organizando Datos!...\n");
Sleep(2000);
p("Nombre: %s\n Telefono: %s \n Edad: %d\n\n",&a.nombre[i], &a.telefono[i], a.edad[i]);
}}return;}
 
main(){
 
int i, buscar,n, op;
char buscarl[25];
p("---Bienvenido A Su Agenda Virtual!---\n");
p("Cuantas Personas Registrara en la Agenda: \n");
s("%d", &a.n);
 
for(i=0 ; i<a.n ; i++){
p("Introduzca Nombre: \n");
s("%s", a.nombre[i]);
p("Introduzca Telefono: \n");
s("%s", a.telefono[i]);
p("Introduzca Edad: \n");
s("%s", a.edad);}
do{
p("Generando Menu...\n");
Sleep(1000);
p("\nEscoja Que Hacer: \n");
p("1. Mostrar Lista de Todas las personas: \n");
p("2. Buscar Por Nombre: \n");
p("3. Buscar Por Telefono: \n");
p("4. Buscar Por Edad: \n");
p("5. Salir \n");
p("Opcion: ");
s("%d", &op);
 
switch(op){
 
case 1:{
for(i=0 ; i<a.n ; i++){
p("\nNombre: %s Telefono: %s Edad %d \n", &a.nombre[i], &a.telefono[i], a.edad[i]);
}}break;
 
case 2: {
    p("Que Desea Buscar: \n");
    s("%s", &buscarl);
buscarcosas(buscarl);
}break;
 
case 3:{
    p("Que Desea Buscar: \n");
    s("%s", &buscarl);
buscarcosas(buscarl);
}break;
 
case 4:{
    p("Que Desea Buscar: \n");
    s("%s", &buscarl);
buscarcosas(buscarl);
}break;}
}while(op!=5);
s("%d");
}
 

El problema que tengo es como dice el nombre, Con la funcion buscar, ya que la misma no busca, Espero me puedan ayudar, Saludos!

[sam90]

Primero que todo al definir el struct los indices estan al revez:

Código C:

struct agenda{
char nombre[100][25];
char telefono[100][25];
int edad[100];
int n;
}a;
 

Como lo habias hecho vos solo permitias 25 cotactos con nombre de 100 caracteres. Pero viendo como definiste el vector edad supongo que queres un maximo de 100 contactos.

Otro error, cuando pides los datos de la edad tambien tienes que ponerle indice y ademas el operador & para pasar la direccion de memoria:

Código C:

s("%s", &a.edad[i])
 

Cuando imprimes datos no es necesario que usses el operador &.

Código C:

p("\nNombre: %s Telefono: %s Edad %d \n", a.nombre[i], a.telefono[i], a.edad[i]);
 

Con respecto a tu error fijate que para comparar cadenas no podes hacer
buscarl==a.nombre[i]

Eso solo compara las direcciones de memoria que en este caso van a ser diferentes. Lo que tienes uqe usar es la funcion strcmp. Esta va comparando caracter a caracter para ver si es la misma cadena.

Por el momento eso es todo. Fijate de leer un buen manual de punteros.

[Bladsmickey]

Bien, Ahora el programa Corre, Pero al utilizar el indice en la edad, el programa se Congela, Ademas, la funcion buscar no Funciona Con el telefono, y la Funcion de Organizar de errores:

Código C:

void reordenar(char x[]){
int i2,i, temp1;
char temp2,temp3;
for(i=0 ; x[i] ; i++)
for(i2=0 ; x[i2] ; i++){
if(a.nombre[i2]<a.nombre[i]){
    temp2=a.nombre[i2];
    a.nombre[i2]=a.nombre[i];
    a.nombre[i]=temp2;
    temp3=a.telefono[i2];
    a.telefono[i2]=a.telefono[i];
    a.telefono[i]=temp3;
    temp1=a.edad[i2];
    a.edad[i2]=a.edad[i];
    a.edad[i]=temp1;
}}}
 

No estoy seguro si la sintaxis del la funcion de ordenar es correcta... Tiene que organizar los Nombres y al mismo tiempo Mover cada telefono con su respectivo nombre al igual que la edad.

Código C:

void buscarcosas(char buscarl[]){
int i;
for(i=0 ; i<a.n ; i++){
if(strcmp(buscarl, a.nombre[i])==0){
p("\nEncontrado!\n");
p("Organizando Datos!...\n");
Sleep(2000);
p("Nombre: %s\n Telefono: %s\n Edad: %d\n\n",&a.nombre[i], &a.telefono[i], a.edad[i]);
}}return;}
 

Al buscar el telefono no muestra el mensaje encontrado.

Código C:

p("Introduzca Edad: \n");
s("%s", a.edad);}
 

Si pongo "a.edad[i];" el programa se congela.

Espero me puedas ayudar, Saludos!

[sam90]

En el scanf de la edad tienes que poner &(a.edad[i])

Fijate que en mi respuesta si puse el operador &. Este se usa para pasar una direccion de memoria. En este caso estas pasando la direccion de memoria de la variable edad en el indice i. De la forma que lo haces vos estarias pasando la direccion de la primer variable. Por eso funciona. nada mas uqe te modifica siempre el primer contacto.

Fijate que en los tres cases usas la misma funcion buscar, asi que siempre va a estar buscando por nombre. Deberias hacer una funcion por busqueda un pasar un parametro que indique que busqueda hacer.

Sobre la funcion reordenar no entiendo que es ese array de caracteres "x" que le estas pasando..para que lo usas y para que iteras sobre el.
Deberias aprender un poco mas sobre cadenas de caracteres. No es como en otros lenguajes que puedes usar los operadores <,>,= para compara cadenas. Eso se usan para compara caracteres. Despues tienes que iterar caracter a caracter de la cadena para ver si son iguales o menor.
Fijate que en la libreria string.h vienen un monton de funciones para el manejo de arrays.

Y por ultimo leete algun manual sobre punteros, porque veo que no tener muy buena idea de que se trata y como se manejan,