NPE on 'empty' line D:

This is from one of my schedulers! Can anyone tell me why it is giving me an error on line 25 when there is just a parentheses

Even though the content of line 25 has changed, it STILL always gives me an error there

Code:java

package com.puffybugs.CloneZ.Runnables;
 
import net.minecraft.server.MobEffect;
import net.minecraft.server.Packet42RemoveMobEffect;
 
import org.bukkit.Bukkit;
import org.bukkit.craftbukkit.entity.CraftLivingEntity;
import org.bukkit.craftbukkit.entity.CraftPlayer;
import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
import org.bukkit.entity.Zombie;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
import com.puffybugs.CloneZ.CloneZ;
 
public class TargetFinder implements Runnable {
 
    CloneZ cloneZ;
 
    public TargetFinder(CloneZ instance) {
 
        cloneZ = instance;
 
    }
 
    /**
    * Finds the target for zombies
    */
    @Override
    public void run() {
 
        for (final Player player : Bukkit.getServer().getOnlinePlayers()) {
 
            if(!CloneZ.playerVisibility.containsKey(player.getName())) {
 
                CloneZ.playerVisibility.put(player.getName(), 1);
 
            }
 
            int playerVisibilityRadius = CloneZ.playerVisibility.get(player.getName()) * 5;
 
            for (Entity e : player.getNearbyEntities(playerVisibilityRadius, playerVisibilityRadius, playerVisibilityRadius)) {
                if (e instanceof Zombie) {
                    if (CloneZ.alphas.containsKey(e)) {
                        ((Zombie) e).setTarget(player);
 
                        final PotionEffect effect = new PotionEffect(
                                PotionEffectType.SPEED, 60,
                                (int) (CloneZ.zombieSpeed / 2));
 
                        ((CraftLivingEntity) e).addPotionEffect(effect);
 
                        Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(cloneZ, new Runnable() {
 
                            @Override
                            public void run() {
                                ((CraftPlayer) player).getHandle().netServerHandler
                                .sendPacket(new Packet42RemoveMobEffect(
                                        ((CraftPlayer) player).getHandle().id,
                                        new MobEffect(effect.getType().getId(), 0,
                                                0)));
                            }
 
                        }, 1L);
                    }
                }
            }
 
        }
 
    }
 
}
 

 

My programming teacher once told me that stack traces aren't always accurate. It could be from the cloneZ = instance

 

Oh OK

 

Where do you call that method?

 

It is a scheduler! (scheduleSyncRepeatingTask)

All of my other ones work