Как правильно использовать Optional?

Замысел. Метод findById возвращает Optional<User> — вызывающий видит в сигнатуре, что пользователя может не быть, и не забудет обработать этот случай.

Идиоматичное использование:

String city = repo.findById(id)
    .map(User::address)
    .map(Address::city)
    .orElse("неизвестно");

User u = repo.findById(id)
    .orElseThrow(() -> new NotFoundException(id));

repo.findById(id).ifPresentOrElse(this::notify, this::log);

Антипаттерны:

• if (opt.isPresent()) opt.get() — тот же null-check, только длиннее;
• голый get() — NoSuchElementException вместо NPE (с Java 10 есть честное имя orElseThrow());
• Optional как поле — не сериализуем, лишний объект на экземпляр;
• Optional как параметр — вызывающий всё равно должен создать Optional, проще перегрузка или null-допускающий параметр;
• Optional<List<T>> — пустая коллекция уже означает «ничего нет»;
• orElse(expensive()) — дорогое значение вычисляется всегда.