Помогите Сборник багов спигота

1)Делаю плагин, в котором есть GUI. Делаю с помощью холдера. То есть у меня есть главный класс GUI Display он имплементирует InventoryHolder. Я создаю другие классы наследующие класс Display. В InventoryClickEvent делаю нечто следующее

PHP:

@EventHandler
    public void click(InventoryClickEvent event){
        InventoryHolder holder = event.getClickedInventory().getHolder();
        if(event.getSlotType() == InventoryType.SlotType.OUTSIDE) return;
        if(holder != null) {
            if (holder instanceof Display) {
                event.setCancelled(true);
                ((Display)holder).click(event);
            }
        }
    }

И когда я кликаю в область где нет инвентаря

Спойлер: Область в которой я кликаю(помечена красным)

Мне вылетает NPE и жалуется на строчку: InventoryHolder holder = event.getClickedInventory().getHolder();
2) Делаю переключатель. Нажимаешь и появляется следующая функция. Делаю так.
Есть массив с именами функций. Есть переменная текущего индекса. Когда я нажимаю на кнопку, индекс увеличивается и имя кнопки меняется. Вчера ещё работало. А сегодня нажимаю ничего не происходит. Помогите решить эти 2 проблемы.

 

UP

 

Причем тут баги спигота, ты проверку на OUTSIDE делаешь после получения холдера, npe вылетает, т. к. твой getClickedInventory() == null.

Получай лучше не clickedInventory, а Inventory и даже если в нижнем инвентаре или вне клик будет, то вернет тот инвентарь, что сверху и отменяй эвент, если холдер твой. Ну а метод click свой запускай только если rawSlot < inventory.getSize().

Ну и убери проверку холдера на нулл, null instanceof всегда false.

 

Нормально скажи или лучше код скидывай, посмотрим шо за хрень ты там настрочил.

 

Я outside поэтому и добавил, что у меня NPE вылетал.

 

Код:

public void render(){
        btn = new ItemButton(new Item(Material.STONE).setName("§aФорма блока: "+names[current_index]),event -> {
            try {
                if(current_index < names.length) {
                    current_index++;
                } else {
                    current_index=0;
                }
                addComponent(event.getSlot() + 1, btn);
                open((Player) event.getWhoClicked());
            } catch (ArrayIndexOutOfBoundsException e){
                e.printStackTrace();
            }
        });
        addComponent(12,btn);
    }

 

Глупый что ли, у тебя NPE вылетает до проверки, делай как я написал.

Что за костыли, нормально напиши, что ты хочешь сделать в итоге.

 

Нажимаешь на кнопку, меняется функция, нажимаешь ещё раз, ещё раз меняется.

 

Ну так делай абстракцию состояния того, что меняется и се, а не костыль с массивом.

 

Я, к сожалению, говнокодер. И не владею программистскими терминами.

 

Расплывчатые требования. У меня есть вот такая хренотень.

 

Мне вот такие ровно кнопки нужны.

 

Ну ты понимаешь, что у тебя массив в котором имя предмета хранится и меняется это костыль и при малейшем изменении требований все переписывать нужно? Используй функциональные интерфейсы и абстракция поведение предмета при клике с разными реализациями. Одинаковый для всех, для конкретного игрока и т. п.

 

Спойлер: Всё что я понял.

Почему?

Как это?

Какие?

Что это значит?

Мне бы лит-ру какую-нибудь. Для осмысления.

 

Ясно все с тобой, иди учи для начала java и ооп.
Литература schildt java 8
И докуменатацию jdk тоже почитай.
Ну и если сложно букавки читать параллельно можешь смотреть видео этого челика, для новичков сойдет:
https://www.youtube.com/playlist?list=PL786bPIlqEjRDXpAKYbzpdTaOYsWyjtCX

 

Ну и посмотри как пример исходники Sightmenu - https://github.com/hyndor/SightMenu

 

Спасибо.

 

Подходит да?
https://github.com/alexandrage/Example/tree/master/src/Example/node

 

Немного почитав, я могу предположить, что делать абстракцию состояния того, что меняется, значит сделать так называемый "чертёж" этого состояния?

 

Че?)