[SOLVED] My Plugin Spams Console Like Crazy! Please Help.

SnowGears · Aug 1, 2012

My plugin keeps throwing errors like crazy and I have been looking into this trying to figure out what is causing them and why it will not work but I just can't get it to work. If someone would help me look through my class and find any errors that would be super helpful. I have been looking through this for hours and am starting to get very annoyed so you would help me with a big headache. Thanks in advance.

This is my listener class.

http://pastebin.com/M82Hn000

This is the error message it keeps spamming.

Code:

 
16:14:20 [SEVERE] Could not pass event PlayerMoveEvent to ArmorAbilities
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:304)
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:477)
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:462)
at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:198)
at net.minecraft.server.Packet10Flying.handle(SourceFile:126)
at net.minecraft.server.NetworkManager.b(NetworkManager.java:246)
at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:102)
at net.minecraft.server.NetworkListenThread.a(NetworkListenThread.java:82)
at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:559)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:451)
at net.minecraft.server.ThreadServerApplication.run(SourceFile:492)
Caused by: java.lang.NullPointerException
at com.pluggertech.armorabilities.ArmorListener.onPlayerMove(ArmorListener.java:136)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:597)
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:302)
... 11 more
16:14:25 [SEVERE] Could not pass event InventoryCloseEvent to ArmorAbilities
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:304)
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:477)
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:462)
at net.minecraft.server.NetServerHandler.handleContainerClose(NetServerHandler.java:1039)
at net.minecraft.server.Packet101CloseWindow.handle(SourceFile:16)
at net.minecraft.server.NetworkManager.b(NetworkManager.java:246)
at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:102)
at net.minecraft.server.NetworkListenThread.a(NetworkListenThread.java:82)
at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:559)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:451)
at net.minecraft.server.ThreadServerApplication.run(SourceFile:492)
Caused by: java.lang.NullPointerException
at com.pluggertech.armorabilities.ArmorListener.eventInventoryClose(ArmorListener.java:115)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:597)
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:302)
... 11 more
16:14:27 [INFO] op lost connection: disconnect.quitting
>

Slayer9x9 · Aug 1, 2012

Tooner101 said: ↑

My plugin keeps throwing errors like crazy and I have been looking into this trying to figure out what is causing them and why it will not work but I just can't get it to work. If someone would help me look through my class and find any errors that would be super helpful. I have been looking through this for hours and am starting to get very annoyed so you would help me with a big headache. Thanks in advance.

This is my listener class.

Code:

package com.pluggertech.armorabilities;
 
 
import java.util.ArrayList;
import java.util.HashMap;
import java.util.Map;
 
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.block.Block;
import org.bukkit.entity.Entity;
import org.bukkit.entity.LivingEntity;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.event.entity.EntityDamageEvent;
import org.bukkit.event.inventory.InventoryCloseEvent;
import org.bukkit.event.player.PlayerMoveEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
 
public class ArmorListener implements Listener{
 
private int amountOfLeather = 0;
private int amountOfIron = 0;
private int amountOfGold = 0;
private int amountOfDiamond = 0;
private int amountOfChainmail = 0;
private ArrayList<Integer> allAmounts = new ArrayList<Integer>();
 
private String leatherAbility = ArmorAbilities.leather;
private String ironAbility = ArmorAbilities.iron;
private String goldAbility = ArmorAbilities.gold;
private String diamondAbility = ArmorAbilities.diamond;
private String chainmailAbility = ArmorAbilities.chainmail;
 
private final ArrayList<Player> noLavaDamage = new ArrayList<Player>();
private final ArrayList<Player> noFireDamage = new ArrayList<Player>();
//private final ArrayList<Player> noWaterDamage = new ArrayList<Player>();
 
Map<String, String> powerMap = new HashMap<String, String>(); //name, ability
 
public ArmorAbilities plugin = ArmorAbilities.plugin;
 
public ArmorListener(ArmorAbilities instance)
    {
        plugin = instance;
    }
 
public String getAbility(Player player)
{
  if(powerMap.containsValue(player.getName()))
    return powerMap.get(player.getName());
  else
      return null; // Or whatever a string of whatever you want, like "none"
}
 
public void setAbility(Player player, String ability)
{
  powerMap.put(player.getName(), ability); // Done!
  player.sendMessage("Ability set is " + ability);
}
 
@EventHandler
public void eventInventoryClose(final InventoryCloseEvent event) //when inventory is closed
{
Player player = (Player)event.getPlayer();
 
amountOfLeather = checkLeather(player);
amountOfIron = checkIron(player);
amountOfGold = checkGold(player);
amountOfDiamond = checkDiamond(player);
amountOfChainmail = checkChainmail(player);
 
allAmounts.clear();
 
allAmounts.add(amountOfLeather);
allAmounts.add(amountOfIron);
allAmounts.add(amountOfGold);
allAmounts.add(amountOfDiamond);
allAmounts.add(amountOfChainmail);
 
for(int i =0; i< allAmounts.size(); i++){
if(allAmounts.get(i) == 4){
switch(i){
case 0: //LEATHER
setAbility(player, leatherAbility);
player.sendMessage("Setting your ability to "+leatherAbility);
break;
case 1: //IRON
setAbility(player, ironAbility);
player.sendMessage("Setting your ability to "+ironAbility);
break;
case 2: //GOLD
setAbility(player, goldAbility);
player.sendMessage("Setting your ability to "+goldAbility);
break;
case 3: //DIAMOND
setAbility(player, diamondAbility);
player.sendMessage("Setting your ability to "+diamondAbility);
break;
case 4: //CHAINMAIL
setAbility(player, chainmailAbility);
player.sendMessage("Setting your ability to "+chainmailAbility);
break;
default: System.out.println("[ERROR] No Armor Ability Was Set!");
break;
}
}
}
if(player.hasPermission("armorabilities.speed") && getAbility(player).equalsIgnoreCase("SPEED")){
int speedNum = ArmorAbilities.speedNum;
 
      PotionEffect speedAbility = new PotionEffect(PotionEffectType.SPEED, 20000, 4*speedNum);
      speedAbility.apply(player);
}
else if(player.hasPermission("armorabilities.jump") && getAbility(player).equalsIgnoreCase("JUMP")){
int jumpNum = ArmorAbilities.jumpNum;
 
      PotionEffect jumpAbility = new PotionEffect(PotionEffectType.JUMP, 20000, 4 * jumpNum);
      jumpAbility.apply(player);
}
}
 
@EventHandler
public void onPlayerMove(PlayerMoveEvent event){
 
Player player = event.getPlayer();
Location loc = player.getLocation();
Location headLoc = player.getEyeLocation();
 
if(player.hasPermission("armorabilities.waterhaste") && getAbility(player).equalsIgnoreCase("SCUBA")){
if(headLoc.getBlock().getType() == Material.WATER || headLoc.getBlock().getType() == Material.STATIONARY_WATER){
 
int fastDig = ArmorAbilities.scubaHasteNum;
PotionEffect scubaHaste = new PotionEffect(PotionEffectType.FAST_DIGGING,20000, fastDig);
scubaHaste.apply(player);
}
else{ //player is not in water
player.addPotionEffect(new PotionEffect(PotionEffectType.FAST_DIGGING, 0, 0), true); //Remove it for the client
}
}
else if(player.hasPermission("armorabilities.lavaswim") && getAbility(player).equalsIgnoreCase("LAVA")){
  //player is not in lava
if ( ! (loc.getBlock().getType() == Material.LAVA || loc.getBlock().getType() == Material.STATIONARY_LAVA)){
if(!noLavaDamage.contains(player))
noLavaDamage.add(player);
if(!noFireDamage.contains(player))
noFireDamage.add(player);
}
}
}
 
@EventHandler
public void onEntityDamage(EntityDamageEvent event) {
 
if(!(event.getEntity() instanceof Player)) //if not a player
return;                              //break out of method
 
final Player player = (Player)event.getEntity();
 
  if ((event.getCause() == EntityDamageEvent.DamageCause.FALL)
  && getAbility(player).equalsIgnoreCase("JUMP") && player.hasPermission("armorabilities.nofalldamage"))
      event.setCancelled(true);
 
  else if((event.getCause() == EntityDamageEvent.DamageCause.DROWNING)
  && getAbility(player).equalsIgnoreCase("SCUBA") && player.hasPermission("armorabilities.nodrown"))
  event.setCancelled(true);
 
  else if(event.getCause() == EntityDamageEvent.DamageCause.LAVA){ //if player is in lava
  if(getAbility(player).equalsIgnoreCase("LAVA") && player.hasPermission("armorabilities.lavaswim"))
 
  if(noLavaDamage.contains(player)) //player is in arraylist
  event.setCancelled(true);  //do not take damage from lava
 
  int lavaTime = ArmorAbilities.lavaTime;
 
  if(noLavaDamage.contains(player)){ //if player is in arraylist
  plugin.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
 
    public void run() {        //create a delayed event to remove player from arraylist after some amount of time
        noLavaDamage.remove(player);
    }
  }, (long)(20*lavaTime));//delay of time before hurt by lava
  } //end of if
      }
 
  else if((event.getCause() == EntityDamageEvent.DamageCause.FIRE || event.getCause() == EntityDamageEvent.DamageCause.FIRE_TICK) //if player is hurt by fire
  && getAbility(player).equalsIgnoreCase("LAVA")){
 
  if(player.hasPermission("armorabilities.nofiredamage"))
  event.setCancelled(true); //not hurt by fire
 
  Location loc = player.getLocation();
  if(noLavaDamage.contains(player) && (loc.getBlock().getType() == Material.LAVA || loc.getBlock().getType() == Material.STATIONARY_LAVA)) //player in lava array and in lava
  event.setCancelled(true);
    }
}
 
@EventHandler
public void onEntityAttack(EntityDamageByEntityEvent event) {
 
if( ! (event.getEntity() instanceof Player))
return;
 
Player player = (Player)event.getEntity();
 
if(getAbility(player).equalsIgnoreCase("RAGE") && player.hasPermission("armorabilities.rage")){
Entity attacker = event.getDamager();
Entity defender = event.getEntity();
int rageLightningDamage = ArmorAbilities.rageLightningDamage;
int rageFireTime = ArmorAbilities.rageFireTime;
 
if(! (defender instanceof Player)) //defender is not a player
return;
Player defend = (Player)event.getEntity();
if(!defend.hasPermission("armorabilities.rage")) //does not have permission
return;
 
if( ! (attacker instanceof org.bukkit.entity.Arrow)){
if(attacker instanceof LivingEntity){
LivingEntity attack = (LivingEntity)event.getDamager();
strikeLightning(attack); //strike attacker with lightning
attack.damage(rageLightningDamage, defender);
attack.setFireTicks(rageFireTime*20); //MAKE THIS INT CONFIGURABLE
}
}
else{ //is a skeleton or arrow shot
org.bukkit.entity.Arrow arrow =(org.bukkit.entity.Arrow)attacker;
LivingEntity shooter = arrow.getShooter();
strikeLightning(shooter); //strike attacker with lightning
shooter.damage(rageLightningDamage, defender);
shooter.setFireTicks(rageFireTime*20);
}
}
}
 
 
 
public int checkLeather(Player p){
 
ItemStack[] allArmor = p.getInventory().getArmorContents();
int amountOfLeather = 0;
 
for(int i=0; i<allArmor.length; i++){
//if allArmor[i] is a type of leather armor
if(allArmor[i].getType() == Material.LEATHER_HELMET || allArmor[i].getType() == Material.LEATHER_CHESTPLATE || allArmor[i].getType() == Material.LEATHER_LEGGINGS || allArmor[i].getType() == Material.LEATHER_BOOTS)
amountOfLeather++;
}
return amountOfLeather;
}
 
public int checkIron(Player p){
 
ItemStack[] allArmor = p.getInventory().getArmorContents();
int amountOfIron = 0;
 
for(int i=0; i<allArmor.length; i++){
//if allArmor[i] is a type of iron armor
if(allArmor[i].getType() == Material.IRON_HELMET || allArmor[i].getType() == Material.IRON_CHESTPLATE || allArmor[i].getType() == Material.IRON_LEGGINGS || allArmor[i].getType() == Material.IRON_BOOTS)
amountOfIron++;
}
return amountOfIron;
}
 
public int checkGold(Player p){
 
ItemStack[] allArmor = p.getInventory().getArmorContents();
int amountOfGold = 0;
 
for(int i=0; i<allArmor.length; i++){
//if allArmor[i] is a type of gold armor
if(allArmor[i].getType() == Material.GOLD_HELMET || allArmor[i].getType() == Material.GOLD_CHESTPLATE || allArmor[i].getType() == Material.GOLD_LEGGINGS || allArmor[i].getType() == Material.GOLD_BOOTS)
amountOfGold++;
}
p.sendMessage("Amount of gold wearing is "+amountOfGold);
return amountOfGold;
}
 
public int checkDiamond(Player p){
 
ItemStack[] allArmor = p.getInventory().getArmorContents();
int amountOfDiamond = 0;
 
for(int i=0; i<allArmor.length; i++){
//if allArmor[i] is a type of diamond armor
if(allArmor[i].getType() == Material.DIAMOND_HELMET || allArmor[i].getType() == Material.DIAMOND_CHESTPLATE || allArmor[i].getType() == Material.DIAMOND_LEGGINGS || allArmor[i].getType() == Material.DIAMOND_BOOTS)
amountOfDiamond++;
}
return amountOfDiamond;
}
 
public int checkChainmail(Player p){
 
ItemStack[] allArmor = p.getInventory().getArmorContents();
int amountOfChainmail = 0;
 
for(int i=0; i<allArmor.length; i++){
//if allArmor[i] is a type of chainmail armor
if(allArmor[i].getType() == Material.CHAINMAIL_HELMET || allArmor[i].getType() == Material.CHAINMAIL_CHESTPLATE || allArmor[i].getType() == Material.CHAINMAIL_LEGGINGS || allArmor[i].getType() == Material.CHAINMAIL_BOOTS)
amountOfChainmail++;
}
return amountOfChainmail;
}
 
 
public void strikeLightning(Entity p){
        Location coords = p.getLocation();
        coords.getWorld().strikeLightningEffect(coords);
        Block block = coords.getBlock();
        block.setType(Material.AIR);
    }
}

This is the error message it keeps spamming.

Code:

 
2012-08-01 15:09:20 [SEVERE] Could not pass event PlayerMoveEvent to ArmorAbilities
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:304)
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:477)
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:462)
at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:198)
at net.minecraft.server.Packet10Flying.handle(SourceFile:126)
at net.minecraft.server.NetworkManager.b(NetworkManager.java:246)
at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:102)
at net.minecraft.server.NetworkListenThread.a(NetworkListenThread.java:82)
at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:559)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:451)
at net.minecraft.server.ThreadServerApplication.run(SourceFile:492)
Caused by: java.lang.NullPointerException
at com.pluggertech.armorabilities.ArmorListener.onPlayerMove(ArmorListener.java:136)
at sun.reflect.GeneratedMethodAccessor6.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:597)
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:302)
... 11 more
2012-08-01 15:09:20 [SEVERE] Could not pass event PlayerMoveEvent to ArmorAbilities
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:304)
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:477)
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:462)
at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:198)
at net.minecraft.server.Packet10Flying.handle(SourceFile:126)
at net.minecraft.server.NetworkManager.b(NetworkManager.java:246)
at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:102)
at net.minecraft.server.NetworkListenThread.a(NetworkListenThread.java:82)
at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:559)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:451)
at net.minecraft.server.ThreadServerApplication.run(SourceFile:492)
Caused by: java.lang.NullPointerException
at com.pluggertech.armorabilities.ArmorListener.onPlayerMove(ArmorListener.java:136)
at sun.reflect.GeneratedMethodAccessor6.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:597)
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:302)
... 11 more

Click to expand...

If you read the error message, the problem is in your "onPlayerMove" event.
Post line numbers so we can see exactly what line throws the Null Pointer exception.

SnowGears · Aug 1, 2012

Slayer9x9 said: ↑

If you read the error message, the problem is in your "onPlayerMove" event.
Post line numbers so we can see exactly what line throws the Null Pointer exception.
Click to expand...

I know it is with onPlayerMove but I don't know what is causing it. It also throws exceptions for onInventoryClose. How do I post line numbers?

lazertester · Aug 1, 2012

Your offending method call:
Code:
if(player.hasPermission("armorabilities.waterhaste") && getAbility(player).equalsIgnoreCase("SCUBA")){
Which is trying a string comparison on the result of this:
Code:
public String getAbility(Player player)
{
if(powerMap.containsValue(player.getName()))
return powerMap.get(player.getName());
else
return null; // Or whatever a string of whatever you want, like "none"
}
Which returns null if a player does not have the ability.

I would recommend creating an AbilityManager with a hasAbility(Player, Ability) method where Ability is an enum constant instead of using a bunch of string comparisons. This would allow you to make your call something like
Code:
if(player.hasPermission("armorabilities.waterhaste") && hasAbility(player,Ability.SCUBA)){

Slayer9x9 · Aug 1, 2012

Tooner101 said: ↑

I know it is with onPlayerMove but I don't know what is causing it. It also throws exceptions for onInventoryClose. How do I post line numbers?
Click to expand...

Eclipse should show you line numbers, just post the content of line "ArmorListener.java:136".

JjPwN1 · Aug 1, 2012

lazertester
Post all of your code onto a new paste at Pastebin and it will automatically put line numbers. Give us the link once you've pasted it there.

SnowGears · May 27, 2016

JjPwN1 said: ↑

lazertester
Post all of your code onto a new paste at Pastebin and it will automatically put line numbers. Give us the link once you've pasted it there.
Click to expand...

http://pastebin.com/M82Hn000

EDIT: I also updated the error message up in the top
lazertester said: ↑
Your offending method call:
Code:
if(player.hasPermission("armorabilities.waterhaste") && getAbility(player).equalsIgnoreCase("SCUBA")){
Which is trying a string comparison on the result of this:
Code:
public String getAbility(Player player)
{
if(powerMap.containsValue(player.getName()))
return powerMap.get(player.getName());
else
return null; // Or whatever a string of whatever you want, like "none"
}
Which returns null if a player does not have the ability.

I would recommend creating an AbilityManager with a hasAbility(Player, Ability) method where Ability is an enum constant instead of using a bunch of string comparisons. This would allow you to make your call something like
Code:
if(player.hasPermission("armorabilities.waterhaste") && hasAbility(player,Ability.SCUBA)){
Click to expand...
How would I do that? Would I make an AbilityManager class with those methods in it? Would those comparisons be the reason my plugin is not working?

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

lazertester · Aug 1, 2012

Your issue is that your comparison method is returning null if the player does not have the ability or is not in the map. When you get null back you are calling null.equalsIgnoreCase("SCUBA")

If you don't want to make the ability manager and use a constant Enumeration to represent abilities then I recommend doing something like:
Code:
public String getAbility(Player player)
{
if(!powerMap.containsKey(player.getName()))
    powerMap.put(player.getName(),"NONE");
return powerMap.get(player.getName());
}
To avoid getting null back from that method.

NEVERMIND

you are calling

powerMap.containsValue(player.getName())

instead of

powerMap.containsKey(player.getName())

I still recommend you change the structure of the method as I posted above to avoid returning null if you do not plan on checking for null every time you call the method. At least make it return an empty string if necessary.

SnowGears · Aug 1, 2012

lazertester

Thanks!

Log in or Sign up

[SOLVED] My Plugin Spams Console Like Crazy! Please Help.

SnowGears

Slayer9x9

SnowGears

lazertester

Slayer9x9

JjPwN1

SnowGears

lazertester

SnowGears

Share This Page

[SOLVED] My Plugin Spams Console Like Crazy! Please Help.

Share This Page

Useful Searches