NisumCL / proyecto_calendario

0 stars 0 forks source link

DT - Encapsular en una unica función #19

Closed KiraElric closed 2 years ago

KiraElric commented 2 years ago
function isValidDate(date) {
    let regex = new RegExp(
        /^([12]\d{3}\/(0[1-9]|1[0-2])\/(0[1-9]|[12]\d|3[01]))$/
    );
    let dateOk = regex.test(date);
    return dateOk;
}
if (isValidDate(fechaInicio) && isValidDate(fechaFin)) {
    console.log("Formato correcto");
} else if (!isValidDate(fechaInicio)) {
    console.log("Formato incorrecto de la primera fecha ingresada");
    return false;
} else if (!isValidDate(fechaFin)) {
    console.log("Formato incorrecto de la segunda fecha ingresada");
    return false;
} else {
    console.log("Formato de ambas fechas es invalido");
    return false;
}

Se podria dejar todo dentro de la función isValidDate, para que esta tenga la unica responsabilidad de validar que el formato de la fecha ingresada sea el correcto, disminuyendo la cantidad de validaciones en el programa principal y encapsulando este dentro de una función.

KiraElric commented 2 years ago
function isValidDate(date) {
  const regex = /^([12]\d{3}\/(0[1-9]|1[0-2])\/(0[1-9]|[12]\d|3[01]))$/;
  const dateOk = regex.test(date);
  if (!dateOk) {
    // eslint-disable-next-line no-console
    console.log('Formato incorrecto', date);
  }
  return dateOk;
}