InventoryClickEvent fehler (beim klick ausserhalb des Inventars)

[Sprax]

Hi Leute! Ich habe diese Frage bereits im Nitrado-Forum gestellt aber da konnte mir leider nicht geholfen werden:

Ich code ein Plugin für Spigot 1.8.8. Dabei nutzte ich ein InventoryClickEvent. Nun bekomme ich einen Fehler in der Konsole, wenn ich ausserhalb des Inventars klicke. Das beeinträchtigt nicht das Plugin sieht aber nicht schön aus und nervt!

Spoiler: Mein EventHandler

  @EventHandler
   private void InventoryClick(InventoryClickEvent e) {
     Player p = (Player)e.getWhoClicked();
     
     if(e.getClickedInventory().getName().equals("§eInv")) {
       if(e.getClickedInventory().getItem(4).getType() == Material.RED_ROSE) {
         e.setCancelled(true);
         
         if(e.getCurrentItem().hasItemMeta() && e.getCurrentItem().getItemMeta().hasDisplayName()) {
           if(e.getCurrentItem().getItemMeta().getDisplayName() == "Lava") {
             p.sendMessage("Lava Partikel");
           }
         }
       }
     }
   }

Spoiler: Der Fehler

[17:48:21] [Server thread/ERROR]: Could not pass event InventoryClickEvent to CoinShop v1.0
org.bukkit.event.EventException
   at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:310) ~[spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62) ~[spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:502) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:487) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at net.minecraft.server.v1_8_R3.PlayerConnection.a(PlayerConnection.java:1630) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at net.minecraft.server.v1_8_R3.PacketPlayInWindowClick.a(SourceFile:31) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at net.minecraft.server.v1_8_R3.PacketPlayInWindowClick.a(SourceFile:9) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at net.minecraft.server.v1_8_R3.PlayerConnectionUtils$1.run(SourceFile:13) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_51]
   at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_51]
   at net.minecraft.server.v1_8_R3.SystemUtils.a(SourceFile:44) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at net.minecraft.server.v1_8_R3.MinecraftServer.B(MinecraftServer.java:714) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at net.minecraft.server.v1_8_R3.DedicatedServer.B(DedicatedServer.java:374) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at net.minecraft.server.v1_8_R3.MinecraftServer.A(MinecraftServer.java:653) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at net.minecraft.server.v1_8_R3.MinecraftServer.run(MinecraftServer.java:556) [spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   at java.lang.Thread.run(Thread.java:745) [?:1.8.0_51]
Caused by: java.lang.NullPointerException
   at me.sprax2013.coin_shop.Inventorys.Trails.InventoryClick(Trails.java:97) ~[?:?]
   at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_51]
   at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:1.8.0_51]
   at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_51]
   at java.lang.reflect.Method.invoke(Method.java:497) ~[?:1.8.0_51]
   at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:306) ~[spigot-1.8.8-R0.1-SNAPSHOT-latest.jar:git-Spigot-fdc1440-53fac9f]
   ... 15 more

Ich hoffe einer weiß wie ich den Fehler beheben kann

 

[Victini151]

Auf den ersten Blick: Entweder das getClickedInventory oder das getCurrentItem ist null,
wo liegt denn Zeile 97?

Das das Nitradoforum nicht mit Kompetenz herhalten kann ist vielleicht nicht so verwunderlich, formulier dein Problem auf Englisch, und versuchs an "der Quelle des Übels", dem Spigot-Forum. Dort wirst du rauhe Antworten bekommen, aber am Ende ist das Problem mit einigen Rückschlägen für die Selbstachtung gelöst.

 

[Sprax]

Meine Zeile 97

if(e.getCurrentItem().hasItemMeta() && e.getCurrentItem().getItemMeta().hasDisplayName()) {

 

[Victini151]

Mit ziemlicher Sicherheit ist bei der beschriebenen Vorgehensweise das "current Item" null, was du aber nicht überprüfst

 

[GPSforLEGENDS]

Nun bekomme ich einen Fehler in der Konsole, wenn ich ausserhalb des Inventars klicke

Also wenn kein Inventar offen ist oder wenn du ein anderes Inventar offen hast?

if(e.getCurrentItem().getItemMeta().getDisplayName()=="Lava"){

Wird nie true zurückgeben.

Und ich denke victinis Antwort ist richtig.

 

[Victini151]

if(e.getCurrentItem().getItemMeta().getDisplayName()=="Lava"){

Wird nie true zurückgeben.

Exkurs: Dass es eine schlechte Praxis ist, ist unbestritten.
Es kann aber durchaus true evaluieren, sofern beide Strings durch Literale besetzt sind, das ist sogar eine Vorraussetzung der Spezifikation der JVM, siehe http://docs.oracle.com/javase/specs/jls/se7/html/jls-3.html#jls-3.10.5

TL;DR: Abändern auf equals() - wird einige potentielle Fehlerquellen ausmerzen.

 

[GPSforLEGENDS]

Exkurs: Dass es eine schlechte Praxis ist, ist unbestritten.
Es kann aber durchaus true evaluieren, sofern beide Strings durch Literale besetzt sind, das ist sogar eine Vorraussetzung der Spezifikation der JVM, siehe http://docs.oracle.com/javase/specs/jls/se7/html/jls-3.html#jls-3.10.5

TL;DR: Abändern auf equals() - wird einige potentielle Fehlerquellen ausmerzen.

Ich meinte ja auch, dass es nie true zurückgeben wird, wenn es so ist.
Das es true zurück gibt, wenn auf beiden Seiten auf das gleiche Objekt verwiesen wird, ist mir durchaus bewusst.

 

[Sprax]

Danke für eure bisherige Hilfe aber das Problem ist nicht gelöst :/

Das abfrage ob es null ist etc. habe ich bereits versucht aber hat nicht geholfen. Da ich keine Lust habe alles zu kopieren und die BB-Tags an das Forum anzupassen hägnge ich keinen Beitrag aus dem Nitrado-Forum an wo ich den Code/Fehler etc. gepostet habe ---> Mein Beitrag

 

[Victini151]

(Das nennt man Cross-Posten, an anderen Orten in unserem Internetz ist das sehr verpönt)

In dem verlinkten Code sehe ich keine Überprüfung ob das CurrentInventory null ist.

 

[Sprax]

Ja da es nicht geholfen hat habe ich es wieder rasugenommen

 

[LukBukkit]

Wenn du ein Inv offen hast und daneben klickst, ist das Inventar null, also (inv != null) als If-Abfrage.

 

[GPSforLEGENDS]

if((e.getClickedInventory().getName() == "§eCoinShop §8| §bPets") && !(e.getClickedInventory() == null)) {

Bitte vernünftig Java lernen...
Wechsel beide seiten
Und

e.getClickedInventory().getName() == "§eCoinShop §8| §bPets")

Statt == solltest du equals benutzen