Comments 4
UFO just landed and posted this here
UFO just landed and posted this here
Я бы предположил, что по смыслу при null-аргументе лучше всё-таки кидать IllegalArgumentException, т.к. вызвать этот метод, не подав в него нормальный идентификатор - это таки ошибка.
0
а почему нельзя просто аргумент метода проверять с помощью javax.validation?
И никакие optional тут не нужны. Читаемый рабочий код
public BigDecimal getOrderPrice(@NotNull Long orderId) {
return orderRepository.findByOrderId(orderId)
.stream()
.map(OrderLine::getPrice)
.reduce(BigDecimal.ZERO, BigDecimal::add);
}
И никакие optional тут не нужны. Читаемый рабочий код
+1
Sign up to leave a comment.
Optional.stream()