LGhosn / TakeAwayNow-DCNT

1 stars 0 forks source link

dudoso este codigo dentro de una entidad #9

Open maurociancio opened 8 months ago

maurociancio commented 8 months ago
public boolean estaAbierto(LocalDateTime FechaYHora) {
    // Obtener la zona horaria de Buenos Aires, y posteriormente la utilizamos para comparar los horarios.
    ZoneId zonaHorariaArgentina = ZoneId.of("America/Argentina/Buenos_Aires");
    ZonedDateTime fechaYHoraArgentina = FechaYHora.atZone(zonaHorariaArgentina);

    // Obtenemos el horario y el día en Argentina.
    LocalTime horarioIngresado = fechaYHoraArgentina.toLocalTime();
    DayOfWeek diaIngresado = fechaYHoraArgentina.getDayOfWeek();

    // Resto del código sigue igual
    return (
            ((diaIngresado.compareTo(diaDeApertura) >= 0) && (diaIngresado.compareTo(diaDeCierre) <= 0))
                    && (horarioIngresado.isAfter(horarioDeApertura) || horarioIngresado.equals(horarioDeApertura))
                    && (horarioIngresado.isBefore(horarioDeCierre) || horarioIngresado.equals(horarioDeCierre))
    );
}

xq estas jugando con zonas horarias acá?

ruizsugliani commented 8 months ago

Porque en el front, cuando estábamos probando el tema de los negocios cerrados y abiertos vimos que no respetaban el horario y al buscar un poco la razón vimos que teníamos que que especificar la zona horaria.

ruizsugliani commented 8 months ago

Por otro lado, dónde debería estar ? No tenemos el material de la materia como para consultarlo.

maurociancio commented 8 months ago

En la clase hablamos varias veces del java.time. No usen zona horaria y ya.

On Wed, Feb 21, 2024 at 12:52 AM Santiago Nahuel Ruiz Sugliani < @.***> wrote:

Por otro lado, dónde debería estar ? No tenemos el material de la materia como para consultarlo.

— Reply to this email directly, view it on GitHub https://github.com/LGhosn/TakeAwayNow-DCNT/issues/9#issuecomment-1955840019, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABGBDJA5EOW2FAKQ3ERKCDYUVVRBAVCNFSM6AAAAABDSGJSCSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJVHA2DAMBRHE . You are receiving this because you authored the thread.Message ID: @.***>

-- Mauro Ciancio http://about.me/maurociancio