Redstone... with timer...

I am making a plugin that gives vanishes you for 7 seconds once you right click a redstone...
this is my first project really working with a timer.

Here is my code:

Code:java

package me.mrgermanrain.scheal;
 
import java.util.logging.Logger;
 
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
 
 
public class scheal extends JavaPlugin implements Listener{
public final Logger logger = Logger.getLogger("Minecraft");
public static scheal plugin;
public int number = 7;
 
    @Override
    public void onDisable() {
        PluginDescriptionFile pdfFile = this.getDescription();
        this.logger.info(pdfFile.getName() + " has been Disabled!");
}
 
    @Override
    public void onEnable() {
        PluginDescriptionFile pdfFile = this.getDescription();
        this.logger.info(pdfFile.getName() + " Version " + pdfFile.getVersion() + " has been Enabled!");
        getServer().getPluginManager().registerEvents(this, this);
}
 
    @EventHandler
    public void PlayerIteractEvent(PlayerInteractEvent e){
 
        if(e.getItem().getType() == Material.REDSTONE){
            if(e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK){
                final Player player = e.getPlayer();   
                for (final Player p : plugin.getServer().getOnlinePlayers()) {       
                        p.hidePlayer(p);
                        player.sendMessage(ChatColor.GRAY + "[" + ChatColor.DARK_AQUA + "Infamy" + ChatColor.GRAY + "] " + "You have now been vanished for 7 seconds");
                        this.getServer().getScheduler().scheduleSyncRepeatingTask(
                                this,
                                new Runnable() {
                                public void run(){
                                    if(number != -1){                                                       
                                    if(number != 7){
                                    player.sendMessage(ChatColor.GRAY + "[" + ChatColor.DARK_AQUA + "Infamy"+ ChatColor.GRAY + "] " + number);
                                    number--;                                 
                                    }else{
                                        p.showPlayer(player);
                                        p.sendMessage(ChatColor.GRAY + "[" + ChatColor.DARK_AQUA + "Infamy"
                                                + ChatColor.GRAY + "] " + "You are now unvanished!");
                                        number--;
                                    }
                                    }
                                }                               
                                },
                                0L,
                                20L);
 
            }
 
            }
 
 
        }
    }
 
    }
 
 

I don't get error messages though when I right click nothing happens.

 

MrGermanrain
Okay... first thing is first
.hidePlayer(Player p) hides the player inputted from the player its called for... so to hide that player from everyone you'd do:

Code:java

Player p = (Player) event.getEntity();
for(Player pl : Bukkit.getOnlinePlayers()){
pl.hidePlayer(p);
}

MrGermanrain
Okay... after some work I rewrote this, and I hope you can learn from it
Task:

Code:

        getServer().getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
            @Override
            public void run() {
                for (Entry<String, Long> en : players.entrySet()) {
                    Player p = Bukkit.getPlayerExact(en.getKey());
                    if (System.currentTimeMillis() - en.getValue() >= 7000) {
                        for (Player pl : Bukkit.getOnlinePlayers()) {
                            pl.showPlayer(Bukkit.getPlayerExact(en.getKey()));
                        }
                        players.remove(en.getKey());
                        p.sendMessage(ChatColor.GRAY + "[" + ChatColor.DARK_AQUA + "Infamy" + ChatColor.GRAY + "] " + "You are now unvanished!");
                    } else {
                        p.sendMessage(ChatColor.GRAY + "[" + ChatColor.DARK_AQUA + "Infamy" + ChatColor.GRAY + "] " + (7 - ((System.currentTimeMillis() - en.getValue()) / 1000)));
                    }
                }
            }
        }, 0L, 20L);

Listener:

Code:

@EventHandler
    public void PlayerIteractEvent(PlayerInteractEvent e) {
        if (e.getItem().getType() == Material.REDSTONE) {
            if (e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK) {
                final Player p = e.getPlayer();
                for (Player pl : Bukkit.getOnlinePlayers()) {
                    pl.hidePlayer(p);
                }
                players.put(p.getName(), System.currentTimeMillis());
                p.sendMessage(ChatColor.GRAY + "[" + ChatColor.DARK_AQUA + "Infamy" + ChatColor.GRAY + "] " + "You have now been vanished for 7 seconds");
            }
 
        }
 
    }

HashMap:

Code:

final HashMap<String, Long> players = new HashMap<String, Long>();

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

 

Thanks Ill see if this works!

 

MrGermanrain
I tested it and it is working for me (Well the timer part is... and I THINK the hiding is too... but I don't have a second player to view it so... you'll have to test that out yourself )

 

thanks I am terrible with HashMaps I need to figure this out


EDIT: Yes, I cant figure this out D: