If (value == null || value.equals(null)))?

Question

[kidar2]

Увидел в чужом коде такое странное условие:

static String valueToString(Object value) throws JSONException {
        if (value == null || value.equals(null)) {
            return "null";
        }
}

Как-то может сработать второе условие без первого?

Comments

[Евгений]

А Вы сами ка думаете? :) Я ставлю на то, что не может, ибо в случае класса Object эти проверки идентичны.

[kidar2]

Я думаю, что если такое условие value.equals(null) - истинно, то оно должно падать с NullPointerException. Нет?

[SaertiS]

kidar2: Ну по спеке истинно оно никогда не будет, разве только вы его так переопределите, иквалс нулл для объекта всегда возвращает фолс. От NPE защищает первая проверка.

[Евгений]

kidar2: Т.е. при приведении значения к строке, мы должны выкинуть NPE, если строка == null? Или я Вас не правильно понимаю?

[SaertiS]

kidar2: судя по ошибке в сигнатуре, скорей всего в иквалс json объекта проверяется пустая строка, и тогда возвращается нулл. В общем смотрите код класса который передают этому методу.

[SaertiS]

Евгений: мне кажется человек имел ввиду что логичная на первый взгляд ситуация когда в качестве объекта передадут null и вызов value.equals(null) должен вызвать NPE никогда не произойдет. Вот только человек забыл про overriding )

[Евгений]

SaertiS:
Ну NPE тут не возможен, как Вы правильно заметили, именно из-за перврй проверки. А вот если заменить ИЛИ на И, то вот он NPE во всей своей красе :)

Answer 1

[Виктор Гогильчин]

Для определенного объекта может быть переопределен метод equals().
Например, по задачам бизнес-логики объект считается null если его "самоеГлавноеПоле" есть null.

В данном же случе - либо старый неисправленный код (раньше параметром приходил не Object а что-то другое) либо банальная ошибка.

А вообще, в случае с пользовательским классом второе условие без первого может сработать.
Например:

package tk.lslayer.temp;

public class NullComparator {
    public Integer someField = null;

    @Override
    public boolean equals(Object obj) {
        if (obj == null && someField == null) return true;
        if (obj != null && obj instanceof NullComparator) {
            return ((NullComparator) obj).someField == someField ? 
                true : false;
        } else return false;
    }

    public static void main(String[] args) {
        NullComparator comparator1 = new NullComparator();
        
        if (comparator1 == null)
            System.out.println("!= null");
        if (comparator1.equals(null)) 
            System.out.println(".equals(null)");
    }
}

Answer 2

[Moxa]

я думаю, что тут специальная магия для обработчки json'a.. предположим, что value - некий JsonItem, который хранит в себе значение, это значение может быть null'ом, для этого и сделана вторая проверка