delet123

README.md

@@ -51,7 +51,7 @@
 | **GET**         | `api/<CODE>/auth`   | Проверка авторизации   | `<CODE>` - код для входа                                                                               | `400` - что-то пошло не так<br>`401` - кода не существует<br>`200` - данный код существует - можно пользоваться приложением                                                                                                                                                                                                                                                                  |
 | **GET**         | `api/<CODE>/info`   | Получение информации о пользователе   | `<CODE>` - код для входа                                                                               | `400` - что-то пошло не так<br>`401` - кода не существует<br>`200` - ОК<br><pre>{<br>  "name":"Иванов Петр Федорович",<br>  "photoUrl":"<https://funnyducks.ru/upload/iblock/0cd/0cdeb7ec3ed6fddda0f90fccee05557d.jpg>",<br>  "booking":{<br>    "2025-01-05": {"id":1,"place":"102"},<br>    "2025-01-06": {"id":2,"place":"209.13"},<br>    "2025-01-09": {"id":3,"place":"Зона 51. 50"}<br>  }<br>}</pre> |
 | **GET**         | `api/<CODE>/booking`   | Получение доступных для бронирования мест   | `<CODE>` - код для входа                                                                               | `400` - что-то пошло не так<br>`401` - кода не существует<br>`200` - ОК<br><pre>{<br>  "2025-01-05": [{"id": 1, "place": "102"},{"id": 2, "place": "209.13"}],<br>  "2025-01-06": [{"id": 3, "place": "Зона 51. 50"}],<br>  "2025-01-07": [{"id": 1, "place": "102"},{"id": 2, "place": "209.13"}],<br>  "2025-01-08": [{"id": 2, "place": "209.13"}]<br>}</pre> **Список дат ограничен текущим + 3 днями** (ответ от сервера содержит 4 дня со свободными местами для каждого)                                                                                                                                                                                                                                                                                       
-| **POST**        | `api/<CODE>/book`   | Создание нового бронирования   | `<CODE>` - код для входа<br>Тело: <br> <pre>{<br>  “date”: “2025-01-05”,<br>  “placeID”: 1 <br>}</pre> |`400` - что-то пошло не так<br>`401` - кода не существует<br>`409` - уже забронировано<br>`201` - бронирование успешно создано                                                                                                                                                                                                                                                                                        
+| **POST**        | `api/<CODE>/book`   | Создание нового бронирования   | `<CODE>` - код для входа<br>Тело: <br> <pre>{<br>  “date”: “2025-01-05”,<br>  “placeId”: 1 <br>}</pre> |`400` - что-то пошло не так<br>`401` - кода не существует<br>`409` - уже забронировано<br>`201` - бронирование успешно создано                                                                                                                                                                                                                                                                                        
 
 
 ## 📊 Пример данных

src/main/java/com/example/nto/controller/BookingController.java

@@ -1,8 +1,5 @@
 package com.example.nto.controller;
 
-import com.example.nto.entity.Booking;
-import com.example.nto.entity.Employee;
-import com.example.nto.entity.Place;
 import com.example.nto.excepation.EmployeeNotFoundException;
 import com.example.nto.repository.BookingRepository;
 import com.example.nto.repository.PlaceRepository;
@@ -12,9 +9,6 @@ import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.*;
-
-import java.time.LocalDate;
-import java.util.LinkedHashMap;
 import java.util.Map;
 
 /**
@@ -34,10 +28,12 @@ public class BookingController {
     private BookingRepository bookingRepository;
     @Autowired
     private EmployeeService employeeService;
-
     @GetMapping("/{code}/booking")
     public ResponseEntity<?> getAvailableBookings(@PathVariable String code) {
         try {
+            if (code == null || code.trim().isEmpty()) {
+                return ResponseEntity.status(HttpStatus.BAD_REQUEST).body("Employee code cannot be null or empty");
+            }
             Map<String, Object> response = bookingService.findAvailableBookings(code);
             return ResponseEntity.ok(response);
         } catch (IllegalArgumentException e) {
@@ -48,59 +44,32 @@ public class BookingController {
             return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).body("Unexpected error");
         }
     }
-
     @PostMapping("/{code}/book")
     public ResponseEntity<?> createBooking(
             @PathVariable String code,
-            @RequestParam Long placeId,
-            @RequestParam String date
+            @RequestBody Map<String, Object> body
     ) {
         try {
+            Long placeId = body.get("placeId") != null ? Long.valueOf(body.get("placeId").toString()) : null;
+            String date = body.get("date") != null ? body.get("date").toString() : null;
             if (placeId == null || date == null || date.isEmpty()) {
-                return ResponseEntity.status(HttpStatus.BAD_REQUEST)
-                        .body("Place ID and date must be provided");
+                return ResponseEntity.status(HttpStatus.BAD_REQUEST).body("Place ID and date must be provided");
             }
-            Employee employee = employeeService.getEmployeeWithBookings(code);
-            if (employee == null) {
-                return ResponseEntity.status(HttpStatus.UNAUTHORIZED)
-                        .body("Employee not found");
-            }
-            Place place = placeRepository.findById(placeId).orElse(null);
-            if (place == null) {
-                return ResponseEntity.status(HttpStatus.BAD_REQUEST)
-                        .body("Place not found");
-            }
-            LocalDate bookingDate;
-            try {
-                bookingDate = LocalDate.parse(date);
-            } catch (Exception ex) {
-                return ResponseEntity.status(HttpStatus.BAD_REQUEST)
-                        .body("Invalid date format. Use yyyy-MM-dd");
-            }
-            boolean exists = bookingRepository.findByDateBetween(bookingDate, bookingDate)
-                    .stream()
-                    .anyMatch(b -> b.getPlace().getId() == placeId);
-            if (exists) {
-                return ResponseEntity.status(HttpStatus.CONFLICT)
-                        .body("Place already booked for this date");
-            }
-            Booking booking = new Booking();
-            booking.setEmployee((Map<String, Object>) employee);
-            booking.setPlace(place);
-            booking.setDate(bookingDate);
-            bookingRepository.save(booking);
-            Map<String, Object> response = new LinkedHashMap<>();
-            response.put("id", booking.getId());
-            response.put("date", booking.getDate());
-            response.put("placeId", place.getId());
-            response.put("employeeId", employee.getId());
-            return ResponseEntity.status(HttpStatus.CREATED).body(response);
-        } catch (java.lang.IllegalArgumentException e) {
+            String message = bookingService.createBooking(code, placeId, date);
+            return ResponseEntity.ok(message);
+        } catch (IllegalArgumentException e) {
             return ResponseEntity.status(HttpStatus.BAD_REQUEST).body(e.getMessage());
+        } catch (EmployeeNotFoundException e) {
+            return ResponseEntity.status(HttpStatus.UNAUTHORIZED).body(e.getMessage());
         } catch (Exception e) {
             return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).body("Unexpected error");
         }
     }
+    @DeleteMapping("/{id}")
+    public ResponseEntity<Void> deleteBooking(@PathVariable Long id) {
+        bookingService.deleteBooking(id);
+        return ResponseEntity.noContent().build();
+    }
 }
 
 

src/main/java/com/example/nto/controller/EmployeeController.java

@@ -1,12 +1,19 @@
 package com.example.nto.controller;
+import com.example.nto.entity.Booking;
 import com.example.nto.entity.Employee;
 import com.example.nto.excepation.EmployeeNotFoundException;
+import com.example.nto.repository.BookingRepository;
 import com.example.nto.service.EmployeeService;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.*;
 
+import java.util.LinkedHashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.stream.Collectors;
+
 /**
  * TODO: ДОРАБОТАТЬ в рамках задания
  * =================================
@@ -18,16 +25,18 @@ import org.springframework.web.bind.annotation.*;
 public class EmployeeController {
     @Autowired
     private EmployeeService employeeService;
+    @Autowired
+    private BookingRepository bookingRepository;
+
     @GetMapping("/{code}/auth")
-    public ResponseEntity<Void> checkAuth(
-            @PathVariable String code,
-            @RequestHeader(value = "Authorization", required = false) String authHeader) {
-        if (authHeader == null || !authHeader.equals("Bearer valid-token")) {
-            return ResponseEntity.status(HttpStatus.UNAUTHORIZED).build();
+    public ResponseEntity<String> checkAuth(
+            @PathVariable String code) {
+        if (code == null) {
+            throw new IllegalArgumentException("Employee code is required");
         }
         try {
-            employeeService. getEmployeeWithBookings(code);
-            return ResponseEntity.ok().build();
+            employeeService.getEmployeeByCode(code);
+            return ResponseEntity.ok("Authorized Employee");
         } catch (IllegalArgumentException e) {
             return ResponseEntity.status(HttpStatus.BAD_REQUEST).build();
         } catch (EmployeeNotFoundException e) {
@@ -37,12 +46,25 @@ public class EmployeeController {
     @GetMapping("/{code}/info")
     public ResponseEntity<?> getEmployeeInfo(@PathVariable String code) {
         try {
-            Employee response = employeeService.getEmployeeWithBookings(code);
+            if (code == null || code.trim().isEmpty()) {
+                return ResponseEntity.status(HttpStatus.BAD_REQUEST)
+                        .body("Employee code cannot be null or empty");
+            }
+            Employee employee = employeeService.getEmployeeByCode(code);
+            List<Booking> bookings = bookingRepository.findByEmployeeId(employee.getId());
+            Map<String, List<Map<String, Object>>> bookingMap = bookings.stream().collect(Collectors.groupingBy(b -> b.getDate().toString(),LinkedHashMap::new,Collectors.mapping(b -> Map.of("id", b.getId(),"place", b.getPlace().getPlace()), Collectors.toList())));
+            Map<String, Object> response = new LinkedHashMap<>();
+            response.put("name", employee.getName());
+            response.put("photoUrl", employee.getPhotoUrl());
+            response.put("booking", bookingMap);
             return ResponseEntity.ok(response);
         } catch (IllegalArgumentException e) {
-            return ResponseEntity.status(400).body(e.getMessage());
+            return ResponseEntity.status(HttpStatus.BAD_REQUEST).body(e.getMessage());
         } catch (EmployeeNotFoundException e) {
-            return ResponseEntity.status(404).body(e.getMessage());
+            return ResponseEntity.status(HttpStatus.UNAUTHORIZED).body(e.getMessage());
+        } catch (Exception e) {
+            return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR)
+                    .body("Unexpected error");
         }
     }
-}
+}

src/main/java/com/example/nto/controller/EmployeeFullInfoResponseDto.java

@@ -1,27 +0,0 @@
-package com.example.nto.controller;
-
-import lombok.AllArgsConstructor;
-import lombok.Builder;
-import lombok.Data;
-import lombok.NoArgsConstructor;
-
-import java.util.Map;
-
-@Data
-@Builder
-@NoArgsConstructor
-@AllArgsConstructor
-public class EmployeeFullInfoResponseDto {
-    private String name;
-    private String photoUrl;
-    private Map<String, BookingInfo> booking;
-
-    @Data
-    @Builder
-    @NoArgsConstructor
-    @AllArgsConstructor
-    public static class BookingInfo {
-        private Long id;
-        private String place;
-    }
-}

src/main/java/com/example/nto/entity/Booking.java

@@ -7,7 +7,6 @@ import lombok.Data;
 import lombok.NoArgsConstructor;
 
 import java.time.LocalDate;
-import java.util.Map;
 
 
 /**
@@ -57,8 +56,7 @@ public class Booking {
     public void setPlace(Place place) {
         this.place = place;
     }
-
-    public void setEmployee(Map<String, Object> employee) {
+    public void setEmployee(Employee employee) {
         this.employee = (Employee) employee;
     }
 }

src/main/java/com/example/nto/entity/Employee.java

@@ -6,6 +6,9 @@ import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.NoArgsConstructor;
+
+import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.List;
 
 
@@ -31,28 +34,11 @@ public class Employee {
     private String code;
 
     private String photoUrl;
+    @OneToMany(mappedBy = "employee", fetch = FetchType.LAZY, cascade = CascadeType.ALL)
+    private List<Booking> bookings = new ArrayList<>();
 
-    @OneToMany(mappedBy = "employee", cascade = CascadeType.ALL, fetch = FetchType.LAZY)
-    @JsonIgnore
-    private List<Booking> bookingList;
-
-    public List<Booking> getBookingList() {
-        return bookingList;
-    }
-
-    public Object getCode() {
-        return code;
-    }
-
-    public Object getPhotoUrl() {
-        return photoUrl;
-    }
-
-    public Object getName() {
-        return name;
-    }
-
-    public Object getId() {
-        return id;
-    }
+    public long getId() { return id; }
+    public String getName() { return name; }
+    public String getCode() { return code; }
+    public String getPhotoUrl() { return photoUrl; }
 }

src/main/java/com/example/nto/excepation/EmployeeNotFoundException.java

@@ -1,7 +1,7 @@
 package com.example.nto.excepation;
 
 public class EmployeeNotFoundException extends RuntimeException{
-    public EmployeeNotFoundException(String msg) {
-        super(msg);
+    public EmployeeNotFoundException(String message) {
+        super(message);
     }
 }

src/main/java/com/example/nto/repository/BookingRepository.java

@@ -1,7 +1,6 @@
 package com.example.nto.repository;
 
 import com.example.nto.entity.Booking;
-import com.example.nto.entity.Employee;
 import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.stereotype.Repository;
 
@@ -16,6 +15,6 @@ import java.util.List;
  */
 @Repository
 public interface BookingRepository extends JpaRepository<Booking, Long> {
-    List<Booking> findByEmployee(Employee employee);
     List<Booking> findByDateBetween(LocalDate start, LocalDate end);
+    List<Booking> findByEmployeeId(long id);
 }

src/main/java/com/example/nto/repository/EmployeeRepository.java

@@ -4,6 +4,8 @@ import com.example.nto.entity.Employee;
 import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.stereotype.Repository;
 
+import java.util.Optional;
+
 /**
  * TODO: ДОРАБОТАТЬ в рамках задания
  * =================================
@@ -12,5 +14,5 @@ import org.springframework.stereotype.Repository;
  */
 @Repository
 public interface EmployeeRepository extends JpaRepository<Employee, Long> {
-    Employee findByCode(String code);
-}
+    Optional<Employee> findByCode(String code);
+}

src/main/java/com/example/nto/service/BookingService.java

@@ -6,4 +6,6 @@ import java.util.Map;
 @Service
 public interface BookingService {
     Map<String, Object> findAvailableBookings(String employeeCode);
+    String createBooking(String code, Long placeId, String date);
+    void deleteBooking(long id);
 }

src/main/java/com/example/nto/service/EmployeeService.java

@@ -10,5 +10,5 @@ import org.springframework.stereotype.Service;
  */
 @Service
 public interface EmployeeService {
-    Employee  getEmployeeWithBookings(String code);
+    Employee getEmployeeByCode(String code);
 }

src/main/java/com/example/nto/service/PlaceServise.java

@@ -2,15 +2,13 @@ package com.example.nto.service;
 import com.example.nto.entity.Place;
 import com.example.nto.repository.PlaceRepository;
 import jakarta.annotation.PostConstruct;
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 
 @Service
 public class PlaceServise {
-    private final PlaceRepository placeRepository;
-    public PlaceServise(PlaceRepository placeRepository) {
-        this.placeRepository = placeRepository;
-    }
-
+    @Autowired
+    private  PlaceRepository placeRepository;
     @PostConstruct
     public void Places() {
         if(placeRepository.count() == 0) {

src/main/java/com/example/nto/service/impl/BookingServiceImpl.java

@@ -5,6 +5,7 @@ import com.example.nto.entity.Employee;
 import com.example.nto.entity.Place;
 import com.example.nto.excepation.EmployeeNotFoundException;
 import com.example.nto.repository.BookingRepository;
+import com.example.nto.repository.EmployeeRepository;
 import com.example.nto.repository.PlaceRepository;
 import com.example.nto.service.BookingService;
 import com.example.nto.service.EmployeeService;
@@ -16,22 +17,20 @@ import java.util.*;
 
 @Service
 public class BookingServiceImpl implements BookingService {
-
     @Autowired
     private BookingRepository bookingRepository;
-
     @Autowired
     private PlaceRepository placeRepository;
-
     @Autowired
     private EmployeeService employeeService;
-
+    @Autowired
+    private EmployeeRepository employeeRepository;
     @Override
     public Map<String, Object> findAvailableBookings(String employeeCode) {
         if (employeeCode == null || employeeCode.isEmpty()) {
             throw new IllegalArgumentException("Employee code cannot be null or empty");
         }
-        Employee employee = employeeService.getEmployeeWithBookings(employeeCode);
+        Employee employee = employeeService.getEmployeeByCode(employeeCode);
         if (employee == null) {
             throw new EmployeeNotFoundException("Employee not found with code: " + employeeCode);
         }
@@ -44,8 +43,7 @@ public class BookingServiceImpl implements BookingService {
             LocalDate date = today.plusDays(i);
             List<Map<String, Object>> freePlaces = new ArrayList<>();
             for (Place place : places) {
-                boolean isBooked = bookings.stream()
-                        .anyMatch(b -> b.getDate().isEqual(date) && b.getPlace().getId() == place.getId());
+                boolean isBooked = bookings.stream().anyMatch(b -> b.getDate().isEqual(date) && b.getPlace().getId() == place.getId());
                 if (!isBooked) {
                     Map<String, Object> placeInfo = new LinkedHashMap<>();
                     placeInfo.put("id", place.getId());
@@ -57,7 +55,48 @@ public class BookingServiceImpl implements BookingService {
         }
         return Collections.unmodifiableMap(result);
     }
+    public String createBooking(String code, Long placeId, String date) {
+
+        if (code == null || code.isBlank()) {
+            throw new IllegalArgumentException("Employee code is empty");
+        }
+        if (placeId == null || date == null || date.isBlank()) {
+            throw new IllegalArgumentException("Place ID and date must be provided");
+        }
+        Employee employee = employeeRepository.findByCode(code).orElseThrow(() -> new EmployeeNotFoundException("Employee not found"));
+        Place place = placeRepository.findById(placeId).orElseThrow(() -> new IllegalArgumentException("Place not found"));
+        LocalDate bookingDate;
+        try {
+            bookingDate = LocalDate.parse(date);
+        } catch (Exception e) {
+            throw new IllegalArgumentException("Invalid date format. Use yyyy-MM-dd");
+        }
+        boolean exists = bookingRepository.findByDateBetween(bookingDate, bookingDate).stream().anyMatch(
+                b -> b.getPlace()
+                        .getId() == placeId);
+        if (exists) {
+            throw new IllegalArgumentException("Place already booked for this date");
+        }
+        Booking booking = new Booking();
+        booking.setEmployee((Employee) employee);
+        booking.setPlace(place);
+        booking.setDate(bookingDate);
+        bookingRepository.save(booking);
+        Map<String, Object> result = new LinkedHashMap<>();
+        result.put("id", booking.getId());
+        result.put("date", booking.getDate());
+        result.put("placeId", place.getId());
+        result.put("employeeId", employee.getId());
+        return "the booking was created successfully";
+    }
+    public void deleteBooking(long id) {
+        if (!bookingRepository.existsById(id)) {
+            throw new IllegalArgumentException("Booking not found with id: " + id);
+        }
+        bookingRepository.deleteById(id);
+    }
 }
 
 
 
+

src/main/java/com/example/nto/service/impl/EmployeeServiceImpl.java

@@ -1,9 +1,9 @@
 package com.example.nto.service.impl;
 import com.example.nto.entity.Employee;
 import com.example.nto.excepation.EmployeeNotFoundException;
-import com.example.nto.repository.BookingRepository;
 import com.example.nto.repository.EmployeeRepository;
 import com.example.nto.service.EmployeeService;
+import lombok.RequiredArgsConstructor;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 
@@ -15,19 +15,15 @@ import org.springframework.stereotype.Service;
  * НЕЛЬЗЯ: Изменять название класса и пакета
  */
 @Service
+@RequiredArgsConstructor
 public class EmployeeServiceImpl implements EmployeeService {
     @Autowired
-    private EmployeeRepository employeeRepository;
+    private  EmployeeRepository employeeRepository;
     @Override
-    public Employee getEmployeeWithBookings(String code) {
+    public Employee getEmployeeByCode(String code) {
         if (code == null || code.isEmpty()) {
-            throw new IllegalArgumentException("Employee code cannot be null or empty");
+            throw new IllegalArgumentException("Employee code is required");
         }
-        Employee employee = employeeRepository.findByCode(code);
-        if (employee == null) {
-            throw new EmployeeNotFoundException("Employee not found with code: " + code);
-        }
-        return employee;
+        return employeeRepository.findByCode(code).orElseThrow(() -> new EmployeeNotFoundException("Employee not found"));
     }
-
 }