PlayerRespawnEvent not working

Code:java

package duell;
 
import java.util.HashMap;
import net.milkbowl.vault.economy.Economy;
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.event.player.PlayerRespawnEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.RegisteredServiceProvider;
import org.bukkit.plugin.java.JavaPlugin;
 
/**
*
* @author Marcel
*/
public class Duell extends JavaPlugin implements Listener {
 
    @Override
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this, this);
        RegisteredServiceProvider economyProvider = getServer().getServicesManager().getRegistration(Economy.class);
        if(economyProvider != null) {
            economy = (Economy)economyProvider.getProvider();
        }
    }
    public static HashMap<Player, Integer> induell = new HashMap<>();
    public static HashMap<Player, Integer> hatanfrage = new HashMap<>();
    public static HashMap<Player, Player> anfrage = new HashMap<>();
    public static HashMap<Player, String> invent = new HashMap<>();
    public Location LastDeath;
    private HashMap<String, ItemStack[]> mySavedItems = new HashMap<String, ItemStack[]>();
    public static Economy economy = null;
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String commandlabel, String[] args) {
        final Player p = (Player) sender;
        if(cmd.getName().equalsIgnoreCase("duell")) {
            if(args.length == 1) {
                if(args[0].equalsIgnoreCase("annehmen")) {
                    try {
                    if (!anfrage.get(p).isOnline()) {
                            throw new NullPointerException();
                        }
                    } catch (NullPointerException e) {
                        p.sendMessage("§6[EE1BSystem | Duell]§e" + args[0] + "§b ist nicht mehr online.");
                        return false;
                    }
                    p.sendMessage("§6[EE1BSystem | Duell]§bDu hast das Duell angenommen");
                    anfrage.get(p).sendMessage("§6[EE1BSystem | Duell]§e" + p.getName() + "§b hat deine Duell Herrausforderung angenommen.");
                    induell.put(p, 1);
                    induell.put(anfrage.get(p),1);
                    hatanfrage.put(anfrage.get(p),0);
                    hatanfrage.put(p,0);
                    Location p1 = new Location(p.getWorld(), -234, 163, 308);
                    Location p2 = new Location(p.getWorld(), -306, 163, 308);
                    p.teleport(p1);
                    anfrage.get(p).teleport(p2);
                    p.getInventory().clear();
                    anfrage.get(p).getInventory().clear();
                    ItemStack sword1 = new ItemStack(Material.STONE_SWORD, 1);
                    p.getInventory().addItem(sword1);
                    anfrage.get(p).getInventory().addItem(sword1);
                    p.setFoodLevel(20);
                    anfrage.get(p).setFoodLevel(20);
                    p.setHealth(20);
                    anfrage.get(p).setHealth(20);
                    ItemStack helmet = new ItemStack(Material.LEATHER_HELMET, 1);
                    ItemStack chest = new ItemStack(Material.LEATHER_CHESTPLATE, 1);
                    p.getEquipment().setHelmet(helmet);
                    p.getEquipment().setChestplate(chest);
                    anfrage.get(p).getEquipment().setHelmet(helmet);
                    anfrage.get(p).getEquipment().setChestplate(chest);
                } else if(args[0].equalsIgnoreCase("ablehnen")) {
                    anfrage.remove(anfrage.get(p));
                    hatanfrage.put(anfrage.get(p),0);
                    hatanfrage.put(p,0);
                    anfrage.remove(p);
                } else if(args[0].equalsIgnoreCase("abbrechen")) {
                    p.sendMessage("§6[EE1BSystem | Duell]§bDu hast deine Duell Herrausforderung zurückgezogen");
                    anfrage.get(p).sendMessage("§6[EE1BSystem | Duell]§bHat seine Duell Herrausforderung zurückgezogen.");
                    anfrage.remove(anfrage.get(p));
                    hatanfrage.put(anfrage.get(p),0);
                    hatanfrage.put(p,0);
                    anfrage.remove(p);
                } else {
                    final Player target = p.getServer().getPlayerExact(args[0]);
                    if (sender.getName().equalsIgnoreCase(args[0])) {
                        p.sendMessage("§6[EE1BSystem | Duell]§bDu kannst dich nicht selber zu einem Duell herrausfordern.");
                        return false;
                    }
                    try {
                        if (!target.isOnline()) {
                            throw new NullPointerException();
                        }
                    } catch (NullPointerException e) {
                        p.sendMessage("§6[EE1BSystem | Duell]§e" + args[0] + " §bist nicht Online.");
                        return false;
                    }
                    try {
                        if(induell.get(target) == 1) {
                            p.sendMessage("§6[EE1BSystem | Duell]§bDieser Spieler ist bereits in einem Duell!");
                            return false;
                        }
                        if(induell.get(p) == 1) {
                            p.sendMessage("§6[EE1BSystem | Duell]§bDu bist bereits in einem Duell!");
                            return false;
                        }                   
                    } catch (NullPointerException ex) {
 
                    }
                    try {
                        if(hatanfrage.get(target) == 1) {
                            p.sendMessage("§6[EE1BSystem | Duell]§bDieser Spieler hat bereits eine Anfrage!");
                            return false;
                        }
                        if(hatanfrage.get(p) == 1) {
                            p.sendMessage("§6[EE1BSystem | Duell]§bDu hast bereits eine Anfrage!");
                            return false;
                        }                   
                    } catch (NullPointerException ex) {
 
                    }
                    anfrage.put(p, target);
                    anfrage.put(target, p);
                    hatanfrage.put(p,1);
                    hatanfrage.put(target,1);
                    p.sendMessage("§6[EE1BSystem | Duell]§bDu hast den Spieler " + target.getDisplayName()  + "§b zu einem Duell herrausgefordert.");
                    target.sendMessage("§6[EE1BSystem | Duell]§bDer Spieler " + p.getDisplayName() + "§b fordert dich zu einem Duell herraus.");
                    target.sendMessage("§6[EE1BSystem | Duell]§e/duell annehmen/ablehnen!");
                }
                return true;
            }
        } else {
            p.sendMessage("§6[EE1BSystem | Duell]§bCommands:"); 
            p.sendMessage("§b/duell annehmen");
            p.sendMessage("§b/duell ablehnen");
            p.sendMessage("§b/duell abbrechen");           
            p.sendMessage("§b/duell [Spieler]");     
        }
        return false;
    }
 
    @EventHandler(priority = EventPriority.HIGHEST)
    public void onPlayerRespawn(PlayerRespawnEvent eve) {   
        final Player p = eve.getPlayer();
        if(induell.get(p) == 1) {
            induell.put(p,0);
            anfrage.remove(anfrage.get(p));
            anfrage.remove(p);
        }
        p.sendMessage("§6[EE1BSystem]&bLetze Position vor dem Tod: X=" + LastDeath.getBlockX() + " Y=" + LastDeath.getBlockY() + " Z=" + LastDeath.getBlockZ());
        Bukkit.broadcastMessage("Test");
    }   
 
    @EventHandler
    public void onPlayerDeath(PlayerDeathEvent event) {
        Bukkit.getServer().broadcastMessage("§6[EE1BSystem]§b" + event.getEntity().getName() + " starb.");
        Player p = event.getEntity().getPlayer();
        LastDeath = p.getLocation();
        Player killer = anfrage.get(p);
        if(induell.get(p) == 1) {
            p.sendMessage("§6[EE1BSystem | Duell]§bDu hast das Duell gegen " + killer.getName() + " verloren.");
            killer.sendMessage("§6[EE1BSystem | Duell]§bDu hast das Duell gegen " + p.getName() + " gewonnen.");
            killer.getInventory().clear();
            induell.put(killer,0);
        }
    }
}

Hello,

My PlayerRespawnEvent just won't work i do not know why everything seems fine to me ... here is my whole Plugin, I hope someone can help me.

 

MinecraftMarc3l Replace

Code:java

getServer().getPluginManager().registerEvents(this, this);

with

Code:java

PluginManager pm = Bukkit.getPluginManager();
pm.registerEvents(this, this);

 

My other Event is working so that is not the Matter i think but ill try

The Server is not working anymore if i do what u said ^^

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

First off, Ouch. That code hurts my eyes
Second off, xepisolonxx Your code is the exact same except saving the first method to a variable, the reason MinecraftMarc3l s plugin broke is because you forgot getServer(), Your code should be

Code:java

PluginManager pm = getServer().getPluginManager();
pm.registerEvents(this, this);

Is the console throwing errors?
What part of the plugin isn't working?

 

Umm as with @acer5999's yes mine hurt too
Please stop abusing static!
2. Read my signature about casting sender to player
3. I would sugest the use of more than one classes for listeners so code doesn't pile up in the main class but that's an option
4. Avoid storage of player object in hashmap... (Google it for the reason why)
5. Why in the command, if the player is not online, you throw a null pointer exception, then catch it and then send the message? You could use !=null and if this is true do whatever else send the message

And about your main problem, have you tried debugging ?

Hope I helped

 

What should i use instead of Hashmaps?

 

i use this code and it work fine with me :/



@EventHandler
public void onRespawn(PlayerRespawnEvent e)
{
e.getPlayer().getInventory().clear();
e.getPlayer().getInventory().addItem(new ItemStack[] { new ItemStack(Material.STONE_SWORD) });
}

 

imLore70 it is the same it just Looks diffrent

by the way it is still not working ^^

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

MinecraftMarc3l Don't catch NullPointers. Check for null and stop it before the error happens. Get rid of those static variables, and use encapsulation.

 

I dont get what These ancapsulation shit is but i do not really care i just want this Event to be called