Ya estoy inscrito ¿Todavía no tienes acceso? Nuestros Planes
Ya estoy inscrito ¿Todavía no tienes acceso? Nuestros Planes
1
respuesta

[Sugerencia] El retorno del método 'elegirMedicoAleatorioDisponibleEnLaFecha' debería ser Optional<Medico>

Imaginemos este caso hipotético:

  1. Agendo una consulta de ortopedia este sábado a las 10:00 horas.
  2. Es mi primer consulta así que no importa cuál médico me atienda y dejo que se escoja de forma aleatoria (cae en este método).
  3. Hay tres ortopedistas en la clínica VollMed.
  4. Pero como hay demasiadas consultas agendadas ese día ninguno de los 3 ortopedistas me puede atender este sábado a las 10:00 así que el método regresa un Medico = null.
    Lo correcto sería tratar ese problema con el uso de Optional<>
        Optional<Medico> medicoAleatorio = medicoRepository.elegirMedicoAleatorioDisponibleEnLaFecha(datos.especialidad(), datos.fecha());
        if (medicoAleatorio.isPresent()) {
            return medicoAleatorio.get();
        } else {
            throw new ValidacionException("Lo siento, ningún especialista te puede atender en este día ni horario, por favor selecciona otro día y horario");
        }

Pero me gustaría saber... ¿Qué opinan?, compañeras y compañeros

1 respuesta

Me parece adecuada tu sugerencia Yahadad, tuve la misma impresión durante el curso.

Saludos.