SteamWar/SchematicSystem
SteamWar/SchematicSystem
Archiviert
13
0
Fork 0
Code Issues 3 Pull-Requests 1 Releases Wiki Aktivität

showRanks #57

Manuell gemergt
Lixfel hat 2 Commits von showRanks nach master 2020-05-31 17:14:02 +02:00 zusammengeführt
Diskussion 0 Commits 2 Geänderte Dateien 2 +9 -1
2 geänderte Dateien mit 9 neuen und 1 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen

7
SchematicSystem_Main/src/de/steamwar/schematicsystem/commands/GUI.java
Datei anzeigen

@ -1,5 +1,6 @@
package de.steamwar.schematicsystem.commands;
import com.google.common.collect.Lists;
import de.steamwar.inventory.SWInventory;
import de.steamwar.inventory.SWItem;
import de.steamwar.inventory.SWListInv;
@ -100,7 +101,9 @@ class GUI {
if(schem.getSchemOwner() == SteamwarUser.get(p.getUniqueId()).getId()){
if(schem.getSchemType().writeable()){
List<CheckedSchematic> checkedSchematics = CheckedSchematic.getLastDeclined(p.getUniqueId());
for(CheckedSchematic checkedSchematic : checkedSchematics) {
ListIterator<CheckedSchematic> i = checkedSchematics.listIterator(checkedSchematics.size());
Jan9103 hat 2020-05-27 13:51:45 +02:00 kommentiert
Review
Link kopieren

ich kenn micj nicht mit iteratoren aus, aber ich dachte sie gehen mittels eines befehls von vorne nach hinten durch -> while(hasPrevious) ist immer true und du erhöhst es auch nie

kann sein, dass ich die komplett falsch verstanden habe oder, dass du hier while(true) hast

ich kenn micj nicht mit iteratoren aus, aber ich dachte sie gehen mittels eines befehls von vorne nach hinten durch -> while(hasPrevious) ist immer true und du erhöhst es auch nie kann sein, dass ich die komplett falsch verstanden habe oder, dass du hier while(true) hast
Lixfel hat 2020-05-27 14:04:38 +02:00 kommentiert
Review
Link kopieren

Nein, ich iteriere mit i.previous() ja rückwärts durch die Liste durch.

Nein, ich iteriere mit i.previous() ja rückwärts durch die Liste durch.
Lixfel hat 2020-05-27 14:16:01 +02:00 kommentiert
Review
Link kopieren

und starte ja überhaupt erstmal am Ende der Liste

und starte ja überhaupt erstmal am Ende der Liste
while(i.hasPrevious()) {
CheckedSchematic checkedSchematic = i.previous();
if(checkedSchematic.getSchemName().equals(schem.getSchemName()) &&
checkedSchematic.getSchemOwner() == schem.getSchemOwner()) {
inv.setItem(1, SWItem.getDye(10), (byte) 10, "§eStatus " + schem.getSchemType().name(), Collections.singletonList("§7" + checkedSchematic.getDeclineReason()), false, click -> {});
@ -165,6 +168,8 @@ class GUI {
m = SWItem.getMaterial(schem.getItem());
SWItem item = new SWItem(m, "§e" + schem.getSchemName(), Collections.singletonList("§7" + schem.getSchemType().name()), !schem.getSchemType().writeable(), click -> {});
if(schem.getRank() > 0)
item.setLore(Lists.newArrayList("§7" + schem.getSchemType().name(), "§8Rang " + schem.getRank()));
schemList.add(new SWListInv.SWListEntry<>(item, schem));
}

3
SchematicSystem_Main/src/de/steamwar/schematicsystem/commands/SchematicCommand.java
Datei anzeigen

@ -292,6 +292,9 @@ public class SchematicCommand implements CommandExecutor {
type.setHoverEvent(new HoverEvent(HoverEvent.Action.SHOW_TEXT, TextComponent.fromLegacyText("§eTyp ändern")));
type.setClickEvent(new ClickEvent(ClickEvent.Action.RUN_COMMAND, "/schem changetype " + schematic.getSchemName()));
}
if(schematic.getRank() > 0){
type.addExtra("§8Rang " + schematic.getRank());
}
player.spigot().sendMessage(type);
if(SteamwarUser.get(schematic.getSchemOwner()).getUUID().equals(player.getUniqueId())){