jpa 연습중입니다. service 부분을 이런 포맷으로 만들었는데 피드백 부탁드립니다..!

Question

@Slf4j @RequiredArgsConstructor @Service public class OrderService { private final OrderRepository orderRepository; private final OrderProductsRepository orderProductsRepository; @Transactional public Long saveOrders(OrdersReqDto reqDto) { Orders orders = new Orders(reqDto.getUserName(), false); orderRepository.save(orders); List orderProducts = reqDto.getOrderProductsReqDtos().stream(). map(resDto -> OrderProducts.builder() .name(resDto.getName()) .price(resDto.getPrice()) .orders(orders) .build()).collect(Collectors.toList()); orderProductsRepository.saveAll(orderProducts); return orders.getId(); } @Transactional public Long updateIsCompleted(Long orderId) throws BaseException { Orders orders = orderRepository.findById(orderId) .orElseThrow(() -> new BaseException(ApiCode.DATA_NOT_FOUND)); orders.updateIsCompleted(true); return orderRepository.save(orders).getId(); } public OrdersResDto findOrders(Long orderId) throws BaseException { return new OrdersResDto(orderRepository.findById(orderId).orElseThrow(() -> new BaseException(ApiCode.DATA_NOT_FOUND))); } public List findOrdersAllByUserName(String userName) { List orders = orderRepository.findAllByUserName(userName); List ordersResDtos = orders.stream() .map(OrdersResDto::new) .collect(Collectors.toList()); return ordersResDtos; } public List findOrdersAll() { List orders = orderRepository.findAll(); List ordersResDtos = orders.stream() .map(OrdersResDto::new) .collect(Collectors.toList()); return ordersResDtos; }

jpa 연습중입니다. service 부분을 이런 포맷으로 만들었는데 피드백 부탁드립니다..!

0개의 답변

Q&A 목록