Hey there, this might be a much asked question, but i need it for my new plugin which im coding, So, anyways. im coding a warp plugin with a delay of 2 seconds. I coded everything with the commands, everything works except for the delay, here is my code: http://pastebin.com/9zE1mdaJ btw: events are registered in Main class, if u need just say Anyways, thanks for viewing, Regards, iAmGuus
Code:java examplearraylist.add(p.getName());Bukkit.getScheduler().scheduleSyncDelayedTask(this , new Runnable() {public void run() {examplearraylist.remove(p.getName());}}, 40);
Code:java Main.getServer().getScheduler().scheduleSyncDelayedTask(Main, new Runnable(){ public void run() { //Everything here is delayed }},(gethowevermanaysecondsyouwanttodelayby)*20);
Sorry, but wanted it so if he moves a block, not with his head, then the teleportation will be cancelled? I already tried it but doesnt work?
iAmGuus You will probably want to use what DarkRangerMC posted... The scheduleAsyncDelayedTask(), not Sync. Just considering what you will be doing. Here's an idea.... When the player sends the command to teleport, put him in an arraylist, and using event listeners... Listen for the playermove event. On that event, check and see if the player is in that arraylist. If he is, remove him. Then, in the delay, check and see if the player is in the arraylist before doing anything. If he moved, then he wont be anymore, and therefor wont be teleported. EDIT by Moderator: merged posts, please use the edit button instead of double posting.
DarkRangerMC xP Well it doesn't really matter either way. Not in his case. Sorry if I'm overcomplicating things.
xYourFreindx That is exactly what i did in my class? isnt it, or am i wrong? What i did here: Code:java ArrayList<String> delay = new ArrayList<String>(); public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) { if(sender instanceof Player) { final Player p = (Player) sender; if(label.equalsIgnoreCase("setwarp")) { if(p.hasPermission("experiencepvp.warps.setwarp")) { if(args.length == 0) { p.sendMessage(ChatColor.BLUE + "ExperiencePvP> " + ChatColor.RED + "You must specify a warp name!"); return true; } Main.warpsConfig.set("warps." + args[0] + ".world", p.getLocation().getWorld().getName()); Main.warpsConfig.set("warps." + args[0] + ".x", p.getLocation().getX()); Main.warpsConfig.set("warps." + args[0] + ".y", p.getLocation().getY()); Main.warpsConfig.set("warps." + args[0] + ".z", p.getLocation().getZ()); Main.warpsConfig.set("warps." + args[0] + ".yaw", p.getLocation().getYaw()); Main.warpsConfig.set("warps." + args[0] + ".pitch", p.getLocation().getPitch()); Main.saveNewConfig(); p.sendMessage(ChatColor.BLUE + "ExperiencePvP> " + ChatColor.WHITE + "Warp " + args[0] + " has been set at your location!"); } else { p.sendMessage("You may not run this command!"); } } if(label.equalsIgnoreCase("warp")) { if(p.hasPermission("experiencepvp.warps.warp")) { if(args.length == 0) { p.sendMessage(ChatColor.BLUE + "ExperiencePvP> " + ChatColor.WHITE + "Warps: " + Main.warpsConfig.getConfigurationSection("warps").getKeys(false)); return true; } if(Main.warpsConfig.getConfigurationSection("warps." + args[0] + "") == null) { p.sendMessage(ChatColor.BLUE + "ExperiencePvP> " + ChatColor.RED + "Warp " + args[0] + " does not exist!"); return true; } final World w = Bukkit.getServer().getWorld(Main.warpsConfig.getString("warps." + args[0] + ".world")); final double x = Main.warpsConfig.getDouble("warps." + args[0] + ".x"); final double z = Main.warpsConfig.getDouble("warps." + args[0] + ".z"); final double y = Main.warpsConfig.getDouble("warps." + args[0] + ".y"); p.sendMessage(ChatColor.BLUE + "ExperiencePvP> " + ChatColor.WHITE + "Stop moving, you will be teleported in 3 seconds!"); delay.add(p.getName()); new BukkitRunnable() { public void run() { if(delay.contains(p.getName())) { p.teleport(new Location(w, x, y, z)); p.sendMessage(ChatColor.BLUE + "ExperiencePvP> " + ChatColor.WHITE + "Teleported successfully!"); delay.remove(p.getName()); } } }.runTaskLater(Main.getPlugin(), 2 * 20); return true; } else { p.sendMessage(ChatColor.RED + "You may not warp!"); return true; } } if(label.equalsIgnoreCase("delwarp")) { if(p.hasPermission("experiencepvp.warp.delwarp")) { if(args.length == 0) { p.sendMessage(ChatColor.BLUE + "ExperiencePvP> " + ChatColor.RED + "You must specify a warp name!"); return true; } if(Main.warpsConfig.getConfigurationSection("warps." + args[0]) == null) { p.sendMessage(ChatColor.BLUE + "ExperiencePvP> " + ChatColor.WHITE + "Warp " + args[0] + " does not exist!"); return true; } Main.warpsConfig.set("warps." + args[0], null); p.sendMessage(ChatColor.BLUE + "ExperiencePvP> " + ChatColor.WHITE + "Warp " + args[0] + " has been successfully deleted!"); Main.saveNewConfig(); } } else { p.sendMessage(ChatColor.RED + "You may not warp!"); } } return true; } @EventHandler public void onMove(PlayerMoveEvent e) { Player p = e.getPlayer(); if(delay.contains(p.getName())) { if(e.getFrom().getX() != e.getTo().getX() || e.getFrom().getY() != e.getTo().getY() || e.getFrom().getZ() != e.getTo().getZ()) { delay.remove(p.getName()) ; p.sendMessage(ChatColor.BLUE + "ExperiencePvP> " + ChatColor.WHITE + "You moved, so you wont get teleported!"); } } } EDIT by Moderator: merged posts, please use the edit button instead of double posting.
iAmGuus That is exactly what you wrote. You know what they say about great minds... Anyways, you said it didn't work.... Is there a stacktrace I could look at?
Nop, there is no stacktrace, if i do /warp lol (for example) everything just works fine, except for that i can move while im in the arraylist, no message says that the teleport has been cancelled...
What's all this for? if(e.getFrom().getX()!= e.getTo().getX()|| e.getFrom().getY()!= e.getTo().getY()|| e.getFrom().getZ()!= e.getTo().getZ()){
That is to check if the player has moved a block, in the player move event? atleast, i thought. If i do remove it, then it will do nothing, tho
AdamQpzm I've got the definition of Async and Sync down.... But due to your condescending tone, I suppose I probably got the definition of what a "thread safe meathod" is.... Wrong.
xYourFreindx I don't mean to sound condescending But the vast majority of the Bukkit API isn't thread safe - your best bet is to never use any of it async (aside from the scheduler), especially when it's something like this, there's not really a justification for why it should be async.
iAmGuus It's 6 AM, I haven't slept... and I don't trust anything I say, but I don't want to leave you hanging here... If the player triggered the playermove event then he has obviously moved... So the only purpose I can see in thaat beautiful code there is to see if he moved a whole block. (to lazy to check and see if it returns an int or a double). Ummm... Make sure your events are registered correctly? AdamQpzm , you're right! What was I thinking.... iAmGuus , I'm sorry for my ignorance. *cough* thanks Adam, I need to redo my entire 73kb, 4 class plugin.... P.s I love your condescending tone. It makes me warm and fuzzy on the inside. EDIT by Moderator: merged posts, please use the edit button instead of double posting.
