Анализатор обнаружил фрагмент кода, в котором может произойти автоматическая распаковка null-значения, что приведёт к возникновению 'NullPointerException'.
Данную ошибку достаточно часто можно встретить внутри операторов сравнения. Например, 'Boolean' можно использовать как флаг с тремя значениями: ложным, истинным и неустановленным. Тогда при проверке того, что какой-либо флаг явно выставлен в нужное значение, можно написать:
public void doSomething() { Boolean debugEnabled = isDebugEnabled(); if (debugEnabled == true) { ... } }
Однако, при сравнении примитива с упакованным значением всегда происходит автоматическая распаковка, что и вызывает 'NullPointerException'. Исправить данный фрагмент можно несколькими способами:
public void doSomething() { Boolean debugEnabled = isDebugEnabled(); if (debugEnabled != null && debugEnabled == true) { ... } // or if (Objects.equals(debugEnabled, true)) { ... } }
В отличие от большинства операторов, тернарный оператор позволяет смешивать в выражении примитивы и обёртки, производя автоматическую упаковку при приведении к общему типу. Это позволяет достаточно легко сделать опечатку:
boolean x = httpRequest.getAttribute("DEBUG_ENABLED") != null ? (boolean) httpRequest.getAttribute("DEBUG_ENABLED") : null;
Здесь выражение тернарного оператора имеет тип 'Boolean' как общий для его операндов, после чего распаковывается обратно в примитив при присвоении в переменную 'x'. После исправления:
boolean x = httpRequest.getAttribute("DEBUG_ENABLED") != null ? (boolean) httpRequest.getAttribute("DEBUG_ENABLED") : false;
Данная диагностика классифицируется как:
|