Как исправить NullPointerException?

Question

[dc65k]

Всем привет, как в следующем примере исправить NullPointerException?

// Exception in thread "main" java.lang.NullPointerException
route = mapFrom.get(route.getTo()).get(0);

package main.java;

import java.util.ArrayList;
import java.util.List;
import java.util.Map;
import java.util.Objects;
import java.util.function.Function;
import java.util.stream.Collectors;

enum Properties {
    FROM, TO,
}

public class Solution {

    public Map<String, List<Ticket>> fromEntries(List<Ticket> tickets, Properties prop) {

        /*
        return tickets.stream()
                .collect(Collectors.groupingBy(Ticket::getFrom, Collectors.mapping(Function.identity(), Collectors.toList())));

         */

        Function<Ticket, String> method = null;

        if (prop == Properties.FROM) {
            method = Ticket::getFrom;
        } else if (prop == Properties.TO) {
            method = Ticket::getTo;
        }

        return tickets.stream()
                .collect(Collectors.groupingBy(Objects.requireNonNull(method),
                        Collectors.mapping(Function.identity(), Collectors.toList())));
    }

    public List<Ticket> sortingTickets(List<Ticket> tickets) {

        Map<String, List<Ticket>> mapFrom = fromEntries(tickets, Properties.FROM);
        System.out.println("mapFrom " + mapFrom);

        Map<String, List<Ticket>> mapTo = fromEntries(tickets, Properties.TO);
        System.out.println("mapTo " + mapTo);

        List<Ticket> sorted = new ArrayList<>();

        List<Ticket> routeList = tickets.stream().filter(element -> !mapTo.containsKey(element.getFrom())).collect(Collectors.toList());
        // System.out.println("route " + route); // [Ticket{from='Череповец', to='Москва'}]
        Ticket route = routeList.get(0);
        System.out.println(route); // Ticket{from='Череповец', to='Москва'}

        while (route != null) {
            System.out.println("route " + route);
            /*
                route Ticket{from='Череповец', to='Москва'}
                route Ticket{from='Москва', to='С.Петербург'}
                route Ticket{from='С.Петербург', to='Минск'}
                route Ticket{from='Минск', to='Киев'}
                route Ticket{from='Киев', to='Новосибирск'}
             */

            sorted.add(route);

            // Exception in thread "main" java.lang.NullPointerException
            route = mapFrom.get(route.getTo()).get(0);
        }

        System.out.println("sorted " + sorted);

//        return sorted;

        return tickets;
    }
}

class Ticket {

    private String from;

    private String to;

    public Ticket(String from, String to) {
        this.from = from;
        this.to = to;
    }

    public String getFrom() {
        return from;
    }

    public void setFrom(String from) {
        this.from = from;
    }

    public String getTo() {
        return to;
    }

    public void setTo(String to) {
        this.to = to;
    }

    @Override
    public String toString() {
        return "Ticket{" +
                "from='" + from + '\'' +
                ", to='" + to + '\'' +
                '}';
    }
}

class Main {

    public static void main(String[] args) {

        List<Ticket> tickets = new ArrayList<>();

        Ticket ticket1 = new Ticket("С.Петербург", "Минск");
        Ticket ticket2 = new Ticket("Киев", "Новосибирск");
        Ticket ticket3 = new Ticket("Череповец", "Москва");
        Ticket ticket4 = new Ticket("Минск", "Киев");
        Ticket ticket5 = new Ticket("Москва", "С.Петербург");

        tickets.add(ticket1);
        tickets.add(ticket2);
        tickets.add(ticket3);
        tickets.add(ticket4);
        tickets.add(ticket5);

        System.out.println(new Solution().sortingTickets(tickets));
    }
}

Comments

[Sand]

Это всё у Вас в одном файле?

[dc65k]

Пример в вопросе, да.

[Jacen11]

dc65k, не надо так делать. Тем более там подразумевается public везде, а с ним ничего даже не соберется при такой реализации

Answer 1

[User700]

Значит на некотором этапе в mapFrom не оказывается элемента с ключём route.getTo(). А .get(0) нельзя взять от null. Попробуйте перейти вместо Map'а List'ов к multimap, когда одному ключу может несколько значений соотв. Или проверяйте и вызодите из цикла когда mapFrom.getTo вернёт null.

Comments

[сергей кузьмин]

добавьте услови выхода из цикла

String fromKey = route.getTo();
			if (mapFrom.containsKey(fromKey))
				route = mapFrom.get(fromKey).get(0);
			else
				route = null;

[dc65k]

Ниже с условием рабочий вариант. Возможно есть более лучшее решение?

while (route != null) {

    sorted.add(route);

    if (mapFrom.get(route.getTo()) == null) {
        break;
    }

    route = mapFrom.get(route.getTo()).get(0);
}

[Jacen11]

dc65k,

while (route != null) {

    sorted.add(route);

    if (mapFrom.get(route.getTo()) != null) {
       route = mapFrom.get(route.getTo()).get(0);
    }
}

но это сработает если mapFrom не null, можно еще в трай кетч обернуть

[сергей кузьмин]

почему не использовать containsKey ?

Answer 2

[Jacen11]

Exception in thread "main" java.lang.NullPointerException

писать нудные цепочки типа if((route.getTo()!=null && mapFrom.get(route.getTo())!=null )
Можно еще опшинал использовать.

если думаешь что нигде там не может быть нала, значит нужно дебажить. Ну и там пишет где именно вылетает