В чем разница между двумя кодами ниже

Question

Вот две реализации одной и той же функции. Первый, написанный мной, и второй, я наткнулся на http://developer.android.com/guide/topics/location/strategies.html.

**1st code:** 
private boolean isSameProvider(String provider1, String provider2) { 
    if (provider1.equals(provider2)) { 
     return true; 
    } 
    else return false; 
} 

**2nd code:** 
private boolean isSameProvider(String provider1, String provider2) { 
    if (provider1 == null) { 
     return provider2 == null; 
    } 
    return provider1.equals(provider2); 
} 

Answer 1

Первая часть кода может потенциально бросить NullPionterException если параметр provider1 равно нулю. И мы вызываем provider1.equals(provider2) на нулевом объекте.

Вторая часть кода проверяет, является ли параметр provider1 нулевым или нет, если он действительно равен нулю, тогда проверьте, является ли параметр provider2 нулевым.

Если оба поставщика1 и провайдер2 являются нулевыми, они считаются одним и тем же провайдером, и этот метод возвращает true.

Answer 2

Второй код более безопасен для вашего приложения, потому что он проверяет, есть ли provider1 == null.

Howevr, если Provider1 и provider2 имеют тенденцию быть пустым в приложении, я предпочел бы написать код вроде этого:

private boolean isSameProvider(String provider1, String provider2) { 
    if (provider1 != null && provider2 != null) { 
     return provider1.equals(provider2); 
    } 
    return false; 
}