Я хочу знать, что есть способ избавиться от предупреждения в findFirst().get() без использования .orElse(), если я знаю, что каждый раз есть результат, поэтому я никогда не получаю NoSuchElementException. Например, см. Следующий код:
List<String> myList = new ArrayList<>();
myList.add("Test");
myList.add("Example");
myList.add("Sth");
String fisrstString = myList.stream().findFirst().get(); // here I surely get "Test"
Я не знаю, как это делают другие IDE, но моя IDE (IntelliJ) рассматривает это как предупреждение ('Optional.get()' without 'isPresent()'). Я думаю, вероятно, он не знает, когда вы можете получить NoSuchElementException там а когда нет, или я не знаю, почему. Я знаю, что есть способы решить это предупреждение (isPresent() check, .orElse(something)), но с бесполезным кодом, поэтому просто я не хочу использовать эти решения, потому что они так ненужным.
Есть ли у вас какие-либо идеи, что я могу сделать или объяснить, как это обрабатывается theIDE?
Изменить: Извините за NPE, его NoSuchElementException У меня была эта ошибка, но я думаю, что вопрос по-прежнему доступен для него.
Хорошо, что касается меня, лучший способ - использовать функциональную программирование и продолжать работать с дополнительным. Так, например, если вам нужно передать эту строку какой-либо службе, вы можете сделать:
String fisrstString = myList.stream().findFirst().get();
service.doSomething(fisrstString);
Но это выглядит не очень хорошо. Вместо этого вы можете использовать профи функционального программирования и делать:
myList.stream().findFirst().ifPresent(service::doSomething);
Сначала вы не получите NPE, но NoSuchElementException. Во-вторых, это вы, кто может быть уверен; но другие люди могут прийти и не осознают, что это не вызовет исключения.
Для проекта с песочницей - да, вам все равно, и вы можете игнорировать предупреждение; для производственного кода я бы не отключил его (даже если бы мог).
И последнее: если вы так уверены, почему бы не выбрасывать исключение?
orElseThrow(IAmSureThisWillNotHappenException::new)
Вы должны использовать Optional, возвращаемый findFirst(), вместо того, чтобы пытаться получить его значение (если оно действительно присутствует).
myList.stream()
.findFirst()
.ifPresent(/* consume the string here, if present */);
Optional.ifPresent получает Consumer, который будет использоваться, только если Optional содержит ненулевое значение.
Проблема заключается в том, что мы, разработчики Java, так привыкли к императивной парадигме... В частности, мы привыкли получать объект и нажимать, то есть на метод:
String myString = "hello"; // getting an object here
System.out.println(myString); // pushing the object to System.out here
// (via the println method)
С Optional, возвращаемым Stream.findFirst(), вы делали то же самое, что и выше:
String myString = myList.stream()
.findFirst()
.get(); // getting a string here
System.out.println(myString); // pushing the string here
С другой стороны, функциональная парадигма (включая Optional) обычно работает по-другому:
myList.stream()
.findFirst()
.ifPresent(myString -> System.out.println(myString));
Здесь вы не получаете строку, а затем нажимаете ее на какой-то метод. Вместо этого вы предоставляете аргумент для операции Optional ifPresent и позволяете реализации Optional вставлять значение в свой аргумент. Другими словами, вы вытащите значение, обернутое Optional с помощью аргумента ifPresent. ifPresent будет использовать этот аргумент Consumer, только если это значение присутствует.
Этот шаблон тяги очень часто встречается в функциональном программировании и очень полезен, как только вы привыкнете к нему. Это просто требует от разработчиков, чтобы начать думать (и программировать) по-другому.
вы можете передать пустой список без проблем, но если вы попытаетесь получить 1-й элемент в пустом списке, вы получите NoSuchElementException
Stream API знает об этом безупречном, поэтому они предлагают вам несколько способов справиться с этим:
Option1: orElse вы можете вернуть значение по умолчанию, если не найден 1-й элемент
String firstString = myList.stream().findFirst().orElse("Ups!");
Option2: orElseGet вы можете использовать Supplier<String>, который возвращает строку, если не найден 1-й элемент
firstString = myList.stream().findFirst().orElseGet(mySupplier);
Option3: orElseThrow вы можете создать исключение, если не найден 1-й элемент
firstString = myList.stream().findFirst().orElseThrow(WhatTerribleFailException::new);
System.out.println(fisrstString);
IF, вы знаете, что ваш Optional никогда не будет пустым, вы можете использовать аннотацию @SuppressWarnings, как показано ниже:
@SuppressWarnings("ConstantConditions") String foo = Optional.of("bar").get();
Иногда Optional.get будет поднимать NullPointerException, например:
Optional<String> it = Optional.empty();
String foo = it.get();
// ^--- throws NullPointerException when this method is invoked
SO, когда это выражение используется Intellij будет сообщать об обнаружении предупреждения.
IF вы хотите отключить всю проверку контрактов, вы можете сделать следующие действия: Настройки → Инспекции → не отмечены Постоянное условие и исключения → не забудьте нажать кнопку Применить внизу, чтобы сохранить свои настройки.
IF вы не хотите отключать всю проверку контракта, кроме Optional.get() предупреждений, вы можете сделать следующие действия: Настройки → Инспекции > → проверили условие Константа и исключения → в правой нижней части рамки для настройки Optional.get() предупреждений → не забудьте нажать Применить нажмите кнопку внизу для сохранения настроек.
