BookService.java

package bookShop.service;

import bookShop.model.Book;
import bookShop.model.request.BookRequest;
import bookShop.repository.BookRepository;
import bookShop.exception.BookNotFoundException;
import bookShop.exception.ForbiddenActionException;
import bookShop.exception.BookAlreadyExistsException;
import lombok.RequiredArgsConstructor;
import org.springframework.stereotype.Service;
import java.util.List;
import lombok.extern.slf4j.Slf4j;

@Slf4j
@Service
@RequiredArgsConstructor
public class BookService {
    private final BookRepository bookRepository;

    public List<Book> getAllBooks() {
        List<Book> books = bookRepository.findAll();
        if (books.isEmpty()) throw new BookNotFoundException("Книги не найдены");
        return books;
    }

    public Book getBookById(Long id) {
        return bookRepository.findById(id)
                .orElseThrow(BookNotFoundException::new);
    }

    public List<Book> getBooksByTitle(String title) {
        title = (title != null) ? title.trim() : null; //Можно еще добавить проверку на ввод пустой строки, возможно при запросе на пустую строку оператор LIKE выдаст все записи
        List<Book> books = bookRepository.findByTitleIgnoreCaseLike(title);
        if (books.isEmpty()) throw new BookNotFoundException("Книги с указанным названием не найдены");
        return books;
    }

    public List<Book> getBooksByAuthor(String author) {
        author = (author != null) ? author.trim() : null; //Можно еще добавить проверку на ввод пустой строки, возможно при запросе на пустую строку оператор LIKE выдаст все записи
        List<Book> books = bookRepository.findByAuthorIgnoreCaseLike(author);
        if (books.isEmpty()) throw new BookNotFoundException("Книги с указанным автором не найдены");
        return books;
    }

    public Book addBook(BookRequest request) {
        log.info("Добавление книги: [{}] [{}]", request.getTitle(), request.getAuthor());
        validateBookUnique(request.getTitle(), request.getAuthor());
        Book book = Book.builder()
                .title(request.getTitle())
                .author(request.getAuthor())
                .price(request.getPrice())
                .copiesAvailable(request.getCopiesAvailable())
                .build();
        log.info("Книга успешно добавлена: [{}] [{}]", request.getTitle(), request.getAuthor());
        return bookRepository.save(book);
    }

    public Book updateBook(Long id, BookRequest request) {
        log.info("Обновление книги [{}]", id);
        Book book = bookRepository.findById(id)
                .orElseThrow(BookNotFoundException::new);
        validateUpdateNotChangeMainFields(book, request);
        book.setPrice(request.getPrice());
        book.setCopiesAvailable(request.getCopiesAvailable());
        log.info("Книга [{}] успешно обновлена", id);
        return bookRepository.save(book);
    }

    public void deleteBook(Long id) {
        log.warn("Попытка удаления админом книги [{}]", id);
        if (!bookRepository.existsById(id)) { // я бы вообще не делал запрос на существование записи в этом случае, потому что это лишний запрос в бд если книга существует. Лучше просто удалять и либо проверять 0 или 1 вернулось, либо просто игнорировать ответ
            throw new BookNotFoundException();
        }
        bookRepository.deleteById(id);
        log.info("Книга [{}] удалена админом", id);
    }

    private void validateBookUnique(String title, String author) {
        if (bookRepository.existsByTitleAndAuthorIgnoreCase(title, author)) {
            throw new BookAlreadyExistsException("Книга с таким названием и автором уже существует");
        }
    }

    private void validateUpdateNotChangeMainFields(Book book, BookRequest request) {
        if (!book.getTitle().equals(request.getTitle())) {
            throw new ForbiddenActionException("Изменение названия книги запрещено");
        }
        if (!book.getAuthor().equals(request.getAuthor())) {
            throw new ForbiddenActionException("Изменение автора книги запрещено");
        }
    }
}