Как исправить java.lang.NullPointerExeption?

Question

[Антон]

Вот код:

try {
            DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance();
            DocumentBuilder db = dbf.newDocumentBuilder();
            final Document doc = db.parse(new File("config.xml"));

            for (Node n = doc.getFirstChild(); n != null; n = n.getNextSibling()) {
                for (Node d = n.getFirstChild(); d != null; d = d.getNextSibling()) {
                    for (Node m = d.getFirstChild(); m != null; m = m.getNextSibling()) {

                        //изменить по возможности этот кусок кода.
                         String stringValue;
                        if ("variable".equals(m.getNodeName())) {
                            stringValue = m.getAttributes().getNamedItem("storiesNumber").getNodeValue().trim();
                            System.out.println(stringValue);
                            stringValue = m.getAttributes().getNamedItem("elevatorCapacity").getNodeValue().trim();
                            System.out.println(stringValue);

                        }
                    }
                }
            }

        }
        catch (Exception e) {
            System.out.println (e.getClass().getName() + " with message : " + e.getMessage());
            e.printStackTrace();
        }

Вот вывод:

25
java.lang.NullPointerException with message : null
java.lang.NullPointerException

Вот xml-файл:

<?xml version="1.0" encoding="UTF-8"?>
<Lift>
    <variables>
        <variable storiesNumber = "25"/>
        <variable elevatorCapacity = "30"/>
        <variable passengerNumber = "13"/>
        <variable animationBoost = "2"/>
    </variables>

Я попытался отдебажить, нов по переменным все правильно.

Answer 1

[Евгений]

Ну смотрите, Вы берете variable, и просите у него такие атрибуты как storiesNumber и elevatorCapacity. Т.е. Ваш код ожидает что запись в XML будет вида

<variable storiesNumber = "25" elevatorCapacity = "30"/>

А в Вашем XML атрибуты storiesNumber и elevatorCapacity принадлежат разным variable. Вот и получается, что вы берете первый variable, просите у него атрибут storiesNumber и он есть, затем вы у того же самого variable просите атрибут elevatorCapacity и его нет, вот Вам и NPE.

Comments

[Антон]

А что изменить в коде, что бы она читал мой xml файл?

[Евгений]

Для работы с XML, если структура XML заранее известна, я бы посоветовал Вам использовать JAXB.
Конкретно в Вашем текущем коде нужно пробовать брать у variable один из описанных атрибутов, и когда нашли хотя бы один, переходить к следующему variable

if ("variable".equals(m.getNodeName())) {
    if (m.getAttributes().getNamedItem("storiesNumber").getNodeValue() != null) {
        stringValue = m.getAttributes().getNamedItem("storiesNumber").getNodeValue().trim();
    } else if (m.getAttributes().getNamedItem("elevatorCapacity") != null) {
        stringValue = m.getAttributes().getNamedItem("elevatorCapacity").getNodeValue().trim();
    } // и так далее
    System.out.println(stringValue);
    continue; // для продолжения цикла в котором мы крутимся
}

как-то так

[Антон]

Евгений, опять NPE, но уже на строчке:

if (m.getAttributes().getNamedItem("storiesNumber").getNodeValue() != null)

[Евгений]

Антон Сорокин, Уберите .getNodeValue(), т.е. должно быть так

if (m.getAttributes().getNamedItem("storiesNumber") != null)

Вы понимаете почему так?

Answer 2

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

Посмотреть по стектрейсу, где null, и понять, почему он там.

Comments

[Антон]

Стектрейс - это: at TLB.main(TLB.java:27)?
Сама 27ая -

stringValue = m.getAttributes().getNamedItem("elevatorCapacity").getNodeValue().trim();

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

Антон Сорокин, значит getAttributes возвращает null

[Антон]

Денис Загаевский, и что делать?

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

Антон Сорокин, точнее, один из методов на этой строке возвращает null. Что делать, я уже сказал - понять, почему это происходит.