Abfragen, ob sich Spieler XY in Region XY befindet. (WorldGuardAPI)

[Kraetzy]

Hey, wie frage ich ab ob sich Spieler XY in Region XY befindet?

Aktuelle Versuche:

public void isInArea(Player player, String region) {
            @SuppressWarnings("unused")
            Plugin worldguard = player.getServer().getPluginManager().getPlugin("WorldGuard");
           
            @SuppressWarnings("unused")
            ProtectedRegion r = ((WorldGuardPlugin) Bukkit.getPluginManager().getPlugin("WorldGuard")).getRegionManager(player.getWorld()).getRegion(region);
           
           
        }

 

[ausgebildet]

Gibt es Fehlermeldungen?

 

[SirYwell]

Der aktuelle Code ist absolut sinnlos. Eine Methode, die mit "is" beginnt, sollte einen boolean zurückgeben (zumindest erwartet man das als Leser). Innerhalb einer Methode "@SupressWarnings("unused")" zu verwenden, sollte einem auch zu denken geben. Die erste Zeile in der Methode ruft das WorldGuardPlugin auf, allerdings wird damit nichts gemacht. Stattdessen wird der Vorgang im nächsten Statement wiederholt. Diese Zeile sollte man zur besseren Übersichtlichkeit auf mehrere Zeilen aufteilen. Wenn die zurückgegebene Region nicht null ist, kannst du dir nur sicher sein, dass die Region in der Welt existiert. Sinnvoller ist es, "getApplicableRegionSet()", soweit ich weiß auch eine Methode in RegionManager, zu verwenden.

 

[Kraetzy]

Der aktuelle Code ist absolut sinnlos. Eine Methode, die mit "is" beginnt, sollte einen boolean zurückgeben (zumindest erwartet man das als Leser). Innerhalb einer Methode "@SupressWarnings("unused")" zu verwenden, sollte einem auch zu denken geben. Die erste Zeile in der Methode ruft das WorldGuardPlugin auf, allerdings wird damit nichts gemacht. Stattdessen wird der Vorgang im nächsten Statement wiederholt. Diese Zeile sollte man zur besseren Übersichtlichkeit auf mehrere Zeilen aufteilen. Wenn die zurückgegebene Region nicht null ist, kannst du dir nur sicher sein, dass die Region in der Welt existiert. Sinnvoller ist es, "getApplicableRegionSet()", soweit ich weiß auch eine Methode in RegionManager, zu verwenden.

Ich arbeite aktuell das erste mal mit der WorldGuardAPI.

 

[Figz]

Ich arbeite aktuell das erste mal mit der WorldGuardAPI.

Na und? Bei seiner Aussage geht es eher um allgemeine Java Konventionen und vernünftige Programmierung.

Man kann versuchen mittels Learning by Doing durch Pluginprogrammierung Java zu lernen. Allerdings sollte man auch mehr wollen als durch irgendeinen Weg zu einer Lösung zu kommen.