[SOLUCIONADO] Como podría Refactorizar para que sea mas legible

[Jose789]

Como podría refactorizar para que sea mas legible

Código Java:

package fecha;
 
public class Fecha {
 
    private int dia;
    private int mes;
    private int anio;
 
    public Fecha(int dia, int mes, int anio) {
 
        this.dia = dia;
        this.mes = mes;
        this.anio = anio;
 
    }
 
    public boolean valida() {
 
        if (dia < 1 || dia > 31)
 
            return false;
 
        if (mes < 1 || mes > 12)
 
            return false;
 
        // determinamos la cantidad de días del mes:
 
        int diasMes = 0;
 
        switch (mes) {
 
        case 1:
 
        case 3:
 
        case 5:
 
        case 7:
 
        case 8:
 
        case 10:
 
        case 12:
            diasMes = 31;
            break;
 
        case 4:
 
        case 6:
 
        case 9:
 
        case 11:
            diasMes = 30;
            break;
 
        case 2: // verificación de año bisiesto
 
            if ((anio % 400 == 0) ||
 
                    ((anio % 4 == 0) && (anio % 100 != 0)))
 
                diasMes = 29;
 
            else
                diasMes = 28;
 
            break;
 
        }
 
        if (dia > diasMes)
 
            return false;
 
        else
            return true;
 
    }
 
    
 
}
 

[CalgaryCorpus]

En vez de tener un gran switch con las distintas posibilidades, define un arreglo de enteros con esas posibilidades

int[] diasPorMes = { 31, 28 ,31, 30 ... };

luego usas el arreglo para obtener los dias de acuerdo al mes

// suponiendo que los meses van de 1 a 12, no de 0 a 11.
dias = diasPorMes[ mes - 1 ];

Eso es solo si quieres hacerlo a mano tu mismo, pues existen clases que permiten crear y usar fechas que podrian hacer el chequeo por ti.

[Jose789]

Iniciado por CalgaryCorpus

En vez de tener un gran switch con las distintas posibilidades, define un arreglo de enteros con esas posibilidades

int[] diasPorMes = { 31, 28 ,31, 30 ... };

luego usas el arreglo para obtener los dias de acuerdo al mes

// suponiendo que los meses van de 1 a 12, no de 0 a 11.
dias = diasPorMes[ mes - 1 ];

Eso es solo si quieres hacerlo a mano tu mismo, pues existen clases que permiten crear y usar fechas que podrian hacer el chequeo por ti.

Pero al definirlo de esa manera al ser un año bisiesto le estoy diciendo que febrero siempre sera tendrá 28 dias no=?

[CalgaryCorpus]

Si, pero nada impide que corrijas despues.

Puedes poner la logica de los anos bisiestos justo despues.

Código:

dias = diasPorMes[ mes - 1 ];
if( esFebrero(mes) && esBisiesto(ano) ) {
   dias++;  // o dias = 29;
}