Solved Get location of offline player

Hello,

I'm looking for the location of an offline player and this does not work:

Code:java

Player offlinePlayer = Bukkit.getOfflinePlayer(playerUUID).getPlayer();
offlinePlayer.loadData();
offlinePlayer.getLocation();

.getPlayer() returns a null value.
And the UUID was of a player who played before but was not online

Is there another way? Of should I save all the locations of the players like this:

Code:java

import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.OfflinePlayer;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.event.player.PlayerLoginEvent;
import org.bukkit.event.player.PlayerQuitEvent;
 
import java.util.HashMap;
import java.util.UUID;
 
public class OfflineLocationManager implements Listener {
 
    public static HashMap<UUID, Location> locMap = new HashMap<>();
 
    @EventHandler
    public void onJoin(PlayerJoinEvent e) {
        locMap.remove(e.getPlayer().getUniqueId());
    }
   //which is better? PlayerJoinEvent or PlayerLoginEvent?
    public void onJoin(PlayerLoginEvent e) {
        locMap.remove(e.getPlayer().getUniqueId());
    }
 
    @EventHandler
    public void onLeave(PlayerQuitEvent e) {
        locMap.put(e.getPlayer().getUniqueId(), e.getPlayer().getLocation());
    }
 
    public static Location getLocation(UUID uuid) {
        OfflinePlayer player = Bukkit.getOfflinePlayer(uuid);
        if (player.isOnline()) {
            return player.getPlayer().getLocation();
        } else if (player.hasPlayedBefore()) {
            return locMap.getOrDefault(player.getUniqueId(), null);
        } else {
            return null;
        }
    }
 
    public static void saveMap() {
        //save
    }
    public static void loadMap() {
        //load
    }
}

Thank you

 

the .getPlayer() method only returns a Player if said user is online. I don't know how one gets a location for an offline player, if it's possible at all.

The log at quit method seems reasonable, it's what Essentials does after all. I wouldn't screw with a map at all though, I'd just use straight config. You don't have to worry about a crash making you lose data then, and as config is cached it isn't a performance hit to query it.

PS PlayerJoinEvent is what you want, not LoginEvent.

 

Alright, thank you!