Eso es, exacto, pero realmente me parece muy recargado meterle tanta cosa para un programa así, me parece que tal y como esta esta bien.

es como decir en el caso de enero, que tiene 31 días:

while(fecha.mes == 1 && fecha.dia > 0 && fecha.dia < 32)
{
    //instrucciones de ejecucion
    //validar registro
    //comparar fechas
    //determinar si es bisiesto
 
   if(fecha.dia < 0 || fecha.dia > 31)
   {
            printf("\nError en el registro de fecha, formato incorrecto, intentalo otra vez besta!!\n");
   }
}
 

Ahora, el problema es que practicamente hay que hacer un while para todos los meses, y luego realizar uno para determinar si es bisiesto en el mismo ciclo de febrero, y pues se hace muy tedioso. Claro que se puede resumir a meses que tengan una cantidad de días iguales así como ya has hecho antes, y seguro que si sacas una, sacas todas las apreciaciones para determinar la mecanica del proceso.

Aunque la verdad, yo lo dejaria hasta ahi, esta muy bien hecho...

Pues lo que pasa es que debes usar la variable, no el tipo de variable:

while(f1.mes == 1 && f1.dia > 0 && f1.dia < 32)

Pero de todas maneras, tu ya tienes esos datos de comprobación listos, no entiendo realmente cuando mencionas lo del formato, porque el formato de una fecha es el siguiente:


Entiendo tu punto de vista, pero realmente lo que necesitas es simplemente indicar si es un bisiesto o no, porque el formato de fechas es correcto, le pones un número menor a 2 digitos para el día y el mes, y un año correcto dentro del estandar de las fechas, ahora lo que debe hacerse es indicarle al usuario si la fecha es correcta o no. Lo unico que debes hacer, es que si quieres restringir en el caso de las fechas:


Es que en el exclusivo caso de que se comparen años biciestos y uno no sea correcto, no por el día, no por el mes, sino porque el año no corresponde a un año biciesto es realizar la restriccion de que si se comparan fechas con meses de febrero, en el cuaĺ, si no es una fecha correcta, pues no se hace el proceso de comparar las fechas, sino simplemente arrojar el resultado que indique "no es biciesto" pero el formato de fechas si corresponde con el que se usa para una fecha, si el usuario indica una fecha en el cual no es un año biciesto, pues el mismo programa le indica, "no es un año biciesto".

Creo que la falla no esta en la programación del programa, de hecho, ya todo esta ahi, solo es reacomodarlo y restructurarlo, no se necesita más ciclos ni condiciones, ya esta todo ahi en las funciones de fecha.c

#include <stdio.h>
#include "fecha.h"
 
struct fecha leeFecha(void)
{
    struct fecha Fech;
    printf("Introduzca la fecha en formato dd/mm/aaaa: ");
    scanf("%d/%d/%d", &Fech.dia, &Fech.mes, &Fech.anyo);
    return Fech;
}
 
 
int compruebaFecha(struct fecha *pFecha)
{
    int compru;
    int bis;
    {
        if (pFecha->dia < 1 || pFecha->dia > 31) // por defecto los meses tienen un promedio de 31
        compru = 0;
 
        else if (pFecha->mes < 1 || pFecha->mes > 12) // por defecto cada año tiene 12 meses
            compru = 0;
 
        // Caso particular para meses:
        // Abril
        // Junio
        // Septiembre
        // Noviembre
        // Que solo poseen 30 dias
        else if ((pFecha->mes == 4 || pFecha->mes == 6 || pFecha->mes == 9 || pFecha->mes == 11) && (pFecha->dia > 30))
            compru = 0;
 
        // Caso Particular Febrero
        // Por defecto solo tiene 28 dìas
        // Excepto en el año biciesto
        else if (pFecha->mes == 2)
        {
            if ((bis = 1) && (pFecha->dia > 29))
                compru = 0;
 
            else if ((bis = 0) && (pFecha->dia > 28))
                compru = 0;
        }
 
        else
            compru = 1;
    }
    return compru;
}
 
 
int comparaFecha(struct fecha f1, struct fecha f2)
    /* Función comparaFecha, devuelve:
       -1 si fecha1 < fecha2
       0 si fecha1 = fecha2
       1 si fecha1 > fecha2
    */
{
    int compara;
 
    if (f1.anyo < f2.anyo)
        compara = -1;
    else if (f1.anyo > f2.anyo)
        compara = 1;
    else
    {
        if (f1.mes < f2.mes)
            compara = -1;
 
        else if (f1.mes > f2.mes)
            compara = 1;
 
        else
        {
            if (f1.dia < f2.dia)
                compara = -1;
 
            else if (f1.dia > f2.dia)
                compara = 1;
 
            else
                compara = 0;
        }
    }
 
    return compara;
}
 
 
void imprimeFecha(struct fecha f)
{
    printf("%02d/%02d/%04d", f.dia, f.mes, f.anyo);
}
 
 
 
// Funcion para comprobar si un año es bisiesto:
int esBisiesto(struct fecha f)
{
    int bis = 0;
 
    // año bisiesto: {divisible por 4 y no por 100} o {divisible por 400}
    if (((f.anyo % 4 == 0) && (f.anyo % 100 != 0)) || (f.anyo % 400 == 0))
        bis = 1;
 
    return bis;
}
 
 

no ha cambiado nada, solo deje claro los casos de los meses pues ya estan hechos, solo es para que tengas en cuenta de que los formatos de fecha son correcto, lo que se necesita es solo decirle  al usuario si la fecha que escribio es un año biciesto o no, no es un error de programación, es un error de sentido común, el usuario piensa que es año bisiesto, pero el programa realiza el cálculo de los modulos y los mutiplos y le indica que no es así.

Solo necesitas ajustar la perspectiva, porque todo esta bien ahi, creo yo.

¡Buenas!

En la función que comprueba la corrección de una fecha tienes un error muy gordo de lógica en el caso de febrero:

    else if (pFecha->mes == 2)
 
      {
        if ((bis = 1) && (pFecha->dia > 29))
 
          compru = 0;
 
        else if ((bis = 0) && (pFecha->dia > 28))
 
          compru = 0;
      }
 

29/2/1992 (año bisiesto):
- bis = 1: true dia> 29: false --> pasamos al else
- bis = 0: false --> pasamos al else ---> ¡NO HAY ELSE! ---> compru queda indefinido.

Supongo que tu intención no era asignar valores uno o cero a bis, sino asignar el valor de la función que evalúa si un año es o no bisiesto. Por cierto, asigna a compru el valor 1 desde el principio, ya que en tu código estás buscando los casos de error y has visto que al menos en el caso de febrero su valor queda indefinido.

Por cierto, en una función que compruebe cosas sobre datos, nunca imprimas mensajes ni pidas datos. No hagas ninguna de las dos cosas porque no sabes la interfaz que va a tener tu programa, lo mismo puede ser un entrono de ventanas que de consola que ser un código empotrado en algún sistema... si una función comprueba la validez de unos datos, si quieres indicar que algo ha salido mal, devuelve datos numéricos que indiquen el error. Así la función que haya llamado a la función de comprobación podrá saber si algo ha salido mal, y actuar en consecuencia.

¡Saludos!

No lo se, puede que sea una mala costumbre, pero no le veo problema pedirle datos en pantalla para comprobación de funciones, al menos, yo lo hago mientras construyo correctamente las funciones y comprobando que efectivamente si realiza la llamada a funciones de una manera que espero, al final, dejo de lado tales impresiones y realizo el retorno correspondiente, excepto en casos particulares donde es necesaria una que otra impresión, eso depende del uso, no me parece un error así macro, más bien es una costumbre adquirida o un habito que no termina que gustar en general 

Es por ello que existen excepciones no?? me gusta relacionar ese tipo de cosas para C, porque no existen excepciones (así como en una forma explicita como en C++) que se manejen de manera, como decirlo, automatica por parte del compilador o del IDE.

aunque el detalle que mencionas tienes razón, es mejor iniciar la variable para el control.