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

RedstoneWincondition #274

Geschlossen
YoyoNow möchte 18 Commits von RedstoneWincondition nach master mergen
pullen von: RedstoneWincondition
Ziel: SteamWar:master
Diskussion 6 Commits 18 Geänderte Dateien 9 +41 -64
5 geänderte Dateien mit 41 neuen und 64 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 21ff7fddaf werden angezeigt - Alle Commits anzeigen

28
FightSystem_Main/src/de/steamwar/fightsystem/winconditions/PercentWincondition.java
Datei anzeigen

@ -19,11 +19,14 @@
package de.steamwar.fightsystem.winconditions;
import de.steamwar.fightsystem.Config;
import de.steamwar.fightsystem.fight.FightTeam;
import de.steamwar.fightsystem.states.FightState;
import de.steamwar.fightsystem.states.OneShotStateDependent;
import de.steamwar.fightsystem.states.StateDependentListener;
import org.bukkit.Bukkit;
import org.bukkit.Material;
import org.bukkit.World;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityExplodeEvent;
@ -37,8 +40,13 @@ public abstract class PercentWincondition extends Wincondition implements Printa
protected final Map<FightTeam, TeamPercent> teamMap = new HashMap<>();
protected PercentWincondition(String windescription) {
protected PercentWincondition(String windescription, Winconditions winconditions) {
super(windescription);
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2021-05-16 20:45:12 +02:00 kommentiert
Veraltet
Review
Link kopieren

Jeder Prozentmodus braucht irgendwo beide Teams, die Map FightTeam -> TeamPercent müsste daher hierher abstrahierbar sein.

Aka: Die Teammap schon hier in PercentWincondition zur Verfügung stellen und füllen (parameter im Konstruktor übergeben lassen), getPercent gleich hier implementieren.

In der PointWincondition hast du dann einfach noch eine zweite Map

Jeder Prozentmodus braucht irgendwo beide Teams, die Map FightTeam -> TeamPercent müsste daher hierher abstrahierbar sein. Aka: Die Teammap schon hier in PercentWincondition zur Verfügung stellen und füllen (parameter im Konstruktor übergeben lassen), getPercent gleich hier implementieren. In der PointWincondition hast du dann einfach noch eine zweite Map
if (Config.ActiveWinconditions.contains(winconditions)) {
printableWinconditions.add(this);
percentWincondition = this;
}
}
public String getDisplay(FightTeam team) {
@ -51,6 +59,8 @@ public abstract class PercentWincondition extends Wincondition implements Printa
public static class TeamPercent implements Listener {
private static final World world = Bukkit.getWorlds().get(0);
public final FightTeam fightTeam;
private final Predicate<Material> testType;
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2021-05-16 20:52:49 +02:00 kommentiert
Veraltet
Review
Link kopieren

OneShotStateDependents werden schon im Konstruktor registered, bitte nicht dopplet registern.

OneShotStateDependents werden schon im Konstruktor registered, bitte nicht dopplet registern.
@ -59,21 +69,33 @@ public abstract class PercentWincondition extends Wincondition implements Printa
private final Consumer<TeamPercent> explodeConsumer;
public TeamPercent(FightTeam fightTeam, Winconditions wincondition, Predicate<Material> testType, Consumer<TeamPercent> enableConsumer, Consumer<TeamPercent> explodeConsumer) {
public TeamPercent(PercentWincondition percentWincondition, FightTeam fightTeam, Winconditions wincondition, Predicate<Material> testType, Consumer<TeamPercent> explodeConsumer) {
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2021-05-23 09:20:51 +02:00 kommentiert
Veraltet
Review
Link kopieren

Hier am Ende des Konstruktors würde ich gleich dieses TeamPercent in die teamMap einfügen, dann musst du das nicht an den ganzen anderen Stellen machen (und kannst es nicht vergessen)

Hier am Ende des Konstruktors würde ich gleich dieses TeamPercent in die teamMap einfügen, dann musst du das nicht an den ganzen anderen Stellen machen (und kannst es nicht vergessen)
this.fightTeam = fightTeam;
this.testType = testType;
this.explodeConsumer = explodeConsumer;
new OneShotStateDependent(wincondition, FightState.Running, () -> {
enableConsumer.accept(TeamPercent.this);
fightTeam.getSchemRegion().forEach((x, y, z) -> {
if (testType.test(world.getBlockAt(x, y, z).getType())) {
totalBlocks++;
}
});
currentBlocks = totalBlocks;
});
new StateDependentListener(wincondition, FightState.Running, this).register();
if (percentWincondition != null) {
percentWincondition.teamMap.put(fightTeam, this);
}
}
@EventHandler
public void onEntityExplode(EntityExplodeEvent event) {
// Entern active
if (!Config.EnterStages.isEmpty() && Config.EnterStages.get(0) >= Wincondition.getTimeOverCountdown().getTimeLeft()) {
return;
}
if (!fightTeam.getExtendRegion().inRegion(event.getEntity().getLocation())) {
return;
}

17
FightSystem_Main/src/de/steamwar/fightsystem/winconditions/WinconditionPercentSystem.java
Datei anzeigen

@ -28,20 +28,13 @@ import org.bukkit.Bukkit;
public class WinconditionPercentSystem extends PercentWincondition {
public WinconditionPercentSystem() {
super("Percent");
teamMap.put(Fight.getBlueTeam(), create(Fight.getBlueTeam()));
teamMap.put(Fight.getRedTeam(), create(Fight.getRedTeam()));
if (Config.ActiveWinconditions.contains(Winconditions.PERCENT_SYSTEM)) {
printableWinconditions.add(this);
percentWincondition = this;
}
super("Percent", Winconditions.PERCENT_SYSTEM);
create(Fight.getBlueTeam());
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2021-05-16 20:53:24 +02:00 kommentiert
Veraltet
Review
Link kopieren

Ist kein Listener mehr.

Ist kein Listener mehr.
create(Fight.getRedTeam());
}
private TeamPercent create(FightTeam fightTeam) {
return new TeamPercent(fightTeam, Winconditions.PERCENT_SYSTEM, material -> true, teamPercent -> {
teamPercent.totalBlocks = teamPercent.fightTeam.getSchemRegion().volume();
}, teamPercent -> {
private void create(FightTeam fightTeam) {
new TeamPercent(this, fightTeam, Winconditions.PERCENT_SYSTEM, material -> true, teamPercent -> {
if (teamPercent.getPercent() >= Config.PercentWin) {
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2021-05-23 09:17:32 +02:00 kommentiert
Veraltet
Review
Link kopieren

Dieser Codeblock sollte in die PercentWincondition verschoben werden (und dann aus den Subklassen entfernt werden). Points braucht ja nur ein @Override getDisplay

Dieser Codeblock sollte in die PercentWincondition verschoben werden (und dann aus den Subklassen entfernt werden). Points braucht ja nur ein @Override getDisplay
YoyoNow hat 2021-05-23 16:51:53 +02:00 kommentiert
Veraltet
Review
Link kopieren

Kannst du mir das nochmal genauer erklären?

Kannst du mir das nochmal genauer erklären?
Lixfel hat 2021-05-23 19:06:12 +02:00 kommentiert
Veraltet
Review
Link kopieren

Diesen if-Block kannst du so in den Konstruktor der PercentWincondition verschieben.

Diesen if-Block kannst du so in den Konstruktor der PercentWincondition verschieben.
Bukkit.broadcastMessage(FightSystem.PREFIX + "§cTeam " + teamPercent.fightTeam.getColoredName() + " §chat zu viel Schaden erlitten!");
win(Fight.getOpposite(teamPercent.fightTeam));

12
FightSystem_Main/src/de/steamwar/fightsystem/winconditions/WinconditionPoints.java
Datei anzeigen

@ -38,7 +38,6 @@ import org.bukkit.event.player.PlayerQuitEvent;
import java.util.HashMap;
import java.util.Map;
import java.util.concurrent.atomic.AtomicInteger;
public class WinconditionPoints extends Wincondition implements PrintableWincondition, Listener {
@ -121,16 +120,7 @@ public class WinconditionPoints extends Wincondition implements PrintableWincond
TeamPoints(FightTeam team){
this.team = team;
this.percent = new PercentWincondition.TeamPercent(team, Winconditions.POINTS, material -> true, teamPercent -> {
this.points = 0;
AtomicInteger currentBlocks = new AtomicInteger();
teamPercent.fightTeam.getSchemRegion().forEach((x, y, z) -> {
if (!Config.Blocks.contains(world.getBlockAt(x, y, z).getType())) {
currentBlocks.getAndIncrement();
}
});
teamPercent.totalBlocks = currentBlocks.get();
}, teamPercent -> {});
this.percent = new PercentWincondition.TeamPercent(null, team, Winconditions.POINTS, material -> true, teamPercent -> {});
}
public void enable() {

24
FightSystem_Main/src/de/steamwar/fightsystem/winconditions/WinconditionRelativePercent.java
Datei anzeigen

@ -24,32 +24,18 @@ import de.steamwar.fightsystem.FightSystem;
import de.steamwar.fightsystem.fight.Fight;
import de.steamwar.fightsystem.fight.FightTeam;
import org.bukkit.Bukkit;
import org.bukkit.World;
public class WinconditionRelativePercent extends PercentWincondition {
private static final World world = Bukkit.getWorlds().get(0);
public WinconditionRelativePercent() {
super("RelativePercent");
teamMap.put(Fight.getBlueTeam(), create(Fight.getBlueTeam()));
teamMap.put(Fight.getRedTeam(), create(Fight.getRedTeam()));
if (Config.ActiveWinconditions.contains(Winconditions.RELATIVE_PERCENT)) {
printableWinconditions.add(this);
percentWincondition = this;
}
super("RelativePercent", Winconditions.RELATIVE_PERCENT);
create(Fight.getBlueTeam());
create(Fight.getRedTeam());
}
private PercentWincondition.TeamPercent create(FightTeam fightTeam) {
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2021-05-16 20:50:17 +02:00 kommentiert
Veraltet
Review
Link kopieren

WÜrde empfehlen, in der COnfig diesen Codeblock zu verwenden, 4 Zeilen sind da etwas kürzer :)

WÜrde empfehlen, in der COnfig diesen Codeblock zu verwenden, 4 Zeilen sind da etwas kürzer :)
return new PercentWincondition.TeamPercent(fightTeam, Winconditions.PERCENT_SYSTEM, material -> {
private void create(FightTeam fightTeam) {
new TeamPercent(this, fightTeam, Winconditions.RELATIVE_WHITELIST_PERCENT, material -> {
return !Config.Blocks.contains(material);
}, teamPercent -> {
teamPercent.fightTeam.getSchemRegion().forEach((x, y, z) -> {
if (!Config.Blocks.contains(world.getBlockAt(x, y, z).getType())) {
teamPercent.totalBlocks++;
}
});
}, teamPercent -> {
if (teamPercent.getPercent() >= Config.PercentWin) {
Bukkit.broadcastMessage(FightSystem.PREFIX + "§cTeam " + teamPercent.fightTeam.getColoredName() + " §chat zu viel Schaden erlitten!");

24
FightSystem_Main/src/de/steamwar/fightsystem/winconditions/WinconditionRelativeWhitelistPercent.java
Datei anzeigen

@ -24,33 +24,19 @@ import de.steamwar.fightsystem.FightSystem;
import de.steamwar.fightsystem.fight.Fight;
import de.steamwar.fightsystem.fight.FightTeam;
import org.bukkit.Bukkit;
import org.bukkit.World;
import org.bukkit.event.Listener;
public class WinconditionRelativeWhitelistPercent extends PercentWincondition implements Listener {
private static final World world = Bukkit.getWorlds().get(0);
public WinconditionRelativeWhitelistPercent() {
super("RelativeWhitelistPercent");
teamMap.put(Fight.getBlueTeam(), create(Fight.getBlueTeam()));
teamMap.put(Fight.getRedTeam(), create(Fight.getRedTeam()));
if (Config.ActiveWinconditions.contains(Winconditions.RELATIVE_WHITELIST_PERCENT)) {
printableWinconditions.add(this);
percentWincondition = this;
}
super("RelativeWhitelistPercent", Winconditions.RELATIVE_WHITELIST_PERCENT);
create(Fight.getBlueTeam());
create(Fight.getRedTeam());
}
private PercentWincondition.TeamPercent create(FightTeam fightTeam) {
return new PercentWincondition.TeamPercent(fightTeam, Winconditions.PERCENT_SYSTEM, material -> {
private void create(FightTeam fightTeam) {
new PercentWincondition.TeamPercent(this, fightTeam, Winconditions.RELATIVE_WHITELIST_PERCENT, material -> {
return Config.Blocks.contains(material);
}, teamPercent -> {
teamPercent.fightTeam.getSchemRegion().forEach((x, y, z) -> {
if (Config.Blocks.contains(world.getBlockAt(x, y, z).getType())) {
teamPercent.totalBlocks++;
}
});
}, teamPercent -> {
if (teamPercent.getPercent() >= Config.PercentWin) {
Bukkit.broadcastMessage(FightSystem.PREFIX + "§cTeam " + teamPercent.fightTeam.getColoredName() + " §chat zu viel Schaden erlitten!");