Kodları oxuyun və səhvləri vurğulayın /** * Alınmış bileti olan müştəri əlavə ödəniş qarşılığında müəyyən bir yer seçə bilər. * Oturacaqların əsas qiyməti tariflər (xarici xidmət) tərəfindən müəyyən edilir. * Müəyyən tariflərə (PREMIUM, ULTRA) sahib müştərilər üçün ödəniş zamanı endirim tətbiq edilməlidir. * Rezervasiya zamanı müştəri üçün ödəniş üçün faktura çıxarılır. Ödənişin idarə olunması üçüncü tərəf xidmətdə həyata keçirilir. */ @Service public class SeatBookingService { @Autowired private SeatBookingRepository seatBookingRepository; @Autowired private TicketRepository ticketRepository; @Autowired private TariffClient tariffClient; @Autowired private CustomerClient customerClient; @Autowired private PaymentClient paymentClient; /** * Rezervasiya. * @param seatCode yer kodu (məsələn, 19A) * @param ticketId bilet ID-si */ @Transactional public void bookSeat(String seatCode, UUID ticketId) { var ticket = ticketRepository.findById(ticketId); //rezervasiya edirik var seatBooking = new SeatBooking(seatCode, ticket.get().getFlightId(), ticketId, BookingStatus.BOOKED); seatBookingRepository.save(seatBooking); //uçuşdakı seçilmiş yer üçün əsas tarif tapırıq var basePrice = tariffClient.getBasePrice(ticket.get().getPlaneModel(), seatCode); //müştəri məlumatlarını tapırıq Long userId = (Long) SecurityContextHolder.getContext().getAuthentication().getPrincipal(); var userData = customerClient.getCustomer(userId); System.out.println("Tapılan istifadəçi " + userData.getFio() + " sənəd nömrəsi " + userData.getDocument()); var price = basePrice; if (userData.getTariff() == "PREMIUM") { //50% endirim price = basePrice * 0.5d; } if (userData.getTariff() == "ULTRA") { //20% endirim price = basePrice * 0.8d; } var invoice = new Invoice(price, ticketId, userId); //faktura çıxarırıq paymentClient.sendInvoice(invoice); System.out.println("Faktura göndərildi"); } @Data @Table("seat_booking") public class SeatBooking { @Column private String seatCode; @Column private UUID flightId; @Column private UUID ticketId; @Column private BookingStatus status; } public enum BookingStatus { BOOKED, PAID; }
sobes.tech Süni İntellekt
AI-dan cavab
В представленном коде есть несколько ошибок и потенциальных проблем:
-
Неправильное сравнение строк: В Java строки нельзя сравнивать через
==, нужно использовать метод.equals(). В коде:if (userData.getTariff() == "PREMIUM") { //... }нужно заменить на:
if ("PREMIUM".equals(userData.getTariff())) { //... }Аналогично для "ULTRA".
-
Отсутствие проверки наличия билета:
ticketRepository.findById(ticketId)возвращает Optional, но сразу вызываетсяticket.get()без проверки, что билет существует. Это может привести кNoSuchElementException. Нужно проверить наличие:var ticketOpt = ticketRepository.findById(ticketId); if (ticketOpt.isEmpty()) { throw new IllegalArgumentException("Ticket not found"); } var ticket = ticketOpt.get(); -
Получение userId из SecurityContext: Приведение
getPrincipal()кLongможет быть неверным, так как обычно там хранится объект UserDetails или другой тип. Нужно привести к правильному типу или получить ID другим способом. -
Отсутствие обработки ошибок внешних сервисов: Вызовы
tariffClient.getBasePrice(),customerClient.getCustomer(),paymentClient.sendInvoice()могут выбрасывать исключения или возвращать null, что не обрабатывается. -
Отсутствие логирования: Использование
System.out.printlnне рекомендуется в продакшене, лучше использовать логгер. -
Отсутствие проверки доступности места: Нет проверки, что место с
seatCodeсвободно для бронирования. -
Отсутствие обновления статуса после оплаты: В коде нет логики изменения статуса бронирования с BOOKED на PAID.
Пример исправления сравнения строк:
if ("PREMIUM".equals(userData.getTariff())) {
price = basePrice * 0.5d;
}
if ("ULTRA".equals(userData.getTariff())) {
price = basePrice * 0.8d;
}