Почему появляется InvocationTargetException?

Question

[Koshkasobaka]

Здравствуйте, у меня есть класс SudokuGenerator, в конструктор которому среди прочего я передаю экземпляр класса SudokuStorage. В методе fillStorage() я создаю экземпляр Sudoku и передаю его в метод sudokuStorage.add(Sudoku), чтобы тот положил его в Mapy. Но дело в том, что любое использование storage в методе add() вызывает InvocationTargetException. Без storage все работает.. что я делаю не так?

public class SudokuGenerator {

    private BoardFactory boardFactory;
    private CellHider cellHider;
    private Sudokustorage sudokuStorage;
// тут геттеры-сеттеры

    public SudokuGenerator(BoardFactory boardFactory, CellHider cellHider, SudokuStorage sudokuStorage) {
        this.boardFactory = boardFactory;
        this.cellHider = cellHider;
        this.sudokuStorage = sudokuStorage;
    }


    public void fillStorage() {
       for (int i = 0; i < 100; i++) {
            Sudoku sudoku = new Sudoku(cellHider.makeBoardWithHiddenCells(boardFactory.generateBoard()));
            sudokuStorage.add(sudoku);
        }
    }
}




public class SudokuStorage {
    private HashMap<Level, ArrayList<Sudoku>> storage;

   public HashMap<Level, ArrayList<Sudoku>> getStorage() {
       return storage;
    }

    public void setStorage() { this.storage = storage; }

    public SudokuStorage() {
        HashMap<Level, ArrayList<Sudoku>> storage = new HashMap<>();
    }


    public void add(Sudoku sudoku) {
        if (storage.containsKey(sudoku.getLevel())) {             
            ArrayList <Sudoku> list = storage.get(sudoku.getLevel());
            list.add(sudoku);
            storage.put(sudoku.getLevel(), list);
        } else {
            ArrayList<Sudoku> list = new ArrayList<>();
            list.add(sudoku);
            storage.put(sudoku.getLevel(), list);
        } 
    

 
    }
}

Comments

[Koshkasobaka]

Bavashi, извините за невнимательность, исправила (случайно перименовала название метода когда переносила код)

Answer 1

[Dmitry Roo]

Во-первых, вы бы приложили стэк-трейс с ошибкой. Было бы понятно что там у вас происходит.
Во-вторых, в конструкторе у вас что-то непонтяное происходит: зачем-то создается и инициализируется локальная переменная storage. Если вы хотите инициализировать переменную класса, лучше сразу так и писать:

private HashMap<Level, ArrayList<Sudoku>> storage = new HashMap<>();

Comments

[Орхан Гасанлы]

Либо использовать блок динамической инициализации...

[Koshkasobaka]

Орхан Гасанлы, вот что PrintStackTrace выдал:
java.lang.NullPointerException
at com.example.sudoku.logic.SudokuStorage.add(SudokuStorage.java:27)
at com.example.sudoku.logic.SudokuGenerator.fillStorage(SudokuGenerator.java:55)
at com.example.sudoku.SudokuApp.main(SudokuApp.java:36)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at com.sun.javafx.application.LauncherImpl.launchApplicationWithArgs(LauncherImpl.java:389)
at com.sun.javafx.application.LauncherImpl.launchApplication(LauncherImpl.java:328)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at sun.launcher.LauncherHelper$FXHelper.main(LauncherHelper.java:767)

[Dmitry Roo]

Koshkasobaka, ну я и говорю - у вас не инициализирован storage.
Когда вы к нему обращаетесь у вас NullPointerException.

[Koshkasobaka]

Dmitry Roo, понятно, спасибо