Solved NullPointerExpectation Help

Hello everyone,

I am trying to make a plugin that will remove Wither Skull Entities when the server is disabled.

Here is my code:

Code:java

    @Override
    public void onDisable()
    {
        World w = Bukkit.getServer().getWorld(getName());
        for(WitherSkull i = (WitherSkull) w.getEntitiesByClass(WitherSkull.class);;)
        {
            i.remove();
        }
    }

However, I seem to be getting a NullPointerExpectation, either I'm missing something obvious or just being dumb, I don't know.

Could you help me by pointing me in the right direction?

Code:

[21:28:21] [Server thread/INFO]: [WitherSkull] Disabling WitherSkull v1
[21:28:21] [Server thread/ERROR]: Error occurred while disabling WitherSkull v1 (Is it up to date?)
java.lang.NullPointerException
    at com.brad.Wither.Main.onDisable(Main.java:20) ~[?:?]
    at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:318) ~[craftbukkit.jar:git-Spigot-1649]
    at org.bukkit.plugin.java.JavaPluginLoader.disablePlugin(JavaPluginLoader.java:356) [craftbukkit.jar:git-Spigot-1649]
    at org.bukkit.plugin.SimplePluginManager.disablePlugin(SimplePluginManager.java:436) [craftbukkit.jar:git-Spigot-1649]
    at org.bukkit.plugin.SimplePluginManager.disablePlugins(SimplePluginManager.java:429) [craftbukkit.jar:git-Spigot-1649]
    at org.bukkit.craftbukkit.v1_7_R4.CraftServer.disablePlugins(CraftServer.java:412) [craftbukkit.jar:git-Spigot-1649]
    at net.minecraft.server.v1_7_R4.MinecraftServer.stop(MinecraftServer.java:393) [craftbukkit.jar:git-Spigot-1649]
    at net.minecraft.server.v1_7_R4.MinecraftServer.run(MinecraftServer.java:525) [craftbukkit.jar:git-Spigot-1649]
    at net.minecraft.server.v1_7_R4.ThreadServerApplication.run(SourceFile:628) [craftbukkit.jar:git-Spigot-1649]

Thanks, Brad

 

hellobrad100
What is this madness?

Code:java

for(WitherSkull i = (WitherSkull) w.getEntitiesByClass(WitherSkull.class);;)

 

here is what you should do hellobrad100

Code:java

// on disable
for (Entity e : <world>.getEntities()) {
if (e.getType().equals(EntityType.WITHER_SKULL)) {
e.remove();
}
}

 

TheCodingCat Don't check enum values using .equals when you can do ==.

 

TheCodingCat
I used your method in 2 ways and it still comes up with the error.
Used both .equals and ==

Any other ideas?

 

TheCodingCat hellobrad100
Check if the world contains WitherSkulls, then remove them.

 

Sorted.

I was just being really... really stupid..