Problema con un ejercicio de comparar cadenas

[Lunadan]

Hola,
tengo un problema con un ejercicio de programación.
Me piden que introduzca un nombre de atleta, apellidos, fecha nacimiento.. entre otros. Y luego tengo que poder modificar la información, pidiendo un nombre y apellido.

Me falla la parte de comparar una cadena del nombre introducido con el nombre del atleta. Lo he hecho con el STRCMP.
Os pongo la parte de modificar, con negrita la parte que me da problemas...




struct atleta {
char nom[atl][max];
char cog1[atl][max];
char cog2[atl][max];
char sexe[atl];
int codi_cmp[atl][comp]; // el codi de competició
} a;

typedef struct {
int dia[atl];
int mes[atl];
int any[atl];
} data;


char nom[max], cognom[max];




printf ("\nModificar datos de un atleta dado su nombre\n");
printf ("************************************\n\n");

printf ("Nombre del atleta que quieres modificar: ");
scanf ("%s", nom);

i=0;
while (i< atl){
if (strcmp (nom, a.nom[i])==0){
error= 1;
}else{
error=0;
}
i++;
}

if (error == 0){
printf ("Este nombre no exististe");
}else{
printf ("Apellido del atleta que quieres modificar: ");
scanf ("%s", cognom);
if (strcmp (cognom, a.cog1[i])==0){
error= 1;
}else{
error= 0;
}

if (error == 0){
printf ("Este apellido no existe");
}else{
printf ("Fecha de nacimiento: %d / %d / %d", d.dia[i], d.mes[i], d.any[i]);
printf ("\nModificar: ");
scanf ("%d %d %d", &dia, &mes, &any);

error=1;
while (error==1 && dia != '\n'){
if (dia >0 && dia<=31 && mes>0 && mes<=12 && any >1900 && any <2100) {
d.dia[i]= dia;
d.mes[i]= mes;
d.any[i]= any;
error=0;
}else{
printf ("Incorrecto. Vuelve a introducir la fecha: ");
scanf ("%d %d %d", &dia, &mes, &any);
error=1;
}
}

printf ("Fecha de naciemiento: %d / %d / %d\n\n", d.dia[i], d.mes[i], d.any[i]);

if (a.sexe[i] == 'h'){
printf ("Sexo: Hombre\n");
}else{
printf ("Sexo: Mujer\n");
}
printf ("Modificar sexo (h/d): "); // h: hombre, d: mujer
scanf ("%c", &hd);

error=1;
while (error==1 && hd !='\n'){
if (hd == 'h' || hd == 'd'){
a.sexe[i]=hd;
error=0;
}else{
printf ("Incorrecto, vuelve a introducir el sexo: ");
scanf ("%c", &hd);
error=1;
}
}
if (a.sexe[i] == 'h'){
printf ("Sexo: Hombe\n");
}else{
printf ("Sexo: Mujer\n");
}
}

// aun faltan cosas!!
}


break;

[mhax]

que error te sale?
// incluiste <string.h> ?

[Lunadan]

si que incluí el <string.h>
cuando le introduzco un nombre, en teoria tiene que buscar si este nombre está o no en la tupla de cadenas (a.nom[i]).
Pero cuando le introduzco el nombre, dice siempre que no encuentra la palabra. Aunque le cambie el numero del error a 1, me sale lo mismo.

No se si he puesto bien el STRCMP. Porque antes lo he intentado hacerlo sin las tuplas, comparando cadena (nom) con cadena de max (nom[i]), y me ha salido, pero al poner el nom[i] dentro de una tupla... no se como hacerlo, porque no me funciona.

[mhax]

Ya se cual es el error.
debes colocar algo que interrumpa el ciclo cuando encuentre el nombre, por ejemplo un break;
de lo contrario seguira comparando.

Saludos

[mhax]

algo asi:

Código C:

while (i< atl){
if (strcmp (nom, a.nom[i])==0){
error= 1;
break;
}else{
error=0;
}
i++;
}
 

[Lunadan]

eres un genio!! gracias XD

[Lunadan]

El siguiente problema de comparar cadenas es que quiero saber si el nombre, el apellido1, y el apellido2 estan o no introducidos, para saber si aquel atleta ya estaba inscrito anteriormente o no. Si esta inscrito, en teoria debe avisar (printf ("El nombre ya esta escrito\n");) y volver a pedir el nombre, el apellido1, y el apellido2.
No me funciona.

Código C:

error=1;
        while (error==1){
        printf ("Nombre: ");
        fflush(stdin);
        scanf ("%[^\n]", a.nom[i]);
        printf ("Primer apellido: ");
        fflush(stdin);
        scanf ("%[^\n]", a.cog1[i]);
        printf ("Segundo apellido: ");
        fflush(stdin);
        scanf ("%[^\n]", a.cog2[i]);
 
        for (k=0;k<max;k++){
            if (strcmp (a.nom[i], a.nom[k])==0){
                for (k=0;k<max;k++){
                    if (strcmp (a.cog1[i], a.cog1[k])==0){
                      for (k=0;k<max;k++){
                            if (strcmp (a.cog2[i], a.cog2[k])==0){
                                error= 1;
 
                            }else{
                                error=0;
                            }
                        }
                    }else{
                        error=0;
                    }
                }
            } else{
                error=0;
            }
        }
 
 
        if (error == 1){
            printf ("El nombre ya esta escrito\n");
        }
        }
 

[mhax]

la condicion seria esta:

Código C:

if(strcmp (a.nom[i], a.nom[k]) != 0)
   printf ("El nombre ya esta escrito\n");
 

Saludos

[Lunadan]

No me funciona.
O me sale todo el rato "El nombre ya esta escrito" o simplemente no me sale nada, aunque el nombre esté repetido.

[mhax]

Pudes hacer una funcion que compare:

Código C:

bool comparar(char a[]){
  for(int i=0;i<max;i++){
    if(!strcmp(a,a.nom[i]))
         return true;
  }
 return false;
}
 

En el main puedes usar esa funcion y comparar de una el valor que ingresas

[mhax]

Código C:

do{
  cout<<"Ingrese Nombre: ";
  cin>>b;
    if(!comparar(b))
        break;
}while(comparar(nom)); //Mientras ya exista el nombre
 

Espero haberte ayudado