Problem z moim pluginem

[masterolos09]

http://pastebin.com/0UuQSZej

at pl.bum.krysztal.listeners.BlockPlaceListenerKrysztal.onBlockPlaceListsner(BlockPlaceListenerKrysztal.java:21) ~[?:?]

Pomoże ktoś?

Linijka 21 w kodzie:

     if((e.getPlayer().getItemInHand().getItemMeta().hasDisplayName()) && e.getPlayer().getItemInHand().getItemMeta().getDisplayName().equalsIgnoreCase(ChatUtil.fixColors(Settings.name))) {
       KrysztalUtil.getPlacedKrysztal().add(e.getBlock().getLocation());

[MarcinWieczorek]

Problem występuje pewnie przy stawianiu bloku który w ogóle nie ma displayname ani lore itp, czyli nie ma ItemMeta. Tak samo jak hasDisplayName musisz dać sprawdzenie hasItemMeta

[ahilles1999]

if(e.getPlayer().getItemInHand().hasItemMeta() && e.getPlayer().getItemInHand().getItemMeta().hasDisplayName())
            if(e.getPlayer().getItemInHand().getItemMeta().getDisplayName().equalsIgnoreCase(ChatUtil.fixColors(Settings.name)))
                KrysztalUtil.getPlacedKrysztal().add(e.getBlock().getLocation());

Proszę bardzo

[MarcinWieczorek]

A po kiego grzyba ten drugi if? Przecież mamy &&

[ahilles1999]

A po kiego grzyba ten drugi if? Przecież mamy &&

 

Jak kto lubi

[masterolos09]

if(e.getPlayer().getItemInHand().hasItemMeta() && e.getPlayer().getItemInHand().getItemMeta().hasDisplayName())
            if(e.getPlayer().getItemInHand().getItemMeta().getDisplayName().equalsIgnoreCase(ChatUtil.fixColors(Settings.name)))
                KrysztalUtil.getPlacedKrysztal().add(e.getBlock().getLocation());

Proszę bardzo

 

Ok, dziekuje bardzo, sprawdze i dam znac co i jak :P

[MarcinWieczorek]

 

A po kiego grzyba ten drugi if? Przecież mamy &&

 

Jak kto lubi

 

Przyzwoitość nakazuje stosować się do pewnych standardów. Popełniasz błąd i rozpowszechniasz go niepotrzebnie.

[masterolos09]

Zmienilem drugi if na &&, na razie nie ma bledu w logach.
Dziekuje,
Do zamknięcia

[ahilles1999]

 

 

A po kiego grzyba ten drugi if? Przecież mamy &&

 

Jak kto lubi

 

Przyzwoitość nakazuje stosować się do pewnych standardów. Popełniasz błąd i rozpowszechniasz go niepotrzebnie.

 

 

Skoro mamy się stosować do tych standardów to - http://www.oracle.com/technetwork/java/javase/documentation/codeconventions-136091.html#248

Dlaczego sam się nie stosujesz do tego? (NovaGuilds) Popełniasz błąd i rozpowszechniasz go niepotrzebnie.

[GotoFinal]

 

 

A po kiego grzyba ten drugi if? Przecież mamy &&

 

Jak kto lubi

 

Przyzwoitość nakazuje stosować się do pewnych standardów. Popełniasz błąd i rozpowszechniasz go niepotrzebnie.

 

ale ten if to wgl rak...

 

Nawet klamerek nie ma :<

        ItemStack item = e.getPlayer().getItemInHand();
        if (item == null || ! item.hasItemMeta())
        {
            return;
        }
        ItemMeta itemMeta = item.getItemMeta();
        if (itemMeta.hasDisplayName() && itemMeta.getDisplayName().equalsIgnoreCase(ChatUtil.fixColors(Settings.name)))
        {
            KrysztalUtil.getPlacedKrysztal().add(e.getBlock().getLocation());
        }

lub 

        ItemStack item = e.getPlayer().getItemInHand();
        if ((item != null) && item.hasItemMeta())
        {
            ItemMeta itemMeta = item.getItemMeta();
            if (itemMeta.hasDisplayName() && itemMeta.getDisplayName().equalsIgnoreCase(ChatUtil.fixColors(Settings.name)))
            {
                KrysztalUtil.getPlacedKrysztal().add(e.getBlock().getLocation());
            }
        }

łatwiej rozwijać, edytować, szukać będów

[masterolos09]

 

 

 

A po kiego grzyba ten drugi if? Przecież mamy &&

 

Jak kto lubi

 

Przyzwoitość nakazuje stosować się do pewnych standardów. Popełniasz błąd i rozpowszechniasz go niepotrzebnie.

 

ale ten if to wgl rak...

 

Nawet klamerek nie ma :<

        ItemStack item = e.getPlayer().getItemInHand();
        if (item == null || ! item.hasItemMeta())
        {
            return;
        }
        ItemMeta itemMeta = item.getItemMeta();
        if (itemMeta.hasDisplayName() && itemMeta.getDisplayName().equalsIgnoreCase(ChatUtil.fixColors(Settings.name)))
        {
            KrysztalUtil.getPlacedKrysztal().add(e.getBlock().getLocation());
        }

lub 

        ItemStack item = e.getPlayer().getItemInHand();
        if ((item != null) && item.hasItemMeta())
        {
            ItemMeta itemMeta = item.getItemMeta();
            if (itemMeta.hasDisplayName() && itemMeta.getDisplayName().equalsIgnoreCase(ChatUtil.fixColors(Settings.name)))
            {
                KrysztalUtil.getPlacedKrysztal().add(e.getBlock().getLocation());
            }
        }

łatwiej rozwijać, edytować, szukać będów

 

Dziekuję, poprawiłem

Wszystko działa bez błędu

Do zamknięcia.