Блог

Обычно я сталкиваюсь со следующей проверкой IntelliJ

 private boolean bar() {    
    return foo().contains("foo"); // Method invocation 'contains' may produce 'java.lang.NullPointerException'
}

private String foo() {
    return null;
}
  

Проверки кажутся мне нормальными, но одно из (или часто единственное) предлагаемых исправлений от IntelliJ заключается в следующем:

 private boolean bar() {
    return Objects.requireNonNull(foo()).contains("foo");
}
  

После этого предупреждение исчезает. Но я вообще не понимаю, как это помогает? requireNonNull просто выдаст то же самое, NullPointerException что было бы выдано в любом случае при .contains вызове на null .

Обычно IntelliJ делает значимые предложения, и это распространенное, так что я упускаю суть здесь?

Обоснование использования Objects#requireNonNull аналогично тому, что стоит за возвратом Optional : вы делаете это, чтобы управлять ожиданиями.

• Когда вы возвращаете, Optional<Something> вместо того, чтобы просто Something сказать: «Эй, я знаю, что это Something значение может отсутствовать, поэтому вместо того, чтобы возвращать null и позволять вам угадывать, я возвращаю Optional<Something> вместо этого, чтобы дать понять, что вам следует ожидать, что его там может не быть, и что я ожидаю, что вы проверите, есть ли оно там, прежде чем использовать его «.
• Когда вы вызываете Objects.requireNonNull(something) в своем коде перед использованием something , вы говорите: «Эй, если вы читаете это, я просто хотел, чтобы вы знали, что я ожидаю, что этот something параметр не будет null, и что я ожидаю, что вы убедитесь, что это не так, прежде чем вызывать этот код; поэтому вместо того, чтобы просто доверять вам и напрямую вызывать something.contains(...) , я вызываю requireNonNull прямо здесь и сейчас, чтобы прояснить это (в случае каких-либо из нас пропустили @NotNull аннотацию в объявлении метода).»

Решает ли это проблему? Нет, это не так. IntelliJ не может волшебным образом запретить параметру принимать значение null. Но это может заставить любого, кто читает этот код, знать, что параметр не должен быть нулевым, и рассуждать о сценариях, в которых это может произойти.