Pro/вести: Инициализация переменной в условном операторе

Что такое NullPointerException?

Хорошим местом для начала является JavaDocs . Они охватывают это:

Брошено, когда приложение пытается использовать null в случае, когда требуется объект. К ним относятся:

  • Вызов метода экземпляра нулевого объекта.
  • Доступ или изменение поля нулевого объекта.
  • Выполнение длины null, как если бы это был массив.
  • Доступ или изменение слотов с нулевым значением, как если бы это был массив.
  • Бросать нуль, как если бы это было значение Throwable.

Приложения должны бросать экземпляры этого класса для указания других незаконных видов использования нулевого объекта.

blockquote>

Также, если вы попытаетесь использовать нулевую ссылку с synchronized, который также выдаст это исключение, за JLS :

SynchronizedStatement:
    synchronized ( Expression ) Block
  • В противном случае, если значение выражения равно null, NullPointerException.
blockquote>

Как это исправить?

Итак, у вас есть NullPointerException. Как вы это исправите? Возьмем простой пример, который выдает NullPointerException:

public class Printer {
    private String name;

    public void setName(String name) {
        this.name = name;
    }

    public void print() {
        printString(name);
    }

    private void printString(String s) {
        System.out.println(s + " (" + s.length() + ")");
    }

    public static void main(String[] args) {
        Printer printer = new Printer();
        printer.print();
    }
}

Идентифицирует нулевые значения

. Первый шаг - точно определить , значения которого вызывают исключение . Для этого нам нужно выполнить некоторую отладку. Важно научиться читать stacktrace . Это покажет вам, где было выбрано исключение:

Exception in thread "main" java.lang.NullPointerException
    at Printer.printString(Printer.java:13)
    at Printer.print(Printer.java:9)
    at Printer.main(Printer.java:19)

Здесь мы видим, что исключение выбрано в строке 13 (в методе printString). Посмотрите на строку и проверьте, какие значения равны нулю, добавив протоколирующие операторы или используя отладчик . Мы обнаруживаем, что s имеет значение null, а вызов метода length на него вызывает исключение. Мы видим, что программа перестает бросать исключение, когда s.length() удаляется из метода.

Трассировка, где эти значения взяты из

Затем проверьте, откуда это значение. Следуя вызовам метода, мы видим, что s передается с printString(name) в методе print(), а this.name - null.

Трассировка, где эти значения должны быть установлены

Где установлен this.name? В методе setName(String). С некоторой дополнительной отладкой мы видим, что этот метод вообще не вызывается. Если этот метод был вызван, обязательно проверьте порядок , что эти методы вызывают, а метод set не будет называться после методом печати. ​​

Этого достаточно, чтобы дать нам решение: добавить вызов printer.setName() перед вызовом printer.print().

Другие исправления

Переменная может иметь значение по умолчанию setName может помешать ему установить значение null):

private String name = "";

Либо метод print, либо printString может проверить значение null например:

printString((name == null) ? "" : name);

Или вы можете создать класс, чтобы name всегда имел ненулевое значение :

public class Printer {
    private final String name;

    public Printer(String name) {
        this.name = Objects.requireNonNull(name);
    }

    public void print() {
        printString(name);
    }

    private void printString(String s) {
        System.out.println(s + " (" + s.length() + ")");
    }

    public static void main(String[] args) {
        Printer printer = new Printer("123");
        printer.print();
    }
}

См. также:

Я все еще не могу найти проблему

Если вы попытались отладить проблему и до сих пор не имеете решения, вы можете отправить вопрос для получения дополнительной справки, но не забудьте включить то, что вы пробовали до сих пор. Как минимум, включите stacktrace в вопрос и отметьте важные номера строк в коде. Также попробуйте сначала упростить код (см. SSCCE ).

16
задан Community 23 May 2017 в 10:31
поделиться

10 ответов

Важная вещь состоит в том, что объявление в C++ не является выражением.

bool a = (CThing* pThing = GetThing()); // not legit!!

Вы не можете сделать и объявления и булевой логики в, если оператор, спецификация языка C++ конкретно позволяет или выражение или объявление.

if(A *a = new A)
{
    // this is legit and a is scoped here
}

, Как мы можем знать ли определенного между одним термином и другим в выражении?

if((A *a = new A) && a->test())
{
    // was a really declared before a->test?
}

Стиснули зубы и используют внутреннее если. Правила объема полезны, и Ваша логика является явной:

if (CThing* pThing = GetThing())
{
    if(pThing->IsReallySomeThing())
    {
    }
}
19
ответ дан 30 November 2019 в 17:05
поделиться

О преимуществах:

всегда рекомендуется определить переменные при первой необходимости в них, не строке прежде. Это для улучшенной удобочитаемости Вашего кода, так как можно сказать то, что CThing, не прокручивая и ища, где это было определено.

Также уменьшающий объем к блоку цикла/если, заставляет переменную быть не имеющей ссылки после выполнения блока кода, который делает его кандидатом на Сборку "мусора" (если язык поддерживает эту функцию).

4
ответ дан 30 November 2019 в 17:05
поделиться

Одна причина, которую я обычно не делаю, который является из-за общей ошибки от пропущенного '=' в условном тесте. Я использую линт с набором ошибки/предупреждений для ловли тех. Это будет тогда вопить обо всех присвоениях в условных выражениях.

2
ответ дан 30 November 2019 в 17:05
поделиться

Просто к вашему сведению часть более старого Microsoft C ++ компиляторы (Visual Studio 6, и.NET 2003 я думаю) не вполне следует правилу обзора в некоторых случаях.

for(int i = 0; i > 20; i++) {
     // some code
}

cout << i << endl;

я должен быть вне объема, но это было допустимым кодом. Я полагаю, что это игралось прочь как функция, но по-моему это - просто не соответствие. Соблюдение стандартов плохо. Так же, как веб-разработчик о IE и Firefox.

Может кто-то с VS проверять и видеть, допустимо ли это все еще?

2
ответ дан 30 November 2019 в 17:05
поделиться

Этот <забастовка> должен не работает в C++ <забастовка> с тех пор даже при том, что это поддерживает срывающая оценка . <забастовка>, Возможно не пробует следующее:

if ((CThing* pThing = GetThing()) && (pThing->IsReallySomeThing()))
{
}

допускают ошибку.. см. ответ Wesley Tarle

2
ответ дан 30 November 2019 в 17:05
поделиться

Можно также включить присвоение в дополнительный набор () для предотвращения предупреждающего сообщения.

0
ответ дан 30 November 2019 в 17:05
поделиться

Я рассматриваю что как довольно опасного. Код ниже намного более безопасен, и фигурные скобки включения все еще ограничат объем pThing в способе, которым Вы хотите.

я предполагаю, что GetThing () иногда возвращает ПУСТОЙ УКАЗАТЕЛЬ, который является, почему я вставил тот забавный пункт если () оператор. Это предотвращает IsReallySomething () называемый на Нулевом указателе.

{
    CThing *pThing = GetThing();
    if(pThing ? pThing->IsReallySomeThing() : false)
    {
    // Do whatever
    }
}
0
ответ дан 30 November 2019 в 17:05
поделиться

также заметьте, что, если Вы пишете код C++, Вы хотите сделать предупреждение компилятора о "=" в условном операторе (который не является частью объявления), ошибка.

0
ответ дан 30 November 2019 в 17:05
поделиться

Это - приемлемая и хорошая практика кодирования. Однако люди, которые не происходят из низкоуровневой среды кодирования, вероятно, не согласились бы.

0
ответ дан 30 November 2019 в 17:05
поделиться
if (CThing* pThing = GetThing())

Это плохой стиль , так как внутри if вы не предоставляете булевого выражения. Вы предоставляете CThing*.

CThing* pThing = GetThing();
if (pThing != NULL)

Это хороший стиль.

3
ответ дан 30 November 2019 в 17:05
поделиться
Другие вопросы по тегам:

Похожие вопросы: