Stupid Delay Tasks

Hey Guys, I have 2 problem with delays at the moment.

Firstly I am trying to make it so if a player is hit by a snowball they are frozen for 5 seconds, if freezes the player but doesn't seem to last 5 seconds, it lasts forever until /reload

Code:

Code:java

    @EventHandler
    public void onFreeze (EntityDamageByEntityEvent e){
        if(e.getEntity() instanceof Player && e.getDamager() instanceof Snowball) {
            Player p = (Player) e.getEntity();
            Snowball snowball = (Snowball) e.getDamager();
                    plugin.Snow.add(p.getName());
                    p.sendMessage(ChatColor.RED + "You've been frozen");
            }
        }
 
    @EventHandler
    public void onFreeze1 (PlayerMoveEvent e){
        Player p = e.getPlayer();
        if (plugin.Snow.contains(p.getName())) {
            e.setTo(e.getFrom());
            p.sendMessage(ChatColor.RED + "You cannot move, you are frozen!");
            delay(p);
 
        }
    }
 
        public void delay(final Player p){
        Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(this.plugin, new Runnable(){
            @Override
            public void run() {
                  p.sendMessage(ChatColor.RED + "You can now move!");
                  plugin.Snow.remove(p.getName());
 
                }
          }, 200);
        }

Second one is I am trying to hide the player who right clicks sugar goes invisible for 10 seconds then they are visible again. It doesn't seem to make the player invisible nor does it remove the player from 'invisibility'.

Code:

Code:java

    @EventHandler
    public void onSpy(PlayerInteractEvent e){
        Player p = e.getPlayer();
        if ((e.getAction() == Action.RIGHT_CLICK_AIR && p.getItemInHand().getType() == Material.SUGAR ||
                e.getAction() == Action.RIGHT_CLICK_BLOCK && p.getItemInHand().getType() == Material.SUGAR)){
 
            int cooldownTime = 35; // Get number of seconds from wherever you want
            if(spy.containsKey(p.getName())) {
            long secondsLeft = ((spy.get(p.getName())/1000)+cooldownTime) - (System.currentTimeMillis()/1000);
        if(secondsLeft>0) {
// Still cooling down
            p.sendMessage(ChatColor.RED + "Wait " + ChatColor.GOLD + secondsLeft + ChatColor.RED + " seconds to use this again!");
            return;
        }
}
// No cooldown found or cooldown has expired, save new cooldown
            spy.put(p.getName(), System.currentTimeMillis());
 
            for(Player p1 : Bukkit.getOnlinePlayers()){
                p.sendMessage(ChatColor.GOLD + "You have gone invisible!");
                p.hidePlayer(p1);   
 
                delay(p);
 
            }   
        }
    }
 
 
 
    public void delay(final Player p){
        Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable(){
                public void run(){
                    for(Player p1 : Bukkit.getOnlinePlayers()){
                        p.sendMessage(ChatColor.RED + "You are now visible!");
                        if(!(p1.canSee(p))){
                            p.showPlayer(p1); 
                            }
                        }
                    }
            }, 200L); // 20 ticks = 1 second. So 200 ticks = 10 seconds.
        }

 

DatCookiez
1. You set the first one to 10 seconds I believe, make it something like this

Code:java

          }, 5 * 20);

Also does the message you put even show up?

2. Does anything perform? Like messages?

 

Niknea

The "You are unfrozen" message does not show

and the message "You are invisible" shows

Niknea
The error for the snowball one.
http://pastebin.com/Jz4D5UWf

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

 

DatCookiez Check this out and see if you can figure it out

Code:

Caused by: java.lang.IllegalArgumentException: Plugin cannot be null
        at org.apache.commons.lang.Validate.notNull(Validate.java:203) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.validate(CraftScheduler.java:391) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.runTaskTimer(CraftScheduler.java:120) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.scheduleSyncRepeatingTask(CraftScheduler.java:116) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.scheduleSyncDelayedTask(CraftScheduler.java:100) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at me.mrgermanrain.main.FrozoneListener.delay(FrozoneListener.java:47) ~[?:?]
        at me.mrgermanrain.main.FrozoneListener.onFreeze1(FrozoneListener.java:41) ~[?:?]

 

Niknea
I'm not sure what's wrong?

Line 41:
delay(p);

Line 47:
Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable(){

 

DatCookiez Mind showing me your entire code?

 

Niknea
It's the first block of code in this thread

 

DatCookiez The WHOLE class please, including the imports and class name, therefore I can see what the real lines in the code are.

 

Niknea

Code:java

package me.mrgermanrain.main;
 
import java.util.HashMap;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
import org.bukkit.entity.Snowball;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.event.player.PlayerMoveEvent;
 
public class FrozoneListener implements Listener{
    public HashMap<String, Long> snow = new HashMap<String, Long>();
    Main plugin;
 
    public void ListenerClass(Main plugin) {
        this.plugin = plugin;
    }
 
    @EventHandler
    public void onFreeze (EntityDamageByEntityEvent e){
        if(e.getEntity() instanceof Player && e.getDamager() instanceof Snowball) {
            Player p = (Player) e.getEntity();
            Snowball snowball = (Snowball) e.getDamager();
                    plugin.Snow.add(p.getName());
                    p.sendMessage(ChatColor.RED + "You've been frozen");
            }
        }
 
    @EventHandler
    public void onFreeze1 (PlayerMoveEvent e){
        Player p = e.getPlayer();
        if (plugin.Snow.contains(p.getName())) {
            e.setTo(e.getFrom());
            p.sendMessage(ChatColor.RED + "You cannot move, you are frozen!");
            delay(p);
 
        }
    }
 
        public void delay(final Player p){
          Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable(){
            public void run() {
                  p.sendMessage(ChatColor.RED + "You can now move!");
                  plugin.Snow.remove(p.getName());
 
                }
          }, 5 * 20);
        }
 
 
}

*I removed 3 imports because they were not being moved*

 

DatCookiez Stacktraces include line numbers that help you and everyone else narrow down the problem, if you change the code, the line numbers are no longer valid, and makes everyone's life that much harder.

 

DatCookiez Can you show us the new stack trace?

 

Niknea
The error is still the same and it's the same 2 lines

 

DatCookiez I now don't know what line the error is on, as you removed a few things.

 

Niknea

Error:

Code:

[16:17:08 ERROR]: Could not pass event PlayerMoveEvent to DatKitPvP v1.0
org.bukkit.event.EventException
    at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:427) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:481) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:466) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at net.minecraft.server.v1_7_R1.PlayerConnection.a(PlayerConnection.java:227) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at net.minecraft.server.v1_7_R1.PacketPlayInFlying.a(SourceFile:137) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at net.minecraft.server.v1_7_R1.PacketPlayInLook.handle(SourceFile:98) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at net.minecraft.server.v1_7_R1.NetworkManager.a(NetworkManager.java:146) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at net.minecraft.server.v1_7_R1.ServerConnection.c(SourceFile:134) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at net.minecraft.server.v1_7_R1.MinecraftServer.u(MinecraftServer.java:655) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at net.minecraft.server.v1_7_R1.DedicatedServer.u(DedicatedServer.java:250) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at net.minecraft.server.v1_7_R1.MinecraftServer.t(MinecraftServer.java:545) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at net.minecraft.server.v1_7_R1.MinecraftServer.run(MinecraftServer.java:457) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at net.minecraft.server.v1_7_R1.ThreadServerApplication.run(SourceFile:617) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
Caused by: java.lang.IllegalArgumentException: Plugin cannot be null
    at org.apache.commons.lang.Validate.notNull(Validate.java:203) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.validate(CraftScheduler.java:391) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.runTaskTimer(CraftScheduler.java:120) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.scheduleSyncRepeatingTask(CraftScheduler.java:116) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.scheduleSyncDelayedTask(CraftScheduler.java:100) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    at me.mrgermanrain.main.ClassListeners.FrozoneListener.delay(FrozoneListener.java:84) ~[?:?]
    at me.mrgermanrain.main.ClassListeners.FrozoneListener.onFreeze1(FrozoneListener.java:79) ~[?:?]
    at sun.reflect.GeneratedMethodAccessor56.invoke(Unknown Source) ~[?:?]
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.7.0_13]
    at java.lang.reflect.Method.invoke(Method.java:601) ~[?:1.7.0_13]
    at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:425) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
    ... 13 more

Code:

Code:java

package me.mrgermanrain.main.ClassListeners;
 
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
 
import me.mrgermanrain.main.Main;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
import org.bukkit.entity.Snowball;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.player.PlayerMoveEvent;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
public class FrozoneListener implements Listener{
    public HashMap<String, Long> snow = new HashMap<String, Long>();
 
    Main plugin;
 
    public void ListenerClass(Main plugin) {
        this.plugin = plugin;
    }
 
    @SuppressWarnings("unused")
    @EventHandler
    public void onPlayerFreeze(PlayerInteractEvent e){
        Player p = e.getPlayer();
        List<Entity> nearbyEntities = p.getNearbyEntities(10, 10, 10);
        ArrayList<Player> nearbyPlayers = new ArrayList<Player>();
                if ((e.getAction() == Action.LEFT_CLICK_BLOCK && e.getPlayer().getItemInHand().getType()
                        == Material.SNOW_BLOCK
                        || e.getAction() == Action.LEFT_CLICK_AIR && e.getPlayer().getItemInHand().getType()
                        == Material.SNOW_BLOCK)){
 
                    int cooldownTime = 20; // Get number of seconds from wherever you want
                    if(snow.containsKey(p.getName())) {
                    long secondsLeft = ((snow.get(p.getName())/1000)+cooldownTime) - (System.currentTimeMillis()/1000);
                    if(secondsLeft>0) {
            // Still cooling down
                        p.sendMessage(ChatColor.RED + "Wait " + ChatColor.GOLD + secondsLeft + ChatColor.RED + " seconds to use this again!");
                        return;
                    }
            }
            // No cooldown found or cooldown has expired, save new cooldown
                    snow.put(p.getName(), System.currentTimeMillis());
 
                    p.getLocation().getWorld();
                        for (Entity e1 : nearbyEntities) {
                            if (e1 instanceof Player) {
                                nearbyPlayers.add((Player) e1);
                        for (Player p1 : nearbyPlayers) {
                            p1.getLocation().getWorld();
                            Main.Snow.add(p1.getName());
                            p1.sendMessage(ChatColor.GOLD + p.getName() + ChatColor.RED + " has frozen you!!");
 
                    }
                }
            }
        }
    }
 
    @EventHandler
    public void onFreeze1 (PlayerMoveEvent e){
        Player p = e.getPlayer();
        if (Main.Snow.contains(p.getName())) {
            e.setTo(e.getFrom());
            p.sendMessage(ChatColor.RED + "You cannot move, you are frozen!");
            delay(p);
        }
    }
 
        public void delay(final Player p){
          Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable(){
            public void run() {
                  p.sendMessage(ChatColor.RED + "You can now move!");
                  Main.Snow.remove(p.getName());
 
                }
          }, 5 * 20);
        }
 
 
}
 

 

You're scheduling it using the base Bukkit class, which is null because there's no server associated with the base class. You need to do plugin.getServer() and schedule it from that.

 

Maurdekye
Still doesn't seem to be working

 

are you actually passing an instance of your main class into the Listener class or?

 

Bump

 

Bump

 

DatCookiez Can I see your Main class?

 

xTigerRebornx
Main class? What do you want to see in it? I have a lot of personal code in there

 

DatCookiez Well, its getting a null instance of your plugin from somewhere. You shouldn't worry about people taking your code, if thats what you mean when you say "personal code"

EDIT: Nevermind, found the problem

Code:

public void ListenerClass(Main plugin) {
        this.plugin = plugin;
    }

My guess is this is meant to be your constructor, fix this

 

xTigerRebornx ???

Code:

    public void FrozoneClass(Main plugin) {
        this.plugin = plugin;
    }

 

DatCookiez Go learn what a constructor is (and how to make one)