Solved Getting ArmorStand

I want to get the armorstand i've created using this:

Code:Java

 
ArmorStand stand = p.getWorld().spawn(p.getLocation().add(0, +1, 0), ArmorStand.class);
 

how would i do this? (I've been trying for some time now and can't get any ideas of how to do this)
I've tried looking this up but results were to save it in a List or HashMap but how would i get this aswell?

Code:Java

 
Item barrier = (Item) p.getWorld().dropItem(p.getLocation().add(0, +2, 0), new ItemStack(Material.BARRIER));
 

I want to get both. :7
Here is the current code i use for creating it:

Code:Java

 
ArmorStand stand = op.getWorld().spawn(op.getLocation().add(0, +1, 0), ArmorStand.class);
stand.setGravity(false);
stand.setVisible(false);
Item barrier = (Item) op.getWorld().dropItem(op.getLocation().add(0, +2, 0), new ItemStack(Material.BARRIER));
barrier.setPickupDelay(Integer.MAX_VALUE);
stand.setPassenger(barrier);
 

My end goal with this is to create what i already have here but only 1 per player(right now i have it in a loop so i want to be able to check if it is already there(the armor stand and barrier item)) and then remove it using a PlayerMoveEvent or something like that instead of what i currently have which is:

Code:Java

 
ArmorStand stand = op.getWorld().spawn(op.getLocation().add(0, +1, 0), ArmorStand.class);
stand.setGravity(false);
stand.setVisible(false);
Item barrier = (Item) op.getWorld().dropItem(op.getLocation().add(0, +2, 0), new ItemStack(Material.BARRIER));
barrier.setPickupDelay(Integer.MAX_VALUE);
stand.setPassenger(barrier);
new BukkitRunnable() {
    @Override
    public void run() {
        if (plugin.getAfk(op.getUniqueId()) < 60*5) {
            stand.remove();
            barrier.remove();
            cancel();
        }
    }
}.runTaskTimer(plugin, 0, 20);
 

I hope i explained myself well enough.

 

@KarimAKL

I see you chose the armor stand route.

You want to get both from a HashMap/List?

A better way would be to do this:
1. Create a AFK class per player and save it in a HashMap (Unless you do step 5 inside your class).
2. In the class create 3 variables: the stand, the afk boolean to tell if the player is afk, and barrier
3. Create a method that removes the stand and barrer when player is unafk for the move event to call
4. Also in the class have the exact same thing as you have above with all the checks and loops to see if the player is afk or not.
5. On PlayerMoveEvent, get the class check if user is afk with the boolean getter method and activate the method that removes the barrier and stand.

Hope that made sense.

 

@CommonSenze Yes, i tried the armor stand route and it was actually exactly what i wanted. Anyway, about this:

No, i meant that i wanted to get both the armor stand and the barrier. Anyway what do you mean by "Create a AFK class per player"? This might be easier for you to explain if i gave you some more classes to look at.
Here you go:
Main class:

Code:Java

 
public class Main extends JavaPlugin {
 
    private HashMap<UUID, Integer> isAfk = new HashMap<UUID, Integer>();
 
    public void setAfk(UUID uuid, Integer integer) {
        this.isAfk.put(uuid, integer);
    }
 
    public Integer getAfk(UUID uuid) {
        return this.isAfk.get(uuid);
    }
 
    public boolean existAfk(UUID uuid) {
        return this.isAfk.containsKey(uuid);
    }
 
    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);
    }
}
 

RepeatRun class(You can see what it does):

Code:Java

 
public class RepeatRun {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public RepeatRun(Main plugin) {
        this.plugin = plugin;
        new BukkitRunnable() {
            @Override
            public void run() {
                if (!plugin.getServer().getOnlinePlayers().isEmpty()) {
                    for (Player op : Bukkit.getOnlinePlayers()) {
                        if (op.isOnline()) {
                            if (plugin.existAfk(op.getUniqueId())) {
                                if (plugin.getAfk(op.getUniqueId()) >= 60*5) {
                                    ArmorStand stand = op.getWorld().spawn(op.getLocation().add(0, +1, 0), ArmorStand.class);
                                    stand.setGravity(false);
                                    stand.setVisible(false);
                                    Item barrier = (Item) op.getWorld().dropItem(op.getLocation().add(0, +2, 0), new ItemStack(Material.BARRIER));
                                    barrier.setPickupDelay(Integer.MAX_VALUE);
                                    stand.setPassenger(barrier);
                                    new BukkitRunnable() {
                                        @Override
                                        public void run() {
                                            if (plugin.getAfk(op.getUniqueId()) < 60*5) {
                                                stand.remove();
                                                barrier.remove();
                                                cancel();
                                            }
                                        }
                                    }.runTaskTimer(plugin, 0, 20);
                                } else {
                                    plugin.setAfk(op.getUniqueId(), plugin.getAfk(op.getUniqueId())+1);
                                }
                            } else {
                                plugin.setAfk(op.getUniqueId(), 1);
                            }
                        }
                    }
                }
            }
        }.runTaskTimer(plugin, 0, 20);
    }
}
 

PlayerMove class:

Code:Java

 
public class PlayerMove implements Listener {
 
    private Main plugin;
 
    public PlayerMove(Main plugin) {
        this.plugin = plugin;
        Bukkit.getPluginManager().registerEvents(this, plugin);
    }
 
    @EventHandler
    public void onMove(PlayerMoveEvent e) {
        Player p = e.getPlayer();
        if (plugin.getAfk(p.getUniqueId()) >= 60*5) {
            p.sendMessage("You are no longer afk.");
        }
        plugin.setAfk(p.getUniqueId(), 0);
    }
}
 

Hope this helps you explain it to me better because i didn't fully understand what you meant when you explained those 5 steps.
EDIT: (I'm using "if (plugin.getAfk(p.getUniqueId()) >= 60*5) {" to check if the player is afk, should i instead put them in a HashSet when they reach the 60*5?)

 

@KarimAKL
Well what I mean by create an AFK class per player is like this.

Create a class called AFK or something.
In another class called AFKManager or something, make a hashmap variable that stores UUIDs as keys and the AFK class as Values.
Create a addAFK methods that takes in a player and put the players UUID with a new AFK class with the contructor taking in the player.
Also create a removeAFK that takes in a player and removes the players uuid from the hashmap when they leave to conserve space

 

@CommonSenze So what you want is for me to create something like this?:

Code:Java

 
public class AFK {
 
}
 

And then this?:

Code:Java

 
public class AFKManager {
 
    private HashMap<UUID, Class> whatever = new HashMap<UUID, Class>();
 
    public void addAFK(Player p) {
        this.whatever.put(p.getUniqueId(), new AFK().getClass());
    }
 
    public void removeAFK(Player p) {
        this.whatever.remove(p.getUniqueId());
    }
}
 

Or did i misunderstand that completely? (I feel like i misunderstood :7 If not then what do you want me to do with the HashMap?)

 

@KarimAKL
Instead of make a HashMap saving a UUID and Class save the UUID and AFK class like this:

Code:java

private HashMap<UUID, AFK> whatever = new HashMap<>();

Other than that, you have the basic concept. I do the same for a lot of my code. It allows me to have a lot of code in bite sized pieces and allows me to reuse code for many players. I do the same thing for my Users class that gives me more methods for the Player interface.

 

@CommonSenze Okay, but then what should i do with it? (I still don't get why i should make this HashMap)

 

@KarimAKL
In this class you will store all the information for being AFK (ie. The Stand, the barrier, the time of how long is needed for them to stand still to be considered afk, the methods to activate a player being afk and placing the stand and barrier down, etc) So that way you are able to have specific barriers and stands per player.

 

@CommonSenze I have this now:

Code:Java

 
public class AFKManager {
 
    private HashMap<UUID, AFK> whatever = new HashMap<UUID, AFK>();
    public void addAFK(Player p) {
        this.whatever.put(p.getUniqueId(), new AFK());
    }
    public void removeAFK(Player p) {
        this.whatever.remove(p.getUniqueId());
    }
 
    public boolean isAFK(Player p) {
        return this.whatever.containsKey(p.getUniqueId());
    }
}
 

And:

Code:Java

 
public class AFK {
 
    public void stand(Player p) {
        ArmorStand stand = p.getWorld().spawn(p.getLocation().add(0, +1, 0), ArmorStand.class);
        stand.setGravity(false);
        stand.setVisible(false);
        Item barrier = (Item) p.getWorld().dropItem(p.getLocation().add(0, +2, 0), new ItemStack(Material.BARRIER));
        barrier.setPickupDelay(Integer.MAX_VALUE);
        stand.setPassenger(barrier);
    }
}
 

I'm lost on what to do next. :7

 

@KarimAKL

I do understand I am being confusing on my explanation and do sincerely apologize. I am use to talking out my explanation rather than typing it.

I'll try to explain step by step what to do inside the AFK class:
1. Create these instance variables:

Code:java

private ArmorStand stand;
private Item barrier;
private boolean afk = false;

2. Create a method that makes 2 local variables: the stand and barrier; with those two variables, set the boolean afk that we made to true, spawn them in the world, and make the instance variables you made in step one equal them like so:

Code:java

this.stand = stand;
this.barrier = barrier

3. Create a remove method that uses the 2 instance variables: stand and barrier; to remove those two items in the world when the player is not afk, also set afk to false
4. Make a getter for the boolean afk in the AFK class
Now that is all you need for the AFK class. Now in the PlayerMoveEvent do this

Code:java

if (plugin.getAFKManager().getAFK(e.getPlayer()).isAFK()){
    plugin.getAFKManager().getAFK(e.getPlayer()).stopAFK();
}

Now the isAFK method and stopAFK method are what I described in the steps, I just game them names on what I would name them.

As I saw in your code, you have a way to check if the player is afk or not (The plugin.getAFK(p.getUniqueID()) thing and so one), keep all that code except in the part where you create the armor stand and all of that do this

Code:java

if (plugin.getAfk(op.getUniqueId()) >= 60*5) {
    AFK afk = plugin.getAFKManager().getAFK(op).startAFK();
} else {
    plugin.setAfk(op.getUniqueId(), plugin.getAfk(op.getUniqueId())+1);
}

Now to explain the getAFKManager(): You will have a variable in your Main class that will have the AFKManager variable like this:

Code:java

private AFKManager afkManager;

And onEnable() would look like this:

Code:java

this.afkManager = new AFKManager();

And finally the getter method

Code:java

public AFKManager getAFKManager() {
    return afkManager;
}

I hope I explained well enough about how I would go about that problem. I am a little rusty at explaining on forums. Like I said, I'm no forums guy, I'm use to talking out my explanations not typing them out.

 

@CommonSenze Wow, thanks for taking the time to write and explain that. I'll try now, thanks again. (I just hope i understood all of it correctly )
EDIT: I think i did something wrong. :/ (I've tried finding other ways but couldn't) Do you think you could take a look at the current code and point me in the right direction? :7
Current AFK class:

Code:Java

 
public class AFK {
 
    private ArmorStand stand;
    private Item barrier;
    private boolean afk = false;
 
    public void createStand(Player p) {
        this.afk = true;
        ArmorStand stand = p.getWorld().spawn(p.getLocation().add(0, +1, 0), ArmorStand.class);
        stand.setGravity(false);
        stand.setVisible(false);
        Item barrier = (Item) p.getWorld().dropItem(p.getLocation().add(0, +2, 0), new ItemStack(Material.BARRIER));
        barrier.setPickupDelay(Integer.MAX_VALUE);
        stand.setPassenger(barrier);
        this.stand = stand;
        this.barrier = barrier;
    }
 
    public void removeStand() {
        this.afk = false;
        this.stand.remove();
        this.barrier.remove();
    }
 
    public boolean getAFK(Player p) {
        return this.afk;
    }
}
 

Current AFKManager:

Code:Java

 
public class AFKManager {
 
    private HashMap<UUID, AFK> whatever = new HashMap<UUID, AFK>();
    public void addAFK(Player p) {
        this.whatever.put(p.getUniqueId(), new AFK());
    }
    public void removeAFK(Player p) {
        this.whatever.remove(p.getUniqueId());
    }
 
    public boolean isAFK(Player p) {
        return this.whatever.containsKey(p.getUniqueId());
    }
}
 

Current Main:

Code:Java

 
public class Main extends JavaPlugin {
 
    private HashMap<UUID, Integer> isAfk = new HashMap<UUID, Integer>();
 
    public void setAfk(UUID uuid, Integer integer) {
        this.isAfk.put(uuid, integer);
    }
 
    public Integer getAfk(UUID uuid) {
        return this.isAfk.get(uuid);
    }
 
    public boolean existAfk(UUID uuid) {
        return this.isAfk.containsKey(uuid);
    }
 
    private AFKManager afkManager;
 
    public AFKManager getAFKManager() {
        return afkManager;
    }
 
    public void onEnable() {
        this.afkManager = new AFKManager();
        new CommandAfk(this);
        new CommandGetAfk(this);
        new AsyncPlayerChat(this);
        new CommandPreprocess(this);
        new PlayerInteract(this);
        new PlayerMove(this);
        new PlayerQuit(this);
        new RepeatRun(this);
    }
}
 

 

@KarimAKL
You seem to have some of what I said mixed up.

You did start in the right direction for most but here I'll explain everything you got wrong and why.

First,

Code:java

public boolean getAFK(Player p) {
    return this.afk;
}

Is wrong in 3 places:
1. The getAFK method is suppose to be in the AFKManager class because the AFKManager is the Manager for the AFK class. With the AFKManager being the Manager, it should "get the AFK Class" which is why getAFK is suppose to be in that class.
2. That is a boolean "True/False" so you have to word it like a yes or no question thats why in my previous response I said I made a method called "isAFK()". That was the method that needed to be the getter for the boolean.
3. The Player p variable is useless since you never needed it in the first place. Getters are a type of method, all they do is give the variable it is for. In this case, its the boolean named AFK so you won't need any arguments.

Second,

Code:java

private HashMap<UUID, Integer> isAfk = new HashMap<UUID, Integer>();
 
public void setAfk(UUID uuid, Integer integer) {
    this.isAfk.put(uuid, integer);
}
 
public Integer getAfk(UUID uuid) {
    return this.isAfk.get(uuid);
}
 
public boolean existAfk(UUID uuid) {
    return this.isAfk.containsKey(uuid);
}

This is not needed at all. You misread me when I was talking about the "isAFK()". That's going to be the method name of the Getter of the boolean afk in the AFK class.

Other than that. I think you are good. Test it out and see if it works. Come back and send me the errors if any and the class of where it is.

 

@CommonSenze The code you wrote there, is that what i'm supposed to do in the AFKManager class? If so then what about the HashMap<UUID, AFK>? Anyway, i think that cleared up everything you said, after you answer the question in this post i'll try again.

 

@KarimAKL
Change getAFK in AFK class to isAFK()
In afkmanager you wont need the isAFK(Player p) method. You can remove that.
Also on Player Join Event make sure to use the addAFK and on Quit removeAFK methods in the AFK Manager class to add the afk class for that player

 

@CommonSenze I've done that now, here is the current code:
Main class:

Code:Java

 
public class Main extends JavaPlugin {
 
    private HashMap<UUID, Integer> isAfk = new HashMap<UUID, Integer>();
 
    public void setAfk(UUID uuid, Integer integer) {
        this.isAfk.put(uuid, integer);
    }
 
    public Integer getAfk(UUID uuid) {
        return this.isAfk.get(uuid);
    }
 
    public boolean existAfk(UUID uuid) {
        return this.isAfk.containsKey(uuid);
    }
 
    private AFKManager afkManager;
 
    public AFKManager getAFKManager() {
        return afkManager;
    }
 
    public void onEnable() {
        this.afkManager = new AFKManager();
        new CommandAfk(this);
        new CommandGetAfk(this);
        new AsyncPlayerChat(this);
        new CommandPreprocess(this);
        new PlayerInteract(this);
        new PlayerMove(this);
        new PlayerQuit(this);
        new RepeatRun(this);
    }
}
 

AFK class:

Code:Java

 
public class AFK {
 
    private ArmorStand stand;
    private Item barrier;
    private boolean afk = false;
 
    public void createStand(Player p) {
        this.afk = true;
        ArmorStand stand = p.getWorld().spawn(p.getLocation().add(0, +1, 0), ArmorStand.class);
        stand.setGravity(false);
        stand.setVisible(false);
        Item barrier = (Item) p.getWorld().dropItem(p.getLocation().add(0, +2, 0), new ItemStack(Material.BARRIER));
        barrier.setPickupDelay(Integer.MAX_VALUE);
        stand.setPassenger(barrier);
        this.stand = stand;
        this.barrier = barrier;
    }
 
    public void removeStand() {
        this.afk = false;
        this.stand.remove();
        this.barrier.remove();
    }
 
    public boolean isAFK() {
        return this.afk;
    }
}
 

AFKManager class:

Code:Java

 
public class AFKManager {
 
    private HashMap<UUID, AFK> whatever = new HashMap<UUID, AFK>();
    public void addAFK(Player p) {
        this.whatever.put(p.getUniqueId(), new AFK());
    }
    public void removeAFK(Player p) {
        this.whatever.remove(p.getUniqueId());
    }
}
 

About the Main class, the code in there is used for this:

Code:Java

 
if (plugin.existAfk(op.getUniqueId())) {
if (plugin.getAfk(op.getUniqueId()) >= 60*5) {
plugin.setAfk(op.getUniqueId(), 1);
 

I guess i should do it another way, huh? Anyway, about the isAFK(), it should be in the AFKManager class, right? But how do i gt the afk boolean from there? It's private isn't it? I'm a little confused about that right now. :7

 

@KarimAKL
The instance variable might be private but the getter method isn't. That's what getter methods are for. So that isAFK() method in your AFK Class is the getter method for your afk boolean.

So you can use that method in the AFKManager class by doing the following

Code:java

 
    public boolean isAfk(Player p) {
        return this.whatever.get(p.getUniqueId()).isAFK();
    }
 

That method needs to be in the AFK Manager. What it's doing is getting the AFK class for that player and checking what the boolean afk is; true or false.

And its that method that you're going to use in the PlayerMoveEvent when a player moves you will call the method like so:

Code:java

 
    public class PlayerMove implements Listener {
 
        private Main plugin;
 
        public PlayerMove(Main plugin) {
            this.plugin = plugin;
            Bukkit.getPluginManager().registerEvents(this, plugin);
        }
 
        @EventHandler
        public void onMove(PlayerMoveEvent e) {
            Player p = e.getPlayer();
            if (plugin.getAfk(p.getUniqueId()) >= 60*5&&plugin.getAFKManager().isAFK(p)) {
                p.sendMessage("You are no longer afk.");
                plugin.getAFKManager().getAFK(p).removeStand();
            }
            plugin.setAfk(p.getUniqueId(), 0);
        }
    }
 

 

@CommonSenze Quick question. What does the getAFK(p) do? I don't have that in my AFKManager class so what should i make it do?

 

@KarimAKL
I realized 2 things were missing in your AFKManager and AFK class.
1. AFKManager needs the getAFK(Player p) method that returns the AFK class for that player. This is saved in the whatever variable HashMap. You have the AFK class as the value so all you do is get the UUID and it will return the value of the AFK class for that specific player

2. The AFK Class should have the constuctor that takes in the Player like so:

Code:java

 
    private Player p;
    public AFK(Player p) {
        this.p = p;
    }

Now, I SINCERELY APOLOGIZE, but I forgot to tell you to make that constuctor and also make another instance variable containing the player like I did above.

This way you can remove the Player p argument in the createStand methods so it won't need to take in a player. All it will do is use the instance variable of the player.

Also, you will need to make sure that in your AFKManager in your addAFK method that the:

Code:java

new AFK()

is turned into:

Code:java

new AFK(p)

Because the AFK class will need the player to save them in the class as the instance variable.

I sincerely apologize for not explaining this as well as I should. I hope I'm getting my message across correctly.

 

@CommonSenze Okay, thanks for that. It doesn't come with any errors so i tried it but it doesn't spawn the stand and barrier and it also comes with a null pointer exception every second if i'm afk. I also tested and it seems the time doesn't go down to 0 when i move. :7
Here is the full error:

Error (open) Error (close)

Code:

[20:53:49 WARN]: [AFK] Task #2 for AFK v0.1 generated an exception
java.lang.NullPointerException
        at me.karim.afk.listeners.RepeatRun$1.run(RepeatRun.java:24) ~[?:?]
        at org.bukkit.craftbukkit.v1_8_R3.scheduler.CraftTask.run(CraftTask.java:71) ~[spigot-1.8.8.jar:git-Spigot-21fe707-e1ebe52]
        at org.bukkit.craftbukkit.v1_8_R3.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:350) [spigot-1.8.8.jar:git-Spigot-21fe707-e1ebe52]
        at net.minecraft.server.v1_8_R3.MinecraftServer.B(MinecraftServer.java:723) [spigot-1.8.8.jar:git-Spigot-21fe707-e1ebe52]
        at net.minecraft.server.v1_8_R3.DedicatedServer.B(DedicatedServer.java:374) [spigot-1.8.8.jar:git-Spigot-21fe707-e1ebe52]
        at net.minecraft.server.v1_8_R3.MinecraftServer.A(MinecraftServer.java:654) [spigot-1.8.8.jar:git-Spigot-21fe707-e1ebe52]
        at net.minecraft.server.v1_8_R3.MinecraftServer.run(MinecraftServer.java:557) [spigot-1.8.8.jar:git-Spigot-21fe707-e1ebe52]
        at java.lang.Thread.run(Unknown Source) [?:1.8.0_161]

As you can see it has something to do with line 24 in my RepeatRun class:

Code:Java

 
plugin.getAFKManager().getAFK(op).createStand();
 

Here is the full class just in case you need it:

Code:Java

 
public class RepeatRun {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public RepeatRun(Main plugin) {
        this.plugin = plugin;
        new BukkitRunnable() {
            @Override
            public void run() {
                if (!plugin.getServer().getOnlinePlayers().isEmpty()) {
                    for (Player op : Bukkit.getOnlinePlayers()) {
                        if (op.isOnline()) {
                            if (plugin.existAfk(op.getUniqueId())) {
                                if (plugin.getAfk(op.getUniqueId()) >= 60*5) {
                                    plugin.getAFKManager().getAFK(op).createStand();
                                } else {
                                    plugin.setAfk(op.getUniqueId(), plugin.getAfk(op.getUniqueId())+1);
                                }
                            } else {
                                plugin.setAfk(op.getUniqueId(), 1);
                            }
                        }
                    }
                }
            }
        }.runTaskTimer(plugin, 0, 20);
    }
}
 

And here is the AFK class:

Code:Java

 
public class AFK {
 
    private ArmorStand stand;
    private Item barrier;
    private boolean afk = false;
    private Player p;
 
    public AFK(Player p) {
        this.p = p;
    }
 
    public void createStand() {
        this.afk = true;
        ArmorStand stand = p.getWorld().spawn(p.getLocation().add(0, +1, 0), ArmorStand.class);
        stand.setGravity(false);
        stand.setVisible(false);
        Item barrier = (Item) p.getWorld().dropItem(p.getLocation().add(0, +2, 0), new ItemStack(Material.BARRIER));
        barrier.setPickupDelay(Integer.MAX_VALUE);
        stand.setPassenger(barrier);
        this.stand = stand;
        this.barrier = barrier;
    }
 
    public void removeStand() {
        this.afk = false;
        this.stand.remove();
        this.barrier.remove();
    }
 
    public boolean isAFK() {
        return this.afk;
    }
}
 

And the AFKManager class here:

Code:Java

 
public class AFKManager {
 
    private HashMap<UUID, AFK> whatever = new HashMap<UUID, AFK>();
    public void addAFK(Player p) {
        this.whatever.put(p.getUniqueId(), new AFK(p));
    }
    public void removeAFK(Player p) {
        this.whatever.remove(p.getUniqueId());
    }
 
    public boolean isAFK(Player p) {
        return this.whatever.get(p.getUniqueId()).isAFK();
    }
 
    public AFK getAFK(Player p) {
        return this.whatever.get(p.getUniqueId());
    }
}
 

Do you see what is returning null and if i did it correctly? Thanks again. :7

 

@KarimAKL
So NullPointerExceptions are when something is null. Now what it's saying is you're trying to access something when it is null. And I assume it is null because you never added the player to the afk Manager when they join.

You need to make 2 events: a Join Event and a QuitEvent that adds and removes the player from the whatever hashmap.

If you have added and removed the players to and from the hashmap in the events then the problem could lie in the getAFKManager returning a null variable because you didn't set it to anything yet.

Those are the only 2 ways that would give a NPE. The reason it didnt spawn the items is because the NPE happened before it can spawn the items itself.

 

@CommonSenze I believe i'm adding them on join and removing them on quit, just to be sure i'll show you the classes.
Join class:

Code:Java

 
public class PlayerJoin implements Listener {
 
    private Main plugin;
 
    public PlayerJoin(Main plugin) {
        this.plugin = plugin;
        Bukkit.getPluginManager().registerEvents(this, plugin);
    }
 
    @EventHandler
    public void onJoin(PlayerJoinEvent e) {
        plugin.getAFKManager().addAFK(e.getPlayer());
    }
}
 

Quit class:

Code:Java

 
public class PlayerQuit implements Listener {
 
    private Main plugin;
 
    public PlayerQuit(Main plugin) {
        this.plugin = plugin;
        Bukkit.getPluginManager().registerEvents(this, plugin);
    }
 
    @EventHandler
    public void onQuit(PlayerQuitEvent e) {
        Player p = e.getPlayer();
        plugin.setAfk(p.getUniqueId(), 0);
        plugin.getAFKManager().removeAFK(p);
    }
}
 

You can see what the methods do in the AFKManager class above.
So if it isn't that then do you have any idea where in the AFKManager class i'm trying to get something that's null? You can see the classes above. :/ I can't seem to find it.

 

@KarimAKL
Did you register the constructors in the onEnable method in the Main class? If you didn't they wouldn't be registered.

 

@CommonSenze Should i do what i did with the AFKManager class with the AFK class aswell?
Like this?(Main class):

Code:Java

 
private AFK afk;
public AFK getAFK() {
    return afk;
}
public void onEnable() {
    this.afk = new AFK();
}
 

If not then what should i do?
EDIT: Nvm that didn't work. :/
Reason why it didn't work:

Code:

The constructor AFK() is undefined

 

@KarimAKL

Code:java

 
    private AFKManager afkManager;
 
    public void onEnable() {
        this.afkManager = new AFKManager();
    }
 
    public AFKManager getAFKManager() {
        return afkManager;
    }
 

That should be in your main

 

@CommonSenze It is.
Here is the whole Main class just to be sure:

Code:Java

 
public class Main extends JavaPlugin {
 
    private HashMap<UUID, Integer> isAfk = new HashMap<UUID, Integer>();
 
    public void setAfk(UUID uuid, Integer integer) {
        this.isAfk.put(uuid, integer);
    }
 
    public Integer getAfk(UUID uuid) {
        return this.isAfk.get(uuid);
    }
 
    public boolean existAfk(UUID uuid) {
        return this.isAfk.containsKey(uuid);
    }
 
    private AFKManager afkManager;
 
    public AFKManager getAFKManager() {
        return afkManager;
    }
 
    public void onEnable() {
        this.afkManager = new AFKManager();
        new CommandAfk(this);
        new CommandGetAfk(this);
        new AsyncPlayerChat(this);
        new CommandPreprocess(this);
        new PlayerInteract(this);
        new PlayerMove(this);
        new PlayerQuit(this);
        new RepeatRun(this);
    }
}
 

What else could it be?
EDIT: This line in the PlayerMove class also comes with a NullPointerException if it helps at all:

Code:Java

 
if (plugin.getAfk(p.getUniqueId()) >= 60*5 && plugin.getAFKManager().isAFK(p)) {
 

 

@KarimAKL
You forgot to register your PlayerJoin class

 

@CommonSenze Omg, i feel like an idiot right now.. Thanks for pointing it out, sorry for the trouble. :7 I'll register it and try again.
EDIT: It shows up now without any errors, just 2 things.
1. It keeps making new ones every second.
2. When i move they don't get removed. :/
What should i do to fix these things? I thought this:

Code:Java

 
    public void createStand() {
        this.afk = true;
        ArmorStand stand = p.getWorld().spawn(p.getLocation().add(0, +1, 0), ArmorStand.class);
        stand.setGravity(false);
        stand.setVisible(false);
        Item barrier = (Item) p.getWorld().dropItem(p.getLocation().add(0, +2, 0), new ItemStack(Material.BARRIER));
        barrier.setPickupDelay(Integer.MAX_VALUE);
        stand.setPassenger(barrier);
        this.stand = stand;
        this.barrier = barrier;
    }
 

Would do the trick because of the "this.afk = true;". Also a few posts back you had some .startAFK and .stopAFK methods, what did they do?
Here is my RepeatRun class:

Code:Java

 
public class RepeatRun {
 
    @SuppressWarnings("unused")
    private Main plugin;
 
    public RepeatRun(Main plugin) {
        this.plugin = plugin;
        new BukkitRunnable() {
            @Override
            public void run() {
                if (!plugin.getServer().getOnlinePlayers().isEmpty()) {
                    for (Player op : Bukkit.getOnlinePlayers()) {
                        if (op.isOnline()) {
                            if (plugin.existAfk(op.getUniqueId())) {
                                if (plugin.getAfk(op.getUniqueId()) >= 60*5) {
                                    plugin.getAFKManager().getAFK(op).createStand();
                                } else {
                                    plugin.setAfk(op.getUniqueId(), plugin.getAfk(op.getUniqueId())+1);
                                }
                            } else {
                                plugin.setAfk(op.getUniqueId(), 1);
                            }
                        }
                    }
                }
            }
        }.runTaskTimer(plugin, 0, 20);
    }
}
 

 

@KarimAKL
No it's alright. I make silly mistakes like that on my server plugin core. But then again, I do have like 70 classes in my plugin so feel like it's more common with me.

 

@CommonSenze Thanks for being patient with me. And wow, that's alot of classes.

 

@KarimAKL
Yeah it is.

Well tell me if the code works and send me the errors, if any.