elhacker.net cabecera Bienvenido(a), Visitante. Por favor Ingresar o Registrarse
¿Perdiste tu email de activación?.
 
Inicio Ayuda Buscar Ingresar Registrarse
29 Mayo 2012, 00:48  


Tema destacado: Suscripción al boletín mensual de elhacker.net

+  Foro de elhacker.net
|-+  Programación
| |-+  Programación C/C++ (Moderadores: Eternal Idol, Littlehorse)
| | |-+  Comparando cadenas
0 Usuarios y 1 Visitante están viendo este tema.
Páginas: [1] 2 Ir Abajo Respuesta Imprimir
Autor Tema: Comparando cadenas  (Leído 1,040 veces)
DickGumshoe

Desconectado Desconectado

Mensajes: 145


Ver Perfil WWW
Comparando cadenas
« en: 22 Enero 2012, 01:17 »

Hola.

Estoy haciendo un problema que dice lo siguiente:

Un programa que pida 10 nombres y los memorice. Después deberá pedir que se teclee un nombre y dirá si se encuentra o no entre los 10 que se han tecleado antes. Volverá a pedir otro nombre y a decir si se encuentra entre ellos, y así sucesivamente hasta que se teclee “fin”.

Yo tengo esto:

Código
 /*Un programa que pida 10 nombres y los memorice. Después deberá pedir que se teclee
un nombre y dirá si se encuentra o no entre los 10 que se han tecleado antes. Volverá
a pedir otro nombre y a decir si se encuentra entre ellos, y así sucesivamente hasta que
se teclee “fin”.*/

 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(){
 
   char n1[20], n2[20], n3[20], n4[20], n5[20], n6[20], n7[20], n8[20], n9[20], n10[20], n11[20];
   char n12[20]={"fin"};
 
   printf("Introduce un nombre");
   gets(n1);
   printf("Introduce otro nombre");
   gets(n2);
   printf("Introduce otro nombre");
   gets(n3);
   printf("Introduce otro nombre");
   gets(n4);
   printf("Introduce otro nombre");
   gets(n5);
   printf("Introduce otro nombre");
   gets(n6);
   printf("Introduce otro nombre");
   gets(n7);
   printf("Introduce otro nombre");
   gets(n8);
   printf("Introduce otro nombre");
   gets(n9);
   printf("Introduce otro nombre");
   gets(n10);
 
 
   do{
   printf("Introduce un nombre para comprobar si se encuentra entre los anteriores");
   gets(n11);
   if((strcmp(n11,n1)||
   strstr(n11,n2)||
   strstr(n3,n11)||
   strstr(n4,n11)||
   strstr(n5,n11)||
   strstr(n6,n11)||
   strstr(n7,n11)||
   strstr(n8,n11)||
   strstr(n9,n11)||
   strstr(n10,n11))==0) printf("El nombre se encuentra entre los anteriores\n");
   else printf("El nombre no se encuentra entre los anteriores\n");
 
   }
   while(n11!=n12);
 
   system("pause");
   return 0;
 
}
 
 

Lo que me hace es que me reconoce si la primera palabra introducida (n1) es igual que la última (n11), pero con las demás no hace nada.

Además, al poner "fin", no finaliza el programa.

¿Alguien podría decirme cuál es mi problema?

Muchísimas gracias.

Saludos.


En línea
RHL


Desconectado Desconectado

Mensajes: 968


mental


Ver Perfil
Re: Comparando cadenas
« Respuesta #1 en: 22 Enero 2012, 02:19 »

Well, si te fijas la funcion strstr devuelve 0 si no encuentra una instancia en el puntero indicado de lo contrario devuelve un puntero a la instancia pero veo que haces una comparacion con el 0 osea esta diciendo que escriba que hay una instancia si
el valor es 0 y en las comparaciones el valor es distinto de cero si encuentra una instancia ( si es correcto) entonces bueno
lo que quiero decir es que estas haciendo lo contrario de lo que queres que haga, si es distinto de cero entonces te va a mostrar
que no hay instancia sabiendo que si hay lo que podes hacer es usar el operador !=

tambien poder simplicar eso de obtener nombres haciendo un bucle


En línea
rir3760


Desconectado Desconectado

Mensajes: 382


Ver Perfil
Re: Comparando cadenas
« Respuesta #2 en: 22 Enero 2012, 03:05 »

El programa tiene errores básicos, se mantiene el mismo consejo que en tu otro tema: consigue material de referencia de calidad.

Como ya se comento deberías utilizar un bucle para verificar si existe una coincidencia y también evitar el uso de la función "gets", utilizar un array de arrays de caracteres para evitar tantas variables, etc.

Por ultimo no hay consistencia en las comparaciones, primero utilizas strcmp, después strstr y por ultimo:
Código
char n10[20], n11[20];
 
/* ... */
 
do {
 
/* ... */
 
}while (n11! = n12);
Si bien es valido (en el sentido de respetar la sintaxis del lenguaje) no hace lo esperado ya que esta comparando las direcciones en memoria de los objetos (los arrays "n11" y "n12") y siendo objetos distintos tienen distintas direcciones. Para compararlas hay que utilizar strcmp.

Para darte una idea, para obtener diez nombres eliminando el '\n' que almacena fgets puedes utilizar:
Código
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define NUM_NOMBRES   10
#define CHARS_NOMBRE  20
 
int main(void)
{
  char nombre[NUM_NOMBRES][CHARS_NOMBRE];
  int i;
  char *p;
  int ch;
 
  for (i = 0; i < NUM_NOMBRES; i++){
     printf("Nombre %2d:", i + 1);
     fflush(stdout);
     fgets(nombre[i], CHARS_NOMBRE, stdin);
 
     /* Buscamos el '\n' y lo eliminamos */
     if ((p = strchr(nombre[i], '\n')) != NULL)
        *p = '\0';
     else
        /* Linea demasiado larga, eliminamos el resto */
        while ((ch = getchar()) != EOF && ch != '\n')
           ;
  }
 
  for (i = 0; i < NUM_NOMBRES; i++)
     printf("Nombre %d: %s\n", i + 1, nombre[i]);
 
  return EXIT_SUCCESS;
}

Un saludo
En línea

The capacity to learn is a gift;
The ability to learn is a skill;
The willingness to learn is a choice.
--
Rebec of Ginaz
DickGumshoe

Desconectado Desconectado

Mensajes: 145


Ver Perfil WWW
Re: Comparando cadenas
« Respuesta #3 en: 22 Enero 2012, 11:45 »

Bueno, acabo de hacer que al insertar "fin" se cierre el programa poniendo:

Código
while((strcmp(n11, "fin")!=0));

Por otra parte, RHL, he puesto ahora

Código
  do{
   printf("Introduce un nombre para comprobar si se encuentra entre los anteriores");
   gets(n11);
   if((strcmp(n1,n11)||
   strcmp(n2,n11)||
   strcmp(n3,n11)||
   strcmp(n4,n11)||
   strcmp(n5,n11)||
   strcmp(n6,n11)||
   strcmp(n7,n11)||
   strcmp(n8,n11)||
   strcmp(n9,n11)||
   strcmp(n10,n11))!=0) printf("El nombre se encuentra entre los anteriores\n");
   else printf("El nombre no se encuentra entre los anteriores\n");

en vez de si es igual a 0. El resultado ahora es que todo lo que inserto se encuentra entre los anteriores...

También, rir3760, creo que al final del manual que estoy siguiendo dan técnicas más avanzadas. Quizá por ahora pueda hacer algo parecido a lo que estoy haciendo, y cuando llegue a las técnicas, corregir el programa (porque con tantas funciones, me he perdido un poco...).

Bueno, muchas gracias a los dos. Voy a seguir intentando que me salga.

Saludos.


EDITO:

Para reducir el código, trato de hacer esto:

Código
do{
   for (i=0;i<=10;i++){
   printf("Introduce un nombre");
   gets(n1[i][20]);
   }
}
   while(n1!=n1[9][20]);

Pero da errores. Así, solo tendría que meter una variable, tal y como dice rir3760.

Gracias.

Saludos.
« Última modificación: 22 Enero 2012, 12:21 por DickGumshoe » En línea
rir3760


Desconectado Desconectado

Mensajes: 382


Ver Perfil
Re: Comparando cadenas
« Respuesta #4 en: 22 Enero 2012, 15:05 »

EDITO:

Para reducir el código, trato de hacer esto:

Código
do{
   for (i=0;i<=10;i++){
   printf("Introduce un nombre");
   gets(n1[i][20]);
   }
}
   while(n1!=n1[9][20]);

Pero da errores. Así, solo tendría que meter una variable, tal y como dice rir3760.
Si declaras el array de esta forma:
Código
char n1[10][20];

Primero: no cuesta nada darle un nombre descriptivo al array, por ejemplo "nombre", así cuando utilizas "nombre[7]" ...

La función gets espera la dirección del primer elemento del array, para obtenerla basta con utilizar "nombre[X]" donde "X" es el indice dentro del rango valido.

Se puede decir mas pero serian suposiciones, mejor publica el código fuente completo para revisarlo.

Un saludo
En línea

The capacity to learn is a gift;
The ability to learn is a skill;
The willingness to learn is a choice.
--
Rebec of Ginaz
DickGumshoe

Desconectado Desconectado

Mensajes: 145


Ver Perfil WWW
Re: Comparando cadenas
« Respuesta #5 en: 22 Enero 2012, 18:01 »

Bueno, he editado mi código fuente con nombres más descriptivos y con comentarios.

Código
/*Un programa que pida 10 nombres y los memorice. Después deberá pedir que se teclee
un nombre y dirá si se encuentra o no entre los 10 que se han tecleado antes. Volverá
a pedir otro nombre y a decir si se encuentra entre ellos, y así sucesivamente hasta que
se teclee “fin”.*/

 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(){
 
   char nombre[10][20], i, aux;
   //aux es el nombre que comprobaremos si esta en la lista o no
 
   //damos un valor a 10 nombres --> ERROR
   do{
   for (i=0;i<=10;i++){
   printf("Introduce un nombre");
   gets(nombre[i][20]);
   }
   }
   while(nombre!=nombre[9][20]);
 
 
   //ERROR, todo lo que comparo sale como que se encuentra entre los anteriores
   do{
   printf("Introduce un nombre para comprobar si se encuentra entre los anteriores");
   gets(aux);
   if((strcmp(nombre[0][20],aux)||
   strcmp(nombre[1][20],aux)||
   strcmp(nombre[2][20],aux)||
   strcmp(nombre[3][20],aux)||
   strcmp(nombre[4][20],aux)||
   strcmp(nombre[5][20],aux)||
   strcmp(nombre[6][20],aux)||
   strcmp(nombre[7][20],aux)||
   strcmp(nombre[8][20],aux)||
   strcmp(nombre[9][20],aux))!=0) printf("El nombre se encuentra entre los anteriores\n");
   else printf("El nombre no se encuentra entre los anteriores\n");
 
   }
   while((strcmp(n11, "fin")!=0));
 
   system("pause");
   return 0;
 
}
 


Por lo que parece, el error está en que en gets no puedo poner
Código
gets(nombre[i][20]);
Aunque no estoy seguro.

Muchísimas gracias.

Saludos.
En línea
satu

Desconectado Desconectado

Mensajes: 263

Siempre aprendiendo


Ver Perfil WWW
Re: Comparando cadenas
« Respuesta #6 en: 22 Enero 2012, 19:05 »

Hola

Citar
Por lo que parece, el error está en que en gets no puedo poner
Código
gets(nombre[i][20]);
 
Aunque no estoy seguro.

No puedes, si te fijas, al poner nombre[ i][20] estás accediendo a un carácter, pero en cada nombre[ i] hay un char[20], por lo tanto debes poner:
Código
gets(nombre[i]);
 

Lo mismo pasa con todos los strcmp(), por eso te falla. Debes poner:
Código
strcmp(nombre[0], aux) ||
strcmp(nombre[1], aux) ||
strcmp(nombre[2], aux) ||
etc...
 

Además no es recomendable el uso de gets(), utiliza mejor fgets()

Saludos
En línea

Breakbeat como forma de vida
BlackZeroX (Astaroth)
Wiki

Desconectado Desconectado

Mensajes: 2.831


I'Love...!¡.


Ver Perfil WWW
Re: Comparando cadenas
« Respuesta #7 en: 22 Enero 2012, 19:34 »

Busca un libro o revisa si el que estas leyendo mas adelante ya usa nombres de variables explicitas o que venga codigo o cuesiones de buena practica.

Código
 
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
 
int main()
{
   char    szNombres[10][20],  //  Array de PAlabras (cada una de 20 letras)
           i,                  //  Indice Contador.
           szLine[20];         //  PAlabra q buscar en el array...
 
   //damos un valor a 10 nombres --> ERROR
   for (i = 0; i < 10; i++)
   {
       printf("Introduce un nombre para el elemento: %d\n", i);
       fgets(szNombres[i], 20, stdin);
   }
 
   while(1)
   {
       printf("Introduce un nombre para comprobar si se encuentra entre los anteriores\n");
       fgets(szLine, 20, stdin);
 
       if (!strcmp(szLine, "fin\n"))   //  si no se le pone \n  y szLine = fin retornara strcmp 1...
           break;  //  Exit do...
 
       for (i = 0; i < 10; ++i)
       {
           if (!strcmp(szNombres[i], szLine))
           {
               printf("Cadena encontrada en el indice: %d\n", i);
               break;  //  Exit For...
           }
       }
   }
   printf("Programa terminado\n");
   getchar();
 
   return EXIT_SUCCESS;
}
 
 

Dulces Lunas!¡.
En línea

Web Principal-->[ Blog(VB6) | Host File (Public & Private) | Scan Port | (New)MyInfraPC (Descubre mi Contraseña venefi. $) ]



The Dark Shadow is my passion.
El infierno es mi Hogar, mi novia es Lilith y el metal mi
DickGumshoe

Desconectado Desconectado

Mensajes: 145


Ver Perfil WWW
Re: Comparando cadenas
« Respuesta #8 en: 23 Enero 2012, 22:40 »

Incluso quitando el [20] del final, me da errores.

Ahora tengo este código:

Código
int main(){
 
   char nombre[10][20], i, aux;
   //aux es el nombre que comprobaremos si esta en la lista o no
 
   //damos un valor a 10 nombres --> ERROR
 
   for (i=0;i<=10;i++){
   printf("Introduce un nombre");
   gets(nombre[i]);
   }
 
 
 
 
   //ERROR, todo lo que comparo sale como que se encuentra entre los anteriores
   do{
   printf("Introduce un nombre para comprobar si se encuentra entre los anteriores");
   gets(aux);
   if((strcmp(nombre[0],aux)||
   strcmp(nombre[1],aux)||
   strcmp(nombre[2],aux)||
   strcmp(nombre[3],aux)||
   strcmp(nombre[4],aux)||
   strcmp(nombre[5],aux)||
   strcmp(nombre[6],aux)||
   strcmp(nombre[7],aux)||
   strcmp(nombre[8],aux)||
   strcmp(nombre[9],aux))!=0) printf("El nombre se encuentra entre los anteriores\n");
   else printf("El nombre no se encuentra entre los anteriores\n");
 
   }
   while((strcmp(n11, "fin")!=0));
 
   system("pause");
   return 0;
 
}
 

Los errores que me da el compilador están todos en el do-while.

Son:

Código:
27 invalid conversion from `char' to `char*'

27  initializing argument 1 of `char* gets(char*)'

28 invalid conversion from `char' to `const char*'

28 initializing argument 2 of `int strcmp(const char*, const char*)'

Hay más, pero son exactamente iguales (para cada comparación me da esos errores que acabo de poner).

Muchas gracias.

Saludos.

En línea
RHL


Desconectado Desconectado

Mensajes: 968


mental


Ver Perfil
Re: Comparando cadenas
« Respuesta #9 en: 23 Enero 2012, 23:45 »

yo dejo una, devuelve true si es igual de lo contrario false:

Código
bool AFUNC(void* op1,void* op2)
{
__asm
{
push edx
push ecx
push ebx
push edi
push esi
mov edi,op1
mov esi,op2
mov eax,0x1
l: mov dl,byte ptr[edi]
mov bl,byte ptr[esi]
cmp dl,0
jne c
cmp bl,0
je f
c: cmp dl,bl
jne n
inc esi
inc edi
jmp l
n: mov eax,0x0
f: pop esi
pop edi
pop ebx
pop ecx
pop edx
}
}
En línea
armizh

Desconectado Desconectado

Mensajes: 182

His brain caught in a hole...


Ver Perfil WWW
Re: Comparando cadenas
« Respuesta #10 en: 24 Enero 2012, 01:09 »

Incluso quitando el [20] del final, me da errores.

Ahora tengo este código:

Código
char nombre[10][20], i, aux;
system("pause");
Primero que todo, lee http://foro.elhacker.net/programacion_cc/lo_que_no_hay_que_hacer_en_cc_nivel_basico-t277729.0.html. Te explica porque no utilizar ni gets() ni system("pause");
Por otro lado, i es un int por lo que debes declararlo como tal ademas de que aux lo estas declarando como "char" y no como "*char".
Por lo tanto debes declarar.
Código
char nombre[10][20], *aux;
int i;
Esos son los errores que yo he visto, no lo he analizado profundamente. Revisa el link que escribi antes, es de aqui mismo en el foro en los mensajes fijos.
Saludos
En línea

Visita mi blog:
http://armizh.10h.us/
Fnx, fork de FreeNOS:
Código
git clone https://github.com/armizh/Fnx
Hamster, Entorno Integrado de Desarrollo C/C++:
Código
git clone https://github.com/armizh/Hamster
satu

Desconectado Desconectado

Mensajes: 263

Siempre aprendiendo


Ver Perfil WWW
Re: Comparando cadenas
« Respuesta #11 en: 24 Enero 2012, 10:59 »

Hola

Tienes 2 fallos:
Código
char nombre[10][20], i, aux;
 
for (i=0;i<=10;i++)
 

1- Tienes declarado aux como char, debes declararlo como: char aux[20];
2- En el for donde introduces los nombres te sobra el = en i<=10, debe ser i<10

Saludos
En línea

Breakbeat como forma de vida
armizh

Desconectado Desconectado

Mensajes: 182

His brain caught in a hole...


Ver Perfil WWW
Re: Comparando cadenas
« Respuesta #12 en: 24 Enero 2012, 22:04 »

Hola

Tienes 2 fallos:
Código
char nombre[10][20], i, aux;
 
for (i=0;i<=10;i++)
 

1- Tienes declarado aux como char, debes declararlo como: char aux[20];
2- En el for donde introduces los nombres te sobra el = en i<=10, debe ser i<10

Saludos
Como yo dije anteriormente, el aux debe ser una matriz o un puntero, queda a gusto de el, y el = en i<=10 no sobra, ya que i puede ser igual o menor a 10. Ademas debe corregir lo del system("pause") que es bastante desaconsejable.

Saludos
En línea

Visita mi blog:
http://armizh.10h.us/
Fnx, fork de FreeNOS:
Código
git clone https://github.com/armizh/Fnx
Hamster, Entorno Integrado de Desarrollo C/C++:
Código
git clone https://github.com/armizh/Hamster
Eternal Idol
Moderador
***
Desconectado Desconectado

Mensajes: 4.392


Clarin miente, TN opera y los giles regurgitan.


Ver Perfil WWW
Re: Comparando cadenas
« Respuesta #13 en: 24 Enero 2012, 22:29 »

Como yo dije anteriormente, el aux debe ser una matriz o un puntero, queda a gusto de el, y el = en i<=10 no sobra, ya que i puede ser igual o menor a 10. Ademas debe corregir lo del system("pause") que es bastante desaconsejable.

Saludos

Si reemplazara char por un puntero a char asi alegremente sin asignarle ningun valor cometeria un error ya que se lo esta pasando a gets (mejor todavia usar fgets con stdin). Tambien tiene razon satu y ya lo habia corregido BlackZeroX (Astaroth) el otro punto, el ultimo elemento de un array es su tamaño menos uno.
« Última modificación: 24 Enero 2012, 22:32 por Eternal Idol » En línea



La economía nunca ha sido libre: o la controla el Estado en beneficio del Pueblo o lo hacen los grandes consorcios en perjuicio de éste.
Juan Domingo Perón
DickGumshoe

Desconectado Desconectado

Mensajes: 145


Ver Perfil WWW
Re: Comparando cadenas
« Respuesta #14 en: 24 Enero 2012, 22:36 »

Muchas gracias a todos.

Ahora me compila, pero al introducir los 10 nombres, dice: 10nombre.exe dejó de funcionar. Windows está buscando una solución al problema...

Gracias.

Saludos.
En línea
Páginas: [1] 2 Ir Arriba Respuesta Imprimir 

Ir a:  

Mensajes similares
Asunto Iniciado por Respuestas Vistas Último mensaje
Comparando Cortafuegos (firewalls)
Seguridad
Cuantico 7 9,640 Último mensaje 17 Septiembre 2004, 23:32
por Th3_Cr0w
Leer Antes de postear, Comparando Codigos:d
Análisis y Diseño de Malware
Ann 2.0 1 699 Último mensaje 22 Septiembre 2007, 14:03
por Huemulito
Blind Search, comparando motores de búsqueda
Noticias
wolfbcn 0 181 Último mensaje 8 Junio 2009, 23:02
por wolfbcn
Comparando resultados[c]
Programación C/C++
Mini_Nauta 2 1,538 Último mensaje 10 Diciembre 2010, 02:55
por Mini_Nauta
[Solucionado] Comparando 2 arrays
PHP
aaronduran2 2 451 Último mensaje 11 Febrero 2012, 08:09
por aaronduran2
Powered by SMF 1.1.16 | SMF © 2006-2008, Simple Machines