Spieler Location aus einer Hashmap

[gmb32]

Hey,
ich speichere Spieler Locations in eine Hashmap zusätzlich noch ihr Inventar. Bei einen bestimmten befehl will ich das alle Spieler zu Ihrem alten Platz mit ihren Inventar gehen.

Problem ist hier:

if(gericht.containsKey(p)) {
                            for(Player key : gericht.keySet()) {
                                Location loc = gericht.get(key);
                                key.teleport(loc);
                                if(key.hasMetadata("Angeklagter".toString())) {
                                    key.removeMetadata("Angeklagter", Main.getInstace());
                                }
                                gericht.remove(key);
                            }
                            for(Player p2 : items.keySet()) {
                                ItemStack[] contents = items.get(p2);
                                p2.getInventory().setContents(contents);
                                p2.updateInventory();
                                items.remove(p2);
                            }
                          
                            p.sendMessage(Main.getInstace().getPrefix()+"Der Gerichtsprozess wurde beendet!");
                            Bukkit.broadcastMessage(Main.getInstace().getPrefix()+"Das Gericht wurde §cgeschlossen!");
                            gerichtenable = false;
                        }else {
                            p.sendMessage(Main.getInstace().getPrefix()+"Du bist in keinem Gerichtsprozess");
                        }

Der Fehler kommt in der Zeile for(Player key : gericht.keySet()) {

 

[SirYwell]

Um dir produktiv helfen zu können, musst du ein wenig mehr an Informationen herausrücken.
1. Welcher Fehler tritt auf?
2. gericht ist vermutlich deine HashMap. So etwas bitte auch schreiben. Dann muss man als Leser nicht herausfinden, dass es eine HashMap bestehend aus Spielern und Locations ist.

So, nun zu meinen Vorschlägen:
Bau dir ein eigenes Objekt, das aus der UUID des Spielers, der Location und dem Inventar besteht. Wenn nun jemand vor Gericht kommt, kannst du ihn zu einer Liste oder einem Set hinzufügen. Somit musst du schon mal nicht zwei HashMaps anlegen. Mit der Stream API, die mit Java 8 kam, lässt sich die Liste auch bequem auf ein Objekt mit der UUID eines bestimmten Spielers untersucht werden.

 

[moinless]

1: Nein, einfach nein

"Angeklagter".toString()

2: Folgende Map wäre noch einfacher umzusetzen, als das von @SirYwell vorgeschlagene Objekt.

Map<UUID, Pair<Location, Inventory>> map = new HashMap<>();

 

[BlackHole]

Wobei das Pair Apache Commons 3 benötigen würde. Hier wäre auch eine einfache Klasse mit Feldern für Location und Inventar geeignet.

Du solltest dir auch überlegen, was passiert soll, wenn der beteiligten Spieler gerade offline ist.

 

[Figz]

1: Nein, einfach nein

"Angeklagter".toString()

2: Folgende Map wäre noch einfacher umzusetzen, als das von @SirYwell vorgeschlagene Objekt.

Map<UUID, Pair<Location, Inventory>> map = new HashMap<>();

Hier sollte man beachten, dass man eine Kopie des Inventares speichert und nicht die Referenz, die man vom Player-Object erhält.

 

[SirYwell]

Wobei das Pair Apache Commons 3 benötigen würde.

Da das meines Wissens in Spigot mitkompiliert wird, wäre das kein großes Problem. Allerdings wäre diese Lösung weniger übersichtlich.

 

[BlackHole]

Die Spigot-API enthält Apache Commons 2.6, aber der Minecraft-Server enthält Apache Commons 3.