Solved Afk plugin doesn't seem to work

I'm trying to make a AFK plugin using a HashMap<UUID, Integer> but when i try using this:

Code:Java

 
public class RepeatRun {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public RepeatRun(Main plugin) {
        this.plugin = plugin;
        HashMap<UUID, Integer> isAfk = new AfkCheck().isAfk;
        new BukkitRunnable() {
            @Override
            public void run() {
                if (!plugin.getServer().getOnlinePlayers().isEmpty()) {
                    for (Player op : Bukkit.getOnlinePlayers()) {
                        if (op.isOnline()) {
                            if (isAfk.containsKey(op.getUniqueId())) {
                                isAfk.put(op.getUniqueId(), isAfk.get(op.getUniqueId())+1);
                                if (isAfk.get(op.getUniqueId()) >= 20*60*5) {
                                    WitherSkull skull = (WitherSkull) op.getWorld().spawn(op.getLocation().add(0, +1, 0), WitherSkull.class);
                                    skull.setDirection(new Vector(0, 0, 0));
                                    skull.setVelocity(new Vector(0, 0, 0));
                                    Entity barrier = op.getWorld().dropItem(op.getLocation().add(0, +2, 0), new ItemStack(Material.BARRIER));
                                    skull.setPassenger(barrier);
                                    new BukkitRunnable() {
                                        @Override
                                        public void run() {
                                            if (isAfk.get(op.getUniqueId()) < 20*60*5) {
                                                skull.remove();
                                                barrier.remove();
                                                cancel();
                                            }
                                        }
                                    }.runTaskTimer(plugin, 0, 20);
                                }
                            } else {
                                isAfk.put(op.getUniqueId(), 1);
                            }
                        }
                    }
                }
            }
        }.runTaskTimer(plugin, 0, 20);
    }
}
 

The count doesn't seem to go up, i have made a "/getafk" command to test if it does go up like this:

Code:Java

 
public class CommandGetAfk implements CommandExecutor {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public CommandGetAfk(Main plugin) {
        this.plugin = plugin;
        plugin.getCommand("getafk").setExecutor(this);
    }
 
    private HashMap<UUID, Integer> isAfk = new AfkCheck().isAfk;
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if (cmd.getName().equalsIgnoreCase("getafk")) {
            if (sender instanceof Player) {
                Player p = (Player) sender;
                if (isAfk.containsKey(p.getUniqueId())) {
                    sender.sendMessage("Afk time: "+isAfk.get(p.getUniqueId()));
                    return true;
                } else {
                    isAfk.put(p.getUniqueId(), 0);
                    sender.sendMessage("Afk time: "+isAfk.get(p.getUniqueId()));
                    return true;
                }
            } else {
                if (args.length == 0) {
                    sender.sendMessage("Only players may execute this command!");
                    return true;
                } else {
                    for (Player op : Bukkit.getOnlinePlayers()) {
                        if (op.getName().equalsIgnoreCase(args[0])) {
                            if (isAfk.containsKey(op.getUniqueId())) {
                                sender.sendMessage("Afk time: "+isAfk.get(op.getUniqueId()));
                                return true;
                            } else {
                                isAfk.put(op.getUniqueId(), 0);
                                sender.sendMessage("Afk time: "+isAfk.get(op.getUniqueId()));
                                return true;
                            }
                        }
                    }
                }
            }
        }
        return false;
    }
}
 

And it always returns "Afk time: 0", i've tried to find something that keeps setting it to 0 and i've also tried finding something that makes it so it doesn't go up but i've had no luck. :/
Here are some classes to set the time to 0 if you move/interact with something, i might've made a mistake in one of them but i can't seem to find it, maybe you can see what's wrong.

Code:Java

 
public class AsyncPlayerChat implements Listener {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public AsyncPlayerChat(Main plugin) {
        this.plugin = plugin;
        Bukkit.getPluginManager().registerEvents(this, plugin);
    }
 
    private HashMap<UUID, Integer> isAfk = new AfkCheck().isAfk;
 
    @EventHandler
    public void onChat(AsyncPlayerChatEvent e) {
        Player p = e.getPlayer();
        isAfk.put(p.getUniqueId(), 0);
        if (isAfk.get(p.getUniqueId()) >= 20*60*5) {
            p.sendMessage("You are no longer afk.");
        }
    }
}
 

Code:Java

 
public class CommandPreprocess implements Listener {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public CommandPreprocess(Main plugin) {
        this.plugin = plugin;
        Bukkit.getPluginManager().registerEvents(this, plugin);
    }
 
    private HashMap<UUID, Integer> isAfk = new AfkCheck().isAfk;
 
    @EventHandler
    public void onCommand(PlayerCommandPreprocessEvent e) {
        if (!e.getMessage().equalsIgnoreCase("/afk") || !e.getMessage().equalsIgnoreCase("/getafk")) {
            Player p = e.getPlayer();
            isAfk.put(p.getUniqueId(), 0);
            if (isAfk.get(p.getUniqueId()) >= 20*60*5) {
                p.sendMessage("You are no longer afk.");
            }
        }
    }
}
 

Code:Java

 
public class PlayerInteract implements Listener {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public PlayerInteract(Main plugin) {
        this.plugin = plugin;
        Bukkit.getPluginManager().registerEvents(this, plugin);
    }
 
    private HashMap<UUID, Integer> isAfk = new AfkCheck().isAfk;
 
    @EventHandler
    public void onInteract(PlayerInteractEvent e) {
        Player p = e.getPlayer();
        isAfk.put(p.getUniqueId(), 0);
        if (isAfk.get(p.getUniqueId()) >= 20*60*5) {
            p.sendMessage("You are no longer afk.");
        }
    }
}
 

Code:Java

 
public class PlayerMove implements Listener {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public PlayerMove(Main plugin) {
        this.plugin = plugin;
        Bukkit.getPluginManager().registerEvents(this, plugin);
    }
 
    private HashMap<UUID, Integer> isAfk = new AfkCheck().isAfk;
 
    @EventHandler
    public void onMove(PlayerMoveEvent e) {
        Player p = e.getPlayer();
        isAfk.put(p.getUniqueId(), 0);
        if (isAfk.get(p.getUniqueId()) >= 20*60*5) {
            p.sendMessage("You are no longer afk.");
        }
    }
}
 

Code:Java

 
public class PlayerQuit implements Listener {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public PlayerQuit(Main plugin) {
        this.plugin = plugin;
        Bukkit.getPluginManager().registerEvents(this, plugin);
    }
 
    private HashMap<UUID, Integer> isAfk = new AfkCheck().isAfk;
 
    @EventHandler
    public void onQuit(PlayerQuitEvent e) {
        Player p = e.getPlayer();
        isAfk.put(p.getUniqueId(), 0);
    }
}
 

EDIT: Seems i forgot to post a "/afk" command class(Just in case that is doing something):

Code:Java

 
public class CommandAfk implements CommandExecutor {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public CommandAfk(Main plugin) {
        this.plugin = plugin;
        plugin.getCommand("afk").setExecutor(this);
    }
 
    private HashMap<UUID, Integer> isAfk = new AfkCheck().isAfk;
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if (cmd.getName().equalsIgnoreCase("afk")) {
            if (sender instanceof Player) {
                Player p = (Player) sender;
                if (isAfk.containsKey(p.getUniqueId())) {
                    if (isAfk.get(p.getUniqueId()) < 20*60*5) {
                        isAfk.put(p.getUniqueId(), 20*60*5);
                        sender.sendMessage("You are now afk.");
                        return true;
                    } else {
                        isAfk.put(p.getUniqueId(), 0);
                        sender.sendMessage("You are no longer afk.");
                        return true;
                    }
                } else {
                    isAfk.put(p.getUniqueId(), 20*60*5);
                    sender.sendMessage("You are now afk.");
                    return true;
                }
            } else {
                sender.sendMessage("Only players may execute this command!");
                return true;
            }
        }
        return false;
    }
}
 

Btw i have no errors or anything.

 

A few hours ago you ran the BukkitRunnable every tick, now changed it to once per second. You probably have to change some numbers inside it doesn't take 100 minutes to become afk xD. The second runnable looks a bit unnecessary. Create listeners for movement and stuff and remove afk scull there

 

How? I tried moving this:

Code:Java

 
public HashMap<UUID, Integer> isAfk = new HashMap<UUID, Integer>();
 

to the main class and do this:

Code:Java

 
HashMap<UUID, Integer> isAfk = plugin.isAfk;
 

to get it but that comes with a NullPointerException at these lines:
In onEnable:

Code:Java

 
new CommandAfk(this);
 

In CommandAfk:

Code:Java

 
HashMap<UUID, Integer> isAfk = plugin.isAfk;
 

I'm sorry but i don't understand what you mean. :/

I don't know what you are talking about at first..? :7 But about the second runnable i don't know how i would remove the skull in another class(probably pretty easy when i think about it) but i wanna make the rest work before i fix that, anyway thanks for the tip.

 

Before calling new CommandAfk(this); you have to initialize your hashmap isAfk.
Maybe you can show us your whole Main class.

A clean code also barely contains any public variables. Create a getter to make it look more professional.

 

@MightyOne Here you go:

Code:Java

 
public class Main extends JavaPlugin {
 
    public HashMap<UUID, Integer> isAfk = new HashMap<UUID, Integer>();
 
    public void onEnable() {
        new CommandAfk(this);
        new CommandGetAfk(this);
        new AsyncPlayerChat(this);
        new CommandPreprocess(this);
        new PlayerInteract(this);
        new PlayerMove(this);
        new PlayerQuit(this);
        new RepeatRun(this);
    }
}
 

This is my current main class.
About creating a getter, i don't know what it is but from a quick google search i think it looks like this?:

Code:Java

 
public void whatever() { //Can add something in the ()
    //Do something
}
 

Not sure if that's a getter tho and if it is i would just get it like this, right?:

Code:Java

 
plugin.whatever();
 

From what i read about them it seems they are used so no one can update them without you knowing?
EDIT: If this is a getter then i've been using it a few times without knowing the name for it.

Anyway, what do you mean "initialize your hashmap"? I tried looking it up on google and the forum but had no luck. :/ (Sorry, this might be a dumb question)

 

Ok I am really impatient at this point. You don't ask dumb questions. It's just like you created a not just basic code and you don't know how to fix it. But if you don't know how to fix it you shouldn't create it.

Everything I can tell you is try to reimplement all of your RepeatRun in steps that small that you can always fix mistakes right away. You will learn a lot by doing that yourself.

Before doing all that you should look up the terms
declaration of a variable
initialization of one
getter methods and
setter methods

That is elementary, you will always need it in java, and without java you can forget bukkit. If anyone else can fix the problem woooh happy end :/

 

@MightyOne I'm sorry but thank you for the terms, i'll look them up and try and learn from what i find.
EDIT: I changed some things and it seems to work now. Thank you for the help MightyOne.