Denn Killer bekommen[CLOSED]

[deleted193557]

Hey,

wie kann man bei einem DeathEvent rausbekommt, wer den Spieler getötet hat,
ich möchte nämlich, dass er ein Smaragd in sein Inventar bekommt, wenn er jemand killt

Lg AdriBloober

 

[KaeseKuchen_DEV]

Du nimmst den Spieler der getötet wurde und machst p.getKiller();
Davor solltest du allerdings abfragen ob der Killer nicht null ist weil du sonst bei einem Tod, bei dem der Spieler ohne einen anderen Spieler geschlagen wird, eine fette Fehlermeldung.

 

[JOO200]

Frei aus dem Kopf raus:

@EventHandler
public void onDeathEvent(PlayerDeathEvent event) {
    if(event.getKiller() instanceof Player) {
        Player killer = (Player)event.getKiller();
        killer.getInventory().addItem(new ItemStack(Material.EMERALD));
        killer.sendMessage("Du hast einen Smaragd erhalten.");
    }
}

 

[OfficialRynec | Max]

NEin.........

Mach das so...

@EventHandler
public void onDeathEvent(PlayerDeathEvent event) {
    if(event.getKiller() instanceof Player) {
        Player killer = (Player)event.getKiller();
if(killer != null){

        killer.getInventory().addItem(new ItemStack(Material.EMERALD));
        killer.sendMessage("Du hast einen Smaragd erhalten.");
return;
}
    }
}

//Dean: Logs, Code & Configs bitte entweder als Code einfügen oder als Dateianhang, bzw.auf pastebin.com hochladen! Danke!

 

[JOO200]

if(event.getKiller() instanceof Player)

beinhaltet bereits die Prüfung, ob event.getKiller() nicht null ist. (Was null ist, kann kein Player sein).
Daher ist deine anschließende Nullprüfung unnötig.

Und wenn du dann dein Ergebnis mit meinem vergleichst, solltest du feststellen, dass meine Lösung deiner entspricht.

 

[LinuxServer]

NEin.........

Mach das so...

@EventHandler
public void onDeathEvent(PlayerDeathEvent event) {
    if(event.getKiller() instanceof Player) {
        Player killer = (Player)event.getKiller();
if(killer != null){

        killer.getInventory().addItem(new ItemStack(Material.EMERALD));
        killer.sendMessage("Du hast einen Smaragd erhalten.");
return;
}
    }
}

//Dean: Logs, Code & Configs bitte entweder als Code einfügen oder als Dateianhang, bzw.auf pastebin.com hochladen! Danke!

Guten Tag,

du brauchst "event.getKiller();" nicht mit casten, weil du schon abfragst, ob der Killer ein Spieler ist. Zudem kannst du dir die Abfrage mit "killer != null" sparen, weil du auch abfragst ob der Killer ein Spieler ist also kann der nicht "null" sein.

Mit freundlichen Grüßen

 

[deleted193557]

Vielen Dank für diese ganzen hilfreichen Antworten.