Problem mit Inventories

[SchnellfeuerXD]

Hallo ich hatte heute schon mal einen Beitrag erstellt da ich einen Fehler bezüglich eines Inventares hatte. Doch da taucht der nächste auf, den ich nicht lösen kann (auch schon recherchiert). Aus meinem Invetar kann Sachen nehmen / verschieben aber im GUI kann ich das genauso obwohl ich das Event gecanceltt hab.
Dann das Hauptproblem

package de.grafplaiz.bangui.events;

import de.grafplaiz.bangui.BanGui;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.inventory.Inventory;

public class BanInventroyEvent implements Listener {

BanGui plugin;

public BanInventroyEvent(BanGui plugin) {
this.plugin = plugin;
}

@EventHandler
public void onMenuClick(InventoryClickEvent e ) {
Player player = (Player) e.getWhoClicked();

if(e.getView().getTitle().equalsIgnoreCase(ChatColor.BLUE + "Spieler Liste")) {
if(e.getCurrentItem().getType().equals(Material.PLAYER_HEAD)); {
Player whoToBan = plugin.getServer().getPlayer(e.getCurrentItem().getItemMeta().getDisplayName());

plugin.openAuswahlMenu(player, whoToBan);

}
e.setCancelled(true);
}
}
}

Bei if(e.getCurrentItem().getType().equals(Material.PLAYER_HEAD)); kommt der Fehler in der Console.
Doch ich kapier einfach nicht was da falsch sein sollte. Ich habe auch schon mit einem anderen Devloper gespochen und dem Fällt auch nichts ein.

 

[Feli490]

getCurrentItem() Kann null sein wenn kein Item angelickt wurde, daher vermute ich mal das der Null Pointer daher kommt.
Du solltest also bevor du auf das Item zu greifst einen Null Check einbauen.

Auch wirkt in Zeile 25 das komisch das du nach dem if direkt ein ';' Dadurch wird deine if Abfrage praktisch nichtig.

 

[SchnellfeuerXD]

getCurrentItem() Kann null sein wenn kein Item angelickt wurde, daher vermute ich mal das der Null Pointer daher kommt.
Du solltest also bevor du auf das Item zu greifst einen Null Check einbauen.

Auch wirkt in Zeile 25 das komisch das du nach dem if direkt ein ';' Dadurch wird deine if Abfrage praktisch nichtig.

Ok danke, ich checke das mal!

 

[Xenoshi]

Oder mach ne NullPointExeption
try {
//Hier dein Code
}catch(NullPointerException e1) {}

 

[Xenoshi]

@LapisMC
Naja ich mache es so bei ein paar Funktionen wo ich weiß das der Fehler kommen wird. Ich sage ja nur das er die Fehlerbehebung so erstmal weg bekommt (sie wird nicht angezeigt). Dann sollte man mehrere Möglichkeiten finden um den Punkt normal zu beheben

 

[Taminoful]

Oder mach ne NullPointExeption
try {
//Hier dein Code
}catch(NullPointerException e1) {}

@LapisMC
Naja ich mache es so bei ein paar Funktionen wo ich weiß das der Fehler kommen wird. Ich sage ja nur das er die Fehlerbehebung so erstmal weg bekommt (sie wird nicht angezeigt). Dann sollte man mehrere Möglichkeiten finden um den Punkt normal zu beheben

Bietet halt absolut keinen Mehrwert sowas als Antwort zu posten, wenn das Problem bereits durch die gepostete Lösung behoben werden kann. Nur weil du etwas so machst, heißt das nicht, dass dies eine gute oder gar richtige Vorgehensweise ist.

Exceptions haben immer einen Ursprung, meist ist der Autor selbst dafür verantwortlich, wenn diese auftreten.
Überall nen leeren Try-Catch drum werfen und das Fehlerbehandlung zu nennen - das findest du halt auch nur bei Anfängern mit Ursprung in der Minecraft Szene.

 

[SchnellfeuerXD]

Danke an alle!