Guarda nombres y comprueba si estan entre los almacenados

[Dempo87]

Hola, estoy realizando un ejercicio que dice:

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.

He realizado decenas de pruebas, pero no doy con el fallo que explico acontinuacion. El programa compila pero si escribo palabras que ya se guardaron en el array, no me dice las muestra como una almacenada, si no que siempre me indica que esa palabra no se encuentra entre las 10 guardadas.

El ejercicio es esto:

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(int argc, char *argv[])
{
    char nombre[][50]={};
    char comparacion[50];
    char num[10][20] = {"primer", "segundo", "tercer", "cuarto", "quinto", "sexto", "septimo", "octavo", "noveno", "decimo"};
    int i;
    
    for (i = 0; i < 10; i++){
        printf ("\n Introduce el %s nombre: ", num[i]);
        scanf ("%s", &nombre);
        }
    do{    
        printf ("\n Introduce un nombre para comprobar si se encuentra entre los almacenados: ");
        scanf ("%s", &comparacion);
        
        if (strcmp(nombre, comparacion) == 0){
           printf ("\n El nombre esta entre los almacenados\n\n\n\n");
           break;}
           else{
                printf ("\n El nombre NO esta entre los almacenados\n\n\n\n");
                }
                }while (strcmp(nombre, comparacion) != 0);
  
  system("PAUSE");  
  return 0;
}
 

[CalgaryCorpus]

Posiblemente la instruccion scanf() que utilizas está incluyendo el \n (cambio de linea) que se ingresa al presionar Enter.

Para comprobarlo, escribir el largo (strlen()) de lo ingresado, o escribir algo como "|%s|", nombre, y si al ingresar A aparece

|A
|

y no aparece

|A|

ese es el problema, y strcmp() tiene la obligacion de decirte que los strings no calzan.

Si esto es el caso, hay que eliminar este \n antes de comparar.

[Dempo87]

gracias por la contestacion amigo. No se si lo entendi bien, pero cambie los "scanf" por "gets", aunque el problema persiste:

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(int argc, char *argv[])
{
    char nombre[][50]={};
    char comparacion[50];
    char num[10][20] = {"primer", "segundo", "tercer", "cuarto", "quinto", "sexto", "septimo", "octavo", "noveno", "decimo"};
    int i;
    
    for (i = 0; i < 10; i++){
        printf ("\n Introduce el %s nombre: ", num[i]);
        gets (nombre);
        //scanf ("%s", &nombre);
        }
    do{    
        printf ("\n Introduce un nombre para comprobar si se encuentra entre los almacenados: ");
        gets (comparacion);
        //scanf ("%s", &comparacion);
        
        if (strcmp(nombre, comparacion) == 0)
           printf ("\n El nombre esta entre los almacenados\n\n\n\n");
           
           else{
                printf ("\n El nombre NO esta entre los almacenados\n\n\n\n");
                }
                }while (strcmp(nombre, comparacion) != 0);
  
  system("PAUSE");  
  return 0;
}
 

[codigo_cpp]

Cuando almacenes un contenido en una variable, utiliza los indices, que en este caso es la letra "i".

Por ejemplo: gets(nombre[i]);

Declara nombre asi: char nombre[10][50]; si no me equivoco esto seria parecido a la declaración que hiciste para num, o sea 10 variables con 50 espacios para cada una.

Para realizar una comparación, también vuelve a utilizar los indices dentro de un ciclo, por que lo que estas comparando con este código: if (strcmp(nombre, comparacion) == 0) es solo el primer valor que almacenaste.

Cualquier duda o pregunta, aquí estamos para ayudarte.

Saludos.

[CalgaryCorpus]

Comparto la opinion previa.

Esta declaracion:

Código C:

char nombre[][50]={};
 

deberia cambiarse por

Código C:

char nombre[10][50];
 

y luego utilizar el indice para llenar cada una de las posiciones del arreglo nombre, como te lo indica sabiamente codigo_cpp.

Otra observacion: dentro del ciclo do {} while, por cada vuelta preguntas por el string "comparacion", Creo que hay que hacerlo justo antes de comenzar el ciclo.

En el ciclo, conviene utilizar nuevamente un indice, que parta en 0 y llegue a alguno de estos casos:

- el indice es mayor o igual a 10 o
- encontraste el string.

Algo parecido a ...

Código PseudoCode:

<leer "comparacion">
i = 0;
do {
   <comparar numero[i] con "comparacion">
   si calzan -> salir
   si no -> incrementar i
} while (i <10);
 

[Dempo87]

Bueno pues despues de casi 2 semanas sin pc, ya estoy de vuelta. Estoy de examenes y no puedo ponerme con estos ejercicios extras que me puse, pero cuanto acabe, vuelvo a este tema pues recuerdo que me seguia fallando.

Gracias gente!