Problema en bucle for

[DickGumshoe]

Hola.

He estado haciendo este ejercicio que venía propuesto en un manual que estoy leyendo:

Código C:

/*Un programa que pida datos de 8 personas: nombre, dia de nacimiento, mes de
nacimiento, y año de nacimiento (que se deben almacenar en una tabla de structs).
Después deberá repetir lo siguiente: preguntar un número de mes y mostrar en
pantalla los datos de las personas que cumplan los años durante ese mes. Terminará
de repetirse cuando se teclee 0 como número de mes.*/ 
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main()
{
    struct
    {
     char nombre[20];
     int dia;
     int mes;
     int edad;
     int ano;
    }datos[8];
    
    int i,comprobar;
    
    for(i=0;i<=7;i++)
    {
      
      printf("Introduce nombre\n");
      scanf("%s",datos[i].nombre);
      
      printf("Introduce edad\n");
      scanf("%d",&datos[i].edad);
      
      printf("Introduce mes de cumpleanos\n");
      scanf("%d",&datos[i].mes);
      
      printf("Introduce ano de nacimiento\n");
      scanf("%d",&datos[i].ano);
      printf("\n\n");
    }
    
    printf("Introduce el mes en el que quieres comprobar si alguna persona cumple anos\n");
    scanf("%d",&comprobar);
    
    for(i=0;i<=7;i++)
    {
      if(comprobar==datos[i].mes)
      {
        printf("%s cumple %d anos este mes\n",datos[i].nombre,datos[i].edad+1);
      }
      if(comprobar!=datos[i].mes) printf("Nadie cumple anos este mes\n");
    }
    
    
    system("pause");
    return 0;
    
}
 

El problema está en el bucle for del final.

Al ejecutar el programa, siempre sale "Nadie cumple años este mes" varias veces. He intentando poner "break", pero no sé si es correcto, o, en caso afirmativo, dónde colocarlo.

Muchas gracias.

Saludos.

[starfix]

podes crear una variable de tipo bool fuera del for con valor inicializado en true, dentro del for, si se cumple la condicion la cambias a false y para cuando termine de ejecutar el for, analizas si la variable que creste sigue valiendo true, si es asi imprimes que nadie cumple en ese mes

Código C:

bool nadie= true;
 
for(i=0;i<=7;i++)
    {
      if(comprobar==datos[i].mes)
      {
        printf("%s cumple %d anos este mes\n",datos[i].nombre,datos[i].edad+1);
        nadie = false;
      }
    }
 
if (nadie) printf("Nadie cumple anos este mes\n");
 

espero que sea lo que buscar, saludos

[DickGumshoe]

Muchas gracias por responder.

Me parece que su código no funciona bien, ya que no dice cuando "Nadie cumple años ese mes". Por otra parte, me gustaría hacerlo sin los valores bool, ya que aún no los he visto en el manual que estoy leyendo...

Gracias.

Saludos.

[starfix]

si funciona, pero lo podes cambiar como sigue:

Código C++:

int nadie= 1;
 
for(i=0;i<=7;i++)
    {
      if(comprobar==datos[i].mes)
      {
        printf("%s cumple %d anos este mes\n",datos[i].nombre,datos[i].edad+1);
        nadie = 0;
      }
    }
if (nadie == 1) printf("Nadie cumple anos este mes\n");
 

es lo mismo pero te cambie el tipo de variable a int, la idea es que la variable nadie te ayude a saber si alguien cumple años o no, si la variable vale 1 (valor por defecto) nadie cumple años y en caso de que dentro del for encontremos a alguien que cumpla años en el mes que se elige se la cambia a 0 para que la ultima condicion no se cumpla y no se imprima "nadie cumple anos este mes"

[DickGumshoe]

Gracias de nuevo.

A lo que me refiero que no funciona bien, es que, por ejemplo, pones varios datos. Después, cuando te pide el dato de comprobar, pones un mes en el que sepas que no haya nadie que cumpla años, y, posteriormente, otro en el que sepas que sí hay alguien. A partir de ahí, todos los que meses que introduzcas y en ese no haya nadie que cumpla años, no te imprime ningún mensaje...

No sé si he explicado bien el problema...

Gracias.

Saludos.

[starfix]

pone el seguimiento que haces el programa, o sea, los datos que introducis linea por linea.... y si modificaste alguna parte del prgrama tambien porque no se donde esta el error, lo he probado y funciona

[DickGumshoe]

Código C:

/*Un programa que pida datos de 8 personas: nombre, dia de nacimiento, mes de
nacimiento, y año de nacimiento (que se deben almacenar en una tabla de structs).
Después deberá repetir lo siguiente: preguntar un número de mes y mostrar en
pantalla los datos de las personas que cumplan los años durante ese mes. Terminará
de repetirse cuando se teclee 0 como número de mes.*/ 
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main()
{
    int nadie=1;
    struct
    {
     char nombre[20];
     int dia;
     int mes;
     int edad;
     int ano;
    }datos[8];
    
    int i,comprobar;
    
    for(i=0;i<=7;i++)
    {
      
      printf("Introduce nombre\n");
      scanf("%s",datos[i].nombre);
      
      printf("Introduce edad\n");
      scanf("%d",&datos[i].edad);
      
      printf("Introduce mes de cumpleanos\n");
      scanf("%d",&datos[i].mes);
      
      printf("Introduce ano de nacimiento\n");
      scanf("%d",&datos[i].ano);
      printf("\n\n");
    }
    do
    {
    printf("Introduce el mes en el que quieres comprobar si alguna persona cumple anos\n");
    scanf("%d",&comprobar);
    
   for(i=0;i<=7;i++)
    {
      if(comprobar==datos[i].mes)
      {
        printf("%s cumple %d anos este mes\n",datos[i].nombre,datos[i].edad+1);
        nadie = 0;
      }
    }
if (nadie == 1) printf("Nadie cumple anos este mes\n");
    }while(comprobar!=0);
    
    
    system("pause");
    return 0;
    
}
 

Si por ejemplo introduzco de datos:
1 2 2 2
2 2 2 2
3 2 2 2
4 2 2 2
5 2 2 2
6 2 2 2
7 2 2 2
8 2 2 2

Y, después, compruebo los meses 1 -> Sale que nadie cumple ese mes
2 -> Sale todos los que cumplen ese mes
3 -> Sale directamente que compruebe otro mes.

Gracias.

[starfix]

claro, cuando entra en el do..while si encuentra personas que cumplan años en ese mes te los imprime en pantalla, si no encuentra, la variable nadie recibe el valor 0, luego habria que volver a colocarla en 1 para que se pueda comprobar nuevamente, si no siempre valdra 0 en lo que queda del programa

Código C++:

do
 
    {
     nadie= 1;
    ....
    }while(comprobar!=0)
 

si agregas esa linea dentro del do...while, cada vez que vuelva a empezar, la variable nadie vuelve a tomar el valor 1 para que se cumpla la ultima condicion en caso de que nadie cumpla años ese mes

[DickGumshoe]

Ah, vale, muchísimas gracias.

Me ha quedado muy claro ^^

Gracias.

Saludos.