Autor
|
Tema: Comparando cadenas (Leído 11,259 veces)
|
armizh
Desconectado
Mensajes: 187
His brain caught in a hole...
|
Incluso quitando el [20] del final, me da errores. Ahora tengo este código: char nombre[10][20], i, aux;
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. 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: git clone https://github.com/armizh/Fnx
Hamster, Entorno Integrado de Desarrollo C/C++: git clone https://github.com/armizh/Hamster
|
|
|
satu
Desconectado
Mensajes: 301
Siempre aprendiendo
|
Hola Tienes 2 fallos: 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
Mensajes: 187
His brain caught in a hole...
|
Hola Tienes 2 fallos: 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: git clone https://github.com/armizh/Fnx
Hamster, Entorno Integrado de Desarrollo C/C++: git clone https://github.com/armizh/Hamster
|
|
|
Eternal Idol
Kernel coder
Moderador
Desconectado
Mensajes: 5.966
Israel nunca torturó niños, ni lo volverá a hacer.
|
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 pm 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
|
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
|
|
|
|
Eternal Idol
Kernel coder
Moderador
Desconectado
Mensajes: 5.966
Israel nunca torturó niños, ni lo volverá a hacer.
|
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.
Vaya uno a saber que frankestein de codigo tendras ahora, ponelo pero antes relee el hilo completo y asegurate de que estas leyendo SOLO 10 elementos en el array (ese es su tamaño, no podes leer mas sino pasa lo que viste).
|
|
|
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
|
El código es: #include <stdio.h> #include <stdlib.h> #include <string.h> int main(){ char nombre[10][20], aux[20]; int i; //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"); } //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"); 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(aux , "fin")!=0)); return 0; }
Y ya no tengo ese problema. Ahora siempre dice que el nombre se ha encontrado entre los 10 primeros... Gracias
|
|
|
En línea
|
|
|
|
Eternal Idol
Kernel coder
Moderador
Desconectado
Mensajes: 5.966
Israel nunca torturó niños, ni lo volverá a hacer.
|
strcmp devuelve 0 cuando las cadenas comparadas son iguales.
¿Viste el codigo que te dejo BlackZeroX (Astaroth)? ¿No te parece un poco mas elegante la comparacion con un bucle?
|
|
|
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
|
Ahora tengo esto: /*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], aux[20]; int i; //aux es el nombre que comprobaremos si esta en la lista o no //damos un valor a 10 nombres for (i=0;i<10;i++){ printf("Introduce un nombre"); } //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"); for (i=0;i<10;i++){ if((strcmp(nombre [i ],aux )==0)){ printf("El nombre se encuentra entre los anteriores\n"); break; } else printf("El nombre no se encuentra entre los anteriores\n"); break; } } while((strcmp(aux , "fin")!=0)); return 0; }
Lo que he editado ha sido el bucle for del final, y ahora, introduzco diez nombres. Cuando pongo el que quiero comprobar: - Si es el que he metido primero (el que equivale a nombre[0]), sale como que lo he introducido --> Correcto - Todos los demás salen como que no los había introducido... Gracias. Saludos.
|
|
|
En línea
|
|
|
|
Eternal Idol
Kernel coder
Moderador
Desconectado
Mensajes: 5.966
Israel nunca torturó niños, ni lo volverá a hacer.
|
Lee el codigo con atencion, publicar 200 mensajes con un solo problema no implica que te lo vayamos a solucionar nosotros. El bucle ese no tiene sentido, si solo funciona con el primer elemento es por algo. Tal vez si usas una identacion como la gente lo veas rapidamente.
|
|
|
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
|
|
|
|
|