Gdzie zapisać zmienna która potrzebuje tylko na chwilę?

Sprawdzanie czy gracz się rusza powinno być asynchroniczne(asynchroniczne zadanie), to poprawi wydajność.

Można to niby sprawdzać również w tasku, ale jak kto woli.

Sposobów jest kilka, ale ja zawsze używam jednego.

W pluginie funnyguilds jest to fajnie zrobione:

cache.setTeleportation(Bukkit.getScheduler().runTaskTimer(FunnyGuilds.getInstance(), () -> {
            if (!player.isOnline()) {
                cache.getTeleportation().cancel();
                cache.setTeleportation(null);
                return;
            }
            
            if (!LocationUtils.equals(player.getLocation(), before)) {
                cache.getTeleportation().cancel();
                player.sendMessage(messages.baseMove);
                cache.setTeleportation(null);
                player.getInventory().addItem(items);
                return;
            }

            if (timeCounter.getAndIncrement() > time) {
                cache.getTeleportation().cancel();
                player.sendMessage(messages.baseTeleport);
                player.teleport(guild.getHome());
                cache.setTeleportation(null);
            }
        }, 0L, 20L));

Czyli zwykły task co sekundę, który będzie się powtarzał przez ilość sekund, za każdym razem sprawdzenie czy gracz jest online i czy się nie ruszył, a jak odczeka czas to go tepnie na home

56 minut temu, kerpson napisał:

Sprawdzanie czy gracz się rusza powinno być asynchroniczne(asynchroniczne zadanie), to poprawi wydajność.

Można to niby sprawdzać również w tasku, ale jak kto woli.

Sposobów jest kilka, ale ja zawsze używam jednego.

W pluginie funnyguilds jest to fajnie zrobione:

cache.setTeleportation(Bukkit.getScheduler().runTaskTimer(FunnyGuilds.getInstance(), () -> {
            if (!player.isOnline()) {
                cache.getTeleportation().cancel();
                cache.setTeleportation(null);
                return;
            }
            
            if (!LocationUtils.equals(player.getLocation(), before)) {
                cache.getTeleportation().cancel();
                player.sendMessage(messages.baseMove);
                cache.setTeleportation(null);
                player.getInventory().addItem(items);
                return;
            }

            if (timeCounter.getAndIncrement() > time) {
                cache.getTeleportation().cancel();
                player.sendMessage(messages.baseTeleport);
                player.teleport(guild.getHome());
                cache.setTeleportation(null);
            }
        }, 0L, 20L));

Czyli zwykły task co sekundę, który będzie się powtarzał przez ilość sekund, za każdym razem sprawdzenie czy gracz jest online i czy się nie ruszył, a jak odczeka czas to go tepnie na home

Skoro sprawdza co sekunde to nie będzie błędu że jak się ruszy w pewnym momencie to nie wykryje? 

wykryje gdyż zapisujesz lokalizację początkową podczas wpisania komendy i porównujesz z lokacją gracza co sekundę