try/catch vs if/else throwing NullPointerException

Question

Что бы я выбрал, когда вы хотите выбросить исключение NullPointerException?

1)

if(parameter != null){ 
    // do something 
} else { 
    Log.error("Reason for Exception."); 
    throw new NullPointerException(); 
} 

2)

try{ 
    // do something 
} catch (NullPointerException npe) { 
    Log.error("Reson for Exception."); 
    throw npe; 
} 

Answer 1

В 2) вы не будете точно знать, какое выражение бросил NPE. В 1) совершенно понятно, почему вы выбрасываете исключение. Если необходимо, выполните ряд отдельных нулевых проверок.

Однако ваш первый подход должен быть переписан:

if (arg == null) { 
    throw new IllegalArgumentException("arg is null"); 
} 
... just go on with the method, no else block needed 

И если вы по крайней мере на Java 7:

Objects.requireNonNull(arg, "arg is null"); 

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

Answer 2

Если вы пытаетесь проверить параметр не null, я бы порекомендовал Guava's Preconditions, в частности checkNotNull.

Preconditions.checkNotNull(parameter, "Please specify a value for parameter"); 

Если параметр пуст, он выдает NPE с указанным сообщением.

Answer 3

Во-первых, я бы использовал IllegalArgumentException более NullPointerException в этом случае. Но в случае выбора между ними, я предпочитаю первый, потому что это ясно.

В противном случае, чтобы избежать такого шаблона, который, на мой взгляд, делает его уродливым. Я бы использовал класс Assert, который проверяет его. Вы можете сделать свой собственный класс Assert или использовать некоторые из них, которые выполняются, например Spring Assert и так далее.

Ваш Assert класс будет что-то вроде этого:

public Assert { 

    public static void isNotNull(Object object) { 
      if(null == object) 
       throw new IllegalArgumentException(".. must be not null"); 
    } 

    .... 

} 

Тогда вам просто нужно использовать его как то (это уменьшает код boilperplate на одну строку):

Assert.isNotNull(parameter); 

Answer 4

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