Define what method to call based on value of Optional

Question

Is it possible, to somehow alter the code below, so in case of no status value provided to return all the PaymentLog objects stored in the database instead of only the ones with status equal with 404? Basically, I would like if the status variable has not been provided to call another method in the service layer logService.getAllPaymentLogs()

@GetMapping(Endpoints.LOGS.PAYMENT_LOGS)
public Page<PaymentLog> getPaymentLog(@RequestParam Optional<Integer> status) {
    return logService.getPaymentLogStatus(status.orElse(404), PageRequest.of(0, 10));
}

These are the getPaymentLogStatus() and getAllPaymentLogs

@Override
public Page<PaymentLog> getPaymentLog(Pageable pageable) {
    return paymentLogRepository.getAllBy(pageable);
}

And

@Override
public Page<PaymentLog> getPaymentLog(int status, Pageable pageable) {
    return paymentLogRepository.getAllByStatus(status, pageable);
}

does your `paymentLogRepository` have a method like `getAll()`? — f1sh, May 28 '20 at 18:59
`status.map(s -> getPaymentLog(s, page)).orElseGet(() -> getPaymentLog(page))`? — 123, May 28 '20 at 19:01
@Dc235 Yeah, call it whatever you want in the lambda, it's just the element inside the Optional — 123, May 28 '20 at 19:20

score 1 · Answer 1 · answered May 28 '20 at 19:18

1

@123 answered the question in the commenting session:

status.map(s -> getPaymentLog(s, page)).orElseGet(() -> getPaymentLog(page))

answered May 28 '20 at 19:18

Define what method to call based on value of Optional

1 Answers1