Java Optional if else logic [duplicate]

Question

I have a method that searches a customer in the DB, if found, it returns that customer, otherwise, it creates a new one.

private Customer getProjectCustomer(String customerCode) {
    Optional<Customer> customerInDB = customerRepository.findByCode(customerCode);

    if (!customerInDB.isPresent()) {
        //call to remote service to get the customer from the remote service
        Customer newCustomer = new Customer();
        ..some sets
        return customerRepository.save(newCustomer);
    }

    return customerInDB.get();
}

I have the feeling that this is not the correct way to use the Java Optional, however, I don't really know if there is a better more functional approach

azro · Accepted Answer · 2022-04-26 17:25:33Z

0

You may use Optional.orElseGet

private Customer getProjectCustomer(String customerCode) {
    return customerRepository.findByCode(customerCode)
            .orElseGet(() -> {
                Customer newCustomer = new Customer();
                // ...
                return customerRepository.save(newCustomer);
            });
}

answered Apr 26, 2022 at 17:25

azro

54.2k9 gold badges39 silver badges75 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Java Optional if else logic [duplicate]

1 Answer 1

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Linked

Related