Java 8 Optional.ofNullable.map, создающая ошибку ссылки на нестатический метод

Question

У меня есть способ проверить получателя электронной почты.

В моем коде .map(Recipient::getId) выдает ошибку:

Нестатический метод не может быть ссылкой из статического контекста.

private Long verifyRecipient(Long recipientId) throws NotFoundException {
    return Optional.ofNullable(recipientRepository.findById(recipientId))
            .map(Recipient::getId)
            .orElseThrow(()-> new NotFoundException("recipient with ID" + recipientId +
                    " was not found"));
}

Recipient class:

@Entity
public class Recipient {
    @Id
    @GeneratedValue
    private Long id;

    @NotBlank
    private String name;

    @NotBlank
    @Email
    @Column(unique = true)
    private String emailAddress;

    public Long getId() {
        return id;
    }

    public void setId(Long id) {
        this.id = id;
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public String getEmailAddress() {
        return emailAddress;
    }

    public void setEmailAddress(String emailAddress) {
        this.emailAddress = emailAddress;
    }
}

Я использую SpringBoot и H2 в базе данных памяти.

Итак, у меня также есть интерфейс RecipientRepository:

public interface RecipientRepository extends JpaRepository<Recipient, Long> {}

ОпределениеfindById() метод:

Optional<T> findById(ID var1);

Answer 1

Метод findById() уже возвращает Optional<T>, поэтому вам не нужно оборачивать результат дополнительными Optional.ofNullable() в этой ситуации.

На самом деле, строка:

Optional.ofNullable(recipientRepository.findById(recipientId));

возвращает Optional<Optional<Recipient>>, что является избыточным.

Вместо этого вы можете просто написать:

private Long verifyRecipient(Long recipientId) throws NotFoundException {
    return recipientRepository.findById(recipientId)
        .map(Recipient::getId)
        .orElseThrow(() ->
            new NotFoundException("Recipient with ID " + recipientId + " was not found"));
}