Нужна ли проверка Map на null?

Question

[Дмитрий Быстров]

Добрый день!

У меня есть некая функция:

public Map<String, Object> getTestMap(MyClass myObject) {
    Map<String, Object> myResultMap = new HashMap<String, Object>();
    myResultMap.put("myKey", myObject.getMyValue());

    return myResultMap;
}

и следующая строка кода:

Map<String, Object> resultMap = getTestMap(myObject);

Подскажите, стоит ли в дальнейшем проверять resultMap на null, когда я захочу получить значения по ключу?
Например так:

Map<String, Object> resultMap = getTestMap(myObject);
if (resultMap != null) {
    Object myObject = resultMap.myKey;
}

Comments

[GavriKos]

Если код такой, как вы привели, то при нормальном поведении Null-а там не будет.
Возможен правда вариант, что new не сможет аллоцировать память (если она закончилась), но я не знаю что в таком случае выдаст джава.

Answer 1

[AndrewStr]

1. Вы неправильно извлекаете значение из map по ключу, надо использовать метод get -> resultMap.get(myKey);
2. Проверка на null при обращении к resultMap не нужна, поскольку вы создаете пустую мепу в методе getTestMap и resultMap никогда не null;
3. Проверка на null необходима в методе getTestMap, потому что вы вызываете myObject.getMyValue() без проверки на null, вот здесь NullPointerException может возникнуть, если метод будет вызван с параметром значение которого равно null.

Comments

[davidnum95]

4. Объявить Map resultMap как final, чтобы обезопасить себя от искушения присвоить null

Answer 2

[Денис Загаевский]

Для того, чтобы не возникало таких вопросов, рекомендую все методы, аргументы и поля помечать @NonNull и @Nullable.
Все поля и локальные переменные, которые по логике не изменяются - final.
Объявление переменных максимально близко к месту их использования.

Ответ на вопрос - нет, в данном случае проверять не надо. А вот если метод библиотечный и не помечен аннотацией - надо.

Ещё лучше взять Kotlin и не мучаться:)