Comparando cadenas

[armizh]

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

Ahora tengo este código:

Código (c) [Seleccionar] Expandir

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 (c) [Seleccionar] Expandir


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

[satu]

Hola

Tienes 2 fallos:

Código (c) [Seleccionar] Expandir


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

[armizh]

Hola

Tienes 2 fallos:

Código (c) [Seleccionar] Expandir


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

[Eternal Idol]

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.

[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]

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

[DickGumshoe]

El código es:

Código (c) [Seleccionar] Expandir

#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?

[DickGumshoe]

Ahora tengo esto:

Código (c) [Seleccionar] Expandir

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