Какова правильная позиция литералов в сравнении строк?

Question

У меня

if (localName.equals("TaxName")) {

, но PMD говорит

Position literals first in String comparisons

Answer 1

"TaxName".equals(localName) лучше, как если бы localName было нулевым, вы не получите исключение нулевого указателя.

Answer 2

PMD также должен сообщать вам , почему генерирует это предупреждение. С документация по правилам на сайте PMD:

Поместите литералы первыми в сравнениях String - таким образом, если String равен нулю, вы не получите NullPointerException, он просто вернет false.

Answer 3

Я предпочитаю сначала размещать литералы, то есть:

if ("TaxName".equals(localName)) { ...

Таким образом, вы получаете правильное сравнение для случая null вместо получения NullPointerException.

Answer 4

Лично для меня это не имеет смысла. Если код перехватывает исключение NullPointerException, значит, он завершил работу, которую вам не придется делать позже. Если localName окажется нулевым, и это позже вызовет проблему, отследить его будет сложнее. Не меняйте код, чтобы сделать компилятор счастливым. Если ваш код генерирует исключение NullPointerException, то он сохраняется, и вы откладываете время позже.

Answer 5

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

if (localName!=null && localName.equals("TaxName")) {
    ...
}