Comparando cadenas

Iniciado por DickGumshoe, 22 Enero 2012, 01:17 AM

0 Miembros y 2 Visitantes están viendo este tema.

armizh

Cita de: DickGumshoe en 23 Enero 2012, 22:40 PM
Incluso quitando el [20] del final, me da errores.

Ahora tengo este 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.

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
Visita mi blog:
http://armizh.10h.us/
Fnx, fork de FreeNOS:
Código (sh) [Seleccionar]
git clone https://github.com/armizh/Fnx
Hamster, Entorno Integrado de Desarrollo C/C++:
Código (sh) [Seleccionar]
git clone https://github.com/armizh/Hamster

satu

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
Breakbeat como forma de vida

armizh

Cita de: satu en 24 Enero 2012, 10:59 AM
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
Visita mi blog:
http://armizh.10h.us/
Fnx, fork de FreeNOS:
Código (sh) [Seleccionar]
git clone https://github.com/armizh/Fnx
Hamster, Entorno Integrado de Desarrollo C/C++:
Código (sh) [Seleccionar]
git clone https://github.com/armizh/Hamster

Eternal Idol

#13
Cita de: armizh en 24 Enero 2012, 22:04 PM
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.
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.

Eternal Idol

Cita de: DickGumshoe en 24 Enero 2012, 22:36 PM
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).
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");
    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(aux, "fin")!=0));
   
    system("pause");
    return 0;
   
}


Y ya no tengo ese problema. Ahora siempre dice que el nombre se ha encontrado entre los 10 primeros...

Gracias

Eternal Idol

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?
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");
    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);
    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));
   
    system("pause");
    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.

Eternal Idol

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.
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