Modificado los servicios y funciones disponibilidad para que devuelva número de…

Modificado los servicios y funciones disponibilidad para que devuelva número de habitaciones disponibles

src/main/java/es/ujaen/dae/reservahoteles/entidades/Hotel.java

@@ -24,6 +24,9 @@ import java.util.List;
  */
 @Entity
 public class Hotel {
+    
+    public static record Disponibilidad(int numHabSimple, int numHabDoble) {}
+    
     @Id
     @GeneratedValue(strategy=GenerationType.IDENTITY)            
     int id;
@@ -102,36 +105,48 @@ public class Hotel {
     }
     
     /**
-     * Comprueba si hay disponibilidad en las fechas indicadas
+     * Devuelve la disponibilidad en las fechas indicadas
      * @param fechaInicio fecha de inicio de la reserva
      * @param fechaFin fecha de final de la reserva
-     * @param numHabSimple número de habitaciones simples solicitadas
-     * @param numHabDoble número de habitaciones dobles solicitadas
-     * @return true si hay disponibilidad, false en caso contrario
+     * @return la disponibilidad en las fechas indicadas
      */
-    public boolean disponible(LocalDate fechaInicio, LocalDate fechaFin, int numHabSimple, int numHabDoble) {
+    public Disponibilidad disponibilidad(LocalDate fechaInicio, LocalDate fechaFin) {
         // Obtener la fecha máxima donde hay registrada una reserva
         final var fechaMax = reservas.stream()
                 .map(r -> r.fechaFin)
                 .max(LocalDate::compareTo)
                 .map(f -> f.isBefore(fechaFin) ? f : fechaFin)
                 .orElse(LocalDate.now());
-                
-        for (var fecha = fechaInicio.plusDays(0); fecha.isBefore(fechaMax); fecha = fecha.plusDays(1))
-            if (!disponible(fecha, numHabSimple, numHabDoble))
-                return false;
-  
-        return true;
+        
+        int numHabSimpleDisponibles = numHabSimple;
+        int numHabDobleDisponibles = numHabDoble;
+        for (var fecha = fechaInicio.plusDays(0); fecha.isBefore(fechaMax); fecha = fecha.plusDays(1)) {
+            var disponibilidadDia = disponibilidad(fecha);
+            numHabSimpleDisponibles = Integer.min(numHabSimpleDisponibles, disponibilidadDia.numHabSimple);
+            numHabDobleDisponibles = Integer.min(numHabDobleDisponibles, disponibilidadDia.numHabDoble);
+        }
+        return new Disponibilidad(numHabSimpleDisponibles, numHabDobleDisponibles);
     }
     
-    /**
-     * Comprueba si hay disponibilidad en un día concreto
-     * @param fecha día a comprobar
+   /**
+     * Indica si el hotel está disponible en las fechas indicadas
+     * @param fechaInicio fecha de inicio de la reserva
+     * @param fechaFin fecha de final de la reserva
      * @param numHabSimple número de habitaciones simples solicitadas
      * @param numHabDoble número de habitaciones dobles solicitadas
-     * @return true si hay disponibilidad, false en caso contrario
+     * @return true si el hotel está disponible
+     */
+    public boolean disponible(LocalDate fechaInicio, LocalDate fechaFin, int numHabSimple, int numHabDoble) {
+        var disponibilidad = disponibilidad(fechaInicio, fechaFin);
+        return disponibilidad.numHabSimple >= numHabSimple && disponibilidad.numHabDoble > numHabDoble;
+    }    
+    
+    /**
+     * Devuelve la disponibilidad en un día concreto
+     * @param fecha día a comprobar
+     * @return la disponibilidad en la fecha dada
      */
-    public boolean disponible(LocalDate fecha, int numHabSimple, int numHabDoble) {
+    public Disponibilidad disponibilidad(LocalDate fecha) {
         int numHabSimpleDisponibles = numHabSimple;
         int numHabDobleDisponibles = numHabDoble;
         
@@ -142,9 +157,9 @@ public class Hotel {
             }
         }
         
-        return numHabSimpleDisponibles >= numHabSimple && numHabDobleDisponibles >= numHabDoble;
+        return new Disponibilidad(numHabSimpleDisponibles, numHabDobleDisponibles);
     }
-    
+        
     /**
      * Devolver las reservas que solapan el intervalo dado
      * @param fechaInicio fecha de inicio

src/main/java/es/ujaen/dae/reservahoteles/rest/ControladorReservas.java

@@ -13,7 +13,6 @@ import jakarta.annotation.PostConstruct;
 import jakarta.validation.ConstraintViolationException;
 import java.time.LocalDate;
 import java.util.List;
-import java.util.Optional;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
@@ -98,12 +97,21 @@ public class ControladorReservas {
         List<Hotel> hoteles;
         if (nombre != null) {
             hoteles = servicioReservas.buscarHotel(nombre, localidad).stream()
-                    .filter(h -> servicioReservas.disponible(h, desdeFinal, hastaFinal, numHabSimple, numHabDoble)).toList();
+                    .filter(h -> h.disponible(desdeFinal, hastaFinal, numHabSimple, numHabDoble)).toList();
         }
         else {
             hoteles = servicioReservas.buscarHotelesDisponiblesPorLocalidad(localidad, desdeFinal, hastaFinal, numHabSimple, numHabDoble);
         }
         
         return ResponseEntity.ok(hoteles.stream().map(h -> mapeador.dto(h)).toList());
-    }        
+    }
+/*    
+    @GetMapping("/hotel/{id}")
+    public ResponseEntity<Hotel> buscarHotel(int id) {
+        try {
+            Hotel hotel =  servicioReservas.buscarHotel(id).orElse(HotelNoRegistrado::new);
+         
+        
+    }
+*/
 }

src/main/java/es/ujaen/dae/reservahoteles/servicios/ServicioReservas.java

@@ -113,31 +113,26 @@ public class ServicioReservas {
     public Hotel hotelConReservas(Hotel hotel) {
         hotel = repositorioHoteles.actualizar(hotel);
         // Usar cualquier operación que acceda a las reservas para que se carguen        
-        hotel.disponible(LocalDate.now(), 1, 1); 
+        hotel.disponibilidad(LocalDate.now()); 
         return hotel;
     }
     
-    /**
-     * Comprueba si hay disponibilidad en las fechas indicadas
+ /**
+     * Devuelve la disponibilidad en las fechas indicadas
      * @param hotel hotel donde se compreuba la disponibilidad
      * @param fechaInicio fecha de inicio de la reserva
      * @param fechaFin fecha de final de la reserva
-     * @param numHabSimple número de habitaciones simples solicitadas
-     * @param numHabDoble número de habitaciones dobles solicitadas
-     * @return true si hay disponibilidad, false en caso contrario
+     * @return la disponibilidad en las fechas indicadas
      */
     @Transactional
-    public boolean disponible(Hotel hotel, 
+    public Hotel.Disponibilidad disponibilidad(Hotel hotel, 
             @FutureOrPresent LocalDate fechaInicio, 
-            @FutureOrPresent LocalDate fechaFin, 
-            @PositiveOrZero int numHabSimple, 
-            @PositiveOrZero int numHabDoble) {
+            @FutureOrPresent LocalDate fechaFin) {
         
         hotel = repositorioHoteles.actualizar(hotel);
-        return hotel.disponible(fechaInicio, fechaFin, numHabSimple, numHabDoble);
-    }
-    
-    
+        return hotel.disponibilidad(fechaInicio, fechaFin);
+    }    
+       
     /**
      * Realiza una reserva en un hotel. La reserva debe ser correcta y haber disponibilidad.
      *