Foro de elhacker.net

Programación => Programación C/C++ => Mensaje iniciado por: Jlrr en 3 Febrero 2012, 21:29 pm


Título: Imprimir una sola vez el mensaje en C
Publicado por: Jlrr en 3 Febrero 2012, 21:29 pm
Hola estoy haciendo un programa en C para comparar varios nombres y saber si son iguales o no,primero escribo el numero de personas a las que le asignare los nombres luego con un ciclo escribo los nombres para las personas que escribi,despues comparo los nombres en un ciclo con dos opciones,la primera es una funcion que compara las cadenas para saber si son iguales y la segunda es la opcion que se da si la primera no se da,ambas con un mensaje diferente que se imprime al final.El problema que tengo es que cuando compilo el programa funciona pero imprime dos mensajes,uno por cada una de las dos opciones y queria saber si alguien me podria decir como cambiar el programa para que solo lo imprima un  mensaje segun sea la opcion

Código
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. struct Datos
  4. {
  5.  char Nombre[25];
  6. };
  7. int main(int argc, char *argv[])
  8. {
  9.   struct Datos P[100];
  10.   int m,i;
  11.   char nom[25]; 
  12.  
  13.   printf("ingrese el n\xA3mero de personas: ");
  14.   scanf("%d",&m);
  15.   for(i=0;i<m;i++)
  16.   {
  17.    fflush(stdin);
  18.    printf("ingrese el nombre de la persona #%d: ",i+1);
  19.    gets(P[i].Nombre);
  20.   }
  21.   printf("Escriba el nombre de la persona que quiere buscar: "); 
  22.   scanf("%s",&nom);
  23.   for(i=0;i<m;i++)
  24.   {
  25.    if(strcmp(nom,P[i].Nombre)==0)
  26.    {
  27.     printf("%s esta en la base de datos\n",nom);
  28.    }
  29.    else
  30.    {
  31.     printf("%s no esta en la base de datos\n",nom);
  32.    }
  33.   }
  34.   system("PAUSE");	
  35.   return 0;
  36. }

Ejemplo:escribo el numero de personas:2
             --Estos son los nombres que se deberian guardar--
             escribo el primer nombre:marco
             escribo el segundo nombre:mateo
             escribo el nombre que quiero buscar:mateo
             --Al final aparece esto--
             mateo no esta en la base de datos
             mateo esta en la base de datos
           --Lo mismo pasa si coloco  3( o mas nombres) solo que  aparece dos veces   
             un mensaje incorrecto y una vez el mensaje correcto--

Título: Re: Imprimir una sola vez el mensaje en C
Publicado por: DickGumshoe en 3 Febrero 2012, 21:39 pm
Hola.

Prueba poniendo "break" en los if y else.

Así:

Código
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <string.h>
  4. struct Datos
  5. {
  6.  char Nombre[25];
  7. };
  8. int main(int argc, char *argv[])
  9. {
  10.   struct Datos P[100];
  11.   int m,i;
  12.   char nom[25]; 
  13.  
  14.   printf("ingrese el n\xA3mero de personas: ");
  15.   scanf("%d",&m);
  16.   for(i=0;i<m;i++)
  17.   {
  18.    fflush(stdin);
  19.    printf("ingrese el nombre de la persona #%d: ",i+1);
  20.    gets(P[i].Nombre);
  21.   }
  22.   printf("Escriba el nombre de la persona que quiere buscar: "); 
  23.   scanf("%s",&nom);
  24.   for(i=0;i<m;i++)
  25.   {
  26.    if(strcmp(nom,P[i].Nombre)==0)
  27.    {
  28.    printf("%s esta en la base de datos\n",nom);
  29.    break;
  30.    }
  31.    else
  32.    {
  33.     printf("%s no esta en la base de datos\n",nom);
  34.     break;
  35.    }
  36.   }
  37.   system("PAUSE");	
  38.   return 0;
  39. }
  40.  

Saludos.

EDITO: Me parece que en tu código faltaba también la librería string.h

Título: Re: Imprimir una sola vez el mensaje en C
Publicado por: Jlrr en 3 Febrero 2012, 21:58 pm
Hola Dick al colocarle los break al final del if y el else el programa ya solo imprime un mensaje el problema es que al realizar la comparacion de cadenas solo hace la comparacion con el primer nombre es decir que el al comparar 2 o mas nombres para saber si estan o no estan el programa solo hace la comparacion con el primer nombre,el resto los toma como si no existieran automaticamente.

Título: Re: Imprimir una sola vez el mensaje en C
Publicado por: DickGumshoe en 3 Febrero 2012, 22:09 pm
Ah, sí. Debes quitar el else, sustituyéndolo por un if, y sacarlo fuera del for. Así:

Código
  1. for(i=0;i<m;i++)
  2.   {
  3.    if(strcmp(nom,P[i].Nombre)==0)
  4.    {
  5.    printf("%s esta en la base de datos\n",nom);
  6.    break;
  7.    }
  8.  
  9.  }
  10.    if(strcmp(nom,P[i].Nombre)!=0)printf("%s no esta en la base de datos\n",nom);
  11.  

Saludos.

Título: Re: Imprimir una sola vez el mensaje en C
Publicado por: Caster en 3 Febrero 2012, 22:16 pm
No me voy a meter en el tema principal pero, un consejo, quita esto:

Código
  1. system("PAUSE");

Y pon esto:

Código

Título: Re: Imprimir una sola vez el mensaje en C
Publicado por: Jlrr en 3 Febrero 2012, 22:23 pm
Gracias dirk cuando converti el else en if y lo saque del for ya se arregla el problema de la comparacion de cadenas ahora ya compara todos los nombres e imprime el mensaje adecuado si estan o no estan en la base de datos,aunque no entiendo bien porque era necesaria la tercera libreria.

Demon al hacer el cambio el programa se termina antes de que muestre el mensaje final creo que es mejor no cambiarlo.

Título: Re: Imprimir una sola vez el mensaje en C
Publicado por: DickGumshoe en 3 Febrero 2012, 22:25 pm
¿La tercera librería?

Esa contiene strcmp, necesario para hacer la comparación. ¿Te compiló sin usarla?

Título: Re: Imprimir una sola vez el mensaje en C
Publicado por: Jlrr en 3 Febrero 2012, 22:29 pm
Si,no aparecio ningun problema especificamente el programa lo estoy haciendo en C con DEV-C++ 4.9.9.2 sin utilizar
Código
  1. #include <string.h>.

Título: Re: Imprimir una sola vez el mensaje en C
Publicado por: DickGumshoe en 3 Febrero 2012, 22:33 pm
Pues yo utilizo la misma versión de tu compilador (DEV-C++ 4.9.9.2), y sin usar esa librería no me compila bien xD (y, de hecho, en los manuales que tengo viene que es necesario usar dicha librería...)

Título: Re: Imprimir una sola vez el mensaje en C
Publicado por: Jlrr en 3 Febrero 2012, 22:44 pm
Que raro no sabia que se necesitaba esa libreria para usar strcmp en esta versión,aunque el DEV-C++ en si lo consegui de una pagina de mi centro de estudios puede que lo hayan alterado o cambiado algo aunque tambien he visto que esta versión actua diferente segun el sistema operativo cuando lo uso en XP en mas facil que cuando lo uso con windows 7,se traba menos.

Título: Re: Imprimir una sola vez el mensaje en C
Publicado por: rir3760 en 4 Febrero 2012, 00:53 am
La forma mas sencilla de realizar la operación es utilizando al contador como bandera de estado, de esta forma:
Código
  1. for (i = 0; i < m; i++)
  2.    if (strcmp(nom, P[i].Nombre) == 0)
  3.       break;
  4. if (i != m)
  5.    printf("%s esta en la base de datos\n", nom);
  6. else
  7.    printf("%s no esta en la base de datos\n", nom);

----

En cuanto al prototipo de las funciones: se recomienda por "buenos modales" y en ciertos casos es obligatorio (por ejemplo con funciones variadicas).

Cuando no se tiene acceso al prototipo de una función se infiere este en base a la primera llamada asumiendo:

1) La función retorna un valor de tipo "signed int".
2) El numero y tipo de argumentos esta dado exactamente en la llamada.

En el caso de DickGumshoe la única llamada a "strcmp" es:
Código:
strcmp(nom,P[i].Nombre) == 0
Y como esta coincide con su prototipo no hay problema.

Pero si se tratara de una función que retorna algún otro tipo de valor (por ejemplo un "double") o una de las funciones printf/scanf/etc si seria un error.

Un saludo


Powered by SMF 1.1.21 | SMF © 2006-2008, Simple Machines