Нужна ли проверка на null перед instanceof в Java?

Краткий ответ: проверка на null перед instanceof не нужна

Оператор instanceof в Java безопасно обрабатывает null. Если левый операнд равен null, оператор всегда возвращает false, не выбрасывая NullPointerException. Это чётко прописано в спецификации языка Java (§15.20.2):

"Если значение ReferenceType равно null, то результат оператора instanceof равен false."

Таким образом, вы можете смело писать:

if (obj instanceof String) {
    String str = (String) obj; // безопасно
}

Почему многие разработчики всё равно проверяют null?

Существует несколько причин, по которым вы можете встретить двойную проверку:

• Привычка с других языков (C#, C++) — в некоторых языках оператор проверки типа может выбросить исключение при null.
• Смешение с приведением типов — если после instanceof идёт явное приведение, разработчик может перестраховаться, хотя instanceof уже гарантирует, что объект не null.
• Устаревший код — в очень старых версиях Java (до JDK 1.0.2) поведение могло отличаться, но это уже неактуально.
• Слепая копипаста — разработчики копируют шаблон "null check + instanceof", не задумываясь.

Когда проверка null всё же может быть полезна?

Хотя для самого instanceof проверка не нужна, есть сценарии, где она делает код понятнее:

1. Явное документирование намерения — если вы пишете код для новичков, явная проверка может служить подсказкой: "здесь ожидается не-null значение".
2. Логическая группировка — когда вы хотите выполнить разные действия для null и для конкретного типа:

if (obj == null) {
    // обработать null
} else if (obj instanceof String) {
    // обработать String
}

Рекомендация

В современной Java (8+) пишите просто if (obj instanceof String) без дополнительной проверки на null. Это лаконично, безопасно и соответствует официальной документации. Если вы используете Java 16+, можно пойти ещё дальше и применить pattern matching for instanceof:

if (obj instanceof String str) {
    // str уже приведён к String, null не пройдёт
    System.out.println(str.length());
}

Эта конструкция автоматически проверяет и null, и тип, избавляя вас от лишнего кода.

// Исходный код
if (obj instanceof String) {
    processString((String) obj);
}

// После рефакторинга кто-то меняет условие:
if (obj != null && obj.getClass() == String.class) { // явная проверка типа
    processString((String) obj);
}