SchematicCommand überarbeiten #1

Neues Issue

Lixfel · 2019-05-28T13:19:39+02:00

Lixfel hat

2019-05-28 13:19:39 +02:00

kommentiert

Besitzer

Schematic Name (hast du öfter in Strings) zu Schematicname ersetzen.

If-Berge abflachen (tritt ein Fehler auf, in den If reingehen und dann returnen, Faustregel für dich: else bei Fehlern VERMEIDEN)

Beim Überschreiben nicht remove() aufrufen! Das löscht alle Member der Schematic! (Und auch wenn du nur member bist, wird die Schem aus der db gelöscht 👎 (weil du die schematic ja auch bekommst, wenn du nur member bist. Du musst also da überprüfen (bevor du sagst, dass sie überschreiben wird), dass der Ausführende auch der Owner der Schematic ist!!! Wenn du es dann wirklich eine neue Schematic ist: new Schematic(), ansonsten (überschreiben) einfach nur schem.setSchemType() (ist ja keine neue Schematic).

Schematic Name (hast du öfter in Strings) zu Schematicname ersetzen. If-Berge abflachen (tritt ein Fehler auf, in den If reingehen und dann returnen, Faustregel für dich: else bei Fehlern VERMEIDEN) Beim Überschreiben nicht remove() aufrufen! Das löscht alle Member der Schematic! (Und auch wenn du nur member bist, wird die Schem aus der db gelöscht :-1: (weil du die schematic ja auch bekommst, wenn du nur member bist. Du musst also da überprüfen (bevor du sagst, dass sie überschreiben wird), dass der Ausführende auch der Owner der Schematic ist!!! Wenn du es dann wirklich eine neue Schematic ist: new Schematic(), ansonsten (überschreiben) einfach nur schem.setSchemType() (ist ja keine neue Schematic).

Yaruma3341 wurde von Lixfel 2019-05-28 13:19:39 +02:00 zugewiesen

Yaruma3341 hat

2019-05-28 19:17:56 +02:00

kommentiert

Mitwirkender

Schematic Namen sind jetzt einheitlich (noch nicht gepusht). Nun aber mal zum Rest:

If-Berge abflachen (tritt ein Fehler auf, in den If reingehen und dann returnen, Faustregel für dich: else bei Fehlern VERMEIDEN)

Auf welche Fehler beziehst du dich?

Könntest du nochmal genau erklären, wie die Schematic sonst überschrieben werden soll? Außerdem: wieso sollte ich nochmal überprüfen ob der Executor der Owner der Schematic ist? Im Vorfeld hole ich mir die Schematic ja bereits über "Schematic schematic = Schematic.getSchemFromDB(args[1], player.getUniqueId());". Dabei kann doch nur eine Owner Schematic returnt werden oder nicht? Und was soll passieren wenn er nicht der Owner ist?

Schematic Namen sind jetzt einheitlich (noch nicht gepusht). Nun aber mal zum Rest: 1. If-Berge abflachen (tritt ein Fehler auf, in den If reingehen und dann returnen, Faustregel für dich: else bei Fehlern VERMEIDEN) Auf welche Fehler beziehst du dich? 2. Könntest du nochmal genau erklären, wie die Schematic sonst überschrieben werden soll? Außerdem: wieso sollte ich nochmal überprüfen ob der Executor der Owner der Schematic ist? Im Vorfeld hole ich mir die Schematic ja bereits über "Schematic schematic = Schematic.getSchemFromDB(args[1], player.getUniqueId());". Dabei kann doch nur eine Owner Schematic returnt werden oder nicht? Und was soll passieren wenn er nicht der Owner ist?

🎉 1 😆 1

Yaruma3341 hat

2019-05-28 22:48:17 +02:00

kommentiert

Mitwirkender

Code cleanup wird noch erfolgen.