7

Возникла ситуация при которой я достаю данные из Intent, необходимо обрабатывать на NullPointerException, но если вдруг кинет это Exception, то всё равно нужно крашить прогу, т.к. данное поведение недопустимо, но добавить комментарий, почему именно так вышло, я в свою очередь делаю так

try{
    mCPath = new CPath(intent.getStringExtra(EXTRA_CPATH)); 
}catch(NullPointerException e){
    throw new NullPointerException("EXTRA_CPATH must be not null");
}

Но мой тимлид говорит, что так не хорошо, необходимо кинуть другой exception, а именно RuntimeException

try{
    mCPath = new CPath(intent.getStringExtra(EXTRA_CPATH)); 
}catch(NullPointerException e){
    throw new RuntimeException("EXTRA_CPATH must be not null");
}

Не совсем понимаю логики, в данном контексте программы, по моему личному мнению, если приходить Null, то и кидать нужно null, просто дополнить комментарием или я не прав?

4
  • 2
    Ответ на ваш вопрос зависит от мнения, что подтверждает различие мнения вашего тимлида и у вас. Это значит, что правильного ответа не существует. 30 дек 2015 в 13:03
  • Хотелось бы, для интереса, узнать конкретное объяснение тимлида, а не просто "нехорошо". Ибо "нехорошо" - не аргумент) 30 дек 2015 в 13:06
  • @АлексейШиманский Разъяснение было такого, что есть другие более подходящие эксепшины, в пример был приведён IllegalStateException, но лично я считаю, что причём тут состояние, если явно нету объекта
    – BORSHEVIK
    30 дек 2015 в 13:38
  • Код, который ловит NPE уже плох. Просто не допускайте выброс этого исключения.
    – VladD
    30 дек 2015 в 14:23

3 ответа 3

13

(Экстраполирую информацию из .NET, могу ошибаться.)

NullPointerException — это системное исключение, которое генерируется в недрах виртуальной машины Java, которое возникает при обращении по околонулевому адресу, что вообще отслеживается на уровне операционной системы. Дурной тон — и ловить исключение, и бросать его.

Ловить исключение не надо, потому что обращение к нулевому объекту — это ошибка в логике приложения. Если какая-то переменная или какое-то поле у вас null, то нужно проверять это значение, а не рассчитывать на то, что рантайм бросит NPE.

Кидать исключение не надо, потому что программисты могут ожидать, что исключение возникнет на системном уровне, а не будет вручную сгенерировано пользовательским кодом.

Как правильно поступить в вашем случае? Проверить intent на null, и если значение отсутствует, то бросить исключение, наиболее подходящее логически, да хотя бы IllegalStateException. Если значение null пришло в виде аргумента, то следует бросать IllegalArgumentException. И уж точно не должно быть никаких try-catch.


Похоже, в Java есть разные взгляды на использование исключения, но в целом преобладает моя точка зрения. См. тж.:

2

Все логично и прекрасно, но в стандартной библиотеке Java принято кидать NullPointerException:

  • java.util.Objects.requireNonNull кидает NPE.
  • java.util.Map.put разрешает кидать NPE, если реализация интерфейса не поддерживает null как ключ или значение.
  • Joshua Bloch в Effective Java (2nd ed) Item 60 пишет:

Если при вызове null передается вместо параметра, для которого null - запрещенное значение, соглашение предписывает бросать NullPointerException, а не IllegalArgumentException.

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

Короче, NPE бросать нормально, NPE бросать похвально.

Тем не менее, демократическим путем установлено, что NullPointerException - это плохо.

Конкретно по коду в вопросе, если вы уверены, что EXTRA_CPATH не будет null никогда-никогда, а оно вдруг null, и остается только убить приложение, кидайте сразу AssertionError. Поскольку вы явно ловите unchecked exception, вы в этом не уверены, и следует проверять и закрывать приложение нормальным способом, с уведомлением пользователя.

2

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

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.