Cause damage in Intervals?

I don't have code at the moment just some snippets. I've tried making cooldowns to cause damage, scheduler, and other various methods of causing damage in intervals. So far I'm unsuccessful. I've tried this:

Code:

            plugin.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
                public void run() {
                    if(plugin.HelmetNull.contains(ply)){
                        ply.damage(plugin.getConfig().getInt("int_Damage"));
                        if(ply.isDead()){
                            plugin.HelmetNull.remove(ply);
                        }
                    }
                }
            }, 60L);

But that doesn't run but returns a null pointer...

Any suggestions?

Background info on Method:

• ply is the parameters for this method (Player ply)
• HelmetNull is a player Array.

Error:

Code:

19:50:54 [SEVERE] Could not pass event PlayerMoveEvent to Contaminate
org.bukkit.event.EventException
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.ja
va:303)
        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.jav
a:62)
        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.j
ava:459)
        at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:209)
        at net.minecraft.server.Packet10Flying.handle(SourceFile:126)
        at net.minecraft.server.NetworkManager.b(NetworkManager.java:229)
        at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:113)
        at net.minecraft.server.NetworkListenThread.a(NetworkListenThread.java:7
8)
        at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:558)
        at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:450)
        at net.minecraft.server.ThreadServerApplication.run(SourceFile:492)
Caused by: java.lang.NullPointerException
        at Orcem.Block.Contamination.AirFactory.NullHelmet(AirFactory.java:13)
        at Orcem.Block.Contamination.PlayerListener.onPlayerMove(PlayerListener.
java:32)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
        at java.lang.reflect.Method.invoke(Unknown Source)
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.ja
va:301)
        ... 10 more
>

 

We need the soruce of the whole file to see where line 13 in Airfactory.java. It is there where the nullpointer is thrown from.

 

I understand this, I tried to provide all the information needed because I've changed the code somewhat.
I'll try to provide everything:

Code:java

 
package Orcem.Block.Contamination;
 
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
 
public class AirFactory {
    private Contaminate plugin;
    public AirFactory(Contaminate plugin){
        this.plugin = plugin;
    }
    public void NullHelmet(final Player p){
        plugin.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
            public void run() {
                if(plugin.HelmetNull.contains(p)){
                    p.damage(plugin.getConfig().getInt("int_Damage"));
                    //p.sendMessage(ChatColor.DARK_RED+"");
                    if(p.isDead()){
                        plugin.HelmetNull.remove(p);
                    }
                }
            }
        }, 60L);
    }
 

Player Listener:

Code:java

 
---
 
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerMoveEvent;
 
public class PlayerListener implements Listener {
    /*
     * -
     * 
     * -
     * 
     */
    private - plugin;
    public PlayerListener(Contaminate plugin){
        this.plugin = plugin;
    }
    AirFactory AirF = new AirFactory(plugin);
    @EventHandler
    public void onPlayerMove(PlayerMoveEvent e){
            final Player p = (Player)e.getPlayer();
            //Check which list is being used.
            if(plugin.getConfig().getBoolean("-")){
                if(plugin.getConfig().getIntegerList("-").contains(Material.AIR.getId())){
                    if(plugin.getConfig().getBoolean("-")){
                        if(p.getInventory().getHelmet() == null){
                            if(plugin.getConfig().getInt("int_Damage") != 0){
                                if(!plugin.HelmentNull.contains(p)){
                                    plugin.HelmentNull.add(p);
                                    AirF.NullHelmet(p);
                                }
                            }
 

 

If you have modified the soruce, can you paste your current stack-trace?

 

Ctrl-z is good friend of mine. I posted Listener as well where the error came from.

 

So, sorry if I do not follow, but where is the error? Is it still on line 13 in Airfactory.java? Or do you have a new stack-trace I can see?

 

Correct, I restored the old code before I started messing with it. I apologize for the confusion. Every line in Stack Trace matches the code provided.

 

Ah, ok, then I know why it does not work.

The plugin reference in Airfactory.java is null, becouse the reference you pass to create the instance is null (the plugin variable). Let me show what is wrong and how to fix it.

In the listener class change

Code:java

 
 
public PlayerListener(Contaminate plugin){
this.plugin = plugin;
}
//This is run before the constructor is run, therefore the passed reference (plugin) is still null.
AirFactory AirF = new AirFactory(plugin);
 
 

to

Code:java

 
AirFactory AirF;
 
public PlayerListener(Contaminate plugin){
this.plugin = plugin;
//Here we have assigned the plugin reference, so now we can create the AirFactory instance and pass a reference that is not null
this.AirF = new AirFactory(plugin);
}
 
 

 

@Ranzdo
You sir, are Java god! Thank you so much! I can't believe how simple some errors really can be...