diff --git a/forge-ai/pom.xml b/forge-ai/pom.xml
index b84adeea7d5..eb808066d02 100644
--- a/forge-ai/pom.xml
+++ b/forge-ai/pom.xml
@@ -6,7 +6,7 @@
forge
forge
- 1.6.28-SNAPSHOT
+ 1.6.29-SNAPSHOT
forge-ai
diff --git a/forge-ai/src/main/java/forge/ai/AIOption.java b/forge-ai/src/main/java/forge/ai/AIOption.java
index 3eea52dfc7e..66925e7f6b8 100644
--- a/forge-ai/src/main/java/forge/ai/AIOption.java
+++ b/forge-ai/src/main/java/forge/ai/AIOption.java
@@ -1,5 +1,5 @@
package forge.ai;
public enum AIOption {
- USE_SIMULATION;
+ USE_SIMULATION
}
diff --git a/forge-ai/src/main/java/forge/ai/AiAttackController.java b/forge-ai/src/main/java/forge/ai/AiAttackController.java
index 1a374d80660..c309b41ac01 100644
--- a/forge-ai/src/main/java/forge/ai/AiAttackController.java
+++ b/forge-ai/src/main/java/forge/ai/AiAttackController.java
@@ -1021,7 +1021,7 @@ public class AiAttackController {
} // stay at home to block
if ( LOG_AI_ATTACKS )
- System.out.println(String.valueOf(this.aiAggression) + " = ai aggression");
+ System.out.println(this.aiAggression + " = ai aggression");
// ****************
// Evaluation the end
@@ -1457,7 +1457,7 @@ public class AiAttackController {
if (artifact != null) {
return artifact;
}
- return null; //should never get here
+ return null;//should never get here
}
private void doLightmineFieldAttackLogic(List attackersLeft, int numForcedAttackers, boolean playAggro) {
diff --git a/forge-ai/src/main/java/forge/ai/AiBlockController.java b/forge-ai/src/main/java/forge/ai/AiBlockController.java
index e948d3cebfc..82df82ef46f 100644
--- a/forge-ai/src/main/java/forge/ai/AiBlockController.java
+++ b/forge-ai/src/main/java/forge/ai/AiBlockController.java
@@ -859,7 +859,7 @@ public class AiBlockController {
damageToPW += ComputerUtilCombat.predictDamageTo((Card) def, pwatkr.getNetCombatDamage(), pwatkr, true);
}
}
- if ((!onlyIfLethal && damageToPW > 0) || damageToPW >= ((Card) def).getCounters(CounterType.LOYALTY)) {
+ if ((!onlyIfLethal && damageToPW > 0) || damageToPW >= def.getCounters(CounterType.LOYALTY)) {
threatenedPWs.add((Card) def);
}
}
@@ -879,7 +879,7 @@ public class AiBlockController {
if (!chumpPWDefenders.isEmpty()) {
for (final Card attacker : attackers) {
GameEntity def = combat.getDefenderByAttacker(attacker);
- if (def instanceof Card && threatenedPWs.contains((Card) def)) {
+ if (def instanceof Card && threatenedPWs.contains(def)) {
if (attacker.hasKeyword(Keyword.TRAMPLE)) {
// don't bother trying to chump a trampling creature
continue;
@@ -914,7 +914,7 @@ public class AiBlockController {
pwDefenders.addAll(combat.getBlockers(pwAtk));
} else {
isFullyBlocked = false;
- damageToPW += ComputerUtilCombat.predictDamageTo((Card) pw, pwAtk.getNetCombatDamage(), pwAtk, true);
+ damageToPW += ComputerUtilCombat.predictDamageTo(pw, pwAtk.getNetCombatDamage(), pwAtk, true);
}
}
if (!isFullyBlocked && damageToPW >= pw.getCounters(CounterType.LOYALTY)) {
@@ -1329,13 +1329,9 @@ public class AiBlockController {
&& ((Card) combat.getDefenderByAttacker(attacker)).isPlaneswalker();
boolean wantToTradeDownToSavePW = chanceToTradeDownToSaveWalker > 0;
- if (((evalBlk <= evalAtk + 1) || (wantToSavePlaneswalker && wantToTradeDownToSavePW)) // "1" accounts for tapped.
+ return ((evalBlk <= evalAtk + 1) || (wantToSavePlaneswalker && wantToTradeDownToSavePW)) // "1" accounts for tapped.
&& powerParityOrHigher
&& (creatureParityOrAllowedDiff || wantToTradeWithCreatInHand)
- && (MyRandom.percentTrue(chance) || wantToSavePlaneswalker)) {
- return true;
- }
-
- return false;
+ && (MyRandom.percentTrue(chance) || wantToSavePlaneswalker);
}
}
diff --git a/forge-ai/src/main/java/forge/ai/AiCardMemory.java b/forge-ai/src/main/java/forge/ai/AiCardMemory.java
index e0794076723..9e203710dd9 100644
--- a/forge-ai/src/main/java/forge/ai/AiCardMemory.java
+++ b/forge-ai/src/main/java/forge/ai/AiCardMemory.java
@@ -137,7 +137,7 @@ public class AiCardMemory {
Set memorySet = getMemorySet(set);
- return memorySet == null ? false : memorySet.contains(c);
+ return memorySet != null && memorySet.contains(c);
}
/**
@@ -291,7 +291,7 @@ public class AiCardMemory {
* @return true, if the given memory set contains no remembered cards.
*/
public boolean isMemorySetEmpty(MemorySet set) {
- return set == null ? true : getMemorySet(set).isEmpty();
+ return set == null || getMemorySet(set).isEmpty();
}
/**
diff --git a/forge-ai/src/main/java/forge/ai/AiController.java b/forge-ai/src/main/java/forge/ai/AiController.java
index bc548e7079f..a027cc9ea1c 100644
--- a/forge-ai/src/main/java/forge/ai/AiController.java
+++ b/forge-ai/src/main/java/forge/ai/AiController.java
@@ -1645,7 +1645,6 @@ public class AiController {
// For non-converted triggers (such as Cumulative Upkeep) that don't have costs or targets to worry about
return true;
}
-
return false;
}
@@ -1690,16 +1689,11 @@ public class AiController {
left = AbilityUtils.calculateAmount(hostCard, svarToCheck, sa);
}
System.out.println("aiShouldRun?" + left + comparator + compareTo);
- if (Expressions.compare(left, comparator, compareTo)) {
- return true;
- }
+ return Expressions.compare(left, comparator, compareTo);
} else if (effect.getMapParams().containsKey("AICheckDredge")) {
return player.getCardsIn(ZoneType.Library).size() > 8 || player.isCardInPlay("Laboratory Maniac");
- } else if (sa != null && doTrigger(sa, false)) {
- return true;
- }
+ } else return sa != null && doTrigger(sa, false);
- return false;
}
public List chooseSaToActivateFromOpeningHand(List usableFromOpeningHand) {
@@ -2078,9 +2072,7 @@ public class AiController {
// AI-specific restrictions specified as activation parameters in spell abilities
if (sa.hasParam("AILifeThreshold")) {
- if (player.getLife() <= Integer.parseInt(sa.getParam("AILifeThreshold"))) {
- return false;
- }
+ return player.getLife() > Integer.parseInt(sa.getParam("AILifeThreshold"));
}
return true;
diff --git a/forge-ai/src/main/java/forge/ai/AiCostDecision.java b/forge-ai/src/main/java/forge/ai/AiCostDecision.java
index c5ae2738f94..47dfbbd4d4b 100644
--- a/forge-ai/src/main/java/forge/ai/AiCostDecision.java
+++ b/forge-ai/src/main/java/forge/ai/AiCostDecision.java
@@ -56,7 +56,7 @@ public class AiCostDecision extends CostDecisionMakerBase {
@Override
public PaymentDecision visit(CostChooseCreatureType cost) {
String choice = player.getController().chooseSomeType("Creature", ability, CardType.getAllCreatureTypes(),
- Lists.newArrayList());
+ Lists.newArrayList());
return PaymentDecision.type(choice);
}
@@ -475,7 +475,7 @@ public class AiCostDecision extends CostDecisionMakerBase {
if (ability.getPayCosts().hasTapCost() && typeList.contains(ability.getHostCard())) {
c--;
}
- source.setSVar("ChosenX", "Number$" + Integer.toString(c));
+ source.setSVar("ChosenX", "Number$" + c);
} else {
if (!isVehicle) {
c = AbilityUtils.calculateAmount(source, amount, ability);
@@ -809,7 +809,7 @@ public class AiCostDecision extends CostDecisionMakerBase {
final String sVar = ability.getSVar(amount);
if (sVar.equals("XChoice")) {
c = AbilityUtils.calculateAmount(source, "ChosenX", ability);
- source.setSVar("ChosenX", "Number$" + String.valueOf(c));
+ source.setSVar("ChosenX", "Number$" + c);
} else if (amount.equals("All")) {
c = source.getCounters(cost.counter);
} else if (sVar.equals("Targeted$CardManaCost")) {
@@ -865,7 +865,7 @@ public class AiCostDecision extends CostDecisionMakerBase {
}
typeList = CardLists.filter(typeList, Presets.TAPPED);
c = typeList.size();
- source.setSVar("ChosenX", "Number$" + Integer.toString(c));
+ source.setSVar("ChosenX", "Number$" + c);
} else {
c = AbilityUtils.calculateAmount(source, amount, ability);
}
diff --git a/forge-ai/src/main/java/forge/ai/AiPlayDecision.java b/forge-ai/src/main/java/forge/ai/AiPlayDecision.java
index c3c41cd20ba..692badb6bd8 100644
--- a/forge-ai/src/main/java/forge/ai/AiPlayDecision.java
+++ b/forge-ai/src/main/java/forge/ai/AiPlayDecision.java
@@ -17,5 +17,5 @@ public enum AiPlayDecision {
WouldBecomeZeroToughnessCreature,
WouldDestroyWorldEnchantment,
BadEtbEffects,
- CurseEffects;
+ CurseEffects
}
\ No newline at end of file
diff --git a/forge-ai/src/main/java/forge/ai/ComputerUtil.java b/forge-ai/src/main/java/forge/ai/ComputerUtil.java
index baf97df8181..f3172be4a43 100644
--- a/forge-ai/src/main/java/forge/ai/ComputerUtil.java
+++ b/forge-ai/src/main/java/forge/ai/ComputerUtil.java
@@ -423,7 +423,7 @@ public class ComputerUtil {
int mana = ComputerUtilMana.getAvailableManaEstimate(ai, false);
boolean cantAffordSoon = activate.getCMC() > mana + 1;
- boolean wrongColor = !activate.determineColor().hasNoColorsExcept(ColorSet.fromNames(ComputerUtilCost.getAvailableManaColors(ai, ImmutableList.of())).getColor());
+ boolean wrongColor = !activate.determineColor().hasNoColorsExcept(ColorSet.fromNames(ComputerUtilCost.getAvailableManaColors(ai, ImmutableList.of())).getColor());
// Only do this for spells, not activated abilities
// We can't pay for this spell even if we play another land, or have wrong colors
@@ -524,7 +524,7 @@ public class ComputerUtil {
typeList = CardLists.filter(typeList, CardPredicates.canBeSacrificedBy(ability));
- if ((target != null) && target.getController() == ai && typeList.contains(target)) {
+ if ((target != null) && target.getController() == ai) {
typeList.remove(target); // don't sacrifice the card we're pumping
}
@@ -554,7 +554,7 @@ public class ComputerUtil {
final Card target, final int amount) {
CardCollection typeList = CardLists.getValidCards(ai.getCardsIn(zone), type.split(";"), activate.getController(), activate, null);
- if ((target != null) && target.getController() == ai && typeList.contains(target)) {
+ if ((target != null) && target.getController() == ai) {
typeList.remove(target); // don't exile the card we're pumping
}
@@ -575,7 +575,7 @@ public class ComputerUtil {
final Card target, final int amount) {
CardCollection typeList = CardLists.getValidCards(ai.getCardsIn(zone), type.split(";"), activate.getController(), activate, null);
- if ((target != null) && target.getController() == ai && typeList.contains(target)) {
+ if ((target != null) && target.getController() == ai) {
typeList.remove(target); // don't move the card we're pumping
}
@@ -704,7 +704,7 @@ public class ComputerUtil {
public static CardCollection chooseReturnType(final Player ai, final String type, final Card activate, final Card target, final int amount) {
final CardCollection typeList =
CardLists.getValidCards(ai.getCardsIn(ZoneType.Battlefield), type.split(";"), activate.getController(), activate, null);
- if ((target != null) && target.getController() == ai && typeList.contains(target)) {
+ if ((target != null) && target.getController() == ai) {
// don't bounce the card we're pumping
typeList.remove(target);
}
@@ -794,11 +794,11 @@ public class ComputerUtil {
if (c.hasSVar("SacMe") || ComputerUtilCard.evaluateCreature(c) < sacThreshold) {
return true;
}
-
+
if (ComputerUtilCard.hasActiveUndyingOrPersist(c)) {
return true;
}
-
+
return false;
}
});
@@ -1925,9 +1925,9 @@ public class ComputerUtil {
if (predictThreatenedObjects(ai, null).contains(source)) {
return true;
}
- if (game.getPhaseHandler().inCombat() &&
- ComputerUtilCombat.combatantWouldBeDestroyed(ai, source, game.getCombat())) {
- return true;
+ if (game.getPhaseHandler().inCombat() &&
+ ComputerUtilCombat.combatantWouldBeDestroyed(ai, source, game.getCombat())) {
+ return true;
}
} else if (zone.getZoneType() == ZoneType.Exile && sa.getMayPlay() != null) {
// play cards in exile that can only be played that turn
@@ -1967,11 +1967,8 @@ public class ComputerUtil {
final CardCollectionView lands = CardLists.filter(handList, new Predicate() {
@Override
public boolean apply(final Card c) {
- if (c.getManaCost().getCMC() > 0 || c.hasSVar("NeedsToPlay")
- || (!c.getType().isLand() && !c.getType().isArtifact())) {
- return false;
- }
- return true;
+ return c.getManaCost().getCMC() <= 0 && !c.hasSVar("NeedsToPlay")
+ && (c.getType().isLand() || c.getType().isArtifact());
}
});
@@ -1986,10 +1983,7 @@ public class ComputerUtil {
final CardCollectionView castables = CardLists.filter(handList, new Predicate() {
@Override
public boolean apply(final Card c) {
- if (c.getManaCost().getCMC() > 0 && c.getManaCost().getCMC() <= landSize) {
- return false;
- }
- return true;
+ return c.getManaCost().getCMC() <= 0 || c.getManaCost().getCMC() > landSize;
}
});
@@ -2186,10 +2180,7 @@ public class ComputerUtil {
CardCollection goodChoices = CardLists.filter(validCards, new Predicate() {
@Override
public boolean apply(final Card c) {
- if (c.hasSVar("DiscardMeByOpp") || c.hasSVar("DiscardMe")) {
- return false;
- }
- return true;
+ return !c.hasSVar("DiscardMeByOpp") && !c.hasSVar("DiscardMe");
}
});
if (goodChoices.isEmpty()) {
@@ -2225,7 +2216,7 @@ public class ComputerUtil {
public static String chooseSomeType(Player ai, String kindOfType, String logic, List invalidTypes) {
if (invalidTypes == null) {
- invalidTypes = ImmutableList.of();
+ invalidTypes = ImmutableList.of();
}
final Game game = ai.getGame();
@@ -2546,8 +2537,7 @@ public class ComputerUtil {
@Override
public boolean apply(final Card c) {
if (c.getController() == ai) {
- if (c.getSVar("Targeting").equals("Dies") || c.getSVar("Targeting").equals("Counter"))
- return false;
+ return !c.getSVar("Targeting").equals("Dies") && !c.getSVar("Targeting").equals("Counter");
}
return true;
}
@@ -2875,7 +2865,6 @@ public class ComputerUtil {
} else if (Iterables.any(list, CardTraitPredicates.hasParam("AiLogic", "LichDraw"))) {
return false;
}
-
return true;
}
diff --git a/forge-ai/src/main/java/forge/ai/ComputerUtilCard.java b/forge-ai/src/main/java/forge/ai/ComputerUtilCard.java
index 843bc7c4e37..21197757d99 100644
--- a/forge-ai/src/main/java/forge/ai/ComputerUtilCard.java
+++ b/forge-ai/src/main/java/forge/ai/ComputerUtilCard.java
@@ -368,7 +368,7 @@ public class ComputerUtilCard {
}
if (hasEnchantmants || hasArtifacts) {
- final List ae = CardLists.filter(list, Predicates.and(Predicates.or(CardPredicates.Presets.ARTIFACTS, CardPredicates.Presets.ENCHANTMENTS), new Predicate() {
+ final List ae = CardLists.filter(list, Predicates.and(Predicates.or(CardPredicates.Presets.ARTIFACTS, CardPredicates.Presets.ENCHANTMENTS), new Predicate() {
@Override
public boolean apply(Card card) {
return !card.hasSVar("DoNotDiscardIfAble");
@@ -1301,7 +1301,7 @@ public class ComputerUtilCard {
combatTrick = true;
final List kws = sa.hasParam("KW") ? Arrays.asList(sa.getParam("KW").split(" & "))
- : Lists.newArrayList();
+ : Lists.newArrayList();
for (String kw : kws) {
if (!kw.equals("Trample") && !kw.equals("First Strike") && !kw.equals("Double Strike")) {
combatTrick = false;
diff --git a/forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java b/forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java
index 4a38704d130..9c030f3a2d4 100644
--- a/forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java
+++ b/forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java
@@ -84,7 +84,7 @@ public class ComputerUtilCombat {
return Iterables.any(defenders, new Predicate() {
@Override public boolean apply(final GameEntity input) {
return ComputerUtilCombat.canAttackNextTurn(attacker, input);
- };
+ }
});
} // canAttackNextTurn(Card)
@@ -119,11 +119,7 @@ public class ComputerUtilCombat {
}
// The creature won't untap next turn
- if (atacker.isTapped() && !Untap.canUntap(atacker)) {
- return false;
- }
-
- return true;
+ return !atacker.isTapped() || Untap.canUntap(atacker);
} // canAttackNextTurn(Card, GameEntity)
/**
@@ -883,20 +879,19 @@ public class ComputerUtilCombat {
} else if (mode == TriggerType.DamageDone) {
willTrigger = true;
if (trigParams.containsKey("ValidSource")) {
- if (CardTraitBase.matchesValid(defender, trigParams.get("ValidSource").split(","), source)
+ if (!(CardTraitBase.matchesValid(defender, trigParams.get("ValidSource").split(","), source)
&& defender.getNetCombatDamage() > 0
&& (!trigParams.containsKey("ValidTarget")
- || CardTraitBase.matchesValid(attacker, trigParams.get("ValidTarget").split(","), source))) {
- return true;
+ || CardTraitBase.matchesValid(attacker, trigParams.get("ValidTarget").split(","), source)))) {
+ return false;
}
- if (CardTraitBase.matchesValid(attacker, trigParams.get("ValidSource").split(","), source)
+ if (!(CardTraitBase.matchesValid(attacker, trigParams.get("ValidSource").split(","), source)
&& attacker.getNetCombatDamage() > 0
&& (!trigParams.containsKey("ValidTarget")
- || CardTraitBase.matchesValid(defender, trigParams.get("ValidTarget").split(","), source))) {
- return true;
+ || CardTraitBase.matchesValid(defender, trigParams.get("ValidTarget").split(","), source)))) {
+ return false;
}
}
- return false;
}
return willTrigger;
@@ -1414,7 +1409,7 @@ public class ComputerUtilCombat {
if (att.matches("[0-9][0-9]?") || att.matches("-" + "[0-9][0-9]?")) {
power += Integer.parseInt(att);
} else {
- String bonus = new String(source.getSVar(att));
+ String bonus = source.getSVar(att);
if (bonus.contains("TriggerCount$NumBlockers")) {
bonus = TextUtil.fastReplace(bonus, "TriggerCount$NumBlockers", "Number$1");
} else if (bonus.contains("TriggeredPlayersDefenders$Amount")) { // for Melee
@@ -1655,7 +1650,7 @@ public class ComputerUtilCombat {
if (def.matches("[0-9][0-9]?") || def.matches("-" + "[0-9][0-9]?")) {
toughness += Integer.parseInt(def);
} else {
- String bonus = new String(source.getSVar(def));
+ String bonus = source.getSVar(def);
if (bonus.contains("TriggerCount$NumBlockers")) {
bonus = TextUtil.fastReplace(bonus, "TriggerCount$NumBlockers", "Number$1");
} else if (bonus.contains("TriggeredPlayersDefenders$Amount")) { // for Melee
@@ -1948,7 +1943,7 @@ public class ComputerUtilCombat {
return defenderDamage >= attackerLife;
} // defender no double strike
- return false; // should never arrive here
+ return false;// should never arrive here
} // canDestroyAttacker
// For AI safety measures like Regeneration
@@ -2195,7 +2190,7 @@ public class ComputerUtilCombat {
return attackerDamage >= defenderLife;
} // attacker no double strike
- return false; // should never arrive here
+ return false;// should never arrive here
} // canDestroyBlocker
diff --git a/forge-ai/src/main/java/forge/ai/ComputerUtilCost.java b/forge-ai/src/main/java/forge/ai/ComputerUtilCost.java
index 43c942f94a2..bdbf2abf61c 100644
--- a/forge-ai/src/main/java/forge/ai/ComputerUtilCost.java
+++ b/forge-ai/src/main/java/forge/ai/ComputerUtilCost.java
@@ -424,7 +424,7 @@ public class ComputerUtilCost {
continue;
}
final int remainingLife = ai.getLife();
- final int lifeCost = ((CostPayLife) part).convertAmount();
+ final int lifeCost = part.convertAmount();
if ((remainingLife - lifeCost) < 10) {
return false; //Don't pay life if it would put AI under 10 life
} else if ((remainingLife / lifeCost) < 4) {
@@ -552,7 +552,7 @@ public class ComputerUtilCost {
final Card source = sa.getHostCard();
final String aiLogic = sa.getParam("UnlessAI");
boolean payForOwnOnly = "OnlyOwn".equals(aiLogic);
- boolean payOwner = sa.hasParam("UnlessAI") ? aiLogic.startsWith("Defined") : false;
+ boolean payOwner = sa.hasParam("UnlessAI") && aiLogic.startsWith("Defined");
boolean payNever = "Never".equals(aiLogic);
boolean shockland = "Shockland".equals(aiLogic);
boolean isMine = sa.getActivatingPlayer().equals(payer);
diff --git a/forge-ai/src/main/java/forge/ai/ComputerUtilMana.java b/forge-ai/src/main/java/forge/ai/ComputerUtilMana.java
index 7aae37dc721..8e9c0f84e2d 100644
--- a/forge-ai/src/main/java/forge/ai/ComputerUtilMana.java
+++ b/forge-ai/src/main/java/forge/ai/ComputerUtilMana.java
@@ -880,10 +880,12 @@ public class ComputerUtilMana {
// For combat tricks, always obey mana reservation
if (curPhase == PhaseType.COMBAT_DECLARE_BLOCKERS || curPhase == PhaseType.CLEANUP) {
- AiCardMemory.clearMemorySet(ai, AiCardMemory.MemorySet.HELD_MANA_SOURCES_FOR_DECLBLK);
- } else if (!(ai.getGame().getPhaseHandler().isPlayerTurn(ai)) && (curPhase == PhaseType.COMBAT_DECLARE_BLOCKERS || curPhase == PhaseType.CLEANUP)) {
- AiCardMemory.clearMemorySet(ai, AiCardMemory.MemorySet.HELD_MANA_SOURCES_FOR_ENEMY_DECLBLK);
- AiCardMemory.clearMemorySet(ai, AiCardMemory.MemorySet.CHOSEN_FOG_EFFECT);
+ if (!(ai.getGame().getPhaseHandler().isPlayerTurn(ai))) {
+ AiCardMemory.clearMemorySet(ai, AiCardMemory.MemorySet.HELD_MANA_SOURCES_FOR_ENEMY_DECLBLK);
+ AiCardMemory.clearMemorySet(ai, AiCardMemory.MemorySet.CHOSEN_FOG_EFFECT);
+ }
+ else
+ AiCardMemory.clearMemorySet(ai, AiCardMemory.MemorySet.HELD_MANA_SOURCES_FOR_DECLBLK);
} else {
if ((AiCardMemory.isRememberedCard(ai, sourceCard, AiCardMemory.MemorySet.HELD_MANA_SOURCES_FOR_DECLBLK)) ||
(AiCardMemory.isRememberedCard(ai, sourceCard, AiCardMemory.MemorySet.HELD_MANA_SOURCES_FOR_ENEMY_DECLBLK))) {
diff --git a/forge-ai/src/main/java/forge/ai/GameState.java b/forge-ai/src/main/java/forge/ai/GameState.java
index a7477639ce4..deb1a7d021b 100644
--- a/forge-ai/src/main/java/forge/ai/GameState.java
+++ b/forge-ai/src/main/java/forge/ai/GameState.java
@@ -1061,7 +1061,7 @@ public abstract class GameState {
}
private void applyCountersToGameEntity(GameEntity entity, String counterString) {
- entity.setCounters(Maps.newEnumMap(CounterType.class));
+ entity.setCounters(Maps.newEnumMap(CounterType.class));
String[] allCounterStrings = counterString.split(",");
for (final String counterPair : allCounterStrings) {
String[] pair = counterPair.split("=", 2);
@@ -1107,7 +1107,7 @@ public abstract class GameState {
Map counters = c.getCounters();
// Note: Not clearCounters() since we want to keep the counters
// var as-is.
- c.setCounters(Maps.newEnumMap(CounterType.class));
+ c.setCounters(Maps.newEnumMap(CounterType.class));
if (c.isAura()) {
// dummy "enchanting" to indicate that the card will be force-attached elsewhere
// (will be overridden later, so the actual value shouldn't matter)
diff --git a/forge-ai/src/main/java/forge/ai/PlayerControllerAi.java b/forge-ai/src/main/java/forge/ai/PlayerControllerAi.java
index 529df9a08c8..7e4923d8c9d 100644
--- a/forge-ai/src/main/java/forge/ai/PlayerControllerAi.java
+++ b/forge-ai/src/main/java/forge/ai/PlayerControllerAi.java
@@ -492,7 +492,7 @@ public class PlayerControllerAi extends PlayerController {
Card toDiscard = Aggregates.itemWithMin(cardsOfType, CardPredicates.Accessors.fnGetCmc);
return new CardCollection(toDiscard);
}
- return getAi().getCardsToDiscard(num, (String[])null, sa);
+ return getAi().getCardsToDiscard(num, null, sa);
}
@@ -612,7 +612,7 @@ public class PlayerControllerAi extends PlayerController {
@Override
public CardCollection chooseCardsToDiscardToMaximumHandSize(int numDiscard) {
- return brains.getCardsToDiscard(numDiscard, (String[])null, null);
+ return brains.getCardsToDiscard(numDiscard, null, null);
}
@Override
@@ -669,7 +669,7 @@ public class PlayerControllerAi extends PlayerController {
throw new InvalidParameterException("SA is not api-based, this is not supported yet");
}
return SpellApiToAi.Converter.get(api).chooseNumber(player, sa, min, max, params);
- };
+ }
@Override
public int chooseNumber(SpellAbility sa, String title, List options, Player relatedPlayer) {
diff --git a/forge-ai/src/main/java/forge/ai/SpecialCardAi.java b/forge-ai/src/main/java/forge/ai/SpecialCardAi.java
index 1b3afb1d8c1..27700e2455b 100644
--- a/forge-ai/src/main/java/forge/ai/SpecialCardAi.java
+++ b/forge-ai/src/main/java/forge/ai/SpecialCardAi.java
@@ -94,12 +94,8 @@ public class SpecialCardAi {
int minCMC = isLowCMCDeck ? 3 : 4; // probably not worth wasting a lotus on a low-CMC spell (<4 CMC), except in low-CMC decks, where 3 CMC may be fine
int paidCMC = cost.getConvertedManaCost();
if (paidCMC < minCMC) {
- if (paidCMC == 3 && numManaSrcs < 3) {
- // if it's a CMC 3 spell and we're more than one mana source short for it, might be worth it anyway
- return true;
- }
-
- return false;
+ // if it's a CMC 3 spell and we're more than one mana source short for it, might be worth it anyway
+ return paidCMC == 3 && numManaSrcs < 3;
}
return true;
@@ -218,11 +214,7 @@ public class SpecialCardAi {
}
}
- if (ai.getLife() <= sa.getHostCard().getNetPower() && !hasUsefulBlocker) {
- return true;
- } else {
- return false;
- }
+ return ai.getLife() <= sa.getHostCard().getNetPower() && !hasUsefulBlocker;
}
public static int getSacThreshold() {
@@ -335,7 +327,7 @@ public class SpecialCardAi {
boolean canTrample = source.hasKeyword(Keyword.TRAMPLE);
if (!isBlocking && combat.getDefenderByAttacker(source) instanceof Card) {
- int loyalty = ((Card)combat.getDefenderByAttacker(source)).getCounters(CounterType.LOYALTY);
+ int loyalty = combat.getDefenderByAttacker(source).getCounters(CounterType.LOYALTY);
int totalDamageToPW = 0;
for (Card atk : (combat.getAttackersOf(combat.getDefenderByAttacker(source)))) {
if (combat.isUnblocked(atk)) {
@@ -411,11 +403,7 @@ public class SpecialCardAi {
Pair predictedPT = getPumpedPT(ai, source.getNetCombatDamage(), source.getNetToughness());
int oppT = Aggregates.sum(potentialBlockers, CardPredicates.Accessors.fnGetNetToughness);
- if (potentialBlockers.isEmpty() || (source.hasKeyword(Keyword.TRAMPLE) && predictedPT.getLeft() - oppT >= oppLife)) {
- return true;
- }
-
- return false;
+ return potentialBlockers.isEmpty() || (source.hasKeyword(Keyword.TRAMPLE) && predictedPT.getLeft() - oppT >= oppLife);
}
public static Pair getPumpedPT(Player ai, int power, int toughness) {
@@ -522,7 +510,7 @@ public class SpecialCardAi {
best.add(sp); // these SAs are prioritized since the AI sees a reason to play them now
}
final List keywords = sp.hasParam("KW") ? Arrays.asList(sp.getParam("KW").split(" & "))
- : Lists.newArrayList();
+ : Lists.newArrayList();
for (String kw : keywords) {
if (!tgtCard.hasKeyword(kw)) {
if ("Indestructible".equals(kw) && ai.getOpponents().getCreaturesInPlay().isEmpty()) {
@@ -568,10 +556,7 @@ public class SpecialCardAi {
@Override
public boolean apply(final Card c) {
// Don't enchant creatures that can survive
- if (!c.canBeDestroyed() || c.getNetCombatDamage() < c.getNetToughness() || c.isEnchantedBy("Guilty Conscience")) {
- return false;
- }
- return true;
+ return c.canBeDestroyed() && c.getNetCombatDamage() >= c.getNetToughness() && !c.isEnchantedBy("Guilty Conscience");
}
});
chosen = ComputerUtilCard.getBestCreatureAI(creatures);
@@ -911,15 +896,14 @@ public class SpecialCardAi {
}
} else if (blackViseOTB && computerHandSize + exiledWithNecro - 1 >= 4) {
// try not to overdraw in presence of Black Vise
- return false;
+ return false;
} else if (computerHandSize + exiledWithNecro - 1 >= maxHandSize) {
// Only draw until we reach max hand size
return false;
} else if (!ph.isPlayerTurn(ai) || !ph.is(PhaseType.MAIN2)) {
// Only activate in AI's own turn (sans the exception above)
return false;
- }
-
+ }
return true;
}
}
@@ -941,11 +925,7 @@ public class SpecialCardAi {
}
// Maybe use it for some important high-impact spells even if there are more cards in hand?
- if (ai.getCardsIn(ZoneType.Hand).size() > 1 && !hasEnsnaringBridgeEffect) {
- return false;
- }
-
- return true;
+ return ai.getCardsIn(ZoneType.Hand).size() <= 1 || hasEnsnaringBridgeEffect;
}
}
@@ -1310,12 +1290,8 @@ public class SpecialCardAi {
}
}
- if (aiHandSize < HAND_SIZE_THRESHOLD || maxOppHandSize - aiHandSize > HAND_SIZE_THRESHOLD) {
- // use in case we're getting low on cards or if we're significantly behind our opponent in cards in hand
- return true;
- }
-
- return false;
+ // use in case we're getting low on cards or if we're significantly behind our opponent in cards in hand
+ return aiHandSize < HAND_SIZE_THRESHOLD || maxOppHandSize - aiHandSize > HAND_SIZE_THRESHOLD;
}
}
@@ -1342,9 +1318,7 @@ public class SpecialCardAi {
if (topGY == null
|| !topGY.isCreature()
|| ComputerUtilCard.evaluateCreature(creatHand) > ComputerUtilCard.evaluateCreature(topGY) + 80) {
- if (numCreatsInHand > 1 || !ComputerUtilMana.canPayManaCost(creatHand.getSpellPermanent(), ai, 0)) {
- return true;
- }
+ return numCreatsInHand > 1 || !ComputerUtilMana.canPayManaCost(creatHand.getSpellPermanent(), ai, 0);
}
}
@@ -1465,8 +1439,7 @@ public class SpecialCardAi {
} else if (!ph.isPlayerTurn(ai)) {
// Only activate in AI's own turn (sans the exception above)
return false;
- }
-
+ }
return true;
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/ActivateAbilityAi.java b/forge-ai/src/main/java/forge/ai/ability/ActivateAbilityAi.java
index 44780ec2e94..18029dd4d7f 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ActivateAbilityAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ActivateAbilityAi.java
@@ -56,12 +56,9 @@ public class ActivateAbilityAi extends SpellAbilityAi {
} else {
final List defined = AbilityUtils.getDefinedPlayers(source, sa.getParam("Defined"), sa);
- if (!defined.contains(opp)) {
- return false;
- }
+ return defined.contains(opp);
}
- return true;
} else {
sa.resetTargets();
sa.getTargets().add(opp);
diff --git a/forge-ai/src/main/java/forge/ai/ability/AddTurnAi.java b/forge-ai/src/main/java/forge/ai/ability/AddTurnAi.java
index 243edf2c3f7..f20a3f2f7fd 100644
--- a/forge-ai/src/main/java/forge/ai/ability/AddTurnAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/AddTurnAi.java
@@ -67,10 +67,8 @@ public class AddTurnAi extends SpellAbilityAi {
return false;
}
}
- if (!StringUtils.isNumeric(sa.getParam("NumTurns"))) {
- // TODO: improve ai for Sage of Hours
- return false;
- }
+ // TODO: improve ai for Sage of Hours
+ return StringUtils.isNumeric(sa.getParam("NumTurns"));
// not sure if the AI should be playing with cards that give the
// Human more turns.
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/AmassAi.java b/forge-ai/src/main/java/forge/ai/ability/AmassAi.java
index 0c98f2b65c6..7699259abd1 100644
--- a/forge-ai/src/main/java/forge/ai/ability/AmassAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/AmassAi.java
@@ -23,9 +23,7 @@ public class AmassAi extends SpellAbilityAi {
final Game game = ai.getGame();
if (!aiArmies.isEmpty()) {
- if (CardLists.count(aiArmies, CardPredicates.canReceiveCounters(CounterType.P1P1)) <= 0) {
- return false;
- }
+ return CardLists.count(aiArmies, CardPredicates.canReceiveCounters(CounterType.P1P1)) > 0;
} else {
final String tokenScript = "b_0_0_zombie_army";
final int amount = AbilityUtils.calculateAmount(host, sa.getParamOrDefault("Num", "1"), sa);
@@ -57,12 +55,9 @@ public class AmassAi extends SpellAbilityAi {
//reset static abilities
game.getAction().checkStaticAbilities(false);
- if (!result) {
- return false;
- }
+ return result;
}
- return true;
}
@Override
diff --git a/forge-ai/src/main/java/forge/ai/ability/AnimateAi.java b/forge-ai/src/main/java/forge/ai/ability/AnimateAi.java
index 0ba446e250e..17748584c9f 100644
--- a/forge-ai/src/main/java/forge/ai/ability/AnimateAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/AnimateAi.java
@@ -207,21 +207,16 @@ public class AnimateAi extends SpellAbilityAi {
return bFlag; // All of the defined stuff is animated, not very useful
} else {
sa.resetTargets();
- if (!animateTgtAI(sa)) {
- return false;
- }
+ return animateTgtAI(sa);
}
- return true;
}
@Override
public boolean chkAIDrawback(SpellAbility sa, Player aiPlayer) {
if (sa.usesTargeting()) {
sa.resetTargets();
- if (!animateTgtAI(sa)) {
- return false;
- }
+ return animateTgtAI(sa);
}
return true;
diff --git a/forge-ai/src/main/java/forge/ai/ability/AnimateAllAi.java b/forge-ai/src/main/java/forge/ai/ability/AnimateAllAi.java
index 8e126db8ae8..79770568fd3 100644
--- a/forge-ai/src/main/java/forge/ai/ability/AnimateAllAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/AnimateAllAi.java
@@ -8,11 +8,7 @@ public class AnimateAllAi extends SpellAbilityAi {
@Override
protected boolean canPlayAI(Player aiPlayer, SpellAbility sa) {
- if ("Always".equals(sa.getParam("AILogic"))) {
- return true;
- }
-
- return false;
+ return "Always".equals(sa.getParam("AILogic"));
} // end animateAllCanPlayAI()
@Override
diff --git a/forge-ai/src/main/java/forge/ai/ability/AttachAi.java b/forge-ai/src/main/java/forge/ai/ability/AttachAi.java
index 91041b9767a..16fe3ec0936 100644
--- a/forge-ai/src/main/java/forge/ai/ability/AttachAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/AttachAi.java
@@ -123,9 +123,7 @@ public class AttachAi extends SpellAbilityAi {
return !(c.hasProtectionFrom(source) || c.hasKeyword(Keyword.SHROUD) || c.hasKeyword(Keyword.HEXPROOF));
}
});
- if (targets.isEmpty()) {
- return false;
- }
+ return !targets.isEmpty();
}
return true;
@@ -239,9 +237,7 @@ public class AttachAi extends SpellAbilityAi {
return false;
}
- if (!(combat.isAttacking(attachTarget) || combat.isBlocking(attachTarget))) {
- return false;
- }
+ return combat.isAttacking(attachTarget) || combat.isBlocking(attachTarget);
}
return true;
@@ -987,9 +983,7 @@ public class AttachAi extends SpellAbilityAi {
return false;
}
// don't equip creatures that don't gain anything
- if (card.hasSVar("NonStackingAttachEffect") && newTarget.isEquippedBy(card.getName())) {
- return false;
- }
+ return !card.hasSVar("NonStackingAttachEffect") || !newTarget.isEquippedBy(card.getName());
}
}
@@ -1353,7 +1347,7 @@ public class AttachAi extends SpellAbilityAi {
CardCollection prefList = list;
// Filter AI-specific targets if provided
- prefList = ComputerUtil.filterAITgts(sa, aiPlayer, (CardCollection)list, true);
+ prefList = ComputerUtil.filterAITgts(sa, aiPlayer, list, true);
Card c = attachGeneralAI(aiPlayer, sa, prefList, mandatory, attachSource, sa.getParam("AILogic"));
@@ -1557,86 +1551,55 @@ public class AttachAi extends SpellAbilityAi {
}
if (evasive) {
- if (card.getNetCombatDamage() + powerBonus <= 0
- || !ComputerUtilCombat.canAttackNextTurn(card)
- || !canBeBlocked) {
- return false;
- }
+ return card.getNetCombatDamage() + powerBonus > 0
+ && ComputerUtilCombat.canAttackNextTurn(card)
+ && canBeBlocked;
} else if (keyword.equals("Haste")) {
- if (!card.hasSickness() || !ph.isPlayerTurn(sa.getActivatingPlayer()) || card.isTapped()
- || card.getNetCombatDamage() + powerBonus <= 0
- || card.hasKeyword("CARDNAME can attack as though it had haste.")
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || !ComputerUtilCombat.canAttackNextTurn(card)) {
- return false;
- }
+ return card.hasSickness() && ph.isPlayerTurn(sa.getActivatingPlayer()) && !card.isTapped()
+ && card.getNetCombatDamage() + powerBonus > 0
+ && !card.hasKeyword("CARDNAME can attack as though it had haste.")
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && ComputerUtilCombat.canAttackNextTurn(card);
} else if (keyword.endsWith("Indestructible")) {
return true;
} else if (keyword.endsWith("Deathtouch") || keyword.endsWith("Wither")) {
- if (card.getNetCombatDamage() + powerBonus <= 0
- || ((!canBeBlocked || !ComputerUtilCombat.canAttackNextTurn(card))
- && !CombatUtil.canBlock(card, true))) {
- return false;
- }
+ return card.getNetCombatDamage() + powerBonus > 0
+ && ((canBeBlocked && ComputerUtilCombat.canAttackNextTurn(card))
+ || CombatUtil.canBlock(card, true));
} else if (keyword.equals("Double Strike") || keyword.equals("Lifelink")) {
- if (card.getNetCombatDamage() + powerBonus <= 0
- || (!ComputerUtilCombat.canAttackNextTurn(card) && !CombatUtil.canBlock(card, true))) {
- return false;
- }
+ return card.getNetCombatDamage() + powerBonus > 0
+ && (ComputerUtilCombat.canAttackNextTurn(card) || CombatUtil.canBlock(card, true));
} else if (keyword.equals("First Strike")) {
- if (card.getNetCombatDamage() + powerBonus <= 0 || card.hasKeyword(Keyword.DOUBLE_STRIKE)
- || (!ComputerUtilCombat.canAttackNextTurn(card) && !CombatUtil.canBlock(card, true))) {
- return false;
- }
+ return card.getNetCombatDamage() + powerBonus > 0 && !card.hasKeyword(Keyword.DOUBLE_STRIKE)
+ && (ComputerUtilCombat.canAttackNextTurn(card) || CombatUtil.canBlock(card, true));
} else if (keyword.startsWith("Flanking")) {
- if (card.getNetCombatDamage() + powerBonus <= 0
- || !ComputerUtilCombat.canAttackNextTurn(card)
- || !canBeBlocked) {
- return false;
- }
+ return card.getNetCombatDamage() + powerBonus > 0
+ && ComputerUtilCombat.canAttackNextTurn(card)
+ && canBeBlocked;
} else if (keyword.startsWith("Bushido")) {
- if ((!canBeBlocked || !ComputerUtilCombat.canAttackNextTurn(card))
- && !CombatUtil.canBlock(card, true)) {
- return false;
- }
+ return (canBeBlocked && ComputerUtilCombat.canAttackNextTurn(card))
+ || CombatUtil.canBlock(card, true);
} else if (keyword.equals("Trample")) {
- if (card.getNetCombatDamage() + powerBonus <= 1
- || !canBeBlocked
- || !ComputerUtilCombat.canAttackNextTurn(card)) {
- return false;
- }
+ return card.getNetCombatDamage() + powerBonus > 1
+ && canBeBlocked
+ && ComputerUtilCombat.canAttackNextTurn(card);
} else if (keyword.equals("Infect")) {
- if (card.getNetCombatDamage() + powerBonus <= 0
- || !ComputerUtilCombat.canAttackNextTurn(card)) {
- return false;
- }
+ return card.getNetCombatDamage() + powerBonus > 0
+ && ComputerUtilCombat.canAttackNextTurn(card);
} else if (keyword.equals("Vigilance")) {
- if (card.getNetCombatDamage() + powerBonus <= 0
- || !ComputerUtilCombat.canAttackNextTurn(card)
- || !CombatUtil.canBlock(card, true)) {
- return false;
- }
+ return card.getNetCombatDamage() + powerBonus > 0
+ && ComputerUtilCombat.canAttackNextTurn(card)
+ && CombatUtil.canBlock(card, true);
} else if (keyword.equals("Reach")) {
- if (card.hasKeyword(Keyword.FLYING) || !CombatUtil.canBlock(card, true)) {
- return false;
- }
+ return !card.hasKeyword(Keyword.FLYING) && CombatUtil.canBlock(card, true);
} else if (keyword.endsWith("CARDNAME can block an additional creature each combat.")) {
- if (!CombatUtil.canBlock(card, true) || card.hasKeyword("CARDNAME can block any number of creatures.")
- || card.hasKeyword("CARDNAME can block an additional ninety-nine creatures each combat.")) {
- return false;
- }
+ return CombatUtil.canBlock(card, true) && !card.hasKeyword("CARDNAME can block any number of creatures.")
+ && !card.hasKeyword("CARDNAME can block an additional ninety-nine creatures each combat.");
} else if (keyword.equals("CARDNAME can attack as though it didn't have defender.")) {
- if (!card.hasKeyword(Keyword.DEFENDER) || card.getNetCombatDamage() + powerBonus <= 0) {
- return false;
- }
+ return card.hasKeyword(Keyword.DEFENDER) && card.getNetCombatDamage() + powerBonus > 0;
} else if (keyword.equals("Shroud") || keyword.equals("Hexproof")) {
- if (card.hasKeyword(Keyword.SHROUD) || card.hasKeyword(Keyword.HEXPROOF)) {
- return false;
- }
- } else if (keyword.equals("Defender")) {
- return false;
- }
- return true;
+ return !card.hasKeyword(Keyword.SHROUD) && !card.hasKeyword(Keyword.HEXPROOF);
+ } else return !keyword.equals("Defender");
}
/**
@@ -1657,17 +1620,11 @@ public class AttachAi extends SpellAbilityAi {
if (keyword.endsWith("CARDNAME can't attack.") || keyword.equals("Defender")
|| keyword.endsWith("CARDNAME can't attack or block.")) {
- if (!ComputerUtilCombat.canAttackNextTurn(card) || card.getNetCombatDamage() < 1) {
- return false;
- }
+ return ComputerUtilCombat.canAttackNextTurn(card) && card.getNetCombatDamage() >= 1;
} else if (keyword.endsWith("CARDNAME attacks each turn if able.") || keyword.endsWith("CARDNAME attacks each combat if able.")) {
- if (!ComputerUtilCombat.canAttackNextTurn(card) || !CombatUtil.canBlock(card, true) || ai.getCreaturesInPlay().isEmpty()) {
- return false;
- }
+ return ComputerUtilCombat.canAttackNextTurn(card) && CombatUtil.canBlock(card, true) && !ai.getCreaturesInPlay().isEmpty();
} else if (keyword.endsWith("CARDNAME can't block.") || keyword.contains("CantBlock")) {
- if (!CombatUtil.canBlock(card, true)) {
- return false;
- }
+ return CombatUtil.canBlock(card, true);
} else if (keyword.endsWith("CARDNAME's activated abilities can't be activated.")) {
for (SpellAbility ability : card.getSpellAbilities()) {
if (ability.isAbility()) {
@@ -1676,18 +1633,12 @@ public class AttachAi extends SpellAbilityAi {
}
return false;
} else if (keyword.endsWith("Prevent all combat damage that would be dealt by CARDNAME.")) {
- if (!ComputerUtilCombat.canAttackNextTurn(card) || card.getNetCombatDamage() < 1) {
- return false;
- }
+ return ComputerUtilCombat.canAttackNextTurn(card) && card.getNetCombatDamage() >= 1;
} else if (keyword.endsWith("Prevent all combat damage that would be dealt to and dealt by CARDNAME.")
|| keyword.endsWith("Prevent all damage that would be dealt to and dealt by CARDNAME.")) {
- if (!ComputerUtilCombat.canAttackNextTurn(card) || card.getNetCombatDamage() < 2) {
- return false;
- }
+ return ComputerUtilCombat.canAttackNextTurn(card) && card.getNetCombatDamage() >= 2;
} else if (keyword.endsWith("CARDNAME doesn't untap during your untap step.")) {
- if (card.isUntapped()) {
- return false;
- }
+ return !card.isUntapped();
}
return true;
}
@@ -1711,12 +1662,8 @@ public class AttachAi extends SpellAbilityAi {
return true;
}
- if (sa.getHostCard().isEquipment() && ComputerUtilCard.isUselessCreature(ai, c)) {
- // useless to equip a creature that can't attack or block.
- return false;
- }
-
- return true;
+ // useless to equip a creature that can't attack or block.
+ return !sa.getHostCard().isEquipment() || !ComputerUtilCard.isUselessCreature(ai, c);
}
public static Card doPumpOrCurseAILogic(final Player ai, final SpellAbility sa, final List list, final String type) {
diff --git a/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAi.java b/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAi.java
index a2906c96b88..93163475b87 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAi.java
@@ -355,9 +355,7 @@ public class ChangeZoneAi extends SpellAbilityAi {
@Override
public boolean apply(final Card c) {
if (c.getType().isLegendary()) {
- if (ai.isCardInPlay(c.getName())) {
- return false;
- }
+ return !ai.isCardInPlay(c.getName());
}
return true;
}
@@ -738,11 +736,7 @@ public class ChangeZoneAi extends SpellAbilityAi {
}
final AbilitySub subAb = sa.getSubAbility();
- if (subAb != null && !SpellApiToAi.Converter.get(subAb.getApi()).chkDrawbackWithSubs(ai, subAb)) {
- return false;
- }
-
- return true;
+ return subAb == null || SpellApiToAi.Converter.get(subAb.getApi()).chkDrawbackWithSubs(ai, subAb);
}
/*
@@ -864,7 +858,7 @@ public class ChangeZoneAi extends SpellAbilityAi {
list = CardLists.getTargetableCards(list, sa);
// Filter AI-specific targets if provided
- list = ComputerUtil.filterAITgts(sa, ai, (CardCollection)list, true);
+ list = ComputerUtil.filterAITgts(sa, ai, list, true);
if (sa.hasParam("AITgtsOnlyBetterThanSelf")) {
list = CardLists.filter(list, new Predicate() {
@Override
@@ -995,11 +989,7 @@ public class ChangeZoneAi extends SpellAbilityAi {
@Override
public boolean apply(final Card c) {
for (Card aura : c.getEnchantedBy()) {
- if (aura.getController().isOpponentOf(ai)) {
- return true;
- } else {
- return false;
- }
+ return aura.getController().isOpponentOf(ai);
}
if (blink) {
return c.isToken();
@@ -1471,16 +1461,12 @@ public class ChangeZoneAi extends SpellAbilityAi {
if (!list.isEmpty()) {
final Card attachedTo = list.get(0);
// This code is for the Dragon auras
- if (attachedTo.getController().isOpponentOf(ai)) {
- return false;
- }
+ return !attachedTo.getController().isOpponentOf(ai);
}
}
} else if (isPreferredTarget(ai, sa, mandatory, true)) {
// do nothing
- } else if (!isUnpreferredTarget(ai, sa, mandatory)) {
- return false;
- }
+ } else return isUnpreferredTarget(ai, sa, mandatory);
return true;
}
@@ -1532,9 +1518,7 @@ public class ChangeZoneAi extends SpellAbilityAi {
@Override
public boolean apply(final Card c) {
if (c.getType().isLegendary()) {
- if (decider.isCardInPlay(c.getName())) {
- return false;
- }
+ return !decider.isCardInPlay(c.getName());
}
return true;
}
@@ -1543,10 +1527,7 @@ public class ChangeZoneAi extends SpellAbilityAi {
fetchList = CardLists.filter(fetchList, new Predicate() {
@Override
public boolean apply(final Card c) {
- if (ComputerUtilCard.isCardRemAIDeck(c) || ComputerUtilCard.isCardRemRandomDeck(c)) {
- return false;
- }
- return true;
+ return !ComputerUtilCard.isCardRemAIDeck(c) && !ComputerUtilCard.isCardRemRandomDeck(c);
}
});
}
@@ -1718,9 +1699,7 @@ public class ChangeZoneAi extends SpellAbilityAi {
@Override
public boolean apply(final Card c) {
if (c.getType().isLegendary()) {
- if (ai.isCardInPlay(c.getName())) {
- return false;
- }
+ return !ai.isCardInPlay(c.getName());
}
return true;
}
@@ -1826,20 +1805,16 @@ public class ChangeZoneAi extends SpellAbilityAi {
&& "Battlefield".equals(causeSub.getParam("Destination"))) {
// A blink effect implemented using ChangeZone API
return false;
- } else if (subApi == ApiType.DelayedTrigger) {
+ } else // This is an intrinsic effect that blinks the card (e.g. Obzedat, Ghost Council), no need to
+ // return the commander to the Command zone.
+ if (subApi == ApiType.DelayedTrigger) {
SpellAbility exec = causeSub.getAdditionalAbility("Execute");
if (exec != null && exec.getApi() == ApiType.ChangeZone) {
- if ("Exile".equals(exec.getParam("Origin")) && "Battlefield".equals(exec.getParam("Destination"))) {
- // A blink effect implemented using a delayed trigger
- return false;
- }
+ // A blink effect implemented using a delayed trigger
+ return !"Exile".equals(exec.getParam("Origin")) || !"Battlefield".equals(exec.getParam("Destination"));
}
- } else if (causeSa.getHostCard() != null && causeSa.getHostCard().equals((Card)sa.getReplacingObject("Card"))
- && causeSa.getActivatingPlayer().equals(aiPlayer)) {
- // This is an intrinsic effect that blinks the card (e.g. Obzedat, Ghost Council), no need to
- // return the commander to the Command zone.
- return false;
- }
+ } else return causeSa.getHostCard() == null || !causeSa.getHostCard().equals(sa.getReplacingObject("Card"))
+ || !causeSa.getActivatingPlayer().equals(aiPlayer);
}
// Normally we want the commander back in Command zone to recast him later
diff --git a/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAllAi.java b/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAllAi.java
index c17506d37b2..fcf0733bf0d 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAllAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAllAi.java
@@ -335,11 +335,8 @@ public class ChangeZoneAllAi extends SpellAbilityAi {
return true;
// if AI creature is better than Human Creature
- if (ComputerUtilCard.evaluateCreatureList(aiCards) >= ComputerUtilCard
- .evaluateCreatureList(humanCards)) {
- return true;
- }
- return false;
+ return ComputerUtilCard.evaluateCreatureList(aiCards) >= ComputerUtilCard
+ .evaluateCreatureList(humanCards);
}
return true;
}
@@ -441,29 +438,21 @@ public class ChangeZoneAllAi extends SpellAbilityAi {
if (sa.getParam("GainControl") != null) {
// Check if the cards are valuable enough
if ((CardLists.getNotType(humanType, "Creature").size() == 0) && (CardLists.getNotType(computerType, "Creature").size() == 0)) {
- if ((ComputerUtilCard.evaluateCreatureList(computerType) + ComputerUtilCard
- .evaluateCreatureList(humanType)) < 1) {
- return false;
- }
+ return (ComputerUtilCard.evaluateCreatureList(computerType) + ComputerUtilCard
+ .evaluateCreatureList(humanType)) >= 1;
} // otherwise evaluate both lists by CMC and pass only if human
// permanents are less valuable
- else if ((ComputerUtilCard.evaluatePermanentList(computerType) + ComputerUtilCard
- .evaluatePermanentList(humanType)) < 1) {
- return false;
- }
+ else return (ComputerUtilCard.evaluatePermanentList(computerType) + ComputerUtilCard
+ .evaluatePermanentList(humanType)) >= 1;
} else {
// don't activate if human gets more back than AI does
if ((CardLists.getNotType(humanType, "Creature").isEmpty()) && (CardLists.getNotType(computerType, "Creature").isEmpty())) {
- if (ComputerUtilCard.evaluateCreatureList(computerType) <= ComputerUtilCard
- .evaluateCreatureList(humanType)) {
- return false;
- }
+ return ComputerUtilCard.evaluateCreatureList(computerType) > ComputerUtilCard
+ .evaluateCreatureList(humanType);
} // otherwise evaluate both lists by CMC and pass only if human
// permanents are less valuable
- else if (ComputerUtilCard.evaluatePermanentList(computerType) <= ComputerUtilCard
- .evaluatePermanentList(humanType)) {
- return false;
- }
+ else return ComputerUtilCard.evaluatePermanentList(computerType) > ComputerUtilCard
+ .evaluatePermanentList(humanType);
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/ChooseCardAi.java b/forge-ai/src/main/java/forge/ai/ability/ChooseCardAi.java
index 6d0cee69a14..1beefd77b09 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ChooseCardAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ChooseCardAi.java
@@ -71,21 +71,15 @@ public class ChooseCardAi extends SpellAbilityAi {
choices = CardLists.filterControlledBy(choices, ai.getOpponents());
}
if (aiLogic.equals("AtLeast1") || aiLogic.equals("OppPreferred")) {
- if (choices.isEmpty()) {
- return false;
- }
+ return !choices.isEmpty();
} else if (aiLogic.equals("AtLeast2") || aiLogic.equals("BestBlocker")) {
- if (choices.size() < 2) {
- return false;
- }
+ return choices.size() >= 2;
} else if (aiLogic.equals("Clone") || aiLogic.equals("Vesuva")) {
final String filter = aiLogic.equals("Clone") ? "Permanent.YouDontCtrl,Permanent.nonLegendary"
: "Permanent.YouDontCtrl+notnamedVesuva,Permanent.nonLegendary+notnamedVesuva";
choices = CardLists.getValidCards(choices, filter, host.getController(), host);
- if (choices.isEmpty()) {
- return false;
- }
+ return !choices.isEmpty();
} else if (aiLogic.equals("Never")) {
return false;
} else if (aiLogic.equals("NeedsPrevention")) {
@@ -103,9 +97,7 @@ public class ChooseCardAi extends SpellAbilityAi {
return ComputerUtilCombat.damageIfUnblocked(c, ai, combat, true) > ref;
}
});
- if (choices.isEmpty()) {
- return false;
- }
+ return !choices.isEmpty();
} else if (aiLogic.equals("Ashiok")) {
final int loyalty = host.getCounters(CounterType.LOYALTY) - 1;
for (int i = loyalty; i >= 0; i--) {
@@ -117,13 +109,9 @@ public class ChooseCardAi extends SpellAbilityAi {
}
}
- if (choices.isEmpty()) {
- return false;
- }
+ return !choices.isEmpty();
} else if (aiLogic.equals("RandomNonLand")) {
- if (CardLists.getValidCards(choices, "Card.nonLand", host.getController(), host).isEmpty()) {
- return false;
- }
+ return !CardLists.getValidCards(choices, "Card.nonLand", host.getController(), host).isEmpty();
} else if (aiLogic.equals("Duneblast")) {
CardCollection aiCreatures = ai.getCreaturesInPlay();
CardCollection oppCreatures = ai.getWeakestOpponent().getCreaturesInPlay();
@@ -139,10 +127,8 @@ public class ChooseCardAi extends SpellAbilityAi {
aiCreatures.remove(chosen);
int minGain = 200;
- if ((ComputerUtilCard.evaluateCreatureList(aiCreatures) + minGain) >= ComputerUtilCard
- .evaluateCreatureList(oppCreatures)) {
- return false;
- }
+ return (ComputerUtilCard.evaluateCreatureList(aiCreatures) + minGain) < ComputerUtilCard
+ .evaluateCreatureList(oppCreatures);
}
return true;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/ChooseColorAi.java b/forge-ai/src/main/java/forge/ai/ability/ChooseColorAi.java
index 8ba40274dcd..2633899924b 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ChooseColorAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ChooseColorAi.java
@@ -52,10 +52,7 @@ public class ChooseColorAi extends SpellAbilityAi {
}
if ("Addle".equals(sourceName)) {
- if (ph.getPhase().isBefore(PhaseType.COMBAT_DECLARE_ATTACKERS) || ai.getWeakestOpponent().getCardsIn(ZoneType.Hand).isEmpty()) {
- return false;
- }
- return true;
+ return !ph.getPhase().isBefore(PhaseType.COMBAT_DECLARE_ATTACKERS) && !ai.getWeakestOpponent().getCardsIn(ZoneType.Hand).isEmpty();
}
if (logic.equals("MostExcessOpponentControls")) {
diff --git a/forge-ai/src/main/java/forge/ai/ability/ChooseDirectionAi.java b/forge-ai/src/main/java/forge/ai/ability/ChooseDirectionAi.java
index 3480f561d70..b3ae74d5274 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ChooseDirectionAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ChooseDirectionAi.java
@@ -33,9 +33,7 @@ public class ChooseDirectionAi extends SpellAbilityAi {
CardCollection right = CardLists.filterControlledBy(all, game.getNextPlayerAfter(ai, Direction.Right));
int leftValue = Aggregates.sum(left, CardPredicates.Accessors.fnGetCmc);
int rightValue = Aggregates.sum(right, CardPredicates.Accessors.fnGetCmc);
- if (aiValue > leftValue || aiValue > rightValue) {
- return false;
- }
+ return aiValue <= leftValue && aiValue <= rightValue;
}
}
return true;
diff --git a/forge-ai/src/main/java/forge/ai/ability/ChooseGenericEffectAi.java b/forge-ai/src/main/java/forge/ai/ability/ChooseGenericEffectAi.java
index 2b608b2acbd..4ecfec696df 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ChooseGenericEffectAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ChooseGenericEffectAi.java
@@ -385,9 +385,7 @@ public class ChooseGenericEffectAi extends SpellAbilityAi {
final Player opp = player.getWeakestOpponent();
if (opp != null) {
// TODO add predict Combat Damage?
- if (opp.getLife() < copy.getNetPower()) {
- return true;
- }
+ return opp.getLife() < copy.getNetPower();
}
// haste might not be good enough?
diff --git a/forge-ai/src/main/java/forge/ai/ability/ChooseSourceAi.java b/forge-ai/src/main/java/forge/ai/ability/ChooseSourceAi.java
index 3478e74b73b..f7a694e0c67 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ChooseSourceAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ChooseSourceAi.java
@@ -97,10 +97,7 @@ public class ChooseSourceAi extends SpellAbilityAi {
return false;
}
int dmg = AbilityUtils.calculateAmount(threatSource, topStack.getParam("NumDmg"), topStack);
- if (ComputerUtilCombat.predictDamageTo(ai, dmg, threatSource, false) <= 0) {
- return false;
- }
- return true;
+ return ComputerUtilCombat.predictDamageTo(ai, dmg, threatSource, false) > 0;
}
if (game.getPhaseHandler().getPhase() != PhaseType.COMBAT_DECLARE_BLOCKERS) {
return false;
@@ -119,9 +116,7 @@ public class ChooseSourceAi extends SpellAbilityAi {
return ComputerUtilCombat.damageIfUnblocked(c, ai, combat, true) > 0;
}
});
- if (choices.isEmpty()) {
- return false;
- }
+ return !choices.isEmpty();
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/CloneAi.java b/forge-ai/src/main/java/forge/ai/ability/CloneAi.java
index 3f49d12fac9..3d6870d033d 100644
--- a/forge-ai/src/main/java/forge/ai/ability/CloneAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/CloneAi.java
@@ -244,9 +244,6 @@ public class CloneAi extends SpellAbilityAi {
}
// don't activate during main2 unless this effect is permanent
- if (ph.is(PhaseType.MAIN2) && !sa.hasParam("Permanent")) {
- return false;
- }
- return true;
+ return !ph.is(PhaseType.MAIN2) || sa.hasParam("Permanent");
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/ControlGainAi.java b/forge-ai/src/main/java/forge/ai/ability/ControlGainAi.java
index eb6e3ed5d9b..630548b00bd 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ControlGainAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ControlGainAi.java
@@ -84,9 +84,7 @@ public class ControlGainAi extends SpellAbilityAi {
if (sa.hasParam("AllValid")) {
CardCollectionView tgtCards = CardLists.filterControlledBy(game.getCardsIn(ZoneType.Battlefield), opponents);
tgtCards = AbilityUtils.filterListByType(tgtCards, sa.getParam("AllValid"), sa);
- if (tgtCards.isEmpty()) {
- return false;
- }
+ return !tgtCards.isEmpty();
}
return true;
} else {
@@ -247,7 +245,7 @@ public class ControlGainAi extends SpellAbilityAi {
break;
}
}
- };
+ }
if (t != null) {
sa.getTargets().add(t);
@@ -296,15 +294,12 @@ public class ControlGainAi extends SpellAbilityAi {
lose.addAll(Lists.newArrayList(sa.getParam("LoseControl").split(",")));
}
- if (lose.contains("EOT")
- && game.getPhaseHandler().getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)) {
- return false;
- }
+ return !lose.contains("EOT")
+ || !game.getPhaseHandler().getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS);
} else {
return this.canPlayAI(ai, sa);
}
- return true;
} // pumpDrawbackAI()
@Override
diff --git a/forge-ai/src/main/java/forge/ai/ability/CountersMoveAi.java b/forge-ai/src/main/java/forge/ai/ability/CountersMoveAi.java
index 34827a4a273..bf142184a33 100644
--- a/forge-ai/src/main/java/forge/ai/ability/CountersMoveAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/CountersMoveAi.java
@@ -90,9 +90,7 @@ public class CountersMoveAi extends SpellAbilityAi {
}
// for Simic Fluxmage and other
- if (!ph.getNextTurn().equals(ai) || ph.getPhase().isBefore(PhaseType.END_OF_TURN)) {
- return false;
- }
+ return ph.getNextTurn().equals(ai) && !ph.getPhase().isBefore(PhaseType.END_OF_TURN);
} else if (CounterType.P1P1.equals(cType) && sa.hasParam("Defined")) {
// something like Cyptoplast Root-kin
@@ -107,9 +105,7 @@ public class CountersMoveAi extends SpellAbilityAi {
}
// Make sure that removing the last counter doesn't kill the creature
if ("Self".equals(sa.getParam("Source"))) {
- if (host != null && host.getNetToughness() - 1 <= 0) {
- return false;
- }
+ return host == null || host.getNetToughness() - 1 > 0;
}
}
return true;
@@ -193,9 +189,7 @@ public class CountersMoveAi extends SpellAbilityAi {
// check for some specific AI preferences
if ("DontMoveCounterIfLethal".equals(sa.getParam("AILogic"))) {
- if (cType == CounterType.P1P1 && src.getNetToughness() - src.getTempToughnessBoost() - 1 <= 0) {
- return false;
- }
+ return cType != CounterType.P1P1 || src.getNetToughness() - src.getTempToughnessBoost() - 1 > 0;
}
}
// no target
@@ -207,9 +201,7 @@ public class CountersMoveAi extends SpellAbilityAi {
public boolean chkAIDrawback(SpellAbility sa, Player ai) {
if (sa.usesTargeting()) {
sa.resetTargets();
- if (!moveTgtAI(ai, sa)) {
- return false;
- }
+ return moveTgtAI(ai, sa);
}
return true;
@@ -287,10 +279,7 @@ public class CountersMoveAi extends SpellAbilityAi {
// do not steal a P1P1 from Undying if it would die
// this way
if (CounterType.P1P1.equals(cType) && srcCardCpy.getNetToughness() <= 0) {
- if (srcCardCpy.getCounters(cType) > 0 || !card.hasKeyword(Keyword.UNDYING) || card.isToken()) {
- return true;
- }
- return false;
+ return srcCardCpy.getCounters(cType) > 0 || !card.hasKeyword(Keyword.UNDYING) || card.isToken();
}
return true;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/CountersMultiplyAi.java b/forge-ai/src/main/java/forge/ai/ability/CountersMultiplyAi.java
index 53b7c4b2619..b20e99ace3b 100644
--- a/forge-ai/src/main/java/forge/ai/ability/CountersMultiplyAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/CountersMultiplyAi.java
@@ -48,6 +48,7 @@ public class CountersMultiplyAi extends SpellAbilityAi {
if (!c.canReceiveCounters(counterType)) {
return false;
}
+
} else {
for (Map.Entry e : c.getCounters().entrySet()) {
// has negative counter it would double
@@ -96,10 +97,7 @@ public class CountersMultiplyAi extends SpellAbilityAi {
@Override
protected boolean doTriggerAINoCost(Player ai, SpellAbility sa, boolean mandatory) {
- if (sa.usesTargeting() && !setTargets(ai, sa) && !mandatory) {
- return false;
- }
- return true;
+ return !sa.usesTargeting() || setTargets(ai, sa) || mandatory;
}
private CounterType getCounterType(SpellAbility sa) {
diff --git a/forge-ai/src/main/java/forge/ai/ability/CountersProliferateAi.java b/forge-ai/src/main/java/forge/ai/ability/CountersProliferateAi.java
index f108106dce3..52538a0827c 100644
--- a/forge-ai/src/main/java/forge/ai/ability/CountersProliferateAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/CountersProliferateAi.java
@@ -83,12 +83,9 @@ public class CountersProliferateAi extends SpellAbilityAi {
}
}));
}
-
- if (cperms.isEmpty() && hperms.isEmpty() && !opponentPoison && !allyExpOrEnergy) {
- return false;
- }
- return true;
+
+ return !cperms.isEmpty() || !hperms.isEmpty() || opponentPoison || allyExpOrEnergy;
}
@Override
diff --git a/forge-ai/src/main/java/forge/ai/ability/CountersPutAi.java b/forge-ai/src/main/java/forge/ai/ability/CountersPutAi.java
index c636ea7b560..d139920d0a4 100644
--- a/forge-ai/src/main/java/forge/ai/ability/CountersPutAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/CountersPutAi.java
@@ -469,7 +469,7 @@ public class CountersPutAi extends SpellAbilityAi {
int left = amount;
for (Card c : list) {
if (ComputerUtilCard.shouldPumpCard(ai, sa, c, i, i,
- Lists.newArrayList())) {
+ Lists.newArrayList())) {
sa.getTargets().add(c);
abTgt.addDividedAllocation(c, i);
left -= i;
@@ -506,7 +506,7 @@ public class CountersPutAi extends SpellAbilityAi {
if (type.equals("P1P1") && !SpellAbilityAi.isSorcerySpeed(sa)) {
for (Card c : list) {
if (ComputerUtilCard.shouldPumpCard(ai, sa, c, amount, amount,
- Lists.newArrayList())) {
+ Lists.newArrayList())) {
choice = c;
break;
}
@@ -595,7 +595,7 @@ public class CountersPutAi extends SpellAbilityAi {
if (ComputerUtil.waitForBlocking(sa)) {
return false;
}
-
+
return true;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/CountersRemoveAi.java b/forge-ai/src/main/java/forge/ai/ability/CountersRemoveAi.java
index 4a2ab325af7..863e3b53c21 100644
--- a/forge-ai/src/main/java/forge/ai/ability/CountersRemoveAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/CountersRemoveAi.java
@@ -101,7 +101,7 @@ public class CountersRemoveAi extends SpellAbilityAi {
}
// Filter AI-specific targets if provided
- list = ComputerUtil.filterAITgts(sa, ai, (CardCollection)list, false);
+ list = ComputerUtil.filterAITgts(sa, ai, list, false);
boolean noLegendary = game.getStaticEffects().getGlobalRuleChange(GlobalRuleChange.noLegendRule);
diff --git a/forge-ai/src/main/java/forge/ai/ability/DamageAiBase.java b/forge-ai/src/main/java/forge/ai/ability/DamageAiBase.java
index d894b2f7a79..fada6ed260e 100644
--- a/forge-ai/src/main/java/forge/ai/ability/DamageAiBase.java
+++ b/forge-ai/src/main/java/forge/ai/ability/DamageAiBase.java
@@ -37,9 +37,7 @@ public abstract class DamageAiBase extends SpellAbilityAi {
}
if ("SelfDamage".equals(sa.getParam("AILogic"))) {
if (comp.getLife() * 0.75 < enemy.getLife()) {
- if (!lifelink) {
- return true;
- }
+ return !lifelink;
}
}
return false;
diff --git a/forge-ai/src/main/java/forge/ai/ability/DamageDealAi.java b/forge-ai/src/main/java/forge/ai/ability/DamageDealAi.java
index f00dffe3205..f4af5ec56dc 100644
--- a/forge-ai/src/main/java/forge/ai/ability/DamageDealAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/DamageDealAi.java
@@ -80,10 +80,7 @@ public class DamageDealAi extends DamageAiBase {
dmg--; // the card will be spent casting the spell, so actual damage is 1 less
}
}
- if (!this.damageTargetAI(ai, sa, dmg, true)) {
- return false;
- }
- return true;
+ return this.damageTargetAI(ai, sa, dmg, true);
}
@Override
@@ -490,9 +487,7 @@ public class DamageDealAi extends DamageAiBase {
for (final Object o : objects) {
if (o instanceof Card) {
final Card c = (Card) o;
- if (hPlay.contains(c)) {
- hPlay.remove(c);
- }
+ hPlay.remove(c);
}
}
hPlay = CardLists.getTargetableCards(hPlay, sa);
@@ -952,9 +947,7 @@ public class DamageDealAi extends DamageAiBase {
final TargetRestrictions tgt = sa.getTargetRestrictions();
if (tgt == null) {
// If it's not mandatory check a few things
- if (!mandatory && !this.damageChooseNontargeted(ai, sa, dmg)) {
- return false;
- }
+ return mandatory || this.damageChooseNontargeted(ai, sa, dmg);
} else {
if (!this.damageChoosingTargets(ai, sa, tgt, dmg, mandatory, true) && !mandatory) {
return false;
diff --git a/forge-ai/src/main/java/forge/ai/ability/DebuffAi.java b/forge-ai/src/main/java/forge/ai/ability/DebuffAi.java
index ca3fae2f3a8..6f5047d533a 100644
--- a/forge-ai/src/main/java/forge/ai/ability/DebuffAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/DebuffAi.java
@@ -124,7 +124,7 @@ public class DebuffAi extends SpellAbilityAi {
final TargetRestrictions tgt = sa.getTargetRestrictions();
sa.resetTargets();
- CardCollection list = getCurseCreatures(ai, sa, kws == null ? Lists.newArrayList() : kws);
+ CardCollection list = getCurseCreatures(ai, sa, kws == null ? Lists.newArrayList() : kws);
list = CardLists.getValidCards(list, tgt.getValidTgts(), sa.getActivatingPlayer(), sa.getHostCard(), sa);
// several uses here:
diff --git a/forge-ai/src/main/java/forge/ai/ability/DelayedTriggerAi.java b/forge-ai/src/main/java/forge/ai/ability/DelayedTriggerAi.java
index cbe954cd663..c5b17785e61 100644
--- a/forge-ai/src/main/java/forge/ai/ability/DelayedTriggerAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/DelayedTriggerAi.java
@@ -27,7 +27,7 @@ public class DelayedTriggerAi extends SpellAbilityAi {
trigsa.setActivatingPlayer(ai);
if (trigsa instanceof AbilitySub) {
- return SpellApiToAi.Converter.get(((AbilitySub) trigsa).getApi()).chkDrawbackWithSubs(ai, (AbilitySub)trigsa);
+ return SpellApiToAi.Converter.get(trigsa.getApi()).chkDrawbackWithSubs(ai, (AbilitySub)trigsa);
} else {
return AiPlayDecision.WillPlay == ((PlayerControllerAi)ai.getController()).getAi().canPlaySa(trigsa);
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/DestroyAi.java b/forge-ai/src/main/java/forge/ai/ability/DestroyAi.java
index cede308bfc7..69c5bf22fc0 100644
--- a/forge-ai/src/main/java/forge/ai/ability/DestroyAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/DestroyAi.java
@@ -48,7 +48,7 @@ public class DestroyAi extends SpellAbilityAi {
return false;
}
- hasXCost = abCost.getCostMana() != null ? abCost.getCostMana().getAmountOfX() > 0 : false;
+ hasXCost = abCost.getCostMana() != null && abCost.getCostMana().getAmountOfX() > 0;
}
if ("AtOpponentsCombatOrAfter".equals(sa.getParam("AILogic"))) {
@@ -132,7 +132,7 @@ public class DestroyAi extends SpellAbilityAi {
}
// Filter AI-specific targets if provided
- list = ComputerUtil.filterAITgts(sa, ai, (CardCollection)list, true);
+ list = ComputerUtil.filterAITgts(sa, ai, list, true);
list = CardLists.getNotKeyword(list, Keyword.INDESTRUCTIBLE);
if (CardLists.getNotType(list, "Creature").isEmpty()) {
@@ -296,8 +296,8 @@ public class DestroyAi extends SpellAbilityAi {
}
if (list.isEmpty()
- || !CardLists.filterControlledBy(list, ai).isEmpty()
- || CardLists.getNotKeyword(list, Keyword.INDESTRUCTIBLE).isEmpty()) {
+ || !CardLists.filterControlledBy(list, ai).isEmpty()
+ || CardLists.getNotKeyword(list, Keyword.INDESTRUCTIBLE).isEmpty()) {
return false;
}
}
@@ -342,7 +342,7 @@ public class DestroyAi extends SpellAbilityAi {
}
// Filter AI-specific targets if provided
- preferred = ComputerUtil.filterAITgts(sa, ai, (CardCollection)preferred, true);
+ preferred = ComputerUtil.filterAITgts(sa, ai, preferred, true);
for (final Card c : preferred) {
list.remove(c);
@@ -400,16 +400,11 @@ public class DestroyAi extends SpellAbilityAi {
}
}
- if (sa.getTargets().getNumTargeted() < tgt.getMinTargets(sa.getHostCard(), sa)) {
- return false;
- }
+ return sa.getTargets().getNumTargeted() >= tgt.getMinTargets(sa.getHostCard(), sa);
} else {
- if (!mandatory) {
- return false;
- }
+ return mandatory;
}
- return true;
}
public boolean doLandForLandRemovalLogic(SpellAbility sa, Player ai, Card tgtLand, String logic) {
diff --git a/forge-ai/src/main/java/forge/ai/ability/DestroyAllAi.java b/forge-ai/src/main/java/forge/ai/ability/DestroyAllAi.java
index 16d426c7f62..a84aa9d8643 100644
--- a/forge-ai/src/main/java/forge/ai/ability/DestroyAllAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/DestroyAllAi.java
@@ -147,7 +147,7 @@ public class DestroyAllAi extends SpellAbilityAi {
block.assignBlockersForCombat(combat);
if (ComputerUtilCombat.lifeInSeriousDanger(ai, combat)) {
- return true;
+ return true;
}
return false;
} // only lands involved
@@ -171,7 +171,6 @@ public class DestroyAllAi extends SpellAbilityAi {
else if ((ComputerUtilCard.evaluatePermanentList(ailist) + 3) >= ComputerUtilCard.evaluatePermanentList(opplist)) {
return false;
}
-
return true;
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/DrainManaAi.java b/forge-ai/src/main/java/forge/ai/ability/DrainManaAi.java
index f6745d73c39..fbb96a1dc91 100644
--- a/forge-ai/src/main/java/forge/ai/ability/DrainManaAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/DrainManaAi.java
@@ -51,12 +51,9 @@ public class DrainManaAi extends SpellAbilityAi {
} else {
final List defined = AbilityUtils.getDefinedPlayers(source, sa.getParam("Defined"), sa);
- if (!defined.contains(opp)) {
- return false;
- }
+ return defined.contains(opp);
}
- return true;
} else {
sa.resetTargets();
sa.getTargets().add(opp);
diff --git a/forge-ai/src/main/java/forge/ai/ability/DrawAi.java b/forge-ai/src/main/java/forge/ai/ability/DrawAi.java
index 094ebe558a1..212f0bbdab6 100644
--- a/forge-ai/src/main/java/forge/ai/ability/DrawAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/DrawAi.java
@@ -69,7 +69,7 @@ public class DrawAi extends SpellAbilityAi {
}
if (!canLoot(ai, sa)) {
- return false;
+ return false;
}
return true;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/EffectAi.java b/forge-ai/src/main/java/forge/ai/ability/EffectAi.java
index 4b83190cc6f..92e8422c41c 100644
--- a/forge-ai/src/main/java/forge/ai/ability/EffectAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/EffectAi.java
@@ -283,11 +283,7 @@ public class EffectAi extends SpellAbilityAi {
return false;
}
final SpellAbility topStack = game.getStack().peekAbility();
- if (topStack.getActivatingPlayer().isOpponentOf(ai) && topStack.getApi() == ApiType.GainLife) {
- return true;
- } else {
- return false;
- }
+ return topStack.getActivatingPlayer().isOpponentOf(ai) && topStack.getApi() == ApiType.GainLife;
} else if (logic.equals("Fight")) {
return FightAi.canFightAi(ai, sa, 0, 0);
} else if (logic.equals("Burn")) {
@@ -301,11 +297,9 @@ public class EffectAi extends SpellAbilityAi {
return false;
}
if (logic.contains(":")) {
- String k[] = logic.split(":");
+ String[] k = logic.split(":");
Integer i = Integer.valueOf(k[1]);
- if (ai.getCreaturesInPlay().size() < i) {
- return false;
- }
+ return ai.getCreaturesInPlay().size() >= i;
}
return true;
} else if (logic.equals("CastFromGraveThisTurn")) {
diff --git a/forge-ai/src/main/java/forge/ai/ability/FightAi.java b/forge-ai/src/main/java/forge/ai/ability/FightAi.java
index 8486fb0ab46..9a3b0965c42 100644
--- a/forge-ai/src/main/java/forge/ai/ability/FightAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/FightAi.java
@@ -273,10 +273,10 @@ public class FightAi extends SpellAbilityAi {
if (!canKill(opponent, fighter, -pumpDefense)) { // can survive
return true;
} else {
- if (MyRandom.getRandom().nextInt(20)<(opponent.getCMC() - fighter.getCMC())) { // trade
- return true;
- }
- }
+ if (MyRandom.getRandom().nextInt(20)<(opponent.getCMC() - fighter.getCMC())) { // trade
+ return true;
+ }
+ }
}
return false;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/FlipACoinAi.java b/forge-ai/src/main/java/forge/ai/ability/FlipACoinAi.java
index 4792fbf0c05..7d61025c40d 100644
--- a/forge-ai/src/main/java/forge/ai/ability/FlipACoinAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/FlipACoinAi.java
@@ -20,9 +20,9 @@ public class FlipACoinAi extends SpellAbilityAi {
if (AILogic.equals("Never")) {
return false;
} else if (AILogic.equals("PhaseOut")) {
- if (!ComputerUtil.predictThreatenedObjects(sa.getActivatingPlayer(), sa).contains(sa.getHostCard())) {
+ if (!ComputerUtil.predictThreatenedObjects(sa.getActivatingPlayer(), sa).contains(sa.getHostCard())) {
return false;
- }
+ }
} else if (AILogic.equals("KillOrcs")) {
if (ai.getGame().getPhaseHandler().getPhase().isBefore(PhaseType.END_OF_TURN) ) {
return false;
diff --git a/forge-ai/src/main/java/forge/ai/ability/GameWinAi.java b/forge-ai/src/main/java/forge/ai/ability/GameWinAi.java
index f5dd9e62318..9586916f190 100644
--- a/forge-ai/src/main/java/forge/ai/ability/GameWinAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/GameWinAi.java
@@ -11,9 +11,7 @@ public class GameWinAi extends SpellAbilityAi {
*/
@Override
protected boolean canPlayAI(Player ai, SpellAbility sa) {
- if (ai.cantWin()) {
- return false;
- }
+ return !ai.cantWin();
// TODO Check conditions are met on card (e.g. Coalition Victory)
@@ -21,7 +19,6 @@ public class GameWinAi extends SpellAbilityAi {
// In general, don't return true.
// But this card wins the game, I can make an exception for that
- return true;
}
@Override
diff --git a/forge-ai/src/main/java/forge/ai/ability/LifeGainAi.java b/forge-ai/src/main/java/forge/ai/ability/LifeGainAi.java
index 353d84b606e..41751f85882 100644
--- a/forge-ai/src/main/java/forge/ai/ability/LifeGainAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/LifeGainAi.java
@@ -106,13 +106,9 @@ public class LifeGainAi extends SpellAbilityAi {
return false;
}
- if (!lifeCritical && !activateForCost
- && (!ph.getNextTurn().equals(ai) || ph.getPhase().isBefore(PhaseType.END_OF_TURN))
- && !sa.hasParam("PlayerTurn") && !SpellAbilityAi.isSorcerySpeed(sa)) {
- return false;
- }
-
- return true;
+ return lifeCritical || activateForCost
+ || (ph.getNextTurn().equals(ai) && !ph.getPhase().isBefore(PhaseType.END_OF_TURN))
+ || sa.hasParam("PlayerTurn") || SpellAbilityAi.isSorcerySpeed(sa);
}
/*
@@ -304,9 +300,7 @@ public class LifeGainAi extends SpellAbilityAi {
hasTgt = true;
}
}
- if (!hasTgt) {
- return false;
- }
+ return hasTgt;
}
return true;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/LifeLoseAi.java b/forge-ai/src/main/java/forge/ai/ability/LifeLoseAi.java
index 96fc5d4f195..087261271c2 100644
--- a/forge-ai/src/main/java/forge/ai/ability/LifeLoseAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/LifeLoseAi.java
@@ -53,9 +53,7 @@ public class LifeLoseAi extends SpellAbilityAi {
}
if (sa.usesTargeting()) {
- if (!doTgt(ai, sa, false)) {
- return false;
- }
+ return doTgt(ai, sa, false);
}
return true;
@@ -187,12 +185,8 @@ public class LifeLoseAi extends SpellAbilityAi {
? new FCollection(sa.getTargets().getTargetPlayers())
: AbilityUtils.getDefinedPlayers(sa.getHostCard(), sa.getParam("Defined"), sa);
- if (!mandatory && tgtPlayers.contains(ai) && amount > 0 && amount + 3 > ai.getLife()) {
- // For cards like Foul Imp, ETB you lose life
- return false;
- }
-
- return true;
+ // For cards like Foul Imp, ETB you lose life
+ return mandatory || !tgtPlayers.contains(ai) || amount <= 0 || amount + 3 <= ai.getLife();
}
protected boolean doTgt(Player ai, SpellAbility sa, boolean mandatory) {
diff --git a/forge-ai/src/main/java/forge/ai/ability/ManaEffectAi.java b/forge-ai/src/main/java/forge/ai/ability/ManaEffectAi.java
index 868f69fd185..60eaeef180f 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ManaEffectAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ManaEffectAi.java
@@ -80,12 +80,9 @@ public class ManaEffectAi extends SpellAbilityAi {
if (sa.hasParam("AILogic")) {
return true; // handled elsewhere, does not meet the standard requirements
}
-
- if (!(sa.getPayCosts() != null && sa.getPayCosts().hasNoManaCost() && sa.getPayCosts().isReusuableResource()
- && sa.getSubAbility() == null && ComputerUtil.playImmediately(ai, sa))) {
- return false;
- }
- return true;
+
+ return sa.getPayCosts() != null && sa.getPayCosts().hasNoManaCost() && sa.getPayCosts().isReusuableResource()
+ && sa.getSubAbility() == null && ComputerUtil.playImmediately(ai, sa);
// return super.checkApiLogic(ai, sa);
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/ManifestAi.java b/forge-ai/src/main/java/forge/ai/ability/ManifestAi.java
index 7adea756b2e..57e70f7fe58 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ManifestAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ManifestAi.java
@@ -185,10 +185,7 @@ public class ManifestAi extends SpellAbilityAi {
CardCollection filtered = CardLists.filter(options, new Predicate() {
@Override
public boolean apply(Card input) {
- if (shouldManyfest(input, ai, sa)) {
- return false;
- }
- return true;
+ return !shouldManyfest(input, ai, sa);
}
});
if (!filtered.isEmpty()) {
diff --git a/forge-ai/src/main/java/forge/ai/ability/MillAi.java b/forge-ai/src/main/java/forge/ai/ability/MillAi.java
index c975d894c29..3656226ee06 100644
--- a/forge-ai/src/main/java/forge/ai/ability/MillAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/MillAi.java
@@ -31,19 +31,13 @@ public class MillAi extends SpellAbilityAi {
PhaseHandler ph = ai.getGame().getPhaseHandler();
if (aiLogic.equals("Main1")) {
- if (ph.getPhase().isBefore(PhaseType.MAIN2) && !sa.hasParam("ActivationPhases")
- && !ComputerUtil.castSpellInMain1(ai, sa)) {
- return false;
- }
+ return !ph.getPhase().isBefore(PhaseType.MAIN2) || sa.hasParam("ActivationPhases")
+ || ComputerUtil.castSpellInMain1(ai, sa);
} else if (aiLogic.equals("EndOfOppTurn")) {
- if (!(ph.is(PhaseType.END_OF_TURN) && ph.getNextTurn().equals(ai))) {
- return false;
- }
+ return ph.is(PhaseType.END_OF_TURN) && ph.getNextTurn().equals(ai);
} else if (aiLogic.equals("LilianaMill")) {
// Only mill if a "Raise Dead" target is available, in case of control decks with few creatures
- if (CardLists.filter(ai.getCardsIn(ZoneType.Graveyard), CardPredicates.Presets.CREATURES).size() < 1) {
- return false;
- }
+ return CardLists.filter(ai.getCardsIn(ZoneType.Graveyard), CardPredicates.Presets.CREATURES).size() >= 1;
}
return true;
}
@@ -62,11 +56,9 @@ public class MillAi extends SpellAbilityAi {
}
}
if (sa.getHostCard().isCreature() && sa.getPayCosts().hasTapCost()) {
- if (!(ph.is(PhaseType.END_OF_TURN) && ph.getNextTurn().equals(ai))) {
- // creatures with a tap cost to mill (e.g. Doorkeeper) should be activated at the opponent's end step
- // because they are also potentially useful for combat
- return false;
- }
+ // creatures with a tap cost to mill (e.g. Doorkeeper) should be activated at the opponent's end step
+ // because they are also potentially useful for combat
+ return ph.is(PhaseType.END_OF_TURN) && ph.getNextTurn().equals(ai);
}
return true;
}
@@ -100,9 +92,7 @@ public class MillAi extends SpellAbilityAi {
// Set PayX here to maximum value.
final int cardsToDiscard = getNumToDiscard(ai, sa);
source.setSVar("PayX", Integer.toString(cardsToDiscard));
- if (cardsToDiscard <= 0) {
- return false;
- }
+ return cardsToDiscard > 0;
}
return true;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/PeekAndRevealAi.java b/forge-ai/src/main/java/forge/ai/ability/PeekAndRevealAi.java
index 7a7c0bdb732..bd6dedc12ec 100644
--- a/forge-ai/src/main/java/forge/ai/ability/PeekAndRevealAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/PeekAndRevealAi.java
@@ -24,10 +24,10 @@ public class PeekAndRevealAi extends SpellAbilityAi {
return false;
}
if ("Main2".equals(sa.getParam("AILogic"))) {
- if (aiPlayer.getGame().getPhaseHandler().getPhase().isBefore(PhaseType.MAIN2)) {
- return false;
- }
- }
+ if (aiPlayer.getGame().getPhaseHandler().getPhase().isBefore(PhaseType.MAIN2)) {
+ return false;
+ }
+ }
// So far this only appears on Triggers, but will expand
// once things get converted from Dig + NoMove
return true;
diff --git a/forge-ai/src/main/java/forge/ai/ability/PermanentAi.java b/forge-ai/src/main/java/forge/ai/ability/PermanentAi.java
index 14a3f8160c5..62012507864 100644
--- a/forge-ai/src/main/java/forge/ai/ability/PermanentAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/PermanentAi.java
@@ -38,10 +38,7 @@ public class PermanentAi extends SpellAbilityAi {
}
// Wait for Main2 if possible
- if (ph.is(PhaseType.MAIN1) && ph.isPlayerTurn(ai) && !ComputerUtil.castPermanentInMain1(ai, sa) && !sa.hasParam("WithoutManaCost")) {
- return false;
- }
- return true;
+ return !ph.is(PhaseType.MAIN1) || !ph.isPlayerTurn(ai) || ComputerUtil.castPermanentInMain1(ai, sa) || sa.hasParam("WithoutManaCost");
}
/**
@@ -259,9 +256,7 @@ public class PermanentAi extends SpellAbilityAi {
}
}
- if (dontCast) {
- return false;
- }
+ return !dontCast;
}
return true;
diff --git a/forge-ai/src/main/java/forge/ai/ability/PermanentCreatureAi.java b/forge-ai/src/main/java/forge/ai/ability/PermanentCreatureAi.java
index 1fa1a224098..fff79b466d4 100644
--- a/forge-ai/src/main/java/forge/ai/ability/PermanentCreatureAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/PermanentCreatureAi.java
@@ -40,9 +40,7 @@ public class PermanentCreatureAi extends PermanentAi {
ComputerUtilCard.applyStaticContPT(game, copy, null);
- if (copy.getNetToughness() <= 0) {
- return false;
- }
+ return copy.getNetToughness() > 0;
}
return true;
}
@@ -225,13 +223,9 @@ public class PermanentCreatureAi extends PermanentAi {
*/
final Card copy = CardUtil.getLKICopy(sa.getHostCard());
ComputerUtilCard.applyStaticContPT(game, copy, null);
- if (copy.getNetToughness() <= 0 && !copy.hasStartOfKeyword("etbCounter") && mana.countX() == 0
- && !copy.hasETBTrigger(false) && !copy.hasETBReplacement() && !copy.hasSVar("NoZeroToughnessAI")) {
- // AiPlayDecision.WouldBecomeZeroToughnessCreature
- return false;
- }
-
- return true;
+ // AiPlayDecision.WouldBecomeZeroToughnessCreature
+ return copy.getNetToughness() > 0 || copy.hasStartOfKeyword("etbCounter") || mana.countX() != 0
+ || copy.hasETBTrigger(false) || copy.hasETBReplacement() || copy.hasSVar("NoZeroToughnessAI");
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/PermanentNoncreatureAi.java b/forge-ai/src/main/java/forge/ai/ability/PermanentNoncreatureAi.java
index a4569fb1c2e..bd7db42f43f 100644
--- a/forge-ai/src/main/java/forge/ai/ability/PermanentNoncreatureAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/PermanentNoncreatureAi.java
@@ -19,10 +19,7 @@ public class PermanentNoncreatureAi extends PermanentAi {
@Override
protected boolean checkAiLogic(final Player ai, final SpellAbility sa, final String aiLogic) {
- if ("Never".equals(aiLogic) || "DontCast".equals(aiLogic)) {
- return false;
- }
- return true;
+ return !"Never".equals(aiLogic) && !"DontCast".equals(aiLogic);
}
/**
@@ -54,10 +51,8 @@ public class PermanentNoncreatureAi extends PermanentAi {
// TODO: consider replacing the condition with host.hasSVar("OblivionRing")
targets = CardLists.filterControlledBy(targets, ai.getOpponents());
}
- if (targets.isEmpty()) {
- // AiPlayDecision.AnotherTime
- return false;
- }
+ // AiPlayDecision.AnotherTime
+ return !targets.isEmpty();
}
return true;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/PhasesAi.java b/forge-ai/src/main/java/forge/ai/ability/PhasesAi.java
index dffcc09e823..ca9dcd4e57c 100644
--- a/forge-ai/src/main/java/forge/ai/ability/PhasesAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/PhasesAi.java
@@ -33,9 +33,7 @@ public class PhasesAi extends SpellAbilityAi {
if (tgtCards.contains(source)) {
// Protect it from something
final boolean isThreatened = ComputerUtil.predictThreatenedObjects(aiPlayer, null, true).contains(source);
- if (isThreatened) {
- return true;
- }
+ return isThreatened;
} else {
// Card def = tgtCards.get(0);
// Phase this out if it might attack me, or before it can be
diff --git a/forge-ai/src/main/java/forge/ai/ability/PlayAi.java b/forge-ai/src/main/java/forge/ai/ability/PlayAi.java
index f023a718fb1..96c22f1d519 100644
--- a/forge-ai/src/main/java/forge/ai/ability/PlayAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/PlayAi.java
@@ -178,11 +178,7 @@ public class PlayAi extends SpellAbilityAi {
// Before accepting, see if the spell has a valid number of targets (it should at this point).
// Proceeding past this point if the spell is not correctly targeted will result
// in "Failed to add to stack" error and the card disappearing from the game completely.
- if (!spell.isTargetNumberValid()) {
- return false;
- }
-
- return true;
+ return spell.isTargetNumberValid();
}
}
return false;
diff --git a/forge-ai/src/main/java/forge/ai/ability/PoisonAi.java b/forge-ai/src/main/java/forge/ai/ability/PoisonAi.java
index a66326ac4e2..0eba3082f50 100644
--- a/forge-ai/src/main/java/forge/ai/ability/PoisonAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/PoisonAi.java
@@ -24,11 +24,8 @@ public class PoisonAi extends SpellAbilityAi {
*/
@Override
protected boolean checkPhaseRestrictions(final Player ai, final SpellAbility sa, final PhaseHandler ph) {
- if (ph.getPhase().isBefore(PhaseType.MAIN2)
- && !sa.hasParam("ActivationPhases")) {
- return false;
- }
- return true;
+ return !ph.getPhase().isBefore(PhaseType.MAIN2)
+ || sa.hasParam("ActivationPhases");
}
/*
@@ -83,7 +80,6 @@ public class PoisonAi extends SpellAbilityAi {
return false;
}
}
-
return true;
}
@@ -99,7 +95,6 @@ public class PoisonAi extends SpellAbilityAi {
} else if (!input.canReceiveCounters(CounterType.POISON)) {
return false;
}
-
return true;
}
@@ -132,10 +127,7 @@ public class PoisonAi extends SpellAbilityAi {
if (input.cantLose()) {
return true;
}
- if (!input.canReceiveCounters(CounterType.POISON)) {
- return true;
- }
- return false;
+ return !input.canReceiveCounters(CounterType.POISON);
}
});
diff --git a/forge-ai/src/main/java/forge/ai/ability/ProtectAi.java b/forge-ai/src/main/java/forge/ai/ability/ProtectAi.java
index 513b8689a0c..13ed8ff2e51 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ProtectAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ProtectAi.java
@@ -162,11 +162,8 @@ public class ProtectAi extends SpellAbilityAi {
@Override
protected boolean checkPhaseRestrictions(final Player ai, final SpellAbility sa, final PhaseHandler ph) {
final boolean notAiMain1 = !(ph.getPlayerTurn() == ai && ph.getPhase() == PhaseType.MAIN1);
- if (SpellAbilityAi.isSorcerySpeed(sa) && notAiMain1) {
- // sorceries can only give protection in order to create an unblockable attacker
- return false;
- }
- return true;
+ // sorceries can only give protection in order to create an unblockable attacker
+ return !SpellAbilityAi.isSorcerySpeed(sa) || !notAiMain1;
}
@Override
@@ -177,9 +174,7 @@ public class ProtectAi extends SpellAbilityAi {
return false;
} else if (cards.size() == 1) {
// Affecting single card
- if ((getProtectCreatures(ai, sa)).contains(cards.get(0))) {
- return true;
- }
+ return (getProtectCreatures(ai, sa)).contains(cards.get(0));
}
/*
* when this happens we need to expand AI to consider if its ok
diff --git a/forge-ai/src/main/java/forge/ai/ability/PumpAi.java b/forge-ai/src/main/java/forge/ai/ability/PumpAi.java
index 3ab163edddc..d29e5c39c0d 100644
--- a/forge-ai/src/main/java/forge/ai/ability/PumpAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/PumpAi.java
@@ -92,10 +92,7 @@ public class PumpAi extends PumpAiBase {
return true;
}
- if (!ph.getNextTurn().equals(ai) || ph.getPhase().isBefore(PhaseType.END_OF_TURN)) {
- return false;
- }
- return true;
+ return ph.getNextTurn().equals(ai) && !ph.getPhase().isBefore(PhaseType.END_OF_TURN);
} else if (logic.equals("Aristocrat")) {
final boolean isThreatened = ComputerUtil.predictThreatenedObjects(ai, null, true).contains(sa.getHostCard());
if (!ph.is(PhaseType.COMBAT_DECLARE_BLOCKERS) && !isThreatened) {
@@ -121,9 +118,7 @@ public class PumpAi extends PumpAiBase {
|| ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS))) {
// Instant-speed pumps should not be cast outside of combat when the
// stack is empty
- if (!sa.isCurse() && !SpellAbilityAi.isSorcerySpeed(sa) && !main1Preferred) {
- return false;
- }
+ return sa.isCurse() || SpellAbilityAi.isSorcerySpeed(sa) || main1Preferred;
}
return true;
}
@@ -134,7 +129,7 @@ public class PumpAi extends PumpAiBase {
final Card source = sa.getHostCard();
final String sourceName = ComputerUtilAbility.getAbilitySourceName(sa);
final List keywords = sa.hasParam("KW") ? Arrays.asList(sa.getParam("KW").split(" & "))
- : Lists.newArrayList();
+ : Lists.newArrayList();
final String numDefense = sa.hasParam("NumDef") ? sa.getParam("NumDef") : "";
final String numAttack = sa.hasParam("NumAtt") ? sa.getParam("NumAtt") : "";
@@ -191,11 +186,8 @@ public class PumpAi extends PumpAiBase {
srcCardCpy.setCounters(cType, srcCardCpy.getCounters(cType) - amount);
if (CounterType.P1P1.equals(cType) && srcCardCpy.getNetToughness() <= 0) {
- if (srcCardCpy.getCounters(cType) > 0 || !card.hasKeyword(Keyword.UNDYING)
- || card.isToken()) {
- return true;
- }
- return false;
+ return srcCardCpy.getCounters(cType) > 0 || !card.hasKeyword(Keyword.UNDYING)
+ || card.isToken();
}
return false;
}
@@ -244,11 +236,8 @@ public class PumpAi extends PumpAiBase {
srcCardCpy.setCounters(cType, srcCardCpy.getCounters(cType) - amount);
if (CounterType.P1P1.equals(cType) && srcCardCpy.getNetToughness() <= 0) {
- if (srcCardCpy.getCounters(cType) > 0 || !card.hasKeyword(Keyword.UNDYING)
- || card.isToken()) {
- return true;
- }
- return false;
+ return srcCardCpy.getCounters(cType) > 0 || !card.hasKeyword(Keyword.UNDYING)
+ || card.isToken();
}
return true;
}
@@ -395,9 +384,7 @@ public class PumpAi extends PumpAiBase {
Card pumped = ComputerUtilCard.getPumpedCreature(ai, sa, card, 0, 0, keywords);
if (game.getPhaseHandler().is(PhaseType.COMBAT_DECLARE_ATTACKERS, ai)
|| game.getPhaseHandler().is(PhaseType.COMBAT_BEGIN, ai)) {
- if (!ComputerUtilCard.doesSpecifiedCreatureAttackAI(ai, pumped)) {
- return false;
- }
+ return ComputerUtilCard.doesSpecifiedCreatureAttackAI(ai, pumped);
}
return true;
@@ -432,7 +419,7 @@ public class PumpAi extends PumpAiBase {
private boolean pumpTgtAI(final Player ai, final SpellAbility sa, final int defense, final int attack, final boolean mandatory,
boolean immediately) {
final List keywords = sa.hasParam("KW") ? Arrays.asList(sa.getParam("KW").split(" & "))
- : Lists.newArrayList();
+ : Lists.newArrayList();
final Game game = ai.getGame();
final Card source = sa.getHostCard();
final boolean isFight = "Fight".equals(sa.getParam("AILogic")) || "PowerDmg".equals(sa.getParam("AILogic"));
@@ -555,7 +542,7 @@ public class PumpAi extends PumpAiBase {
}
// Filter AI-specific targets if provided
- list = ComputerUtil.filterAITgts(sa, ai, (CardCollection)list, true);
+ list = ComputerUtil.filterAITgts(sa, ai, list, true);
if (list.isEmpty()) {
if (ComputerUtil.activateForCost(sa, ai)) {
@@ -788,15 +775,11 @@ public class PumpAi extends PumpAiBase {
if (!source.hasKeyword(Keyword.INDESTRUCTIBLE) && source.getNetToughness() + defense <= source.getDamage()) {
return false;
}
- if (source.getNetToughness() + defense <= 0) {
- return false;
- }
+ return source.getNetToughness() + defense > 0;
}
} else {
//Targeted
- if (!pumpTgtAI(ai, sa, defense, attack, false, true)) {
- return false;
- }
+ return pumpTgtAI(ai, sa, defense, attack, false, true);
}
return true;
@@ -849,9 +832,7 @@ public class PumpAi extends PumpAiBase {
}
}
);
- if (sacFodder.size() >= numCreatsToSac) {
- return true;
- }
+ return sacFodder.size() >= numCreatsToSac;
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/PumpAiBase.java b/forge-ai/src/main/java/forge/ai/ability/PumpAiBase.java
index 3ccc5b81605..3f178539be2 100644
--- a/forge-ai/src/main/java/forge/ai/ability/PumpAiBase.java
+++ b/forge-ai/src/main/java/forge/ai/ability/PumpAiBase.java
@@ -56,24 +56,17 @@ public abstract class PumpAiBase extends SpellAbilityAi {
if (!CardUtil.isStackingKeyword(keyword) && card.hasKeyword(keyword)) {
return false;
} else if (keyword.equals("Defender") || keyword.endsWith("CARDNAME can't attack.")) {
- if (!ph.isPlayerTurn(card.getController()) || !CombatUtil.canAttack(card, ai)
- || (card.getNetCombatDamage() <= 0)
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)) {
- return false;
- }
+ return ph.isPlayerTurn(card.getController()) && CombatUtil.canAttack(card, ai)
+ && (card.getNetCombatDamage() > 0)
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS);
} else if (keyword.endsWith("CARDNAME can't attack or block.")) {
if (sa.hasParam("UntilYourNextTurn")) {
- if (CombatUtil.canAttack(card, ai) || CombatUtil.canBlock(card, true)) {
- return true;
- }
- return false;
+ return CombatUtil.canAttack(card, ai) || CombatUtil.canBlock(card, true);
}
if (!ph.isPlayerTurn(ai)) {
- if (!CombatUtil.canAttack(card, ai)
- || (card.getNetCombatDamage() <= 0)
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)) {
- return false;
- }
+ return CombatUtil.canAttack(card, ai)
+ && (card.getNetCombatDamage() > 0)
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS);
} else {
if (ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS)
|| ph.getPhase().isBefore(PhaseType.MAIN1)) {
@@ -90,9 +83,7 @@ public abstract class PumpAiBase extends SpellAbilityAi {
return CombatUtil.canAttack(c, card.getController()) || (combat != null && combat.isAttacking(c));
}
});
- if (!CombatUtil.canBlockAtLeastOne(card, attackers)) {
- return false;
- }
+ return CombatUtil.canBlockAtLeastOne(card, attackers);
}
} else if (keyword.endsWith("CARDNAME can't block.")) {
if (!ph.isPlayerTurn(ai) || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS)
@@ -112,24 +103,18 @@ public abstract class PumpAiBase extends SpellAbilityAi {
&& card.getController().equals(combat.getDefenderPlayerByAttacker(c)));
}
});
- if (!CombatUtil.canBlockAtLeastOne(card, attackers)) {
- return false;
- }
+ return CombatUtil.canBlockAtLeastOne(card, attackers);
} else if (keyword.endsWith("CantBlockCardUIDSource")) { // can't block CARDNAME this turn
if (!ph.isPlayerTurn(ai) || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS)
|| ph.getPhase().isBefore(PhaseType.MAIN1) || !CombatUtil.canBlock(sa.getHostCard(), card)) {
return false;
}
// target needs to be a creature, controlled by the player which is attacked
- if (sa.getHostCard().isTapped() && (combat == null || !combat.isAttacking(sa.getHostCard())
- || !card.getController().equals(combat.getDefenderPlayerByAttacker(sa.getHostCard())))) {
- return false;
- }
+ return !sa.getHostCard().isTapped() || (combat != null && combat.isAttacking(sa.getHostCard())
+ && card.getController().equals(combat.getDefenderPlayerByAttacker(sa.getHostCard())));
} else if (keyword.endsWith("This card doesn't untap during your next untap step.")) {
- if (ph.getPhase().isBefore(PhaseType.MAIN2) || card.isUntapped() || !ph.isPlayerTurn(ai)
- || !Untap.canUntap(card)) {
- return false;
- }
+ return !ph.getPhase().isBefore(PhaseType.MAIN2) && !card.isUntapped() && ph.isPlayerTurn(ai)
+ && Untap.canUntap(card);
} else if (keyword.endsWith("Prevent all combat damage that would be dealt by CARDNAME.")
|| keyword.endsWith("Prevent all damage that would be dealt by CARDNAME.")) {
if (ph.isPlayerTurn(ai) && (!(CombatUtil.canBlock(card) || combat != null && combat.isBlocking(card))
@@ -139,28 +124,18 @@ public abstract class PumpAiBase extends SpellAbilityAi {
|| CardLists.getNotKeyword(ai.getCreaturesInPlay(), Keyword.DEFENDER).isEmpty())) {
return false;
}
- if (!ph.isPlayerTurn(ai) && (combat == null || !combat.isAttacking(card) || card.getNetCombatDamage() <= 0)) {
- return false;
- }
+ return ph.isPlayerTurn(ai) || (combat != null && combat.isAttacking(card) && card.getNetCombatDamage() > 0);
} else if (keyword.endsWith("CARDNAME attacks each turn if able.")
|| keyword.endsWith("CARDNAME attacks each combat if able.")) {
- if (ph.isPlayerTurn(ai) || !CombatUtil.canAttack(card, ai) || !CombatUtil.canBeBlocked(card, ai)
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)) {
- return false;
- }
+ return !ph.isPlayerTurn(ai) && CombatUtil.canAttack(card, ai) && CombatUtil.canBeBlocked(card, ai)
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS);
} else if (keyword.endsWith("CARDNAME can't be regenerated.")) {
if (card.getShieldCount() > 0) {
return true;
}
- if (card.hasKeyword("If CARDNAME would be destroyed, regenerate it.") && combat != null
- && (combat.isBlocked(card) || combat.isBlocking(card))) {
- return true;
- }
- return false;
- } else if (keyword.endsWith("CARDNAME's activated abilities can't be activated.")) {
- return false; //too complex
- }
- return true;
+ return card.hasKeyword("If CARDNAME would be destroyed, regenerate it.") && combat != null
+ && (combat.isBlocked(card) || combat.isBlocking(card));
+ } else return !keyword.endsWith("CARDNAME's activated abilities can't be activated."); //too complex
}
/**
@@ -187,12 +162,10 @@ public abstract class PumpAiBase extends SpellAbilityAi {
final boolean evasive = (keyword.endsWith("Unblockable") || keyword.endsWith("Shadow") || keyword.startsWith("CantBeBlockedBy"));
// give evasive keywords to creatures that can or do attack
if (evasive) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || newPower <= 0
- || CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && newPower > 0
+ && !CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty();
} else if (keyword.endsWith("Flying")) {
CardCollectionView attackingFlyer = CardCollection.EMPTY;
if (combat != null) {
@@ -221,13 +194,11 @@ public abstract class PumpAiBase extends SpellAbilityAi {
}
}
}
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || newPower <= 0
- || !Iterables.any(CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)),
- Predicates.not(flyingOrReach))) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && newPower > 0
+ && Iterables.any(CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)),
+ Predicates.not(flyingOrReach));
} else if (keyword.endsWith("Horsemanship")) {
if (ph.isPlayerTurn(opp)
&& ph.getPhase().equals(PhaseType.COMBAT_DECLARE_ATTACKERS)
@@ -236,46 +207,35 @@ public abstract class PumpAiBase extends SpellAbilityAi {
&& ComputerUtilCombat.lifeInDanger(ai, game.getCombat())) {
return true;
}
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || newPower <= 0
- || CardLists.getNotKeyword(CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)),
- Keyword.HORSEMANSHIP).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && newPower > 0
+ && !CardLists.getNotKeyword(CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)),
+ Keyword.HORSEMANSHIP).isEmpty();
} else if (keyword.endsWith("Intimidate")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || newPower <= 0
- || CardLists.getNotType(CardLists.filter(
- opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)), "Artifact").isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && newPower > 0
+ && !CardLists.getNotType(CardLists.filter(
+ opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)), "Artifact").isEmpty();
} else if (keyword.endsWith("Fear")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || newPower <= 0
- || CardLists.getNotColor(CardLists.getNotType(CardLists.filter(
- opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)), "Artifact"), MagicColor.BLACK).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && newPower > 0
+ && !CardLists.getNotColor(CardLists.getNotType(CardLists.filter(
+ opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)), "Artifact"), MagicColor.BLACK).isEmpty();
} else if (keyword.endsWith("Haste")) {
- if (!card.hasSickness() || ph.isPlayerTurn(opp) || card.isTapped()
- || newPower <= 0
- || card.hasKeyword("CARDNAME can attack as though it had haste.")
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || !ComputerUtilCombat.canAttackNextTurn(card)) {
- return false;
- }
+ return card.hasSickness() && !ph.isPlayerTurn(opp) && !card.isTapped()
+ && newPower > 0
+ && !card.hasKeyword("CARDNAME can attack as though it had haste.")
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && ComputerUtilCombat.canAttackNextTurn(card);
} else if (keyword.endsWith("Indestructible")) {
// Predicting threatened objects in relevant non-combat situations happens elsewhere,
// so we are only worrying about combat relevance of Indestructible at this point.
- if (combat == null
- || !((combat.isBlocked(card) || combat.isBlocking(card))
- && ComputerUtilCombat.combatantWouldBeDestroyed(ai, card, combat))) {
- return false;
- }
- return true;
+ return combat != null
+ && ((combat.isBlocked(card) || combat.isBlocking(card))
+ && ComputerUtilCombat.combatantWouldBeDestroyed(ai, card, combat));
} else if (keyword.endsWith("Deathtouch")) {
if (ph.isPlayerTurn(opp) && ph.getPhase().equals(PhaseType.COMBAT_DECLARE_ATTACKERS)) {
List attackers = combat.getAttackers();
@@ -297,12 +257,10 @@ public abstract class PumpAiBase extends SpellAbilityAi {
}
return false;
} else if (keyword.equals("Bushido")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS)
- || opp.getCreaturesInPlay().isEmpty()
- || CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS)
+ && !opp.getCreaturesInPlay().isEmpty()
+ && !CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty();
} else if (keyword.equals("First Strike")) {
if (card.hasKeyword(Keyword.DOUBLE_STRIKE)) {
return false;
@@ -321,40 +279,31 @@ public abstract class PumpAiBase extends SpellAbilityAi {
if (!ComputerUtilCombat.canDestroyAttacker(ai, attacker, card, combat, true)
&& ComputerUtilCombat.canDestroyAttacker(ai, attacker, card, combat, false))
return true;
- if (ComputerUtilCombat.canDestroyBlocker(ai, card, attacker, combat, true)
- && !ComputerUtilCombat.canDestroyBlocker(ai, card, attacker, combat, false))
- return true;
+ return ComputerUtilCombat.canDestroyBlocker(ai, card, attacker, combat, true)
+ && !ComputerUtilCombat.canDestroyBlocker(ai, card, attacker, combat, false);
}
return false;
} else if (keyword.equals("Double Strike")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || newPower <= 0
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS)) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && newPower > 0
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS);
} else if (keyword.startsWith("Rampage")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || newPower <= 0
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).size() < 2) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && newPower > 0
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).size() >= 2;
} else if (keyword.startsWith("Flanking")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || newPower <= 0
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || CardLists.getNotKeyword(CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)),
- Keyword.FLANKING).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && newPower > 0
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && !CardLists.getNotKeyword(CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)),
+ Keyword.FLANKING).isEmpty();
} else if (keyword.startsWith("Trample")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || !CombatUtil.canBeBlocked(card, opp)
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || newPower <= 1
- || CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && CombatUtil.canBeBlocked(card, opp)
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && newPower > 1
+ && !CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty();
} else if (keyword.equals("Infect")) {
if (newPower <= 0) {
return false;
@@ -362,11 +311,9 @@ public abstract class PumpAiBase extends SpellAbilityAi {
if (combat != null && combat.isBlocking(card) && !card.hasKeyword(Keyword.WITHER)) {
return true;
}
- if ((ph.isPlayerTurn(opp))
- || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS)) {
- return false;
- }
+ return (!ph.isPlayerTurn(opp))
+ && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS);
} else if (keyword.endsWith("Wither")) {
if (newPower <= 0 || card.hasKeyword(Keyword.INFECT)) {
return false;
@@ -378,20 +325,16 @@ public abstract class PumpAiBase extends SpellAbilityAi {
}
return combat != null && ( combat.isAttacking(card) || combat.isBlocking(card) );
} else if (keyword.equals("Vigilance")) {
- if (ph.isPlayerTurn(opp) || !CombatUtil.canAttack(card, opp)
- || newPower <= 0
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || CardLists.getNotKeyword(opp.getCreaturesInPlay(), Keyword.DEFENDER).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && CombatUtil.canAttack(card, opp)
+ && newPower > 0
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && !CardLists.getNotKeyword(opp.getCreaturesInPlay(), Keyword.DEFENDER).isEmpty();
} else if (keyword.equals("Reach")) {
- if (ph.isPlayerTurn(ai)
- || !ph.getPhase().equals(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || CardLists.getKeyword(game.getCombat().getAttackers(), Keyword.FLYING).isEmpty()
- || card.hasKeyword(Keyword.FLYING)
- || !CombatUtil.canBlock(card)) {
- return false;
- }
+ return !ph.isPlayerTurn(ai)
+ && ph.getPhase().equals(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && !CardLists.getKeyword(game.getCombat().getAttackers(), Keyword.FLYING).isEmpty()
+ && !card.hasKeyword(Keyword.FLYING)
+ && CombatUtil.canBlock(card);
} else if (keyword.endsWith("CARDNAME can block an additional creature each combat.")) {
if (ph.isPlayerTurn(ai)
|| !ph.getPhase().equals(PhaseType.COMBAT_DECLARE_ATTACKERS)) {
@@ -407,63 +350,43 @@ public abstract class PumpAiBase extends SpellAbilityAi {
}
}
}
- if (possibleBlockNum <= canBlockNum) {
- return false;
- }
+ return possibleBlockNum > canBlockNum;
} else if (keyword.equals("Shroud") || keyword.equals("Hexproof")) {
- if (!ComputerUtil.predictThreatenedObjects(sa.getActivatingPlayer(), sa).contains(card)) {
- return false;
- }
+ return ComputerUtil.predictThreatenedObjects(sa.getActivatingPlayer(), sa).contains(card);
} else if (keyword.equals("Persist")) {
- if (card.getBaseToughness() <= 1 || card.hasKeyword(Keyword.UNDYING)) {
- return false;
- }
+ return card.getBaseToughness() > 1 && !card.hasKeyword(Keyword.UNDYING);
} else if (keyword.equals("Islandwalk")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || newPower <= 0
- || CardLists.getType(opp.getLandsInPlay(), "Island").isEmpty()
- || CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && newPower > 0
+ && !CardLists.getType(opp.getLandsInPlay(), "Island").isEmpty()
+ && !CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty();
} else if (keyword.equals("Swampwalk")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || newPower <= 0
- || CardLists.getType(opp.getLandsInPlay(), "Swamp").isEmpty()
- || CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && newPower > 0
+ && !CardLists.getType(opp.getLandsInPlay(), "Swamp").isEmpty()
+ && !CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty();
} else if (keyword.equals("Mountainwalk")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || newPower <= 0
- || CardLists.getType(opp.getLandsInPlay(), "Mountain").isEmpty()
- || CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && newPower > 0
+ && !CardLists.getType(opp.getLandsInPlay(), "Mountain").isEmpty()
+ && !CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty();
} else if (keyword.equals("Forestwalk")) {
- if (ph.isPlayerTurn(opp) || !(CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
- || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
- || newPower <= 0
- || CardLists.getType(opp.getLandsInPlay(), "Forest").isEmpty()
- || CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty()) {
- return false;
- }
+ return !ph.isPlayerTurn(opp) && (CombatUtil.canAttack(card, opp) || (combat != null && combat.isAttacking(card)))
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
+ && newPower > 0
+ && !CardLists.getType(opp.getLandsInPlay(), "Forest").isEmpty()
+ && !CardLists.filter(opp.getCreaturesInPlay(), CardPredicates.possibleBlockers(card)).isEmpty();
} else if (keyword.endsWith("CARDNAME can attack as though it didn't have defender.")) {
- if (!ph.isPlayerTurn(ai) || !card.hasKeyword(Keyword.DEFENDER)
- || ph.getPhase().isAfter(PhaseType.COMBAT_BEGIN)
- || card.isTapped() || newPower <= 0) {
- return false;
- }
+ return ph.isPlayerTurn(ai) && card.hasKeyword(Keyword.DEFENDER)
+ && !ph.getPhase().isAfter(PhaseType.COMBAT_BEGIN)
+ && !card.isTapped() && newPower > 0;
} else if (keyword.equals("Prevent all combat damage that would be dealt to CARDNAME.")) {
- if (combat == null || !(combat.isBlocking(card) || combat.isBlocked(card))) {
- return false;
- }
+ return combat != null && (combat.isBlocking(card) || combat.isBlocked(card));
} else if (keyword.equals("Menace")) {
- if (combat == null || !combat.isAttacking(card)) {
- return false;
- }
+ return combat != null && combat.isAttacking(card);
}
return true;
}
@@ -547,10 +470,7 @@ public abstract class PumpAiBase extends SpellAbilityAi {
return true;
}
//Don't waste a -7/-0 spell on a 1/1 creature
- if (c.getNetPower() + attack > -2 || c.getNetPower() > 3) {
- return true;
- }
- return false;
+ return c.getNetPower() + attack > -2 || c.getNetPower() > 3;
}
});
} else {
diff --git a/forge-ai/src/main/java/forge/ai/ability/PumpAllAi.java b/forge-ai/src/main/java/forge/ai/ability/PumpAllAi.java
index 7f03f31ddf4..9f1a6c7b159 100644
--- a/forge-ai/src/main/java/forge/ai/ability/PumpAllAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/PumpAllAi.java
@@ -141,10 +141,7 @@ public class PumpAllAi extends PumpAiBase {
// evaluate both lists and pass only if human creatures are more
// valuable
- if ((ComputerUtilCard.evaluateCreatureList(comp) + 200) >= ComputerUtilCard.evaluateCreatureList(human)) {
- return false;
- }
- return true;
+ return (ComputerUtilCard.evaluateCreatureList(comp) + 200) < ComputerUtilCard.evaluateCreatureList(human);
} // end Curse
return !CardLists.getValidCards(getPumpCreatures(ai, sa, defense, power, keywords, false), valid, source.getController(), source).isEmpty();
diff --git a/forge-ai/src/main/java/forge/ai/ability/RemoveFromCombatAi.java b/forge-ai/src/main/java/forge/ai/ability/RemoveFromCombatAi.java
index 6c498f1e764..5d1abd1e177 100644
--- a/forge-ai/src/main/java/forge/ai/ability/RemoveFromCombatAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/RemoveFromCombatAi.java
@@ -18,9 +18,7 @@ public class RemoveFromCombatAi extends SpellAbilityAi {
// AI should only activate this during Human's turn
if ("RemoveBestAttacker".equals(sa.getParam("AILogic"))) {
- if (aiPlayer.getGame().getCombat() != null && aiPlayer.getGame().getCombat().getDefenders().contains(aiPlayer)) {
- return true;
- }
+ return aiPlayer.getGame().getCombat() != null && aiPlayer.getGame().getCombat().getDefenders().contains(aiPlayer);
}
// TODO - implement AI
diff --git a/forge-ai/src/main/java/forge/ai/ability/RepeatAi.java b/forge-ai/src/main/java/forge/ai/ability/RepeatAi.java
index 08059c5cace..7a37733b284 100644
--- a/forge-ai/src/main/java/forge/ai/ability/RepeatAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/RepeatAi.java
@@ -32,9 +32,7 @@ public class RepeatAi extends SpellAbilityAi {
// Set PayX here to maximum value.
final int max = ComputerUtilMana.determineLeftoverMana(sa, ai);
source.setSVar("PayX", Integer.toString(max));
- if (max <= 0) {
- return false;
- }
+ return max > 0;
}
return true;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/RepeatEachAi.java b/forge-ai/src/main/java/forge/ai/ability/RepeatEachAi.java
index 7ef2f78aef4..8ba1494b666 100644
--- a/forge-ai/src/main/java/forge/ai/ability/RepeatEachAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/RepeatEachAi.java
@@ -35,9 +35,7 @@ public class RepeatEachAi extends SpellAbilityAi {
List humTokenCreats = CardLists.filter(aiPlayer.getOpponents().getCreaturesInPlay(), Presets.TOKEN);
List compTokenCreats = CardLists.filter(aiPlayer.getCreaturesInPlay(), Presets.TOKEN);
- if (compTokenCreats.size() <= humTokenCreats.size()) {
- return false;
- }
+ return compTokenCreats.size() > humTokenCreats.size();
} else if ("BalanceLands".equals(logic)) {
if (CardLists.filter(aiPlayer.getCardsIn(ZoneType.Battlefield), Presets.LANDS).size() >= 5) {
return false;
@@ -111,9 +109,7 @@ public class RepeatEachAi extends SpellAbilityAi {
}
}
// would not hit oppoent, don't do that
- if (!hitOpp) {
- return false;
- }
+ return hitOpp;
}
// TODO Add some normal AI variability here
diff --git a/forge-ai/src/main/java/forge/ai/ability/RevealHandAi.java b/forge-ai/src/main/java/forge/ai/ability/RevealHandAi.java
index 841a0cdedca..b2484c99f30 100644
--- a/forge-ai/src/main/java/forge/ai/ability/RevealHandAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/RevealHandAi.java
@@ -30,11 +30,7 @@ public class RevealHandAi extends RevealAiBase {
@Override
protected boolean doTriggerAINoCost(Player ai, SpellAbility sa, boolean mandatory) {
- if (!revealHandTargetAI(ai, sa/*, false, mandatory*/)) {
- return false;
- }
-
- return true;
+ return revealHandTargetAI(ai, sa/*, false, mandatory*/);
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/RollPlanarDiceAi.java b/forge-ai/src/main/java/forge/ai/ability/RollPlanarDiceAi.java
index 4dd60206601..09d3859a5c5 100644
--- a/forge-ai/src/main/java/forge/ai/ability/RollPlanarDiceAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/RollPlanarDiceAi.java
@@ -144,7 +144,7 @@ public class RollPlanarDiceAi extends SpellAbilityAi {
}
}
- return decideToRoll ? true : false;
+ return decideToRoll;
}
/* (non-Javadoc)
diff --git a/forge-ai/src/main/java/forge/ai/ability/SacrificeAi.java b/forge-ai/src/main/java/forge/ai/ability/SacrificeAi.java
index f5f1c587075..94e36b479da 100644
--- a/forge-ai/src/main/java/forge/ai/ability/SacrificeAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/SacrificeAi.java
@@ -136,9 +136,7 @@ public class SacrificeAi extends SpellAbilityAi {
// Since all of the cards have AI:RemoveDeck:All, I enabled 1 for 1
// (or X for X) trades for special decks
- if (humanList.size() < amount) {
- return false;
- }
+ return humanList.size() >= amount;
} else if (defined.equals("You")) {
List computerList =
CardLists.getValidCards(ai.getCardsIn(ZoneType.Battlefield), valid.split(","), sa.getActivatingPlayer(), sa.getHostCard(), sa);
diff --git a/forge-ai/src/main/java/forge/ai/ability/ScryAi.java b/forge-ai/src/main/java/forge/ai/ability/ScryAi.java
index ea10f386b9e..cf01c1d8026 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ScryAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ScryAi.java
@@ -145,10 +145,10 @@ public class ScryAi extends SpellAbilityAi {
if (maxToRemove <= 0) {
return false;
}
- sa.setSVar("ChosenX", "Number$" + Integer.toString(maxToRemove));
+ sa.setSVar("ChosenX", "Number$" + maxToRemove);
} else {
// no Instant or Sorceries anymore, just scry
- sa.setSVar("ChosenX", "Number$" + Integer.toString(Math.min(counterNum, libsize)));
+ sa.setSVar("ChosenX", "Number$" + Math.min(counterNum, libsize));
}
}
return true;
diff --git a/forge-ai/src/main/java/forge/ai/ability/SetStateAi.java b/forge-ai/src/main/java/forge/ai/ability/SetStateAi.java
index dc5c4f5e635..79104d062a6 100644
--- a/forge-ai/src/main/java/forge/ai/ability/SetStateAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/SetStateAi.java
@@ -92,9 +92,7 @@ public class SetStateAi extends SpellAbilityAi {
}
}
- if (sa.getTargets().getNumTargeted() < tgt.getMinTargets(source, sa)) {
- return false;
- }
+ return sa.getTargets().getNumTargeted() >= tgt.getMinTargets(source, sa);
}
} else if ("TurnFace".equals(mode)) {
if (!sa.usesTargeting()) {
@@ -123,9 +121,7 @@ public class SetStateAi extends SpellAbilityAi {
}
}
- if (sa.getTargets().getNumTargeted() < tgt.getMinTargets(source, sa)) {
- return false;
- }
+ return sa.getTargets().getNumTargeted() >= tgt.getMinTargets(source, sa);
}
}
return true;
@@ -254,9 +250,7 @@ public class SetStateAi extends SpellAbilityAi {
// for legendary KI counter creatures
if (othercard.getCounters(CounterType.KI) >= source.getCounters(CounterType.KI)) {
// if the other legendary is useless try to replace it
- if (!ComputerUtilCard.isUselessCreature(aiPlayer, othercard)) {
- return false;
- }
+ return ComputerUtilCard.isUselessCreature(aiPlayer, othercard);
}
}
}
@@ -266,10 +260,6 @@ public class SetStateAi extends SpellAbilityAi {
public boolean confirmAction(Player player, SpellAbility sa, PlayerActionConfirmMode mode, String message) {
// TODO: improve the AI for when it may want to transform something that's optional to transform
- if (!isSafeToTransformIntoLegendary(player, sa.getHostCard())) {
- return false;
- }
-
- return true;
+ return isSafeToTransformIntoLegendary(player, sa.getHostCard());
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/SkipTurnAi.java b/forge-ai/src/main/java/forge/ai/ability/SkipTurnAi.java
index 23f55ec9544..14cae9165fc 100644
--- a/forge-ai/src/main/java/forge/ai/ability/SkipTurnAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/SkipTurnAi.java
@@ -11,10 +11,7 @@ public class SkipTurnAi extends SpellAbilityAi {
*/
@Override
protected boolean canPlayAI(Player aiPlayer, SpellAbility sa) {
- if ("Always".equals(sa.getParam("AILogic"))) {
- return true;
- }
- return false;
+ return "Always".equals(sa.getParam("AILogic"));
}
/* (non-Javadoc)
diff --git a/forge-ai/src/main/java/forge/ai/ability/SurveilAi.java b/forge-ai/src/main/java/forge/ai/ability/SurveilAi.java
index 23069294e94..eb2beab480b 100644
--- a/forge-ai/src/main/java/forge/ai/ability/SurveilAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/SurveilAi.java
@@ -76,9 +76,7 @@ public class SurveilAi extends SpellAbilityAi {
if ("Never".equals(aiLogic)) {
return false;
} else if ("Once".equals(aiLogic)) {
- if (AiCardMemory.isRememberedCard(ai, source, AiCardMemory.MemorySet.ACTIVATED_THIS_TURN)) {
- return false;
- }
+ return !AiCardMemory.isRememberedCard(ai, source, AiCardMemory.MemorySet.ACTIVATED_THIS_TURN);
}
// TODO: add card-specific Surveil AI logic here when/if necessary
diff --git a/forge-ai/src/main/java/forge/ai/ability/TapAi.java b/forge-ai/src/main/java/forge/ai/ability/TapAi.java
index 82b55dce044..a4eb933dbe4 100644
--- a/forge-ai/src/main/java/forge/ai/ability/TapAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/TapAi.java
@@ -64,9 +64,7 @@ public class TapAi extends TapAiBase {
bFlag |= c.isUntapped();
}
- if (!bFlag) {
- return false;
- }
+ return bFlag;
} else {
if ("TapForXCounters".equals(sa.getParam("AILogic"))) {
// e.g. Waxmane Baku
@@ -83,12 +81,9 @@ public class TapAi extends TapAiBase {
}
sa.resetTargets();
- if (!tapPrefTargeting(ai, source, tgt, sa, false)) {
- return false;
- }
+ return tapPrefTargeting(ai, source, tgt, sa, false);
}
- return true;
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/TapAiBase.java b/forge-ai/src/main/java/forge/ai/ability/TapAiBase.java
index 77f83383db6..6a408821cbe 100644
--- a/forge-ai/src/main/java/forge/ai/ability/TapAiBase.java
+++ b/forge-ai/src/main/java/forge/ai/ability/TapAiBase.java
@@ -248,12 +248,8 @@ public abstract class TapAiBase extends SpellAbilityAi {
sa.getTargets().add(choice);
}
- if (sa.getTargets().getNumTargeted() == 0) {
- // Nothing was ever targeted, so we need to bail.
- return false;
- }
-
- return true;
+ // Nothing was ever targeted, so we need to bail.
+ return sa.getTargets().getNumTargeted() != 0;
}
/**
@@ -307,11 +303,7 @@ public abstract class TapAiBase extends SpellAbilityAi {
// just tap whatever we can
tapList = list;
- if (tapTargetList(ai, sa, tapList, mandatory)) {
- return true;
- }
-
- return false;
+ return tapTargetList(ai, sa, tapList, mandatory);
}
@Override
diff --git a/forge-ai/src/main/java/forge/ai/ability/TapAllAi.java b/forge-ai/src/main/java/forge/ai/ability/TapAllAi.java
index baf1842e4e0..8361e437dcd 100644
--- a/forge-ai/src/main/java/forge/ai/ability/TapAllAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/TapAllAi.java
@@ -93,9 +93,7 @@ public class TapAllAi extends SpellAbilityAi {
return CombatUtil.canAttack(c) && ComputerUtilCombat.canAttackNextTurn(c);
}
});
- if(!any) {
- return false;
- }
+ return any;
}
return true;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/TokenAi.java b/forge-ai/src/main/java/forge/ai/ability/TokenAi.java
index cee492ffb3c..6704b41e46b 100644
--- a/forge-ai/src/main/java/forge/ai/ability/TokenAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/TokenAi.java
@@ -103,12 +103,9 @@ public class TokenAi extends SpellAbilityAi {
if (actualToken == null) {
final AbilitySub sub = sa.getSubAbility();
- if (pwPlus || (sub != null && SpellApiToAi.Converter.get(sub.getApi()).chkAIDrawback(sub, ai))) {
- return true; // planeswalker plus ability or sub-ability is
- // useful
- } else {
- return false; // no token created
- }
+ // useful
+ // no token created
+ return pwPlus || (sub != null && SpellApiToAi.Converter.get(sub.getApi()).chkAIDrawback(sub, ai)); // planeswalker plus ability or sub-ability is
}
// X-cost spells
@@ -154,10 +151,7 @@ public class TokenAi extends SpellAbilityAi {
&& !haste && !pwMinus) {
return false;
}
- if ((ph.getPhase().isAfter(PhaseType.COMBAT_BEGIN) || !ph.isPlayerTurn(ai)) && oneShot) {
- return false;
- }
- return true;
+ return (!ph.getPhase().isAfter(PhaseType.COMBAT_BEGIN) && ph.isPlayerTurn(ai)) || !oneShot;
}
@Override
@@ -269,10 +263,8 @@ public class TokenAi extends SpellAbilityAi {
list.add(token);
list = CardLists.getValidCards(list, valid.split(","), ai.getWeakestOpponent(), topStack.getHostCard(), sa);
list = CardLists.filter(list, CardPredicates.canBeSacrificedBy(topStack));
- if (ComputerUtilCard.evaluateCreature(token) < ComputerUtilCard.evaluateCreature(list.get(0))
- && list.contains(token)) {
- return true;
- }
+ return ComputerUtilCard.evaluateCreature(token) < ComputerUtilCard.evaluateCreature(list.get(0))
+ && list.contains(token);
}
return false;
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/UnattachAllAi.java b/forge-ai/src/main/java/forge/ai/ability/UnattachAllAi.java
index 6b7ad4eefbe..4f3292f58d4 100644
--- a/forge-ai/src/main/java/forge/ai/ability/UnattachAllAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/UnattachAllAi.java
@@ -82,9 +82,7 @@ public class UnattachAllAi extends SpellAbilityAi {
//don't equip a worse creature
if (card.isEquipping()) {
Card oldTarget = card.getEquipping();
- if (ComputerUtilCard.evaluateCreature(oldTarget) > ComputerUtilCard.evaluateCreature(newTarget)) {
- return false;
- }
+ return ComputerUtilCard.evaluateCreature(oldTarget) <= ComputerUtilCard.evaluateCreature(newTarget);
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ability/UntapAi.java b/forge-ai/src/main/java/forge/ai/ability/UntapAi.java
index 48884ee524f..5e2659fdacc 100644
--- a/forge-ai/src/main/java/forge/ai/ability/UntapAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/UntapAi.java
@@ -45,11 +45,7 @@ public class UntapAi extends SpellAbilityAi {
return false;
}
- if (!ComputerUtilCost.checkDiscardCost(ai, cost, sa.getHostCard())) {
- return false;
- }
-
- return true;
+ return ComputerUtilCost.checkDiscardCost(ai, cost, sa.getHostCard());
}
@Override
@@ -63,16 +59,11 @@ public class UntapAi extends SpellAbilityAi {
if (tgt == null) {
final List pDefined = AbilityUtils.getDefinedCards(source, sa.getParam("Defined"), sa);
- if (pDefined != null && pDefined.get(0).isUntapped() && pDefined.get(0).getController() == ai) {
- return false;
- }
+ return pDefined == null || !pDefined.get(0).isUntapped() || pDefined.get(0).getController() != ai;
} else {
- if (!untapPrefTargeting(ai, tgt, sa, false)) {
- return false;
- }
+ return untapPrefTargeting(ai, tgt, sa, false);
}
- return true;
}
@Override
@@ -86,11 +77,7 @@ public class UntapAi extends SpellAbilityAi {
// TODO: use Defined to determine, if this is an unfavorable result
final List pDefined = AbilityUtils.getDefinedCards(sa.getHostCard(), sa.getParam("Defined"), sa);
- if (pDefined != null && pDefined.get(0).isUntapped() && pDefined.get(0).getController() == ai) {
- return false;
- }
-
- return true;
+ return pDefined == null || !pDefined.get(0).isUntapped() || pDefined.get(0).getController() != ai;
} else {
if (untapPrefTargeting(ai, tgt, sa, mandatory)) {
return true;
@@ -271,11 +258,7 @@ public class UntapAi extends SpellAbilityAi {
// just tap whatever we can
tapList = list;
- if (untapTargetList(source, tgt, sa, mandatory, tapList)) {
- return true;
- }
-
- return false;
+ return untapTargetList(source, tgt, sa, mandatory, tapList);
}
private boolean untapTargetList(final Card source, final TargetRestrictions tgt, final SpellAbility sa, final boolean mandatory,
@@ -438,13 +421,10 @@ public class UntapAi extends SpellAbilityAi {
// no harm in doing this past declare blockers during the opponent's turn and right before our turn,
// maybe we'll serendipitously untap into something like a removal spell or burn spell that'll help
- if (ph.getNextTurn() == ai
- && (ph.is(PhaseType.COMBAT_DECLARE_BLOCKERS) || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS))) {
- return true;
- }
+ return ph.getNextTurn() == ai
+ && (ph.is(PhaseType.COMBAT_DECLARE_BLOCKERS) || ph.getPhase().isAfter(PhaseType.COMBAT_DECLARE_BLOCKERS));
// haven't found any immediate playable options
- return false;
}
}
diff --git a/forge-ai/src/main/java/forge/ai/simulation/PossibleTargetSelector.java b/forge-ai/src/main/java/forge/ai/simulation/PossibleTargetSelector.java
index fcaf0ad073f..3aa48b72945 100644
--- a/forge-ai/src/main/java/forge/ai/simulation/PossibleTargetSelector.java
+++ b/forge-ai/src/main/java/forge/ai/simulation/PossibleTargetSelector.java
@@ -80,7 +80,7 @@ public class PossibleTargetSelector {
}
private static class SimilarTargetSkipper {
- private ArrayListMultimap validTargetsMap = ArrayListMultimap.create();
+ private ArrayListMultimap validTargetsMap = ArrayListMultimap.create();
private HashMap cardTypeStrings = new HashMap();
private HashMap creatureScores;
diff --git a/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityChoicesIterator.java b/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityChoicesIterator.java
index 75225c73cab..36fbde8d365 100644
--- a/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityChoicesIterator.java
+++ b/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityChoicesIterator.java
@@ -49,7 +49,7 @@ public class SpellAbilityChoicesIterator {
// TODO: Do we need to do something special to support cards that have extra costs
// when choosing more modes, like Blessed Alliance?
if (!allowRepeat) {
- modeIterator = CombinatoricsUtils.combinationsIterator(choices.size(), num);;
+ modeIterator = CombinatoricsUtils.combinationsIterator(choices.size(), num);
} else {
// Note: When allowRepeat is true, it does result in many possibilities being tried.
// We should ideally prune some of those at a higher level.
diff --git a/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityPicker.java b/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityPicker.java
index a14f2b94047..f6c9eec8dfa 100644
--- a/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityPicker.java
+++ b/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityPicker.java
@@ -308,9 +308,7 @@ public class SpellAbilityPicker {
return true;
}
List phases = conditions.getPhases();
- if (phases.isEmpty() || phases.contains(PhaseType.MAIN1)) {
- return true;
- }
+ return phases.isEmpty() || phases.contains(PhaseType.MAIN1);
}
return false;
diff --git a/forge-core/pom.xml b/forge-core/pom.xml
index 6d1913363d1..de96bebf9b6 100644
--- a/forge-core/pom.xml
+++ b/forge-core/pom.xml
@@ -6,7 +6,7 @@
forge
forge
- 1.6.28-SNAPSHOT
+ 1.6.29-SNAPSHOT
forge-core
diff --git a/forge-core/src/main/java/forge/CardStorageReader.java b/forge-core/src/main/java/forge/CardStorageReader.java
index 92b30bce7ac..02e4d414277 100644
--- a/forge-core/src/main/java/forge/CardStorageReader.java
+++ b/forge-core/src/main/java/forge/CardStorageReader.java
@@ -64,7 +64,7 @@ public class CardStorageReader {
void report(int current, int total);
// does nothing, used when they pass null instead of an instance
- public final static ProgressObserver emptyObserver = new ProgressObserver() {
+ ProgressObserver emptyObserver = new ProgressObserver() {
@Override public void setOperationName(final String name, final boolean usePercents) {}
@Override public void report(final int current, final int total) {}
};
diff --git a/forge-core/src/main/java/forge/ImageKeys.java b/forge-core/src/main/java/forge/ImageKeys.java
index bd2c20ed0b5..3e5192ba642 100644
--- a/forge-core/src/main/java/forge/ImageKeys.java
+++ b/forge-core/src/main/java/forge/ImageKeys.java
@@ -123,7 +123,7 @@ public final class ImageKeys {
int index = filename.lastIndexOf('_');
if (index != -1) {
String setlessFilename = filename.substring(0, index);
- String setCode = filename.substring(index + 1, filename.length());
+ String setCode = filename.substring(index + 1);
// try with upper case set
file = findFile(dir, setlessFilename + "_" + setCode.toUpperCase());
if (file != null) { return file; }
diff --git a/forge-core/src/main/java/forge/card/CardDb.java b/forge-core/src/main/java/forge/card/CardDb.java
index 0c4071e4cf4..0b157f26ac2 100644
--- a/forge-core/src/main/java/forge/card/CardDb.java
+++ b/forge-core/src/main/java/forge/card/CardDb.java
@@ -41,7 +41,7 @@ public final class CardDb implements ICardDatabase, IDeckGenPool {
public final static char NameSetSeparator = '|';
// need this to obtain cardReference by name+set+artindex
- private final ListMultimap allCardsByName = Multimaps.newListMultimap(new TreeMap>(String.CASE_INSENSITIVE_ORDER), CollectionSuppliers.arrayLists());
+ private final ListMultimap allCardsByName = Multimaps.newListMultimap(new TreeMap>(String.CASE_INSENSITIVE_ORDER), CollectionSuppliers.arrayLists());
private final Map uniqueCardsByName = Maps.newTreeMap(String.CASE_INSENSITIVE_ORDER);
private final Map rulesByName;
private final Map facesByName = Maps.newTreeMap(String.CASE_INSENSITIVE_ORDER);
@@ -62,7 +62,7 @@ public final class CardDb implements ICardDatabase, IDeckGenPool {
Random(false);
final boolean filterSets;
- private SetPreference(boolean filterIrregularSets) {
+ SetPreference(boolean filterIrregularSets) {
filterSets = filterIrregularSets;
}
diff --git a/forge-core/src/main/java/forge/card/CardEdition.java b/forge-core/src/main/java/forge/card/CardEdition.java
index 86653eb178b..b9b88687d97 100644
--- a/forge-core/src/main/java/forge/card/CardEdition.java
+++ b/forge-core/src/main/java/forge/card/CardEdition.java
@@ -189,7 +189,7 @@ public final class CardEdition implements Comparable { // immutable
public String getBoosterMustContain() { return boosterMustContain; }
public CardInSet[] getCards() { return cards; }
- public Map getTokens() { return tokenNormalized; };
+ public Map getTokens() { return tokenNormalized; }
public static final Function FN_GET_CODE = new Function() {
@Override
diff --git a/forge-core/src/main/java/forge/card/CardFace.java b/forge-core/src/main/java/forge/card/CardFace.java
index 001f2eff18d..3e7afb2858b 100644
--- a/forge-core/src/main/java/forge/card/CardFace.java
+++ b/forge-core/src/main/java/forge/card/CardFace.java
@@ -20,7 +20,7 @@ final class CardFace implements ICardFace {
public enum FaceSelectionMethod { //
USE_ACTIVE_FACE,
USE_PRIMARY_FACE,
- COMBINE;
+ COMBINE
}
@@ -87,7 +87,7 @@ final class CardFace implements ICardFace {
void setInitialLoyalty(String value) { this.initialLoyalty = value; }
void setPtText(String value) {
- final String k[] = value.split("/");
+ final String[] k = value.split("/");
if (k.length != 2) {
throw new RuntimeException("Creature '" + this.getName() + "' has bad p/t stats");
diff --git a/forge-core/src/main/java/forge/card/CardFacePredicates.java b/forge-core/src/main/java/forge/card/CardFacePredicates.java
index ee83dda3787..e2b4d2a5815 100644
--- a/forge-core/src/main/java/forge/card/CardFacePredicates.java
+++ b/forge-core/src/main/java/forge/card/CardFacePredicates.java
@@ -85,7 +85,7 @@ public final class CardFacePredicates {
@Override
public boolean apply(ICardFace input) {
- String k[] = valid.split("\\.", 2);
+ String[] k = valid.split("\\.", 2);
if ("Card".equals(k[0])) {
// okay
@@ -110,10 +110,7 @@ public final class CardFacePredicates {
static protected boolean hasProperty(ICardFace input, final String v) {
if (v.startsWith("non")) {
return !hasProperty(input, v.substring(3));
- } else if (!input.getType().hasStringType(v)) {
- return false;
- }
- return true;
+ } else return input.getType().hasStringType(v);
}
}
diff --git a/forge-core/src/main/java/forge/card/CardRarity.java b/forge-core/src/main/java/forge/card/CardRarity.java
index 7ed13dda93c..b6f1f898067 100644
--- a/forge-core/src/main/java/forge/card/CardRarity.java
+++ b/forge-core/src/main/java/forge/card/CardRarity.java
@@ -35,7 +35,7 @@ public enum CardRarity {
private final String shortName, longName;
- private CardRarity(final String shortName0, final String longName0) {
+ CardRarity(final String shortName0, final String longName0) {
shortName = shortName0;
longName = longName0;
}
diff --git a/forge-core/src/main/java/forge/card/CardRulesPredicates.java b/forge-core/src/main/java/forge/card/CardRulesPredicates.java
index e6b200b1a94..4748c821013 100644
--- a/forge-core/src/main/java/forge/card/CardRulesPredicates.java
+++ b/forge-core/src/main/java/forge/card/CardRulesPredicates.java
@@ -435,10 +435,10 @@ public final class CardRulesPredicates {
return this.op(card.getManaCost().getGenericCost(), this.operand);
case POWER:
value = card.getIntPower();
- return value != Integer.MAX_VALUE ? this.op(value, this.operand) : false;
+ return value != Integer.MAX_VALUE && this.op(value, this.operand);
case TOUGHNESS:
value = card.getIntToughness();
- return value != Integer.MAX_VALUE ? this.op(value, this.operand) : false;
+ return value != Integer.MAX_VALUE && this.op(value, this.operand);
default:
return false;
}
diff --git a/forge-core/src/main/java/forge/card/CardSplitType.java b/forge-core/src/main/java/forge/card/CardSplitType.java
index 38902df9db1..a949ec8b975 100644
--- a/forge-core/src/main/java/forge/card/CardSplitType.java
+++ b/forge-core/src/main/java/forge/card/CardSplitType.java
@@ -10,7 +10,7 @@ public enum CardSplitType
Split(FaceSelectionMethod.COMBINE, CardStateName.RightSplit),
Flip(FaceSelectionMethod.USE_PRIMARY_FACE, CardStateName.Flipped);
- private CardSplitType(FaceSelectionMethod calcMode, CardStateName stateName) {
+ CardSplitType(FaceSelectionMethod calcMode, CardStateName stateName) {
method = calcMode;
this.changedStateName = stateName;
}
diff --git a/forge-core/src/main/java/forge/card/CardType.java b/forge-core/src/main/java/forge/card/CardType.java
index ebc12e29266..21cd142f63a 100644
--- a/forge-core/src/main/java/forge/card/CardType.java
+++ b/forge-core/src/main/java/forge/card/CardType.java
@@ -73,7 +73,7 @@ public final class CardType implements Comparable, CardTypeView {
public final boolean isPermanent;
private static final ImmutableList allCoreTypeNames = EnumUtil.getNames(CoreType.class);
- private CoreType(final boolean permanent) {
+ CoreType(final boolean permanent) {
isPermanent = permanent;
}
}
diff --git a/forge-core/src/main/java/forge/card/ICardFace.java b/forge-core/src/main/java/forge/card/ICardFace.java
index 4953f3ddf0e..d73578d6242 100644
--- a/forge-core/src/main/java/forge/card/ICardFace.java
+++ b/forge-core/src/main/java/forge/card/ICardFace.java
@@ -5,5 +5,5 @@ package forge.card;
*
*/
public interface ICardFace extends ICardCharacteristics, ICardRawAbilites, Comparable {
- public String getAltName();
+ String getAltName();
}
diff --git a/forge-core/src/main/java/forge/card/MagicColor.java b/forge-core/src/main/java/forge/card/MagicColor.java
index bf0b4fdbb9f..c21dedeac97 100644
--- a/forge-core/src/main/java/forge/card/MagicColor.java
+++ b/forge-core/src/main/java/forge/card/MagicColor.java
@@ -168,7 +168,7 @@ public final class MagicColor {
private final String name, symbol;
private final byte colormask;
- private Color(String name0, byte colormask0, String symbol0) {
+ Color(String name0, byte colormask0, String symbol0) {
name = name0;
colormask = colormask0;
symbol = symbol0;
diff --git a/forge-core/src/main/java/forge/card/mana/ManaAtom.java b/forge-core/src/main/java/forge/card/mana/ManaAtom.java
index 5a38b3a3310..951f7fbf776 100644
--- a/forge-core/src/main/java/forge/card/mana/ManaAtom.java
+++ b/forge-core/src/main/java/forge/card/mana/ManaAtom.java
@@ -44,7 +44,7 @@ public abstract class ManaAtom {
if (s.length() == 2) { //if name is two characters, check for combination of two colors
return (byte)(fromName(s.charAt(0)) | fromName(s.charAt(1)));
} else if (s.length() == 1) {
- return (byte) fromName(s.charAt(0));
+ return fromName(s.charAt(0));
}
s = s.toLowerCase();
diff --git a/forge-core/src/main/java/forge/card/mana/ManaCost.java b/forge-core/src/main/java/forge/card/mana/ManaCost.java
index a2393429897..90886018108 100644
--- a/forge-core/src/main/java/forge/card/mana/ManaCost.java
+++ b/forge-core/src/main/java/forge/card/mana/ManaCost.java
@@ -69,7 +69,7 @@ public final class ManaCost implements Comparable, IterablenewArrayList());
+ sealClass(Lists.newArrayList());
}
private ManaCost(int cmc, List shards0) {
diff --git a/forge-core/src/main/java/forge/card/mana/ManaCostShard.java b/forge-core/src/main/java/forge/card/mana/ManaCostShard.java
index 84d317554b7..8c97c99e5b8 100644
--- a/forge-core/src/main/java/forge/card/mana/ManaCostShard.java
+++ b/forge-core/src/main/java/forge/card/mana/ManaCostShard.java
@@ -86,7 +86,7 @@ public enum ManaCostShard {
* @param sValue
* the s value
*/
- private ManaCostShard(final int value, final String sValue) {
+ ManaCostShard(final int value, final String sValue) {
this(value, sValue, sValue);
}
@@ -100,7 +100,7 @@ public enum ManaCostShard {
* @param imgKey
* the img key
*/
- private ManaCostShard(final int value, final String sValue, final String imgKey) {
+ ManaCostShard(final int value, final String sValue, final String imgKey) {
this.shard = value;
this.cmc = this.getCMC();
this.cmpc = this.getCmpCost();
diff --git a/forge-core/src/main/java/forge/deck/DeckFormat.java b/forge-core/src/main/java/forge/deck/DeckFormat.java
index 5d9f651828b..f0515c6494b 100644
--- a/forge-core/src/main/java/forge/deck/DeckFormat.java
+++ b/forge-core/src/main/java/forge/deck/DeckFormat.java
@@ -85,10 +85,7 @@ public enum DeckFormat {
if (otherPart != null && otherPart.getManaCost().getCMC() > 3) {
return false; //only cards with CMC less than 3 are allowed
}
- if (bannedCards.contains(rules.getName())) {
- return false;
- }
- return true;
+ return !bannedCards.contains(rules.getName());
}
}) {
private final Set bannedCommanders = ImmutableSet.of("Derevi, Empyrial Tactician", "Erayo, Soratami Ascendant", "Rofellos, Llanowar Emissary");
@@ -120,7 +117,7 @@ public enum DeckFormat {
private final static String ADVPROCLAMATION = "Advantageous Proclamation";
private final static String SOVREALM = "Sovereign's Realm";
- private DeckFormat(Range mainRange0, Range sideRange0, int maxCardCopies0, Predicate cardPoolFilter0, Predicate paperCardPoolFilter0) {
+ DeckFormat(Range mainRange0, Range sideRange0, int maxCardCopies0, Predicate cardPoolFilter0, Predicate paperCardPoolFilter0) {
mainRange = mainRange0;
sideRange = sideRange0;
maxCardCopies = maxCardCopies0;
@@ -128,7 +125,7 @@ public enum DeckFormat {
paperCardPoolFilter = paperCardPoolFilter0;
}
- private DeckFormat(Range mainRange0, Range sideRange0, int maxCardCopies0, Predicate cardPoolFilter0) {
+ DeckFormat(Range mainRange0, Range sideRange0, int maxCardCopies0, Predicate cardPoolFilter0) {
mainRange = mainRange0;
sideRange = sideRange0;
maxCardCopies = maxCardCopies0;
@@ -136,7 +133,7 @@ public enum DeckFormat {
cardPoolFilter = cardPoolFilter0;
}
- private DeckFormat(Range mainRange0, Range sideRange0, int maxCardCopies0) {
+ DeckFormat(Range mainRange0, Range sideRange0, int maxCardCopies0) {
mainRange = mainRange0;
sideRange = sideRange0;
maxCardCopies = maxCardCopies0;
diff --git a/forge-core/src/main/java/forge/deck/DeckSection.java b/forge-core/src/main/java/forge/deck/DeckSection.java
index 90d2b91d17e..59516041ad0 100644
--- a/forge-core/src/main/java/forge/deck/DeckSection.java
+++ b/forge-core/src/main/java/forge/deck/DeckSection.java
@@ -10,7 +10,7 @@ public enum DeckSection {
Conspiracy(0);
private final int typicalSize; // Rules enforcement is done in DeckFormat class, this is for reference only
- private DeckSection(int commonSize) {
+ DeckSection(int commonSize) {
typicalSize = commonSize;
}
diff --git a/forge-core/src/main/java/forge/deck/generation/DeckGeneratorBase.java b/forge-core/src/main/java/forge/deck/generation/DeckGeneratorBase.java
index d3f0a844019..050f1827e61 100644
--- a/forge-core/src/main/java/forge/deck/generation/DeckGeneratorBase.java
+++ b/forge-core/src/main/java/forge/deck/generation/DeckGeneratorBase.java
@@ -142,7 +142,7 @@ public abstract class DeckGeneratorBase {
if(basicLandEdition == null){
if(setBasicLandPool(cp.getEdition())){
basicLandEdition = cp.getEdition();
- };
+ }
}
cardCounts.put(cp.getName(), newCount);
trace.append(String.format("(%d) %s [%s]%n", cp.getRules().getManaCost().getCMC(), cp.getName(), cp.getRules().getManaCost()));
diff --git a/forge-core/src/main/java/forge/item/PaperCard.java b/forge-core/src/main/java/forge/item/PaperCard.java
index be80957515c..4200b240e4d 100644
--- a/forge-core/src/main/java/forge/item/PaperCard.java
+++ b/forge-core/src/main/java/forge/item/PaperCard.java
@@ -238,13 +238,11 @@ public final class PaperCard implements Comparable, InventoryItemFro
// Return true if card is one of the five basic lands that can be added for free
public boolean isVeryBasicLand() {
- if ((this.getName().equals("Swamp"))
+ return (this.getName().equals("Swamp"))
|| (this.getName().equals("Plains"))
|| (this.getName().equals("Island"))
|| (this.getName().equals("Forest"))
- || (this.getName().equals("Mountain"))) {
- return true;
- } else return false;
+ || (this.getName().equals("Mountain"));
}
}
diff --git a/forge-core/src/main/java/forge/util/Base64Coder.java b/forge-core/src/main/java/forge/util/Base64Coder.java
index c78acff79b9..2a53be0d765 100644
--- a/forge-core/src/main/java/forge/util/Base64Coder.java
+++ b/forge-core/src/main/java/forge/util/Base64Coder.java
@@ -36,6 +36,7 @@ import javax.crypto.SecretKey;
import javax.crypto.SecretKeyFactory;
import javax.crypto.spec.PBEKeySpec;
import javax.crypto.spec.PBEParameterSpec;
+import java.nio.charset.StandardCharsets;
/**
* A Base64 encoder/decoder.
@@ -377,7 +378,7 @@ public final class Base64Coder {
SecretKey key = keyFactory.generateSecret(new PBEKeySpec(PASSWORD));
Cipher pbeCipher = Cipher.getInstance("PBEWithMD5AndDES");
pbeCipher.init(Cipher.ENCRYPT_MODE, key, new PBEParameterSpec(SALT, 20));
- return String.valueOf(encode(pbeCipher.doFinal(value.getBytes("UTF-8"))));
+ return String.valueOf(encode(pbeCipher.doFinal(value.getBytes(StandardCharsets.UTF_8))));
}
public static String decrypt(String value) throws Exception {
@@ -385,7 +386,7 @@ public final class Base64Coder {
SecretKey key = keyFactory.generateSecret(new PBEKeySpec(PASSWORD));
Cipher pbeCipher = Cipher.getInstance("PBEWithMD5AndDES");
pbeCipher.init(Cipher.DECRYPT_MODE, key, new PBEParameterSpec(SALT, 20));
- return new String(pbeCipher.doFinal(decode(value)), "UTF-8");
+ return new String(pbeCipher.doFinal(decode(value)), StandardCharsets.UTF_8);
}
// Dummy constructor.
diff --git a/forge-core/src/main/java/forge/util/ComparableOp.java b/forge-core/src/main/java/forge/util/ComparableOp.java
index ff7fb1030fb..ad295dac98e 100644
--- a/forge-core/src/main/java/forge/util/ComparableOp.java
+++ b/forge-core/src/main/java/forge/util/ComparableOp.java
@@ -33,7 +33,7 @@ public enum ComparableOp {
private final String caption;
- private ComparableOp(String caption0) {
+ ComparableOp(String caption0) {
caption = caption0;
}
diff --git a/forge-core/src/main/java/forge/util/ImageUtil.java b/forge-core/src/main/java/forge/util/ImageUtil.java
index fff1213d844..4b2e515f8fd 100644
--- a/forge-core/src/main/java/forge/util/ImageUtil.java
+++ b/forge-core/src/main/java/forge/util/ImageUtil.java
@@ -10,7 +10,7 @@ import forge.item.PaperCard;
public class ImageUtil {
public static float getNearestHQSize(float baseSize, float actualSize) {
//get nearest power of actualSize to baseSize so that the image renders good
- return (float)Math.round(actualSize) * (float)Math.pow(2, (double)Math.round(Math.log((double)(baseSize / actualSize)) / Math.log(2)));
+ return (float)Math.round(actualSize) * (float)Math.pow(2, (double)Math.round(Math.log(baseSize / actualSize) / Math.log(2)));
}
public static PaperCard getPaperCardFromImageKey(String key) {
@@ -119,7 +119,6 @@ public class ImageUtil {
for (int i = 0; i < in.length(); i++) {
c = in.charAt(i);
if ((c == '"') || (c == '/') || (c == ':') || (c == '?')) {
- out.append("");
} else {
out.append(c);
}
diff --git a/forge-core/src/main/java/forge/util/Lang.java b/forge-core/src/main/java/forge/util/Lang.java
index 216bfee2664..531f7237314 100644
--- a/forge-core/src/main/java/forge/util/Lang.java
+++ b/forge-core/src/main/java/forge/util/Lang.java
@@ -102,7 +102,7 @@ public final class Lang {
strCount = startsWithVowel(noun) ? "an " : "a ";
}
else {
- strCount = String.valueOf(cnt) + " ";
+ strCount = cnt + " ";
}
return strCount + countedForm;
}
diff --git a/forge-core/src/main/java/forge/util/LocalizationChangeObserver.java b/forge-core/src/main/java/forge/util/LocalizationChangeObserver.java
index 88887508309..559a2191424 100644
--- a/forge-core/src/main/java/forge/util/LocalizationChangeObserver.java
+++ b/forge-core/src/main/java/forge/util/LocalizationChangeObserver.java
@@ -1,5 +1,5 @@
package forge.util;
public interface LocalizationChangeObserver {
- public void localizationChanged();
+ void localizationChanged();
}
diff --git a/forge-core/src/main/java/forge/util/Localizer.java b/forge-core/src/main/java/forge/util/Localizer.java
index 6569e1a2e91..73076e6b30c 100644
--- a/forge-core/src/main/java/forge/util/Localizer.java
+++ b/forge-core/src/main/java/forge/util/Localizer.java
@@ -1,10 +1,10 @@
package forge.util;
import java.io.File;
-import java.io.UnsupportedEncodingException;
import java.net.MalformedURLException;
import java.net.URL;
import java.net.URLClassLoader;
+import java.nio.charset.StandardCharsets;
import java.text.MessageFormat;
import java.util.*;
@@ -38,7 +38,7 @@ public class Localizer {
try {
//formatter = new MessageFormat(resourceBundle.getString(key.toLowerCase()), locale);
- formatter = new MessageFormat(resourceBundle.getString(key.toString()), locale);
+ formatter = new MessageFormat(resourceBundle.getString(key), locale);
} catch (final IllegalArgumentException | MissingResourceException e) {
e.printStackTrace();
}
@@ -51,14 +51,10 @@ public class Localizer {
formatter.setLocale(locale);
String formattedMessage = "CHAR ENCODING ERROR";
- try {
- //Support non-English-standard characters
- formattedMessage = new String(formatter.format(messageArguments).getBytes("ISO-8859-1"), "UTF-8");
- } catch (final UnsupportedEncodingException e) {
- e.printStackTrace();
- }
-
- return formattedMessage;
+ //Support non-English-standard characters
+ formattedMessage = new String(formatter.format(messageArguments).getBytes(StandardCharsets.ISO_8859_1), StandardCharsets.UTF_8);
+
+ return formattedMessage;
}
diff --git a/forge-core/src/main/java/forge/util/storage/StorageBase.java b/forge-core/src/main/java/forge/util/storage/StorageBase.java
index 5a970878e8d..2c0c959fe20 100644
--- a/forge-core/src/main/java/forge/util/storage/StorageBase.java
+++ b/forge-core/src/main/java/forge/util/storage/StorageBase.java
@@ -65,7 +65,7 @@ public class StorageBase implements IStorage {
@Override
public boolean contains(String name) {
- return name == null ? false : map.containsKey(name);
+ return name != null && map.containsKey(name);
}
@Override
diff --git a/forge-game/pom.xml b/forge-game/pom.xml
index 898e378a0a9..ee375af6f5b 100644
--- a/forge-game/pom.xml
+++ b/forge-game/pom.xml
@@ -6,7 +6,7 @@
forge
forge
- 1.6.28-SNAPSHOT
+ 1.6.29-SNAPSHOT
forge-game
diff --git a/forge-game/src/main/java/forge/GameCommand.java b/forge-game/src/main/java/forge/GameCommand.java
index f0d96122ffe..e67dc9560a3 100644
--- a/forge-game/src/main/java/forge/GameCommand.java
+++ b/forge-game/src/main/java/forge/GameCommand.java
@@ -27,7 +27,7 @@ package forge;
*/
public interface GameCommand extends java.io.Serializable, Runnable {
/** Constant Blank. */
- public final GameCommand BLANK = new GameCommand() {
+ GameCommand BLANK = new GameCommand() {
private static final long serialVersionUID = 2689172297036001710L;
diff --git a/forge-game/src/main/java/forge/game/CardTraitBase.java b/forge-game/src/main/java/forge/game/CardTraitBase.java
index aacc1d389e1..7897017b187 100644
--- a/forge-game/src/main/java/forge/game/CardTraitBase.java
+++ b/forge-game/src/main/java/forge/game/CardTraitBase.java
@@ -159,9 +159,7 @@ public abstract class CardTraitBase extends GameObject implements IHasCardView {
*/
public final boolean isSecondary() {
if (this.mapParams.containsKey("Secondary")) {
- if (this.mapParams.get("Secondary").equals("True")) {
- return true;
- }
+ return this.mapParams.get("Secondary").equals("True");
}
return false;
}
@@ -451,9 +449,7 @@ public abstract class CardTraitBase extends GameObject implements IHasCardView {
if (params.containsKey("ActivateNoLoyaltyAbilitiesCondition")) {
final Player active = game.getPhaseHandler().getPlayerTurn();
- if (active.getActivateLoyaltyAbilityThisTurn()) {
- return false;
- }
+ return !active.getActivateLoyaltyAbilityThisTurn();
}
return true;
}
diff --git a/forge-game/src/main/java/forge/game/ForgeScript.java b/forge-game/src/main/java/forge/game/ForgeScript.java
index c5db86d46ad..1bf735a42d4 100644
--- a/forge-game/src/main/java/forge/game/ForgeScript.java
+++ b/forge-game/src/main/java/forge/game/ForgeScript.java
@@ -29,8 +29,7 @@ public class ForgeScript {
int desiredColor = MagicColor.fromName(colorName);
boolean hasColor = colors.hasAnyColor(desiredColor);
- if (mustHave != hasColor)
- return false;
+ return mustHave == hasColor;
} else if (property.contains("Colorless")) { // ... Card is colorless
boolean non = property.startsWith("non");
@@ -38,66 +37,48 @@ public class ForgeScript {
if (non && withSource && isColorlessSource) {
return false;
}
- if (non == colors.isColorless()) return false;
+ return non != colors.isColorless();
} else if (property.contains("MultiColor")) {
// ... Card is multicolored
if (property.endsWith("Source") && isColorlessSource)
return false;
- if (property.startsWith("non") == colors.isMulticolor())
- return false;
+ return property.startsWith("non") != colors.isMulticolor();
} else if (property.contains("MonoColor")) { // ... Card is monocolored
if (property.endsWith("Source") && isColorlessSource)
return false;
- if (property.startsWith("non") == colors.isMonoColor())
- return false;
+ return property.startsWith("non") != colors.isMonoColor();
} else if (property.startsWith("ChosenColor")) {
if (property.endsWith("Source") && isColorlessSource)
return false;
- if (!source.hasChosenColor() || !colors.hasAnyColor(MagicColor.fromName(source.getChosenColor())))
- return false;
+ return source.hasChosenColor() && colors.hasAnyColor(MagicColor.fromName(source.getChosenColor()));
} else if (property.startsWith("AnyChosenColor")) {
if (property.endsWith("Source") && isColorlessSource)
return false;
- if (!source.hasChosenColor()
- || !colors.hasAnyColor(ColorSet.fromNames(source.getChosenColors()).getColor()))
- return false;
+ return source.hasChosenColor()
+ && colors.hasAnyColor(ColorSet.fromNames(source.getChosenColors()).getColor());
} else if (property.startsWith("non")) {
// ... Other Card types
- if (cardState.getTypeWithChanges().hasStringType(property.substring(3))) {
- return false;
- }
+ return !cardState.getTypeWithChanges().hasStringType(property.substring(3));
} else if (property.equals("CostsPhyrexianMana")) {
- if (!cardState.getManaCost().hasPhyrexian()) {
- return false;
- }
+ return cardState.getManaCost().hasPhyrexian();
} else if (property.startsWith("HasSVar")) {
final String svar = property.substring(8);
- if (!cardState.hasSVar(svar)) {
- return false;
- }
+ return cardState.hasSVar(svar);
} else if (property.equals("ChosenType")) {
- if (!cardState.getTypeWithChanges().hasStringType(source.getChosenType())) {
- return false;
- }
+ return cardState.getTypeWithChanges().hasStringType(source.getChosenType());
} else if (property.equals("IsNotChosenType")) {
- if (cardState.getTypeWithChanges().hasStringType(source.getChosenType())) {
- return false;
- }
+ return !cardState.getTypeWithChanges().hasStringType(source.getChosenType());
} else if (property.startsWith("HasSubtype")) {
final String subType = property.substring(11);
- if (!cardState.getTypeWithChanges().hasSubtype(subType)) {
- return false;
- }
+ return cardState.getTypeWithChanges().hasSubtype(subType);
} else if (property.startsWith("HasNoSubtype")) {
final String subType = property.substring(13);
- if (cardState.getTypeWithChanges().hasSubtype(subType)) {
- return false;
- }
+ return !cardState.getTypeWithChanges().hasSubtype(subType);
} else if (property.equals("hasActivatedAbilityWithTapCost")) {
for (final SpellAbility sa : cardState.getSpellAbilities()) {
if (sa.isAbility() && (sa.getPayCosts() != null) && sa.getPayCosts().hasTapCost()) {
@@ -136,80 +117,47 @@ public class ForgeScript {
x = AbilityUtils.calculateAmount(source, rhs, spellAbility);
}
- if (!Expressions.compare(y, property, x)) {
- return false;
- }
- } else if (!cardState.getTypeWithChanges().hasStringType(property)) {
- return false;
- }
-
- return true;
-
+ return Expressions.compare(y, property, x);
+ } else return cardState.getTypeWithChanges().hasStringType(property);
+
-
}
public static boolean spellAbilityHasProperty(SpellAbility sa, String property, Player sourceController,
Card source, SpellAbility spellAbility) {
if (property.equals("ManaAbility")) {
- if (!sa.isManaAbility()) {
- return false;
- }
+ return sa.isManaAbility();
} else if (property.equals("nonManaAbility")) {
- if (sa.isManaAbility()) {
- return false;
- }
+ return !sa.isManaAbility();
} else if (property.equals("Buyback")) {
- if (!sa.isBuyBackAbility()) {
- return false;
- }
+ return sa.isBuyBackAbility();
} else if (property.equals("Cycling")) {
- if (!sa.isCycling()) {
- return false;
- }
+ return sa.isCycling();
} else if (property.equals("Dash")) {
- if (!sa.isDash()) {
- return false;
- }
+ return sa.isDash();
} else if (property.equals("Flashback")) {
- if (!sa.isFlashBackAbility()) {
- return false;
- }
+ return sa.isFlashBackAbility();
} else if (property.equals("Jumpstart")) {
- if (!sa.isJumpstart()) {
- return false;
- }
+ return sa.isJumpstart();
} else if (property.equals("Kicked")) {
- if (!sa.isKicked()) {
- return false;
- }
+ return sa.isKicked();
} else if (property.equals("Loyalty")) {
- if (!sa.isPwAbility()) {
- return false;
- }
+ return sa.isPwAbility();
} else if (property.equals("Aftermath")) {
- if (!sa.isAftermath()) {
- return false;
- }
+ return sa.isAftermath();
} else if (property.equals("MorphUp")) {
- if (!sa.isMorphUp()) {
- return false;
- }
+ return sa.isMorphUp();
} else if (property.equals("Equip")) {
- if (!sa.hasParam("Equip")) {
- return false;
- }
+ return sa.hasParam("Equip");
} else if (property.equals("MayPlaySource")) {
StaticAbility m = sa.getMayPlay();
if (m == null) {
return false;
}
- if (!source.equals(m.getHostCard())) {
- return false;
- }
+ return source.equals(m.getHostCard());
} else if (property.startsWith("IsTargeting")) {
- String k[] = property.split(" ", 2);
+ String[] k = property.split(" ", 2);
boolean found = false;
for (GameObject o : AbilityUtils.getDefinedObjects(source, k[1], spellAbility)) {
if (sa.isTargeting(o)) {
@@ -217,15 +165,11 @@ public class ForgeScript {
break;
}
}
- if (!found) {
- return false;
- }
+ return found;
} else if (property.equals("YouCtrl")) {
return sa.getActivatingPlayer().equals(sourceController);
} else if (sa.getHostCard() != null) {
- if (!sa.getHostCard().hasProperty(property, sourceController, source, spellAbility)) {
- return false;
- }
+ return sa.getHostCard().hasProperty(property, sourceController, source, spellAbility);
}
return true;
diff --git a/forge-game/src/main/java/forge/game/Game.java b/forge-game/src/main/java/forge/game/Game.java
index 4e58a1aba55..beef5eade46 100644
--- a/forge-game/src/main/java/forge/game/Game.java
+++ b/forge-game/src/main/java/forge/game/Game.java
@@ -845,7 +845,7 @@ public class Game {
}
}
- library.removeAll((Collection>)toRemove);
+ library.removeAll(toRemove);
if (library.size() > 0) { //Make sure that matches were found. If not, use the original method to choose antes
Card ante = library.get(MyRandom.getRandom().nextInt(library.size()));
diff --git a/forge-game/src/main/java/forge/game/GameAction.java b/forge-game/src/main/java/forge/game/GameAction.java
index 2b95afafab1..50d3523676d 100644
--- a/forge-game/src/main/java/forge/game/GameAction.java
+++ b/forge-game/src/main/java/forge/game/GameAction.java
@@ -763,7 +763,7 @@ public class GameAction {
checkStaticAbilities(true);
}
public final void checkStaticAbilities(final boolean runEvents) {
- checkStaticAbilities(runEvents, Sets.newHashSet(), CardCollection.EMPTY);
+ checkStaticAbilities(runEvents, Sets.newHashSet(), CardCollection.EMPTY);
}
public final void checkStaticAbilities(final boolean runEvents, final Set affectedCards, final CardCollectionView preList) {
if (isCheckingStaticAbilitiesOnHold()) {
@@ -929,7 +929,7 @@ public class GameAction {
}
public final void checkStateEffects(final boolean runEvents) {
- checkStateEffects(runEvents, Sets.newHashSet());
+ checkStateEffects(runEvents, Sets.newHashSet());
}
public final void checkStateEffects(final boolean runEvents, final Set affectedCards) {
// sol(10/29) added for Phase updates, state effects shouldn't be
diff --git a/forge-game/src/main/java/forge/game/GameActionUtil.java b/forge-game/src/main/java/forge/game/GameActionUtil.java
index b8fa75f9f13..e425d2fff6e 100644
--- a/forge-game/src/main/java/forge/game/GameActionUtil.java
+++ b/forge-game/src/main/java/forge/game/GameActionUtil.java
@@ -234,7 +234,7 @@ public final class GameActionUtil {
// there is a flashback cost (and not the cards cost)
if (keyword.contains(":")) {
- final String k[] = keyword.split(":");
+ final String[] k = keyword.split(":");
flashback.setPayCosts(new Cost(k[1], false));
}
alternatives.add(flashback);
diff --git a/forge-game/src/main/java/forge/game/GameEntity.java b/forge-game/src/main/java/forge/game/GameEntity.java
index 1e7ecd2d3f9..58ab4d02127 100644
--- a/forge-game/src/main/java/forge/game/GameEntity.java
+++ b/forge-game/src/main/java/forge/game/GameEntity.java
@@ -407,11 +407,7 @@ public abstract class GameEntity extends GameObject implements IIdentifiable {
}
// true for all
- if (hasProtectionFrom(attach, checkSBA)) {
- return false;
- }
-
- return true;
+ return !hasProtectionFrom(attach, checkSBA);
}
protected boolean canBeEquippedBy(final Card aura) {
diff --git a/forge-game/src/main/java/forge/game/GameFormat.java b/forge-game/src/main/java/forge/game/GameFormat.java
index 2790a759a64..bfa3f08efae 100644
--- a/forge-game/src/main/java/forge/game/GameFormat.java
+++ b/forge-game/src/main/java/forge/game/GameFormat.java
@@ -368,7 +368,7 @@ public class GameFormat implements Comparable {
if ( strCars != null ) {
CardRarity cr;
rarities = Lists.newArrayList();
- for (String s: Arrays.asList(strCars.split(", "))) {
+ for (String s: strCars.split(", ")) {
cr = CardRarity.smartValueOf(s);
if (!cr.name().equals("Unknown")) {
rarities.add(cr);
diff --git a/forge-game/src/main/java/forge/game/GameLogEntryType.java b/forge-game/src/main/java/forge/game/GameLogEntryType.java
index 76538fef910..8654fba1a55 100644
--- a/forge-game/src/main/java/forge/game/GameLogEntryType.java
+++ b/forge-game/src/main/java/forge/game/GameLogEntryType.java
@@ -20,7 +20,7 @@ public enum GameLogEntryType {
PHASE("Phase");
private final String caption;
- private GameLogEntryType(String name) {
+ GameLogEntryType(String name) {
this.caption = name;
}
diff --git a/forge-game/src/main/java/forge/game/GameLogFormatter.java b/forge-game/src/main/java/forge/game/GameLogFormatter.java
index 6ad9bd1cf8f..bed7d9a2755 100644
--- a/forge-game/src/main/java/forge/game/GameLogFormatter.java
+++ b/forge-game/src/main/java/forge/game/GameLogFormatter.java
@@ -182,7 +182,7 @@ public class GameLogFormatter extends IGameEventVisitor.Base {
if (event.type == DamageType.LoyaltyLoss) {
additionalLog = " (Removing " + Lang.nounWithAmount(event.amount, "loyalty counter") + ")";
}
- String message = event.source.toString() + " deals " + String.valueOf(event.amount) + " damage" + additionalLog + " to " + event.card.toString() + ".";
+ String message = event.source.toString() + " deals " + event.amount + " damage" + additionalLog + " to " + event.card.toString() + ".";
return new GameLogEntry(GameLogEntryType.DAMAGE, message);
}
@@ -197,7 +197,7 @@ public class GameLogFormatter extends IGameEventVisitor.Base {
@Override
public GameLogEntry visit(GameEventTurnBegan event) {
- String message = "Turn " + String.valueOf(event.turnNumber) + " (" + event.turnOwner.toString() + ")";
+ String message = "Turn " + event.turnNumber + " (" + event.turnOwner.toString() + ")";
return new GameLogEntry(GameLogEntryType.TURN, message);
}
@@ -205,7 +205,7 @@ public class GameLogFormatter extends IGameEventVisitor.Base {
public GameLogEntry visit(GameEventPlayerDamaged ev) {
String extra = ev.infect ? " (as poison counters)" : "";
String damageType = ev.combat ? "combat" : "non-combat";
- String message = ev.source.toString() + " deals " + String.valueOf(ev.amount) + " " + damageType + " damage to " + ev.target.toString() + extra + ".";
+ String message = ev.source.toString() + " deals " + ev.amount + " " + damageType + " damage to " + ev.target.toString() + extra + ".";
return new GameLogEntry(GameLogEntryType.DAMAGE, message);
}
@@ -281,7 +281,7 @@ public class GameLogFormatter extends IGameEventVisitor.Base {
@Override
public GameLogEntry visit(GameEventMulligan ev) {
- String message = ev.player.toString() + " has mulliganed down to " + String.valueOf(ev.player.getZone(ZoneType.Hand).size()) + " cards.";
+ String message = ev.player.toString() + " has mulliganed down to " + ev.player.getZone(ZoneType.Hand).size() + " cards.";
return new GameLogEntry(GameLogEntryType.MULLIGAN, message);
}
diff --git a/forge-game/src/main/java/forge/game/GameStage.java b/forge-game/src/main/java/forge/game/GameStage.java
index 9d32e7ac39f..251fc2e25ee 100644
--- a/forge-game/src/main/java/forge/game/GameStage.java
+++ b/forge-game/src/main/java/forge/game/GameStage.java
@@ -5,5 +5,5 @@ public enum GameStage {
Mulligan,
Play,
RestartedByKarn,
- GameOver;
+ GameOver
}
\ No newline at end of file
diff --git a/forge-game/src/main/java/forge/game/GameType.java b/forge-game/src/main/java/forge/game/GameType.java
index dc63a482d3f..7ef81786c26 100644
--- a/forge-game/src/main/java/forge/game/GameType.java
+++ b/forge-game/src/main/java/forge/game/GameType.java
@@ -71,11 +71,11 @@ public enum GameType {
private final String name, description;
private final Function deckAutoGenerator;
- private GameType(DeckFormat deckFormat0, boolean isCardPoolLimited0, boolean canSideboard0, boolean addWonCardsMidgame0, String name0, String description0) {
+ GameType(DeckFormat deckFormat0, boolean isCardPoolLimited0, boolean canSideboard0, boolean addWonCardsMidgame0, String name0, String description0) {
this(deckFormat0, isCardPoolLimited0, canSideboard0, addWonCardsMidgame0, name0, description0, null);
}
- private GameType(DeckFormat deckFormat0, boolean isCardPoolLimited0, boolean canSideboard0, boolean addWonCardsMidgame0, String name0, String description0, Function deckAutoGenerator0) {
+ GameType(DeckFormat deckFormat0, boolean isCardPoolLimited0, boolean canSideboard0, boolean addWonCardsMidgame0, String name0, String description0, Function deckAutoGenerator0) {
final Localizer localizer = forge.util.Localizer.getInstance();
deckFormat = deckFormat0;
isCardPoolLimited = isCardPoolLimited0;
diff --git a/forge-game/src/main/java/forge/game/GlobalRuleChange.java b/forge-game/src/main/java/forge/game/GlobalRuleChange.java
index aa2e4f1b326..2e7fc2ca2ab 100644
--- a/forge-game/src/main/java/forge/game/GlobalRuleChange.java
+++ b/forge-game/src/main/java/forge/game/GlobalRuleChange.java
@@ -40,7 +40,7 @@ public enum GlobalRuleChange {
private final String ruleText;
- private GlobalRuleChange(String text) {
+ GlobalRuleChange(String text) {
ruleText = text;
}
diff --git a/forge-game/src/main/java/forge/game/IIdentifiable.java b/forge-game/src/main/java/forge/game/IIdentifiable.java
index 8c3bf20272a..6b90c6826e8 100644
--- a/forge-game/src/main/java/forge/game/IIdentifiable.java
+++ b/forge-game/src/main/java/forge/game/IIdentifiable.java
@@ -3,8 +3,8 @@ package forge.game;
import com.google.common.base.Function;
public interface IIdentifiable {
- public abstract int getId();
- public static final Function FN_GET_ID = new Function() {
+ int getId();
+ Function FN_GET_ID = new Function() {
@Override
public Integer apply(final IIdentifiable input) {
return Integer.valueOf(input.getId());
diff --git a/forge-game/src/main/java/forge/game/ability/AbilityFactory.java b/forge-game/src/main/java/forge/game/ability/AbilityFactory.java
index 6fd213ea43c..1e770f1fce9 100644
--- a/forge-game/src/main/java/forge/game/ability/AbilityFactory.java
+++ b/forge-game/src/main/java/forge/game/ability/AbilityFactory.java
@@ -64,7 +64,7 @@ public final class AbilityFactory {
SubAbility("DB");
private final String prefix;
- private AbilityRecordType(String prefix) {
+ AbilityRecordType(String prefix) {
this.prefix = prefix;
}
public String getPrefix() {
diff --git a/forge-game/src/main/java/forge/game/ability/AbilityUtils.java b/forge-game/src/main/java/forge/game/ability/AbilityUtils.java
index 747945ae97b..399515f6755 100644
--- a/forge-game/src/main/java/forge/game/ability/AbilityUtils.java
+++ b/forge-game/src/main/java/forge/game/ability/AbilityUtils.java
@@ -35,7 +35,6 @@ import io.sentry.event.BreadcrumbBuilder;
import org.apache.commons.lang3.StringUtils;
import java.util.ArrayList;
-import java.util.Collection;
import java.util.List;
import java.util.Map;
import java.util.Map.Entry;
@@ -1063,7 +1062,7 @@ public class AbilityUtils {
}
else if (defined.startsWith("OppNon")) {
players.addAll(player.getOpponents());
- players.removeAll((Collection>)getDefinedPlayers(card, defined.substring(6), sa));
+ players.removeAll(getDefinedPlayers(card, defined.substring(6), sa));
}
else if (defined.startsWith("Replaced")) {
final SpellAbility root = sa.getRootAbility();
@@ -1800,7 +1799,7 @@ public class AbilityUtils {
public static final String getSVar(final CardTraitBase ability, final String sVarName) {
String val = null;
if (ability instanceof SpellAbility) {
- val = ((SpellAbility) ability).getSVar(sVarName);
+ val = ability.getSVar(sVarName);
}
if (StringUtils.isEmpty(val)) {
Card host = null;
@@ -1857,7 +1856,7 @@ public class AbilityUtils {
public boolean apply(Card input) {
for (final KeywordInterface inst : input.getKeywords(Keyword.SPLICE)) {
String k = inst.getOriginal();
- final String n[] = k.split(":");
+ final String[] n = k.split(":");
if (source.isValid(n[1].split(","), player, input, sa)) {
return true;
}
@@ -1890,7 +1889,7 @@ public class AbilityUtils {
// This Function thinks that Splice exist only once on the card
for (final KeywordInterface inst : c.getKeywords(Keyword.SPLICE)) {
final String k = inst.getOriginal();
- final String n[] = k.split(":");
+ final String[] n = k.split(":");
spliceCost = new Cost(n[2], false);
}
diff --git a/forge-game/src/main/java/forge/game/ability/effects/ActivateAbilityEffect.java b/forge-game/src/main/java/forge/game/ability/effects/ActivateAbilityEffect.java
index 2811591354c..3114cf48501 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/ActivateAbilityEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/ActivateAbilityEffect.java
@@ -51,7 +51,7 @@ public class ActivateAbilityEffect extends SpellAbilityEffect {
continue;
}
SpellAbility manaAb = p.getController().chooseSingleSpellForEffect(
- possibleAb, sa, "Choose a mana ability:", ImmutableMap.of());
+ possibleAb, sa, "Choose a mana ability:", ImmutableMap.of());
p.getController().playChosenSpellAbility(manaAb);
}
}
diff --git a/forge-game/src/main/java/forge/game/ability/effects/AssignGroupEffect.java b/forge-game/src/main/java/forge/game/ability/effects/AssignGroupEffect.java
index eb53cecf76b..e3e53dc6fa0 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/AssignGroupEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/AssignGroupEffect.java
@@ -38,7 +38,7 @@ public class AssignGroupEffect extends SpellAbilityEffect {
List defined = getDefinedOrTargeted(sa, "Defined");
- final List abilities = Lists.newArrayList(sa.getAdditionalAbilityList("Choices"));
+ final List abilities = Lists.newArrayList(sa.getAdditionalAbilityList("Choices"));
Player chooser = sa.getActivatingPlayer();
if (sa.hasParam("Chooser")) {
diff --git a/forge-game/src/main/java/forge/game/ability/effects/BidLifeEffect.java b/forge-game/src/main/java/forge/game/ability/effects/BidLifeEffect.java
index fdd3365212e..0ea798f50e9 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/BidLifeEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/BidLifeEffect.java
@@ -54,7 +54,7 @@ public class BidLifeEffect extends SpellAbilityEffect {
willBid = false;
for (final Player p : bidPlayers) {
final boolean result = p.getController().confirmBidAction(sa, PlayerActionConfirmMode.BidLife,
- "Do you want to top bid? Current Bid =" + String.valueOf(bid), bid, winner);
+ "Do you want to top bid? Current Bid =" + bid, bid, winner);
willBid |= result;
if (result) { // a different choose number
bid += p.getController().chooseNumber(sa, "Bid life:", 1, 9);
diff --git a/forge-game/src/main/java/forge/game/ability/effects/ChangeTextEffect.java b/forge-game/src/main/java/forge/game/ability/effects/ChangeTextEffect.java
index 91ee11bd4b8..131eecbcff7 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/ChangeTextEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/ChangeTextEffect.java
@@ -71,13 +71,13 @@ public class ChangeTextEffect extends SpellAbilityEffect {
validTypes.addAll(CardType.Constant.CREATURE_TYPES);
kindOfType = "creature";
}
- changedTypeWordOriginal = sa.getActivatingPlayer().getController().chooseSomeType(kindOfType, sa, validTypes, Lists.newArrayList());
+ changedTypeWordOriginal = sa.getActivatingPlayer().getController().chooseSomeType(kindOfType, sa, validTypes, Lists.newArrayList());
} else {
changedTypeWordOriginal = changedTypeWordsArray[0];
}
validTypes.clear();
- final List forbiddenTypes = sa.hasParam("ForbiddenNewTypes") ? Lists.newArrayList(sa.getParam("ForbiddenNewTypes").split(",")) : Lists.newArrayList();
+ final List forbiddenTypes = sa.hasParam("ForbiddenNewTypes") ? Lists.newArrayList(sa.getParam("ForbiddenNewTypes").split(",")) : Lists.newArrayList();
forbiddenTypes.add(changedTypeWordOriginal);
if (changedTypeWordsArray[1].startsWith("Choose")) {
if (changedTypeWordsArray[1].equals("ChooseBasicLandType")) {
diff --git a/forge-game/src/main/java/forge/game/ability/effects/ChangeZoneEffect.java b/forge-game/src/main/java/forge/game/ability/effects/ChangeZoneEffect.java
index 58220d302fb..8cbf3f0ba5b 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/ChangeZoneEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/ChangeZoneEffect.java
@@ -908,7 +908,7 @@ public class ChangeZoneEffect extends SpellAbilityEffect {
}
if (totalcmc != null) {
if (totcmc >= 0) {
- fetchList = CardLists.getValidCards(fetchList, "Card.cmcLE" + Integer.toString(totcmc), source.getController(), source);
+ fetchList = CardLists.getValidCards(fetchList, "Card.cmcLE" + totcmc, source.getController(), source);
}
}
diff --git a/forge-game/src/main/java/forge/game/ability/effects/ChooseCardEffect.java b/forge-game/src/main/java/forge/game/ability/effects/ChooseCardEffect.java
index 5d7083bf5ab..d87d9da4b85 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/ChooseCardEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/ChooseCardEffect.java
@@ -98,7 +98,7 @@ public class ChooseCardEffect extends SpellAbilityEffect {
int chosenP = 0;
while (!creature.isEmpty()) {
Card c = p.getController().chooseSingleEntityForEffect(creature, sa,
- "Select creature(s) with total power less than or equal to " + Integer.toString(totP - chosenP - negativeNum)
+ "Select creature(s) with total power less than or equal to " + (totP - chosenP - negativeNum)
+ "\r\n(Selected:" + chosenPool + ")\r\n" + "(Total Power: " + chosenP + ")", chosenP <= totP);
if (c == null) {
if (p.getController().confirmAction(sa, PlayerActionConfirmMode.OptionalChoose, "Cancel Choose?")) {
diff --git a/forge-game/src/main/java/forge/game/ability/effects/ChooseGenericEffect.java b/forge-game/src/main/java/forge/game/ability/effects/ChooseGenericEffect.java
index 93e5b3bca09..9073e1a9342 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/ChooseGenericEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/ChooseGenericEffect.java
@@ -30,14 +30,14 @@ public class ChooseGenericEffect extends SpellAbilityEffect {
public void resolve(SpellAbility sa) {
final Card host = sa.getHostCard();
- final List abilities = Lists.newArrayList(sa.getAdditionalAbilityList("Choices"));
+ final List abilities = Lists.newArrayList(sa.getAdditionalAbilityList("Choices"));
final SpellAbility fallback = sa.getAdditionalAbility("FallbackAbility");
final List tgtPlayers = getDefinedPlayersOrTargeted(sa);
for (final Player p : tgtPlayers) {
// determine if any of the choices are not valid
- List saToRemove = Lists.newArrayList();
+ List saToRemove = Lists.newArrayList();
for (SpellAbility saChoice : abilities) {
if (!saChoice.getRestrictions().checkOtherRestrictions(host, saChoice, sa.getActivatingPlayer()) ) {
diff --git a/forge-game/src/main/java/forge/game/ability/effects/CloneEffect.java b/forge-game/src/main/java/forge/game/ability/effects/CloneEffect.java
index 19933d913e2..950643e4a9c 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/CloneEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/CloneEffect.java
@@ -124,7 +124,7 @@ public class CloneEffect extends SpellAbilityEffect {
}
if (!pumpKeywords.isEmpty()) {
- tgtCard.addChangedCardKeywords(pumpKeywords, Lists.newArrayList(), false, false, ts);
+ tgtCard.addChangedCardKeywords(pumpKeywords, Lists.newArrayList(), false, false, ts);
}
tgtCard.updateStateForView();
diff --git a/forge-game/src/main/java/forge/game/ability/effects/ControlGainEffect.java b/forge-game/src/main/java/forge/game/ability/effects/ControlGainEffect.java
index 9844f1dd6a2..ec26b621f1d 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/ControlGainEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/ControlGainEffect.java
@@ -142,7 +142,7 @@ public class ControlGainEffect extends SpellAbilityEffect {
}
if (!kws.isEmpty()) {
- tgtC.addChangedCardKeywords(kws, Lists.newArrayList(), false, false, tStamp);
+ tgtC.addChangedCardKeywords(kws, Lists.newArrayList(), false, false, tStamp);
game.fireEvent(new GameEventCardStatsChanged(tgtC));
}
diff --git a/forge-game/src/main/java/forge/game/ability/effects/CopyPermanentEffect.java b/forge-game/src/main/java/forge/game/ability/effects/CopyPermanentEffect.java
index 2ecdfcfd620..612737941e8 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/CopyPermanentEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/CopyPermanentEffect.java
@@ -192,7 +192,7 @@ public class CopyPermanentEffect extends SpellAbilityEffect {
copyInPlay.setCloneOrigin(host);
if (!pumpKeywords.isEmpty()) {
- copyInPlay.addChangedCardKeywords(pumpKeywords, Lists.newArrayList(), false, false, timestamp);
+ copyInPlay.addChangedCardKeywords(pumpKeywords, Lists.newArrayList(), false, false, timestamp);
}
crds.add(copyInPlay);
if (sa.hasParam("RememberCopied")) {
diff --git a/forge-game/src/main/java/forge/game/ability/effects/CountersRemoveEffect.java b/forge-game/src/main/java/forge/game/ability/effects/CountersRemoveEffect.java
index 15a4a69bcf2..5338477cae1 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/CountersRemoveEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/CountersRemoveEffect.java
@@ -32,7 +32,7 @@ public class CountersRemoveEffect extends SpellAbilityEffect {
int amount = 0;
if (!num.equals("All") && !num.equals("Remembered")) {
amount = AbilityUtils.calculateAmount(sa.getHostCard(), num, sa);
- };
+ }
sb.append("Remove ");
if (sa.hasParam("UpTo")) {
diff --git a/forge-game/src/main/java/forge/game/ability/effects/DelayedTriggerEffect.java b/forge-game/src/main/java/forge/game/ability/effects/DelayedTriggerEffect.java
index d3c5ab0bc4c..33cd4a6aaf7 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/DelayedTriggerEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/DelayedTriggerEffect.java
@@ -30,9 +30,7 @@ public class DelayedTriggerEffect extends SpellAbilityEffect {
@Override
public void resolve(SpellAbility sa) {
Map mapParams = Maps.newHashMap(sa.getMapParams());
- if (mapParams.containsKey("Cost")) {
- mapParams.remove("Cost");
- }
+ mapParams.remove("Cost");
if (mapParams.containsKey("SpellDescription")) {
mapParams.put("TriggerDescription", mapParams.get("SpellDescription"));
diff --git a/forge-game/src/main/java/forge/game/ability/effects/ImmediateTriggerEffect.java b/forge-game/src/main/java/forge/game/ability/effects/ImmediateTriggerEffect.java
index a96057b6c34..8fe2f8b4813 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/ImmediateTriggerEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/ImmediateTriggerEffect.java
@@ -29,9 +29,7 @@ public class ImmediateTriggerEffect extends SpellAbilityEffect {
public void resolve(SpellAbility sa) {
Map mapParams = Maps.newHashMap(sa.getMapParams());
- if (mapParams.containsKey("Cost")) {
- mapParams.remove("Cost");
- }
+ mapParams.remove("Cost");
if (mapParams.containsKey("SpellDescription")) {
mapParams.put("TriggerDescription", mapParams.get("SpellDescription"));
diff --git a/forge-game/src/main/java/forge/game/ability/effects/LifeLoseEffect.java b/forge-game/src/main/java/forge/game/ability/effects/LifeLoseEffect.java
index d4af63d3a8e..39693308a68 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/LifeLoseEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/LifeLoseEffect.java
@@ -43,12 +43,12 @@ public class LifeLoseEffect extends SpellAbilityEffect {
lifeLost += p.loseLife(lifeAmount);
}
}
- sa.getHostCard().setSVar("AFLifeLost", "Number$" + Integer.toString(lifeLost));
+ sa.getHostCard().setSVar("AFLifeLost", "Number$" + lifeLost);
// Exceptional case for Extort: must propagate the amount of life lost to subability,
// otherwise the first Extort trigger per game won't work
if (sa.getSubAbility() != null && ApiType.GainLife.equals(sa.getSubAbility().getApi())) {
- sa.getSubAbility().setSVar("AFLifeLost", "Number$" + Integer.toString(lifeLost));
+ sa.getSubAbility().setSVar("AFLifeLost", "Number$" + lifeLost);
}
}
diff --git a/forge-game/src/main/java/forge/game/ability/effects/ProtectAllEffect.java b/forge-game/src/main/java/forge/game/ability/effects/ProtectAllEffect.java
index 2a09ba0eb5b..8929a776217 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/ProtectAllEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/ProtectAllEffect.java
@@ -123,7 +123,7 @@ public class ProtectAllEffect extends SpellAbilityEffect {
final List playerList = AbilityUtils.getDefinedPlayers(host, players, sa);
for (final Player player : playerList) {
for (final String gain : gains) {
- player.addChangedKeywords(ImmutableList.of("Protection from " + gain), ImmutableList.of(), timestamp);
+ player.addChangedKeywords(ImmutableList.of("Protection from " + gain), ImmutableList.of(), timestamp);
}
if (!sa.hasParam("Permanent")) {
diff --git a/forge-game/src/main/java/forge/game/ability/effects/ProtectEffect.java b/forge-game/src/main/java/forge/game/ability/effects/ProtectEffect.java
index 3b2b1fe6445..df30dea7f34 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/ProtectEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/ProtectEffect.java
@@ -27,7 +27,7 @@ public class ProtectEffect extends SpellAbilityEffect {
protected String getStackDescription(SpellAbility sa) {
final List gains = getProtectionList(sa);
- final boolean choose = (sa.hasParam("Choices")) ? true : false;
+ final boolean choose = sa.hasParam("Choices");
final String joiner = choose ? "or" : "and";
final StringBuilder sb = new StringBuilder();
diff --git a/forge-game/src/main/java/forge/game/ability/effects/PumpAllEffect.java b/forge-game/src/main/java/forge/game/ability/effects/PumpAllEffect.java
index ea9dcb5f80a..010914c6775 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/PumpAllEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/PumpAllEffect.java
@@ -51,7 +51,7 @@ public class PumpAllEffect extends SpellAbilityEffect {
boolean redrawPT = false;
- if (a > 0 || d > 0) {
+ if (a != 0 || d != 0) {
tgtC.addPTBoost(a, d, timestamp);
redrawPT = true;
}
@@ -82,6 +82,7 @@ public class PumpAllEffect extends SpellAbilityEffect {
for (String kw : hiddenkws) {
tgtC.removeHiddenExtrinsicKeyword(kw);
}
+ tgtC.updatePowerToughnessForView();
game.fireEvent(new GameEventCardStatsChanged(tgtC));
}
diff --git a/forge-game/src/main/java/forge/game/ability/effects/PumpEffect.java b/forge-game/src/main/java/forge/game/ability/effects/PumpEffect.java
index 591dcf525ad..72c2987dd5c 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/PumpEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/PumpEffect.java
@@ -55,12 +55,12 @@ public class PumpEffect extends SpellAbilityEffect {
}
}
- if (a > 0 || d > 0) {
+ if (a != 0 || d != 0) {
gameCard.addPTBoost(a, d, timestamp);
redrawPT = true;
}
- gameCard.addChangedCardKeywords(kws, Lists.newArrayList(), false, false, timestamp);
+ gameCard.addChangedCardKeywords(kws, Lists.newArrayList(), false, false, timestamp);
if (redrawPT) {
gameCard.updatePowerToughnessForView();
}
@@ -106,7 +106,7 @@ public class PumpEffect extends SpellAbilityEffect {
&& !(host.isInPlay() || host.isInZone(ZoneType.Stack))) {
return;
}
- p.addChangedKeywords(keywords, ImmutableList.of(), timestamp);
+ p.addChangedKeywords(keywords, ImmutableList.of(), timestamp);
if (!sa.hasParam("Permanent")) {
// If not Permanent, remove Pumped at EOT
@@ -252,7 +252,7 @@ public class PumpEffect extends SpellAbilityEffect {
if (defined.equals("ChosenType")) {
replaced = host.getChosenType();
} else if (defined.equals("CardUIDSource")) {
- replaced = "CardUID_" + String.valueOf(host.getId());
+ replaced = "CardUID_" + host.getId();
} else if (defined.equals("ActivatorName")) {
replaced = sa.getActivatingPlayer().getName();
}
@@ -277,9 +277,7 @@ public class PumpEffect extends SpellAbilityEffect {
if (sa.hasParam("NoRepetition")) {
for (KeywordInterface inst : tgtCards.get(0).getKeywords()) {
final String kws = inst.getOriginal();
- if (total.contains(kws)) {
- total.remove(kws);
- }
+ total.remove(kws);
}
}
final int min = Math.min(total.size(), numkw);
diff --git a/forge-game/src/main/java/forge/game/card/Card.java b/forge-game/src/main/java/forge/game/card/Card.java
index 366eb1c8680..1ee0b0f0f17 100644
--- a/forge-game/src/main/java/forge/game/card/Card.java
+++ b/forge-game/src/main/java/forge/game/card/Card.java
@@ -113,7 +113,7 @@ public class Card extends GameEntity implements Comparable {
private final Map mayPlay = Maps.newHashMap();
- private final Multimap withFlash = HashMultimap.create();
+ private final Multimap withFlash = HashMultimap.create();
// changes by AF animate and continuous static effects - timestamp is the key of maps
private final Map changedCardTypes = Maps.newTreeMap();
@@ -1196,9 +1196,7 @@ public class Card extends GameEntity implements Comparable {
if (type == CounterType.DREAM) {
// need to be done extra because it is also a state based action
- if (hasKeyword("CARDNAME can't have more than seven dream counters on it.") && getCounters(CounterType.DREAM) > 6) {
- return false;
- }
+ return !hasKeyword("CARDNAME can't have more than seven dream counters on it.") || getCounters(CounterType.DREAM) <= 6;
}
return true;
}
@@ -1709,7 +1707,7 @@ public class Card extends GameEntity implements Comparable {
sbLong.append(sbx).append("\r\n");
}
} else if (keyword.startsWith("Hexproof:")) {
- final String k[] = keyword.split(":");
+ final String[] k = keyword.split(":");
sbLong.append("Hexproof from ").append(k[2])
.append(" (").append(inst.getReminderText()).append(")").append("\r\n");
} else if (keyword.endsWith(".") && !keyword.startsWith("Haunt")) {
@@ -1803,7 +1801,7 @@ public class Card extends GameEntity implements Comparable {
} else if (keyword.equals("AllNonLegendaryCreatureNames")) {
sbLong.append(getName()).append(" has all names of nonlegendary creature cards.\r\n");
} else if (keyword.startsWith("IfReach")) {
- String k[] = keyword.split(":");
+ String[] k = keyword.split(":");
sbLong.append(getName()).append(" can block ")
.append(CardType.getPluralType(k[1]))
.append(" as though it had reach.\r\n");
@@ -1813,7 +1811,7 @@ public class Card extends GameEntity implements Comparable {
String desc = AbilityFactory.getMapParams(getSVar(k[1])).get("SpellDescription");
sbLong.append(desc);
} else if (keyword.startsWith("Saga")) {
- String k[] = keyword.split(":");
+ String[] k = keyword.split(":");
String desc = "(As this Saga enters and after your draw step, "
+ " add a lore counter. Sacrifice after " + Strings.repeat("I", Integer.valueOf(k[1])) + ".)";
sbLong.append(desc);
@@ -2201,7 +2199,7 @@ public class Card extends GameEntity implements Comparable {
if (n.length > 3) {
desc = n[3];
} else {
- String k[] = n[1].split(",");
+ String[] k = n[1].split(",");
for (int i = 0; i < k.length; i++) {
if (CardType.isACardType(k[i])) {
k[i] = k[i].toLowerCase();
@@ -3392,7 +3390,7 @@ public class Card extends GameEntity implements Comparable {
}
@Override
public String toString() {
- return TextUtil.concatWithSpace("c:"+String.valueOf(currentValue),"tb:"+String.valueOf(tempBoost),"bfc:"+String.valueOf(bonusFromCounters));
+ return TextUtil.concatWithSpace("c:"+ currentValue,"tb:"+ tempBoost,"bfc:"+ bonusFromCounters);
}
}
@@ -4785,7 +4783,7 @@ public class Card extends GameEntity implements Comparable {
CardCollection newCardsInCommand = (CardCollection)getGame().getCardsIn(ZoneType.Command);
newCardsInCommand.removeAll(cardsInCommand);
if (!newCardsInCommand.isEmpty()) {
- newCardsInCommand.get(0).setSVar("PreventedDamage", "Number$" + Integer.toString(dmgToBePrevented));
+ newCardsInCommand.get(0).setSVar("PreventedDamage", "Number$" + dmgToBePrevented);
}
}
subtractPreventNextDamageWithEffect(shieldSource, restDamage);
@@ -5080,7 +5078,7 @@ public class Card extends GameEntity implements Comparable {
addChangedCardTypes(new CardType(Collections.singletonList("Aura")),
new CardType(Collections.singletonList("Creature")),
false, false, false, false, false, false, true, bestowTimestamp, updateView);
- addChangedCardKeywords(Collections.singletonList("Enchant creature"), Lists.newArrayList(),
+ addChangedCardKeywords(Collections.singletonList("Enchant creature"), Lists.newArrayList(),
false, false, bestowTimestamp, updateView);
}
@@ -5384,7 +5382,7 @@ public class Card extends GameEntity implements Comparable {
switch (kw.getOriginal()) {
case "Shroud":
StringBuilder sb = new StringBuilder();
- sb.append("Can target CardUID_").append(String.valueOf(getId()));
+ sb.append("Can target CardUID_").append(getId());
sb.append(" with spells and abilities as though it didn't have shroud.");
if (sa.getActivatingPlayer() == null) {
System.err.println("Unexpected behavior: SA activator was null when trying to determine if the activating player could target a card with Shroud. SA host card = " + source + ", SA = " + sa);
@@ -5447,9 +5445,7 @@ public class Card extends GameEntity implements Comparable {
}
// check valid
- if (!isValid(tgt.getValidTgts(), aura.getController(), aura, sa)) {
- return false;
- }
+ return isValid(tgt.getValidTgts(), aura.getController(), aura, sa);
}
return true;
@@ -5457,18 +5453,12 @@ public class Card extends GameEntity implements Comparable {
@Override
protected final boolean canBeEquippedBy(final Card equip) {
- if (!isCreature() || !isInPlay()) {
- return false;
- }
- return true;
+ return isCreature() && isInPlay();
}
@Override
protected boolean canBeFortifiedBy(final Card fort) {
- if (!isLand() || !isInPlay() || fort.isLand()) {
- return false;
- }
- return true;
+ return isLand() && isInPlay() && !fort.isLand();
}
/* (non-Javadoc)
@@ -5727,11 +5717,7 @@ public class Card extends GameEntity implements Comparable {
}
}
- if (!getController().canSacrificeBy(source)) {
- return false;
- }
-
- return true;
+ return getController().canSacrificeBy(source);
}
public CardRules getRules() {
@@ -6130,10 +6116,7 @@ public class Card extends GameEntity implements Comparable {
if (hasKeyword(Keyword.FLASH)) {
return true;
}
- if (withFlash.containsValue(p)) {
- return true;
- }
- return false;
+ return withFlash.containsValue(p);
}
public void addWithFlash(Long timestamp, Iterable players) {
@@ -6170,11 +6153,7 @@ public class Card extends GameEntity implements Comparable {
return false;
}
- if (!getOwner().canDiscardBy(sa)) {
- return false;
- }
-
- return true;
+ return getOwner().canDiscardBy(sa);
}
public void addAbilityActivated(SpellAbility ability) {
diff --git a/forge-game/src/main/java/forge/game/card/CardFactory.java b/forge-game/src/main/java/forge/game/card/CardFactory.java
index 2a69b3e4567..98dab46962d 100644
--- a/forge-game/src/main/java/forge/game/card/CardFactory.java
+++ b/forge-game/src/main/java/forge/game/card/CardFactory.java
@@ -645,7 +645,7 @@ public class CardFactory {
Trigger t = null;
if (sa.isWrapper()) {
// copy trigger?
- t = ((WrappedAbility) sa).getTrigger();
+ t = sa.getTrigger();
} else { // some keyword ability, e.g. Exalted, Annihilator
return sa.copy();
}
@@ -677,7 +677,7 @@ public class CardFactory {
WrappedAbility wrapperAbility = new WrappedAbility(t, trig, ((WrappedAbility) sa).getDecider());
wrapperAbility.setTrigger(true);
- wrapperAbility.setMandatory(((WrappedAbility) sa).isMandatory());
+ wrapperAbility.setMandatory(sa.isMandatory());
wrapperAbility.setDescription(wrapperAbility.getStackDescription());
t.setTriggeredSA(wrapperAbility);
return wrapperAbility;
@@ -774,7 +774,7 @@ public class CardFactory {
// triggers to add to clone
if (sa.hasParam("AddTriggers")) {
- for (final String s : Arrays.asList(sa.getParam("AddTriggers").split(","))) {
+ for (final String s : sa.getParam("AddTriggers").split(",")) {
if (origSVars.containsKey(s)) {
final String actualTrigger = origSVars.get(s);
final Trigger parsedTrigger = TriggerHandler.parseTrigger(actualTrigger, out, true);
@@ -786,7 +786,7 @@ public class CardFactory {
// SVars to add to clone
if (sa.hasParam("AddSVars") || sa.hasParam("GainTextSVars")) {
final String str = sa.getParamOrDefault("GainTextSVars", sa.getParam("AddSVars"));
- for (final String s : Arrays.asList(str.split(","))) {
+ for (final String s : str.split(",")) {
if (origSVars.containsKey(s)) {
final String actualsVar = origSVars.get(s);
state.setSVar(s, actualsVar);
@@ -797,7 +797,7 @@ public class CardFactory {
// abilities to add to clone
if (sa.hasParam("AddAbilities") || sa.hasParam("GainTextAbilities")) {
final String str = sa.getParamOrDefault("GainTextAbilities", sa.getParam("AddAbilities"));
- for (final String s : Arrays.asList(str.split(","))) {
+ for (final String s : str.split(",")) {
if (origSVars.containsKey(s)) {
final String actualAbility = origSVars.get(s);
final SpellAbility grantedAbility = AbilityFactory.getAbility(actualAbility, out);
diff --git a/forge-game/src/main/java/forge/game/card/CardFactoryUtil.java b/forge-game/src/main/java/forge/game/card/CardFactoryUtil.java
index bf22da56bb6..3c2413b6ed6 100644
--- a/forge-game/src/main/java/forge/game/card/CardFactoryUtil.java
+++ b/forge-game/src/main/java/forge/game/card/CardFactoryUtil.java
@@ -147,7 +147,7 @@ public class CardFactoryUtil {
sbCost.append("— ");
}
// get rid of the ": " at the end
- sbCost.append(costDesc.substring(0, costDesc.length() - 2));
+ sbCost.append(costDesc, 0, costDesc.length() - 2);
StringBuilder sb = new StringBuilder();
sb.append("ST$ SetState | Cost$ ").append(costStr).append(" | CostDesc$ ").append(sbCost);
@@ -292,11 +292,7 @@ public class CardFactoryUtil {
* @return a boolean.
*/
public static boolean isCounterable(final Card c) {
- if (c.hasKeyword("CARDNAME can't be countered.") || !c.getCanCounter()) {
- return false;
- }
-
- return true;
+ return !c.hasKeyword("CARDNAME can't be countered.") && c.getCanCounter();
}
/**
@@ -318,7 +314,7 @@ public class CardFactoryUtil {
for (KeywordInterface k : c.getKeywords()) {
final String o = k.getOriginal();
if (o.startsWith("CantBeCounteredBy")) {
- final String m[] = o.split(":");
+ final String[] m = o.split(":");
if (sa.isValid(m[1].split(","), c.getController(), c, null)) {
return false;
}
@@ -2186,12 +2182,12 @@ public class CardFactoryUtil {
final String abStringAfflict = "DB$ LoseLife | Defined$ TriggeredDefendingPlayer" +
" | LifeAmount$ " + n;
- final Trigger afflictTrigger = TriggerHandler.parseTrigger(trigStr.toString(), card, intrinsic);
+ final Trigger afflictTrigger = TriggerHandler.parseTrigger(trigStr, card, intrinsic);
afflictTrigger.setOverridingAbility(AbilityFactory.getAbility(abStringAfflict, card));
inst.addTrigger(afflictTrigger);
} else if (keyword.startsWith("Afterlife")) {
- final String k[] = keyword.split(":");
+ final String[] k = keyword.split(":");
final String name = StringUtils.join(k, " ");
final StringBuilder sb = new StringBuilder();
@@ -2435,7 +2431,7 @@ public class CardFactoryUtil {
final String effect = "DB$ PutCounter | Defined$ Self | CounterType$ P1P1 | "
+ "CounterNum$ 1 | Evolve$ True";
- final Trigger trigger = TriggerHandler.parseTrigger(trigStr.toString(), card, intrinsic);
+ final Trigger trigger = TriggerHandler.parseTrigger(trigStr, card, intrinsic);
trigger.setOverridingAbility(AbilityFactory.getAbility(effect, card));
inst.addTrigger(trigger);
@@ -2731,7 +2727,7 @@ public class CardFactoryUtil {
" | TriggerDescription$ Melee (" + inst.getReminderText() + ")";
final String effect = "DB$ Pump | Defined$ TriggeredAttackerLKICopy | NumAtt$ MeleeX | NumDef$ MeleeX";
- final Trigger trigger = TriggerHandler.parseTrigger(trigStr.toString(), card, intrinsic);
+ final Trigger trigger = TriggerHandler.parseTrigger(trigStr, card, intrinsic);
SpellAbility sa = AbilityFactory.getAbility(effect, card);
sa.setSVar("MeleeX", "TriggeredPlayersDefenders$Amount");
@@ -2746,7 +2742,7 @@ public class CardFactoryUtil {
final String effect = "DB$ PutCounter | CounterType$ P1P1 | CounterNum$ 1"
+ " | ValidTgts$ Creature.attacking+powerLTX"
+ " | TgtPrompt$ Select target attacking creature with less power";
- final Trigger trigger = TriggerHandler.parseTrigger(trigStr.toString(), card, intrinsic);
+ final Trigger trigger = TriggerHandler.parseTrigger(trigStr, card, intrinsic);
SpellAbility sa = AbilityFactory.getAbility(effect, card);
sa.setSVar("X", "Count$CardPower");
@@ -2854,7 +2850,7 @@ public class CardFactoryUtil {
final String trigStr = "Mode$ DamageDone | ValidSource$ Card.Self | ValidTarget$ Player | CombatDamage$ True | Secondary$ True"
+ " | TriggerZones$ Battlefield | TriggerDescription$ Poisonous " + n + " (" + inst.getReminderText() + ")";
- final Trigger parsedTrigger = TriggerHandler.parseTrigger(trigStr.toString(), card, intrinsic);
+ final Trigger parsedTrigger = TriggerHandler.parseTrigger(trigStr, card, intrinsic);
final String effect = "DB$ Poison | Defined$ TriggeredTarget | Num$ " + n;
parsedTrigger.setOverridingAbility(AbilityFactory.getAbility(effect, card));
@@ -2902,7 +2898,7 @@ public class CardFactoryUtil {
final String effect = "DB$ Pump | Defined$ TriggeredAttackerLKICopy" +
" | NumAtt$ Rampage" + n + " | NumDef$ Rampage" + n;
- final Trigger trigger = TriggerHandler.parseTrigger(trigStr.toString(), card, intrinsic);
+ final Trigger trigger = TriggerHandler.parseTrigger(trigStr, card, intrinsic);
SpellAbility sa = AbilityFactory.getAbility(effect, card);
sa.setSVar("Rampage" + n, "SVar$RampageCount/Times." + n);
@@ -3070,7 +3066,7 @@ public class CardFactoryUtil {
inst.addTrigger(parsedTrigger);
} else if (keyword.startsWith("Suspend")) {
//upkeep trigger
- StringBuilder upkeepTrig = new StringBuilder();;
+ StringBuilder upkeepTrig = new StringBuilder();
upkeepTrig.append("Mode$ Phase | Phase$ Upkeep | ValidPlayer$ You | TriggerZones$ Exile ");
upkeepTrig.append(" | IsPresent$ Card.Self+suspended | PresentZone$ Exile");
@@ -3201,7 +3197,7 @@ public class CardFactoryUtil {
SpellAbility saDelay = AbilityFactory.getAbility(strDelay, card);
saDelay.setAdditionalAbility("Execute", (AbilitySub) AbilityFactory.getAbility(strSac, card));
- final Trigger trigger = TriggerHandler.parseTrigger(strTrig.toString(), card, intrinsic);
+ final Trigger trigger = TriggerHandler.parseTrigger(strTrig, card, intrinsic);
trigger.setOverridingAbility(saDelay);
inst.addTrigger(trigger);
}
@@ -3404,7 +3400,7 @@ public class CardFactoryUtil {
sb.append("| ValidStackSa$ Spell.Flashback | Description$ Flashback");
if (keyword.contains(":")) {
- final String k[] = keyword.split(":");
+ final String[] k = keyword.split(":");
final Cost cost = new Cost(k[1], false);
sb.append( cost.isOnlyManaCost() ? " " : "—");
@@ -3896,7 +3892,7 @@ public class CardFactoryUtil {
// Add the Epic effect as a subAbility
String dbStr = "DB$ Effect | Triggers$ EpicTrigger | SVars$ EpicCopy | StaticAbilities$ EpicCantBeCast | Duration$ Permanent | Epic$ True";
- final AbilitySub newSA = (AbilitySub) AbilityFactory.getAbility(dbStr.toString(), card);
+ final AbilitySub newSA = (AbilitySub) AbilityFactory.getAbility(dbStr, card);
newSA.setSVar("EpicCantBeCast", "Mode$ CantBeCast | ValidCard$ Card | Caster$ You | EffectZone$ Command | Description$ For the rest of the game, you can't cast spells.");
newSA.setSVar("EpicTrigger", "Mode$ Phase | Phase$ Upkeep | ValidPlayer$ You | Execute$ EpicCopy | TriggerDescription$ "
@@ -3964,7 +3960,7 @@ public class CardFactoryUtil {
// don't use SimpleString there because it does has "and" between cost i dont want that
costStr = cost.toString();
// but now it has ": " at the end i want to remove
- sb.append("| CostDesc$ ").append(costStr.substring(0, costStr.length() - 2));
+ sb.append("| CostDesc$ ").append(costStr, 0, costStr.length() - 2);
if (!cost.isOnlyManaCost()) {
sb.append(".");
}
@@ -4542,7 +4538,7 @@ public class CardFactoryUtil {
final StringBuilder sbValid = new StringBuilder();
if (!keyword.equals("Hexproof")) {
- final String k[] = keyword.split(":");
+ final String[] k = keyword.split(":");
sbDesc.append(" from ").append(k[2]);
sbValid.append("| ValidSource$ ").append(k[1]);
diff --git a/forge-game/src/main/java/forge/game/card/CardPlayOption.java b/forge-game/src/main/java/forge/game/card/CardPlayOption.java
index 3c53c4d0e35..b7531dde10d 100644
--- a/forge-game/src/main/java/forge/game/card/CardPlayOption.java
+++ b/forge-game/src/main/java/forge/game/card/CardPlayOption.java
@@ -11,7 +11,7 @@ public final class CardPlayOption {
/** Indicates the mana cost must be paid. */
YES,
/** Indicates the mana cost may not be paid. */
- NO;
+ NO
}
private final Player player;
diff --git a/forge-game/src/main/java/forge/game/card/CardPredicates.java b/forge-game/src/main/java/forge/game/card/CardPredicates.java
index 5d5a415af5c..4f4d00b2010 100644
--- a/forge-game/src/main/java/forge/game/card/CardPredicates.java
+++ b/forge-game/src/main/java/forge/game/card/CardPredicates.java
@@ -214,7 +214,7 @@ public final class CardPredicates {
return c.canBeSacrificedBy(sa);
}
};
- };
+ }
public static final Predicate canBeAttached(final Card aura) {
return new Predicate() {
@@ -223,7 +223,7 @@ public final class CardPredicates {
return c.canBeAttached(aura);
}
};
- };
+ }
public static final Predicate isColor(final byte color) {
return new Predicate() {
diff --git a/forge-game/src/main/java/forge/game/card/CardProperty.java b/forge-game/src/main/java/forge/game/card/CardProperty.java
index 74112fa659e..1411664dd18 100644
--- a/forge-game/src/main/java/forge/game/card/CardProperty.java
+++ b/forge-game/src/main/java/forge/game/card/CardProperty.java
@@ -27,8 +27,7 @@ import java.util.List;
public class CardProperty {
- public static boolean cardHasProperty(Card card, String property, Player sourceController, Card source,
- SpellAbility spellAbility) {
+ public static boolean cardHasProperty(Card card, String property, Player sourceController, Card source, SpellAbility spellAbility) {
final Game game = card.getGame();
final Combat combat = game.getCombat();
// lki can't be null but it does return this
@@ -40,7 +39,7 @@ public class CardProperty {
String name = TextUtil.fastReplace(property.substring(5), ";", ","); // for some legendary cards
if (!card.sharesNameWith(name)) {
return false;
- }
+ }
} else if (property.startsWith("notnamed")) {
if (card.sharesNameWith(property.substring(8))) {
return false;
@@ -1115,9 +1114,9 @@ public class CardProperty {
return false;
}
} else if (property.startsWith("DrawnThisTurn")) {
- if (!card.getDrawnThisTurn()) {
- return false;
- }
+ if (!card.getDrawnThisTurn()) {
+ return false;
+ }
} else if (property.startsWith("enteredBattlefieldThisTurn")) {
if (!(card.getTurnInZone() == game.getPhaseHandler().getTurn())) {
return false;
@@ -1749,7 +1748,7 @@ public class CardProperty {
final ZoneType realZone = ZoneType.smartValueOf(strZone);
// lki last zone does fall back to this zone
final Zone lkiZone = lki.getLastKnownZone();
-
+
if (lkiZone == null || !lkiZone.is(realZone)) {
return false;
}
@@ -1773,4 +1772,4 @@ public class CardProperty {
return true;
}
-}
+}
\ No newline at end of file
diff --git a/forge-game/src/main/java/forge/game/card/CardState.java b/forge-game/src/main/java/forge/game/card/CardState.java
index dbe4ca1546b..b2b4d60e72a 100644
--- a/forge-game/src/main/java/forge/game/card/CardState.java
+++ b/forge-game/src/main/java/forge/game/card/CardState.java
@@ -471,9 +471,7 @@ public class CardState extends GameObject {
view.updateFoilIndex(card.getState(CardStateName.Original));
}
public final void removeSVar(final String var) {
- if (sVars.containsKey(var)) {
- sVars.remove(var);
- }
+ sVars.remove(var);
}
public final int getFoil() {
diff --git a/forge-game/src/main/java/forge/game/card/CardUtil.java b/forge-game/src/main/java/forge/game/card/CardUtil.java
index 014515eb063..8d0c2000fff 100644
--- a/forge-game/src/main/java/forge/game/card/CardUtil.java
+++ b/forge-game/src/main/java/forge/game/card/CardUtil.java
@@ -80,7 +80,7 @@ public final class CardUtil {
}
public static boolean isStackingKeyword(final String keyword) {
- String kw = new String(keyword);
+ String kw = keyword;
if (kw.startsWith("HIDDEN")) {
kw = kw.substring(7);
}
@@ -345,7 +345,7 @@ public final class CardUtil {
// a nice entry point with minimum parameters
public static Set getReflectableManaColors(final SpellAbility sa) {
- return getReflectableManaColors(sa, sa, Sets.newHashSet(), new CardCollection());
+ return getReflectableManaColors(sa, sa, Sets.newHashSet(), new CardCollection());
}
private static Set getReflectableManaColors(final SpellAbility abMana, final SpellAbility sa,
@@ -390,9 +390,7 @@ public final class CardUtil {
// remove anything cards that is already in parents
for (final Card p : parents) {
- if (cards.contains(p)) {
- cards.remove(p);
- }
+ cards.remove(p);
}
if ((cards.size() == 0) && !reflectProperty.equals("Produced")) {
@@ -506,9 +504,7 @@ public final class CardUtil {
// Remove cards already targeted
final List targeted = Lists.newArrayList(ability.getTargets().getTargetCards());
for (final Card c : targeted) {
- if (choices.contains(c)) {
- choices.remove(c);
- }
+ choices.remove(c);
}
// Remove cards exceeding total CMC
diff --git a/forge-game/src/main/java/forge/game/card/CardView.java b/forge-game/src/main/java/forge/game/card/CardView.java
index 5afd53b06ca..8762d2816d2 100644
--- a/forge-game/src/main/java/forge/game/card/CardView.java
+++ b/forge-game/src/main/java/forge/game/card/CardView.java
@@ -345,10 +345,7 @@ public class CardView extends GameEntityView {
return true;
}
col = get(TrackableProperty.PlayerMayLookTemp);
- if (col != null && col.contains(pv)) {
- return true;
- }
- return false;
+ return col != null && col.contains(pv);
}
void setPlayerMayLook(Player p, boolean mayLook, boolean temp) {
TrackableProperty prop = temp ? TrackableProperty.PlayerMayLookTemp : TrackableProperty.PlayerMayLook;
@@ -380,7 +377,7 @@ public class CardView extends GameEntityView {
return Iterables.any(viewers, new Predicate() {
public final boolean apply(final PlayerView input) {
return canBeShownTo(input);
- };
+ }
});
}
@@ -462,10 +459,7 @@ public class CardView extends GameEntityView {
if (mindSlaveMaster != null && canFaceDownBeShownTo(mindSlaveMaster)) {
return true;
}
- if (isInZone(EnumSet.of(ZoneType.Battlefield, ZoneType.Stack, ZoneType.Sideboard)) && getController().equals(viewer)) {
- return true;
- }
- return false;
+ return isInZone(EnumSet.of(ZoneType.Battlefield, ZoneType.Stack, ZoneType.Sideboard)) && getController().equals(viewer);
}
public FCollectionView getEncodedCards() {
@@ -1067,7 +1061,7 @@ public class CardView extends GameEntityView {
TrackableCollection views = get(key);
if (views == null) {
views = new TrackableCollection();
- views.add(cardToAdd.getView());;
+ views.add(cardToAdd.getView());
set(key, views);
}
else if (views.add(cardToAdd.getView())) {
diff --git a/forge-game/src/main/java/forge/game/card/token/TokenInfo.java b/forge-game/src/main/java/forge/game/card/token/TokenInfo.java
index 5d4ff3b79aa..4abd4a7447e 100644
--- a/forge-game/src/main/java/forge/game/card/token/TokenInfo.java
+++ b/forge-game/src/main/java/forge/game/card/token/TokenInfo.java
@@ -320,7 +320,7 @@ public class TokenInfo {
if (!CardUtil.isKeywordModifiable(o)) {
continue;
}
- String r = new String(o);
+ String r = o;
// replace types
for (final Map.Entry e : typeMap.entrySet()) {
final String key = e.getKey();
diff --git a/forge-game/src/main/java/forge/game/combat/AttackConstraints.java b/forge-game/src/main/java/forge/game/combat/AttackConstraints.java
index c3420217e83..f7b59f9413f 100644
--- a/forge-game/src/main/java/forge/game/combat/AttackConstraints.java
+++ b/forge-game/src/main/java/forge/game/combat/AttackConstraints.java
@@ -94,7 +94,7 @@ public class AttackConstraints {
final int globalMax = globalRestrictions.getMax();
final int myMax = Ints.min(globalMax == -1 ? Integer.MAX_VALUE : globalMax, possibleAttackers.size());
if (myMax == 0) {
- return Pair.of(Collections.emptyMap(), Integer.valueOf(0));
+ return Pair.of(Collections.emptyMap(), Integer.valueOf(0));
}
final MapToAmount