Solved Loop causes NPE

I've been fiddling with this for days,
but the result is always the same...
Whenever I get hit by a mob (seems to be effected by whether or not I hit it first) my console loops a NPE. At this line.
main.pvpTimer.put(player.getUniqueId().toString(), main.pvpTimer.get(player.getUniqueId().toString()) - 1);​

​

Code:java

 
    @EventHandler
    public void onEntityDamageByEntity(EntityDamageByEntityEvent e) {
        if (e.getDamager() instanceof Player) {
            final Player player = (Player) e.getDamager();
            if(main.flyplayers.contains(player.getUniqueId().toString())){
                e.setCancelled(true);
                        player.sendMessage(ChatColor.RED + reason);
            }else{
                if (main.getConfig().getLong("CameraMode.PvpTimer") != 0){
                    if (e.getEntity() instanceof Zombie || e.getEntity() instanceof Slime || e.getEntity() instanceof Player || e.getEntity() instanceof Skeleton || e.getEntity() instanceof Creeper || e.getEntity() instanceof Spider || e.getEntity() instanceof Witch || e.getEntity() instanceof Wolf || e.getEntity() instanceof Blaze || e.getEntity() instanceof Ghast || e.getEntity() instanceof MagmaCube || e.getEntity() instanceof Arrow || e.getEntity() instanceof CaveSpider || e.getEntity() instanceof EnderDragon | e.getEntity() instanceof PigZombie || e.getEntity() instanceof Silverfish || e.getEntity() instanceof Fireball || e.getEntity() instanceof WitherSkull || e.getEntity() instanceof Wither || e.getEntity() instanceof IronGolem || e.getEntity() instanceof Giant){
                        //if (!main.pvpTimer.containsKey(player.getUniqueId().toString()))
                            main.pvpTimer.put(e.getDamager().getUniqueId().toString(), main.getConfig().getInt("CameraMode.PvpTimer") +1);
                            ID = main.getServer().getScheduler().scheduleSyncRepeatingTask(main, new Runnable(){
                                public void run() {
                                        main.pvpTimer.put(player.getUniqueId().toString(), main.pvpTimer.get(player.getUniqueId().toString()) - 1);
                                        if (main.pvpTimer.get(player.getUniqueId().toString()) == 1){
                                            Bukkit.getScheduler().cancelTask(ID);
                                        }
                                }
                            }, 0,20);
                            main.getServer().getScheduler().scheduleSyncDelayedTask(main, new Runnable(){
                                public void run() {
                                    main.pvpTimer.remove(player.getUniqueId().toString());
                                }
                            },20 * main.getConfig().getLong("CameraMode.PvpTimer"));
 
                    }
                }
            }
        }
        if (e.getEntity() instanceof Player){
            final Player entiti = (Player) e.getEntity();
            if(main.flyplayers.contains(entiti.getUniqueId().toString())){
                e.setCancelled(true);
            }else{
                if (main.getConfig().getLong("CameraMode.PvpTimer") != 0){
                    if (e.getDamager() instanceof Zombie || e.getEntity() instanceof Slime || e.getEntity() instanceof Player || e.getEntity() instanceof Skeleton || e.getEntity() instanceof Creeper || e.getEntity() instanceof Spider || e.getEntity() instanceof Witch || e.getEntity() instanceof Wolf || e.getEntity() instanceof Blaze || e.getEntity() instanceof Ghast || e.getEntity() instanceof MagmaCube || e.getEntity() instanceof Arrow || e.getEntity() instanceof CaveSpider || e.getEntity() instanceof EnderDragon | e.getEntity() instanceof PigZombie || e.getEntity() instanceof Silverfish || e.getEntity() instanceof Fireball || e.getEntity() instanceof WitherSkull || e.getEntity() instanceof Wither || e.getEntity() instanceof IronGolem || e.getEntity() instanceof Giant){
                        //if (!main.pvpTimer.containsKey(entiti.getUniqueId().toString()))
                            main.pvpTimer.put(e.getEntity().getUniqueId().toString(), main.getConfig().getInt("CameraMode.PvpTimer") + 1);
                            IDS = main.getServer().getScheduler().scheduleSyncRepeatingTask(main, new Runnable(){
                                public void run() {
                                        main.pvpTimer.put(entiti.getUniqueId().toString(), main.pvpTimer.get(entiti.getUniqueId().toString()) - 1);
                                        if (main.pvpTimer.get(entiti.getUniqueId().toString()) == 1){
                                            Bukkit.getScheduler().cancelTask(IDS);
                                        }
                                }
                            }, 0,20);
                            main.getServer().getScheduler().scheduleSyncDelayedTask(main, new Runnable(){
                                public void run() {
                                    main.pvpTimer.remove(entiti.getUniqueId().toString());
                                }
                            },20 * main.getConfig().getLong("CameraMode.PvpTimer"));
 
                    }
                }
            }
        }
    }

Am I doing something obviously wrong? ​

Btw

Code:java

Int ID;
Int IDS;

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

 

xYourFreindx

main variable?

 

ZodiacTheories
Main is just equal to the main class. It is where the hashmaps and config data is retrieved from.
If you'd like a copy, let me know.

 

xYourFreindx

Can I see where you declare/initialize the variable?

 

ZodiacTheories

Code:java

 
 
public class Events implements Listener {
 
    CameraMode main;
 
    int ID;
    int IDS;
    String reason = "You are in CameraMode!";
 
    public Events(CameraMode plugin){
        this.main = plugin;
    }
    //...

I added a "looper" and "loopers" arraylist, and reorganized the loops.
You can see my changes here.
​

Code:java

 
    @EventHandler
    public void onEntityDamageByEntity(EntityDamageByEntityEvent e) {
        if (e.getDamager() instanceof Player) {
            final Player player = (Player) e.getDamager();
            if(main.flyplayers.contains(player.getUniqueId().toString())){
                e.setCancelled(true);
                        player.sendMessage(ChatColor.RED + reason);
            }else{
                if (main.getConfig().getLong("CameraMode.PvpTimer") != 0){
                    if (e.getEntity() instanceof Zombie || e.getEntity() instanceof Slime || e.getEntity() instanceof Player || e.getEntity() instanceof Skeleton || e.getEntity() instanceof Creeper || e.getEntity() instanceof Spider || e.getEntity() instanceof Witch || e.getEntity() instanceof Wolf || e.getEntity() instanceof Blaze || e.getEntity() instanceof Ghast || e.getEntity() instanceof MagmaCube || e.getEntity() instanceof Arrow || e.getEntity() instanceof CaveSpider || e.getEntity() instanceof EnderDragon | e.getEntity() instanceof PigZombie || e.getEntity() instanceof Silverfish || e.getEntity() instanceof Fireball || e.getEntity() instanceof WitherSkull || e.getEntity() instanceof Wither || e.getEntity() instanceof IronGolem || e.getEntity() instanceof Giant){
                        //if (!main.pvpTimer.containsKey(player.getUniqueId().toString())){
                            if (main.looper.contains(player.getUniqueId())){
                                Bukkit.getScheduler().cancelTask(ID);
                                main.looper.remove(player.getUniqueId());
                            }
                            if (main.loopers.contains(player.getUniqueId())){
                                Bukkit.getScheduler().cancelTask(IDS);
                                main.loopers.remove(player.getUniqueId());
                            }
                            main.pvpTimer.put(e.getDamager().getUniqueId().toString(), main.getConfig().getInt("CameraMode.PvpTimer") +1);
                            main.looper.add(player.getUniqueId());
                            ID = main.getServer().getScheduler().scheduleSyncRepeatingTask(main, new Runnable(){
                                public void run() {
                                        main.pvpTimer.put(player.getUniqueId().toString(), main.pvpTimer.get(player.getUniqueId().toString()) - 1);
                                        if (main.pvpTimer.get(player.getUniqueId().toString()) == 1){
                                            Bukkit.getScheduler().cancelTask(ID);
                                            main.pvpTimer.remove(player.getUniqueId().toString());
                                        }
                                }
                            }, 0,20);
 
                    }
                }
            }
        }
        if (e.getEntity() instanceof Player){
            final Player entiti = (Player) e.getEntity();
            if(main.flyplayers.contains(entiti.getUniqueId().toString())){
                e.setCancelled(true);
            }else{
                if (main.getConfig().getLong("CameraMode.PvpTimer") != 0){
                    if (e.getDamager() instanceof Zombie || e.getEntity() instanceof Slime || e.getEntity() instanceof Player || e.getEntity() instanceof Skeleton || e.getEntity() instanceof Creeper || e.getEntity() instanceof Spider || e.getEntity() instanceof Witch || e.getEntity() instanceof Wolf || e.getEntity() instanceof Blaze || e.getEntity() instanceof Ghast || e.getEntity() instanceof MagmaCube || e.getEntity() instanceof Arrow || e.getEntity() instanceof CaveSpider || e.getEntity() instanceof EnderDragon | e.getEntity() instanceof PigZombie || e.getEntity() instanceof Silverfish || e.getEntity() instanceof Fireball || e.getEntity() instanceof WitherSkull || e.getEntity() instanceof Wither || e.getEntity() instanceof IronGolem || e.getEntity() instanceof Giant){
                        //if (!main.pvpTimer.containsKey(entiti.getUniqueId().toString())){
                        if (main.loopers.contains(entiti.getUniqueId())){
                            Bukkit.getScheduler().cancelTask(IDS);
                            main.loopers.remove(entiti.getUniqueId());
                        }
                        if (main.looper.contains(entiti.getUniqueId())){
                            Bukkit.getScheduler().cancelTask(ID);
                            main.looper.remove(entiti.getUniqueId());
                        }
                            main.loopers.add(entiti.getUniqueId());
                            main.pvpTimer.put(e.getEntity().getUniqueId().toString(), main.getConfig().getInt("CameraMode.PvpTimer") + 1);
                            IDS = main.getServer().getScheduler().scheduleSyncRepeatingTask(main, new Runnable(){
                                public void run() {
                                        main.pvpTimer.put(entiti.getUniqueId().toString(), main.pvpTimer.get(entiti.getUniqueId().toString()) - 1);
                                        if (main.pvpTimer.get(entiti.getUniqueId().toString()) == 1){
                                            Bukkit.getScheduler().cancelTask(IDS);
                                            main.pvpTimer.remove(entiti.getUniqueId().toString());
                                        }
                                }
                            }, 0,20);
 
                    }
                }
            }
        }
    }

This system works perfectly.
The timer restarts every time the player is hit or hits something, and there are no more errors.
The loops no longer run on top of each other. ​

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