Repeat a task until a command is run.

What I want is like a toggle command, but Repeating a playEffect. With a slight delay. Essentially what I'm trying to do is once I run /smoke, I want smoke to follow me around. I know how to do the toggling, with Hashmaps, but I need help with the delay.

Every few ticks, when command is toggled on:

Code:java

 
Location loc = player.getLocation();
World world = player.getWorld();
 world.playEffect(loc,Effect.SMOKE,4);
 

Can anyone help?

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

 

This is a rough explanation of how I would do it, but something like this should get the job done.
Have a variable that holds whether a player should have smoke. So for example:

Code:

ArrayList<Player> isSmokeEnabled = new ArrayList<Player>();

Then, when a player calls the /smoke command, add their name to the list. Every tick in the game, call the smoke effect on that player.

 

Thats exactly what I need to know how to do.

 

Make a list that contains the players with smoke enable:

Code:java

ArrayList<Player> smoke = new Arraylist<Player>();

Make a runnable that makes smoke follow them

Code:java

private class SmokeRunnable implements Runnable
{
    public void run()
    {
        for(Player p:getServer().getOnlinePlayers())
        {
            if(smoke.contains(p))p.getWorld().playEffect(p.getLocation(), Effect.SMOKE, 0);
        }
    }
}

Schedule it on your on enable

Code:java

getServer().getScheduler().scheduleSyncRepeatingTask(this, new SmokeRunnable(), 0, 20);//The last argument is ticks, 20 ticks = 1 second

Toggling the list

Code:java

if(smoke.contains(player))smoke.remove(player);
else smoke.add(player);

 

Thank you so much guys!!!!

 

well... you can make it a bit faster. Why don't you just go through the list?

 

I changed it to my needs, also with getting the direction of the player for direction of smoke. But how do u test if the player is standing still?

Maybe a hashmap

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

 

The player could be offline, and it would make the smoke look weird.

Have a HashMap with new locations, and have a HashMap with old locations. If the new location is different from the old location, play the effect.

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

 

Can u tell me how? I dont get how you would.

 

Anybody???

 

Code:java

HashMap<Player, Location> oldloc = new HashMap<Player, Location>();
HashMap<Player, Location> newloc = new HashMap<Player, Location>();
 
@EventHandler(priority = EventPriority.MONITOR)
public void onPlayerMove(PlayerMoveEvent event)
{
    if(event.isCancelled())return;
    //whatever you need to do here...
   //....
    newloc.put(event.getPlayer(), event.getPlayer().getLocation());
    if(oldloc.containsKey(player))
    {
        Location o = oldloc.get(player);
        Location n = newlod.get(player);
        if(o.getX() == n.getX() && o.getY() == n.getY() && o.getZ() == n.getZ())
        {
            //Use the code here
            //.........
 
            //This is optional, but if you want to check if the player's is looking in the same place, move the above here
            if(o.getYaw() == n.getYaw() && o.getPitch() == n.getPitch())
            {
 
            }
        }
    }
}