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..51798704399 100644
--- a/forge-ai/src/main/java/forge/ai/AiAttackController.java
+++ b/forge-ai/src/main/java/forge/ai/AiAttackController.java
@@ -81,7 +81,7 @@ public class AiAttackController {
this.defendingOpponent = choosePreferredDefenderPlayer();
this.oppList = getOpponentCreatures(this.defendingOpponent);
this.myList = ai.getCreaturesInPlay();
- this.attackers = new ArrayList();
+ this.attackers = new ArrayList<>();
for (Card c : myList) {
if (CombatUtil.canAttack(c, this.defendingOpponent)) {
attackers.add(c);
@@ -95,7 +95,7 @@ public class AiAttackController {
this.defendingOpponent = choosePreferredDefenderPlayer();
this.oppList = getOpponentCreatures(this.defendingOpponent);
this.myList = ai.getCreaturesInPlay();
- this.attackers = new ArrayList();
+ this.attackers = new ArrayList<>();
if (CombatUtil.canAttack(attacker, this.defendingOpponent)) {
attackers.add(attacker);
}
@@ -103,8 +103,7 @@ public class AiAttackController {
} // overloaded constructor to evaluate single specified attacker
public static List getOpponentCreatures(final Player defender) {
- List defenders = new ArrayList();
- defenders.addAll(defender.getCreaturesInPlay());
+ List defenders = new ArrayList<>(defender.getCreaturesInPlay());
Predicate canAnimate = new Predicate() {
@Override
public boolean apply(Card c) {
@@ -151,7 +150,7 @@ public class AiAttackController {
*
*/
public final static List sortAttackers(final List in) {
- final List list = new ArrayList();
+ final List list = new ArrayList<>();
// Cards with triggers should come first (for Battle Cry)
for (final Card attacker : in) {
@@ -256,7 +255,7 @@ public class AiAttackController {
}
public final static List getPossibleBlockers(final List blockers, final List attackers) {
- List possibleBlockers = new ArrayList(blockers);
+ List possibleBlockers = new ArrayList<>(blockers);
possibleBlockers = CardLists.filter(possibleBlockers, new Predicate() {
@Override
public boolean apply(final Card c) {
@@ -267,7 +266,7 @@ public class AiAttackController {
}
public final static boolean canBlockAnAttacker(final Card c, final List attackers, final boolean nextTurn) {
- final List attackerList = new ArrayList(attackers);
+ final List attackerList = new ArrayList<>(attackers);
if (!c.isCreature()) {
return false;
}
@@ -280,7 +279,7 @@ public class AiAttackController {
}
public final static Card getCardCanBlockAnAttacker(final Card c, final List attackers, final boolean nextTurn) {
- final List attackerList = new ArrayList(attackers);
+ final List attackerList = new ArrayList<>(attackers);
if (!c.isCreature()) {
return null;
}
@@ -295,9 +294,9 @@ public class AiAttackController {
// this checks to make sure that the computer player doesn't lose when the human player attacks
// this method is used by getAttackers()
public final List notNeededAsBlockers(final Player ai, final List attackers) {
- final List notNeededAsBlockers = new ArrayList(attackers);
+ final List notNeededAsBlockers = new ArrayList<>(attackers);
int fixedBlockers = 0;
- final List vigilantes = new ArrayList();
+ final List vigilantes = new ArrayList<>();
//check for time walks
if (ai.getGame().getPhaseHandler().getNextTurn().equals(ai)) {
return attackers;
@@ -336,7 +335,7 @@ public class AiAttackController {
}
}
- List opponentsAttackers = new ArrayList(oppList);
+ List opponentsAttackers = new ArrayList<>(oppList);
opponentsAttackers = CardLists.filter(opponentsAttackers, new Predicate() {
@Override
public boolean apply(final Card c) {
@@ -548,8 +547,7 @@ public class AiAttackController {
remainingAttackers.removeAll(unblockedAttackers);
for (Card blocker : this.blockers) {
- if (blocker.hasKeyword("CARDNAME can block any number of creatures.")
- || blocker.hasKeyword("CARDNAME can block an additional ninety-nine creatures each combat.")) {
+ if (blocker.canBlockAny()) {
for (Card attacker : this.attackers) {
if (CombatUtil.canBlock(attacker, blocker)) {
remainingAttackers.remove(attacker);
@@ -565,14 +563,19 @@ public class AiAttackController {
if (remainingAttackers.isEmpty() || maxBlockersAfterCrew == 0) {
break;
}
- if (blocker.hasKeyword("CARDNAME can block an additional creature each combat.")) {
- blockedAttackers.add(remainingAttackers.get(0));
- remainingAttackers.remove(0);
- maxBlockersAfterCrew--;
- if (remainingAttackers.isEmpty()) {
- break;
+
+ int numExtraBlocks = blocker.canBlockAdditional();
+ if (numExtraBlocks > 0) {
+ while (numExtraBlocks-- > 0 && !remainingAttackers.isEmpty()) {
+ blockedAttackers.add(remainingAttackers.get(0));
+ remainingAttackers.remove(0);
+ maxBlockersAfterCrew--;
}
}
+
+ if (remainingAttackers.isEmpty()) {
+ break;
+ }
blockedAttackers.add(remainingAttackers.get(0));
remainingAttackers.remove(0);
maxBlockersAfterCrew--;
@@ -681,7 +684,7 @@ public class AiAttackController {
// Determine who will be attacked
GameEntity defender = this.chooseDefender(combat, bAssault);
- List attackersLeft = new ArrayList(this.attackers);
+ List attackersLeft = new ArrayList<>(this.attackers);
// TODO probably use AttackConstraints instead of only GlobalAttackRestrictions?
GlobalAttackRestrictions restrict = GlobalAttackRestrictions.getGlobalRestrictions(ai, combat.getDefenders());
@@ -821,12 +824,12 @@ public class AiAttackController {
int humanForcesForAttritionalAttack = 0;
// examine the potential forces
- final List nextTurnAttackers = new ArrayList();
+ final List nextTurnAttackers = new ArrayList<>();
int candidateCounterAttackDamage = 0;
final Player opp = this.defendingOpponent;
// get the potential damage and strength of the AI forces
- final List candidateAttackers = new ArrayList();
+ final List candidateAttackers = new ArrayList<>();
int candidateUnblockedDamage = 0;
for (final Card pCard : this.myList) {
// if the creature can attack then it's a potential attacker this
@@ -885,7 +888,7 @@ public class AiAttackController {
final int outNumber = computerForces - humanForces;
for (Card blocker : this.blockers) {
- if (blocker.hasKeyword("CARDNAME can block any number of creatures.")) {
+ if (blocker.canBlockAny()) {
aiLifeToPlayerDamageRatio--;
}
}
@@ -908,7 +911,7 @@ public class AiAttackController {
// get player life total
int humanLife = opp.getLife();
// get the list of attackers up to the first blocked one
- final List attritionalAttackers = new ArrayList();
+ final List attritionalAttackers = new ArrayList<>();
for (int x = 0; x < (this.attackers.size() - humanForces); x++) {
attritionalAttackers.add(this.attackers.get(x));
}
@@ -1021,7 +1024,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 +1460,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..5b2caa36636 100644
--- a/forge-ai/src/main/java/forge/ai/AiBlockController.java
+++ b/forge-ai/src/main/java/forge/ai/AiBlockController.java
@@ -148,9 +148,7 @@ public class AiBlockController {
final CardCollection attackers = combat.getAttackersOf(defender);
// Begin with the attackers that pose the biggest threat
CardLists.sortByPowerDesc(attackers);
- for (final Card c : attackers) {
- sortedAttackers.add(c);
- }
+ sortedAttackers.addAll(attackers);
} else if (defender instanceof Player && defender.equals(ai)) {
firstAttacker = combat.getAttackersOf(defender);
}
@@ -163,9 +161,7 @@ public class AiBlockController {
}
} else {
// add creatures attacking the Player to the back of the list
- for (final Card c : firstAttacker) {
- sortedAttackers.add(c);
- }
+ sortedAttackers.addAll(firstAttacker);
}
return sortedAttackers;
}
@@ -481,8 +477,7 @@ public class AiBlockController {
final int damageNeeded = ComputerUtilCombat.getDamageToKill(attacker)
+ ComputerUtilCombat.predictToughnessBonusOfAttacker(attacker, secondBlocker, combat, false);
- List usableBlockersAsThird = new ArrayList<>();
- usableBlockersAsThird.addAll(usableBlockers);
+ List usableBlockersAsThird = new ArrayList<>(usableBlockers);
usableBlockersAsThird.remove(secondBlocker);
// loop over the remaining blockers in search of a good third blocker candidate
@@ -859,7 +854,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 +874,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 +909,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 +1324,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/AiProfileUtil.java b/forge-ai/src/main/java/forge/ai/AiProfileUtil.java
index cfe4bcd61dc..06f60545f52 100644
--- a/forge-ai/src/main/java/forge/ai/AiProfileUtil.java
+++ b/forge-ai/src/main/java/forge/ai/AiProfileUtil.java
@@ -39,7 +39,7 @@ import java.util.Map;
* @version $Id: AIProfile.java 20169 2013-03-08 08:24:17Z Agetian $
*/
public class AiProfileUtil {
- private static Map> loadedProfiles = new HashMap>();
+ private static Map> loadedProfiles = new HashMap<>();
private static String AI_PROFILE_DIR;
private static final String AI_PROFILE_EXT = ".ai";
@@ -74,7 +74,7 @@ public class AiProfileUtil {
* @param profileName a profile to load.
*/
private static final Map loadProfile(final String profileName) {
- Map profileMap = new HashMap();
+ Map profileMap = new HashMap<>();
List lines = FileUtil.readFile(buildFileName(profileName));
for (String line : lines) {
@@ -122,7 +122,7 @@ public class AiProfileUtil {
*/
public static List getAvailableProfiles()
{
- final List availableProfiles = new ArrayList();
+ final List availableProfiles = new ArrayList<>();
final File dir = new File(AI_PROFILE_DIR);
final String[] children = dir.list();
@@ -146,7 +146,7 @@ public class AiProfileUtil {
* available profiles including special random profile tags.
*/
public static List getProfilesDisplayList() {
- final List availableProfiles = new ArrayList();
+ final List availableProfiles = new ArrayList<>();
availableProfiles.add(AI_PROFILE_RANDOM_MATCH);
availableProfiles.add(AI_PROFILE_RANDOM_DUEL);
availableProfiles.addAll(getAvailableProfiles());
diff --git a/forge-ai/src/main/java/forge/ai/ComputerUtil.java b/forge-ai/src/main/java/forge/ai/ComputerUtil.java
index baf97df8181..a5de6d50c5f 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;
}
});
@@ -1514,7 +1514,7 @@ public class ComputerUtil {
*/
public static List predictThreatenedObjects(final Player ai, final SpellAbility sa, boolean top) {
final Game game = ai.getGame();
- final List objects = new ArrayList();
+ final List objects = new ArrayList<>();
if (game.getStack().isEmpty()) {
return objects;
}
@@ -1543,8 +1543,8 @@ public class ComputerUtil {
private static Iterable extends GameObject> predictThreatenedObjects(final Player aiPlayer, final SpellAbility saviour,
final SpellAbility topStack) {
- Iterable extends GameObject> objects = new ArrayList();
- final List threatened = new ArrayList();
+ Iterable extends GameObject> objects = new ArrayList<>();
+ final List threatened = new ArrayList<>();
ApiType saviourApi = saviour == null ? null : saviour.getApi();
int toughness = 0;
boolean grantIndestructible = false;
@@ -1574,7 +1574,7 @@ public class ComputerUtil {
}
} else {
objects = topStack.getTargets().getTargets();
- final List canBeTargeted = new ArrayList();
+ final List canBeTargeted = new ArrayList<>();
for (Object o : objects) {
if (o instanceof Card) {
final Card c = (Card) o;
@@ -1597,7 +1597,7 @@ public class ComputerUtil {
toughness = saviorWithSubs.hasParam("NumDef") ?
AbilityUtils.calculateAmount(saviorWithSubs.getHostCard(), saviorWithSubs.getParam("NumDef"), saviour) : 0;
final List keywords = saviorWithSubs.hasParam("KW") ?
- Arrays.asList(saviorWithSubs.getParam("KW").split(" & ")) : new ArrayList();
+ Arrays.asList(saviorWithSubs.getParam("KW").split(" & ")) : new ArrayList<>();
if (keywords.contains("Indestructible")) {
grantIndestructible = true;
}
@@ -1630,7 +1630,7 @@ public class ComputerUtil {
final SpellAbility sub = topStack.getSubAbility();
boolean noRegen = false;
if (sub != null && sub.getApi() == ApiType.Pump) {
- final List keywords = sub.hasParam("KW") ? Arrays.asList(sub.getParam("KW").split(" & ")) : new ArrayList();
+ final List keywords = sub.hasParam("KW") ? Arrays.asList(sub.getParam("KW").split(" & ")) : new ArrayList<>();
for (String kw : keywords) {
if (kw.contains("can't be regenerated")) {
noRegen = true;
@@ -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;
}
});
@@ -2059,7 +2053,7 @@ public class ComputerUtil {
//Too many lands!
//Init
int cntColors = MagicColor.WUBRG.length;
- List numProducers = new ArrayList(cntColors);
+ List numProducers = new ArrayList<>(cntColors);
for (byte col : MagicColor.WUBRG) {
numProducers.add(col, new CardCollection());
}
@@ -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()) {
@@ -2205,7 +2196,7 @@ public class ComputerUtil {
}
}
}
-
+
Collections.sort(goodChoices, CardLists.TextLenComparator);
CardLists.sortByCmcDesc(goodChoices);
@@ -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();
@@ -2292,8 +2283,7 @@ public class ComputerUtil {
chosen = ComputerUtilCard.getMostProminentType(list, valid);
} else if (logic.equals("MostNeededType")) {
// Choose a type that is in the deck, but not in hand or on the battlefield
- final List basics = new ArrayList();
- basics.addAll(CardType.Constant.BASIC_TYPES);
+ final List basics = new ArrayList<>(CardType.Constant.BASIC_TYPES);
CardCollectionView presentCards = CardCollection.combine(ai.getCardsIn(ZoneType.Battlefield), ai.getCardsIn(ZoneType.Hand));
CardCollectionView possibleCards = ai.getAllCards();
@@ -2546,8 +2536,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;
}
@@ -2598,7 +2587,7 @@ public class ComputerUtil {
int damage = 0;
final Game game = player.getGame();
final Card card = sa.getHostCard();
- final FCollection theTriggers = new FCollection();
+ final FCollection theTriggers = new FCollection<>();
for (Card c : game.getCardsIn(ZoneType.Battlefield)) {
theTriggers.addAll(c.getTriggers());
@@ -2690,7 +2679,7 @@ public class ComputerUtil {
public static int getDamageFromETB(final Player player, final Card permanent) {
int damage = 0;
final Game game = player.getGame();
- final FCollection theTriggers = new FCollection();
+ final FCollection theTriggers = new FCollection<>();
for (Card card : game.getCardsIn(ZoneType.Battlefield)) {
theTriggers.addAll(card.getTriggers());
@@ -2875,7 +2864,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 f562e7a3a01..bb8bc575f55 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");
@@ -564,7 +564,7 @@ public class ComputerUtilCard {
AiBlockController aiBlk = new AiBlockController(ai);
Combat combat = new Combat(ai);
combat.addAttacker(attacker, ai);
- final List attackers = new ArrayList();
+ final List attackers = new ArrayList<>();
attackers.add(attacker);
aiBlk.assignBlockersGivenAttackers(combat, attackers);
return ComputerUtilCombat.attackerWouldBeDestroyed(ai, attacker, combat);
@@ -788,7 +788,7 @@ public class ComputerUtilCard {
public static List getColorByProminence(final List list) {
int cntColors = MagicColor.WUBRG.length;
- final List> map = new ArrayList>();
+ final List> map = new ArrayList<>();
for(int i = 0; i < cntColors; i++) {
map.add(MutablePair.of(MagicColor.WUBRG[i], 0));
}
@@ -809,7 +809,7 @@ public class ComputerUtilCard {
});
// will this part be once dropped?
- List result = new ArrayList(cntColors);
+ List result = new ArrayList<>(cntColors);
for(Pair idx : map) { // fetch color names in the same order
result.add(MagicColor.toLongString(idx.getKey()));
}
@@ -881,7 +881,7 @@ public class ComputerUtilCard {
}
};
public static List chooseColor(SpellAbility sa, int min, int max, List colorChoices) {
- List chosen = new ArrayList();
+ List chosen = new ArrayList<>();
Player ai = sa.getActivatingPlayer();
final Game game = ai.getGame();
Player opp = ai.getWeakestOpponent();
@@ -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;
@@ -1571,7 +1571,7 @@ public class ComputerUtilCard {
Card pumped = CardFactory.copyCard(c, true);
pumped.setSickness(c.hasSickness());
final long timestamp = c.getGame().getNextTimestamp();
- final List kws = new ArrayList();
+ final List kws = new ArrayList<>();
for (String kw : keywords) {
if (kw.startsWith("HIDDEN")) {
pumped.addHiddenExtrinsicKeyword(kw);
@@ -1601,8 +1601,8 @@ public class ComputerUtilCard {
}
pumped.addNewPT(c.getCurrentPower(), c.getCurrentToughness(), timestamp);
- pumped.addTempPowerBoost(c.getTempPowerBoost() + power + berserkPower);
- pumped.addTempToughnessBoost(c.getTempToughnessBoost() + toughness);
+ pumped.setPTBoost(c.getPTBoostMap());
+ pumped.addPTBoost(power + berserkPower, toughness, timestamp);
pumped.addChangedCardKeywords(kws, null, false, false, timestamp);
Set types = c.getCounters().keySet();
for(CounterType ct : types) {
@@ -1648,6 +1648,8 @@ public class ComputerUtilCard {
}
list.add(vCard); // account for the static abilities that may be present on the card itself
for (final Card c : list) {
+ // remove old boost that might be copied
+ vCard.removePTBoost(c.getTimestamp());
for (final StaticAbility stAb : c.getStaticAbilities()) {
final Map params = stAb.getMapParams();
if (!params.get("Mode").equals("Continuous")) {
@@ -1663,26 +1665,25 @@ public class ComputerUtilCard {
if (!vCard.isValid(valid, c.getController(), c, null)) {
continue;
}
+ int att = 0;
if (params.containsKey("AddPower")) {
String addP = params.get("AddPower");
- int att = 0;
if (addP.equals("AffectedX")) {
att = CardFactoryUtil.xCount(vCard, AbilityUtils.getSVar(stAb, addP));
} else {
att = AbilityUtils.calculateAmount(c, addP, stAb);
}
- vCard.addTempPowerBoost(att);
}
+ int def = 0;
if (params.containsKey("AddToughness")) {
String addT = params.get("AddToughness");
- int def = 0;
if (addT.equals("AffectedY")) {
def = CardFactoryUtil.xCount(vCard, AbilityUtils.getSVar(stAb, addT));
} else {
def = AbilityUtils.calculateAmount(c, addT, stAb);
}
- vCard.addTempToughnessBoost(def);
}
+ vCard.addPTBoost(att, def, c.getTimestamp());
}
}
}
diff --git a/forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java b/forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java
index 4a38704d130..262d63705e9 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;
@@ -970,7 +965,7 @@ public class ComputerUtilCombat {
}
}
- final FCollection theTriggers = new FCollection();
+ final FCollection theTriggers = new FCollection<>();
for (Card card : game.getCardsIn(ZoneType.Battlefield)) {
theTriggers.addAll(card.getTriggers());
}
@@ -1105,7 +1100,7 @@ public class ComputerUtilCombat {
}
final Game game = attacker.getGame();
- final FCollection theTriggers = new FCollection();
+ final FCollection theTriggers = new FCollection<>();
for (Card card : game.getCardsIn(ZoneType.Battlefield)) {
theTriggers.addAll(card.getTriggers());
}
@@ -1289,7 +1284,7 @@ public class ComputerUtilCombat {
}
final Game game = attacker.getGame();
- final FCollection theTriggers = new FCollection();
+ final FCollection theTriggers = new FCollection<>();
for (Card card : game.getCardsIn(ZoneType.Battlefield)) {
theTriggers.addAll(card.getTriggers());
}
@@ -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
@@ -1510,7 +1505,7 @@ public class ComputerUtilCombat {
}
final Game game = attacker.getGame();
- final FCollection theTriggers = new FCollection();
+ final FCollection theTriggers = new FCollection<>();
for (Card card : game.getCardsIn(ZoneType.Battlefield)) {
theTriggers.addAll(card.getTriggers());
}
@@ -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
@@ -1731,7 +1726,7 @@ public class ComputerUtilCombat {
}
// check Destroy triggers (Cockatrice and friends)
- final FCollection theTriggers = new FCollection();
+ final FCollection theTriggers = new FCollection<>();
for (Card card : attacker.getGame().getCardsIn(ZoneType.Battlefield)) {
theTriggers.addAll(card.getTriggers());
}
@@ -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
@@ -2011,7 +2006,7 @@ public class ComputerUtilCombat {
}
final Game game = blocker.getGame();
- final FCollection theTriggers = new FCollection();
+ final FCollection theTriggers = new FCollection<>();
for (Card card : game.getCardsIn(ZoneType.Battlefield)) {
theTriggers.addAll(card.getTriggers());
}
@@ -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..1a1b9fc6df5 100644
--- a/forge-ai/src/main/java/forge/ai/ComputerUtilMana.java
+++ b/forge-ai/src/main/java/forge/ai/ComputerUtilMana.java
@@ -170,8 +170,7 @@ public class ComputerUtilMana {
}
return ability1.compareTo(ability2);
- }
- else {
+ } else {
return preOrder;
}
}
@@ -344,6 +343,10 @@ public class ComputerUtilMana {
payMultipleMana(cost, manaProduced, ai);
// remove from available lists
+ /*
+ * Refactoring this code to sourcesForShards.values().removeIf((SpellAbility srcSa) -> srcSa.getHostCard().equals(saPayment.getHostCard()));
+ * causes Android build not to compile
+ * */
Iterator itSa = sourcesForShards.values().iterator();
while (itSa.hasNext()) {
SpellAbility srcSa = itSa.next();
@@ -362,7 +365,7 @@ public class ComputerUtilMana {
private static boolean payManaCost(final ManaCostBeingPaid cost, final SpellAbility sa, final Player ai, final boolean test, boolean checkPlayable) {
adjustManaCostToAvoidNegEffects(cost, sa.getHostCard(), ai);
- List manaSpentToPay = test ? new ArrayList() : sa.getPayingMana();
+ List manaSpentToPay = test ? new ArrayList<>() : sa.getPayingMana();
boolean purePhyrexian = cost.containsOnlyPhyrexianMana();
int testEnergyPool = ai.getCounters(CounterType.ENERGY);
@@ -487,6 +490,10 @@ public class ComputerUtilMana {
payMultipleMana(cost, manaProduced, ai);
// remove from available lists
+ /*
+ * Refactoring this code to sourcesForShards.values().removeIf((SpellAbility srcSa) -> srcSa.getHostCard().equals(saPayment.getHostCard()));
+ * causes Android build not to compile
+ * */
Iterator itSa = sourcesForShards.values().iterator();
while (itSa.hasNext()) {
SpellAbility srcSa = itSa.next();
@@ -517,6 +524,10 @@ public class ComputerUtilMana {
if (hasConverge) { // hack to prevent converge re-using sources
// remove from available lists
+ /*
+ * Refactoring this code to sourcesForShards.values().removeIf((SpellAbility srcSa) -> srcSa.getHostCard().equals(saPayment.getHostCard()));
+ * causes Android build not to compile
+ * */
Iterator itSa = sourcesForShards.values().iterator();
while (itSa.hasNext()) {
SpellAbility srcSa = itSa.next();
@@ -880,10 +891,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))) {
@@ -1346,7 +1359,7 @@ public class ComputerUtilMana {
final ListMultimap manaMap = ArrayListMultimap.create();
final Game game = ai.getGame();
- List replacementEffects = new ArrayList();
+ List replacementEffects = new ArrayList<>();
for (final Player p : game.getPlayers()) {
for (final Card crd : p.getAllCards()) {
for (final ReplacementEffect replacementEffect : crd.getReplacementEffects()) {
@@ -1552,7 +1565,7 @@ public class ComputerUtilMana {
* @return map between creatures and shards to convoke
*/
public static Map getConvokeOrImproviseFromList(final ManaCost cost, List list, boolean improvise) {
- final Map convoke = new HashMap();
+ final Map convoke = new HashMap<>();
Card convoked = null;
if (!improvise) {
for (ManaCostShard toPay : cost) {
diff --git a/forge-ai/src/main/java/forge/ai/GameState.java b/forge-ai/src/main/java/forge/ai/GameState.java
index a7477639ce4..6edd9ffb70e 100644
--- a/forge-ai/src/main/java/forge/ai/GameState.java
+++ b/forge-ai/src/main/java/forge/ai/GameState.java
@@ -41,7 +41,7 @@ import java.util.*;
import java.util.Map.Entry;
public abstract class GameState {
- private static final Map ZONES = new HashMap();
+ private static final Map ZONES = new HashMap<>();
static {
ZONES.put(ZoneType.Battlefield, "battlefield");
ZONES.put(ZoneType.Hand, "hand");
@@ -66,8 +66,8 @@ public abstract class GameState {
private boolean puzzleCreatorState = false;
- private final Map humanCardTexts = new EnumMap(ZoneType.class);
- private final Map aiCardTexts = new EnumMap(ZoneType.class);
+ private final Map humanCardTexts = new EnumMap<>(ZoneType.class);
+ private final Map aiCardTexts = new EnumMap<>(ZoneType.class);
private final Map idToCard = new HashMap<>();
private final Map cardToAttachId = new HashMap<>();
@@ -254,7 +254,7 @@ public abstract class GameState {
newText.append(";");
}
if (c.isToken()) {
- newText.append("t:" + new TokenInfo(c).toString());
+ newText.append("t:").append(new TokenInfo(c).toString());
} else {
if (c.getPaperCard() == null) {
return;
@@ -377,7 +377,7 @@ public abstract class GameState {
newText.append("|Attacking");
GameEntity def = c.getGame().getCombat().getDefenderByAttacker(c);
if (def instanceof Card) {
- newText.append(":" + def.getId());
+ newText.append(":").append(def.getId());
}
}
}
@@ -653,15 +653,15 @@ public abstract class GameState {
}
private String processManaPool(ManaPool manaPool) {
- String mana = "";
+ StringBuilder mana = new StringBuilder();
for (final byte c : MagicColor.WUBRGC) {
int amount = manaPool.getAmountOfColor(c);
for (int i = 0; i < amount; i++) {
- mana += MagicColor.toShortString(c) + " ";
+ mana.append(MagicColor.toShortString(c)).append(" ");
}
}
- return mana.trim();
+ return mana.toString().trim();
}
private void updateManaPool(Player p, String manaDef, boolean clearPool, boolean persistent) {
@@ -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);
@@ -1078,7 +1078,7 @@ public abstract class GameState {
p.getZone(zt).removeAllCards(true);
}
- Map playerCards = new EnumMap(ZoneType.class);
+ Map playerCards = new EnumMap<>(ZoneType.class);
for (Entry kv : cardTexts.entrySet()) {
String value = kv.getValue();
playerCards.put(kv.getKey(), processCardsForZone(value.isEmpty() ? new String[0] : value.split(";"), p));
@@ -1091,7 +1091,7 @@ public abstract class GameState {
for (Entry kv : playerCards.entrySet()) {
PlayerZone zone = p.getZone(kv.getKey());
if (kv.getKey() == ZoneType.Battlefield) {
- List cards = new ArrayList();
+ List cards = new ArrayList<>();
for (final Card c : kv.getValue()) {
if (c.isToken()) {
cards.add(c);
@@ -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..3f461395650 100644
--- a/forge-ai/src/main/java/forge/ai/PlayerControllerAi.java
+++ b/forge-ai/src/main/java/forge/ai/PlayerControllerAi.java
@@ -168,8 +168,8 @@ public class PlayerControllerAi extends PlayerController {
if (delayedReveal != null) {
reveal(delayedReveal.getCards(), delayedReveal.getZone(), delayedReveal.getOwner(), delayedReveal.getMessagePrefix());
}
- FCollection remaining = new FCollection(optionList);
- List selecteds = new ArrayList();
+ FCollection remaining = new FCollection<>(optionList);
+ List selecteds = new ArrayList<>();
T selected;
do {
selected = chooseSingleEntityForEffect(remaining, null, sa, title, selecteds.size()>=min, targetedPlayer);
@@ -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) {
@@ -1072,7 +1072,7 @@ public class PlayerControllerAi extends PlayerController {
}
});
} else {
- return new HashMap();
+ return new HashMap<>();
}
}
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 ae728a26dc7..bbf17fa1069 100644
--- a/forge-ai/src/main/java/forge/ai/ability/AttachAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/AttachAi.java
@@ -31,10 +31,7 @@ import forge.game.trigger.TriggerType;
import forge.game.zone.ZoneType;
import forge.util.MyRandom;
-import java.util.ArrayList;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
+import java.util.*;
public class AttachAi extends SpellAbilityAi {
@@ -123,9 +120,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 +234,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;
@@ -455,7 +448,7 @@ public class AttachAi extends SpellAbilityAi {
*/
private static Player attachToPlayerAIPreferences(final Player aiPlayer, final SpellAbility sa,
final boolean mandatory) {
- List targetable = new ArrayList();
+ List targetable = new ArrayList<>();
for (final Player player : aiPlayer.getGame().getPlayers()) {
if (sa.canTarget(player)) {
targetable.add(player);
@@ -859,7 +852,7 @@ public class AttachAi extends SpellAbilityAi {
int totToughness = 0;
int totPower = 0;
- final List keywords = new ArrayList();
+ final List keywords = new ArrayList<>();
for (final StaticAbility stAbility : attachSource.getStaticAbilities()) {
final Map stabMap = stAbility.getMapParams();
@@ -879,15 +872,11 @@ public class AttachAi extends SpellAbilityAi {
String kws = stabMap.get("AddKeyword");
if (kws != null) {
- for (final String kw : kws.split(" & ")) {
- keywords.add(kw);
- }
+ keywords.addAll(Arrays.asList(kws.split(" & ")));
}
kws = stabMap.get("AddHiddenKeyword");
if (kws != null) {
- for (final String kw : kws.split(" & ")) {
- keywords.add(kw);
- }
+ keywords.addAll(Arrays.asList(kws.split(" & ")));
}
}
}
@@ -910,7 +899,7 @@ public class AttachAi extends SpellAbilityAi {
Card c = null;
if (prefList == null || prefList.isEmpty()) {
- prefList = new ArrayList(list);
+ prefList = new ArrayList<>(list);
} else {
c = ComputerUtilCard.getBestAI(prefList);
if (c != null) {
@@ -964,7 +953,7 @@ public class AttachAi extends SpellAbilityAi {
protected boolean doTriggerAINoCost(final Player ai, final SpellAbility sa, final boolean mandatory) {
final Card card = sa.getHostCard();
// Check if there are any valid targets
- List targets = new ArrayList();
+ List targets = new ArrayList<>();
final TargetRestrictions tgt = sa.getTargetRestrictions();
if (tgt == null) {
targets = AbilityUtils.getDefinedObjects(sa.getHostCard(), sa.getParam("Defined"), sa);
@@ -987,9 +976,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());
}
}
@@ -1156,8 +1143,9 @@ public class AttachAi extends SpellAbilityAi {
int totToughness = 0;
int totPower = 0;
- final List keywords = new ArrayList();
+ final List keywords = new ArrayList<>();
boolean grantingAbilities = false;
+ boolean grantingExtraBlock = false;
for (final StaticAbility stAbility : attachSource.getStaticAbilities()) {
final Map stabMap = stAbility.getMapParams();
@@ -1176,18 +1164,15 @@ public class AttachAi extends SpellAbilityAi {
totPower += AbilityUtils.calculateAmount(attachSource, stabMap.get("AddPower"), stAbility);
grantingAbilities |= stabMap.containsKey("AddAbility");
+ grantingExtraBlock |= stabMap.containsKey("CanBlockAmount") || stabMap.containsKey("CanBlockAny");
String kws = stabMap.get("AddKeyword");
if (kws != null) {
- for (final String kw : kws.split(" & ")) {
- keywords.add(kw);
- }
+ keywords.addAll(Arrays.asList(kws.split(" & ")));
}
kws = stabMap.get("AddHiddenKeyword");
if (kws != null) {
- for (final String kw : kws.split(" & ")) {
- keywords.add(kw);
- }
+ keywords.addAll(Arrays.asList(kws.split(" & ")));
}
}
}
@@ -1209,20 +1194,27 @@ public class AttachAi extends SpellAbilityAi {
}
//only add useful keywords unless P/T bonus is significant
- if (totToughness + totPower < 4 && !keywords.isEmpty()) {
+ if (totToughness + totPower < 4 && (!keywords.isEmpty() || grantingExtraBlock)) {
final int pow = totPower;
+ final boolean extraBlock = grantingExtraBlock;
prefList = CardLists.filter(prefList, new Predicate() {
@Override
public boolean apply(final Card c) {
- for (final String keyword : keywords) {
- if (isUsefulAttachKeyword(keyword, c, sa, pow)) {
- return true;
- }
- if (c.hasKeyword(Keyword.INFECT) && pow >= 2) {
- // consider +2 power a significant bonus on Infect creatures
- return true;
+ if (!keywords.isEmpty()) {
+ for (final String keyword : keywords) {
+ if (isUsefulAttachKeyword(keyword, c, sa, pow)) {
+ return true;
+ }
}
}
+
+ if (c.hasKeyword(Keyword.INFECT) && pow >= 2) {
+ // consider +2 power a significant bonus on Infect creatures
+ return true;
+ }
+ if (extraBlock && CombatUtil.canBlock(c, true) && !c.canBlockAny()) {
+ return true;
+ }
return false;
}
});
@@ -1353,7 +1345,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 +1549,52 @@ 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;
- }
- } 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 !card.hasKeyword(Keyword.FLYING) && CombatUtil.canBlock(card, true);
} 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 +1615,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 +1628,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 +1657,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..0a747747895 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;
}
@@ -484,7 +482,7 @@ public class ChangeZoneAi extends SpellAbilityAi {
}
}
- List origin = new ArrayList();
+ List origin = new ArrayList<>();
if (sa.hasParam("Origin")) {
origin = ZoneType.listValueOf(sa.getParam("Origin"));
}
@@ -559,7 +557,7 @@ public class ChangeZoneAi extends SpellAbilityAi {
*/
private static Card basicManaFixing(final Player ai, final List list) { // Search for a Basic Land
final CardCollectionView combined = CardCollection.combine(ai.getCardsIn(ZoneType.Battlefield), ai.getCardsIn(ZoneType.Hand));
- final List basics = new ArrayList();
+ final List basics = new ArrayList<>();
// what types can I go get?
for (final String name : MagicColor.Constant.BASIC_LANDS) {
@@ -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();
@@ -1291,7 +1281,7 @@ public class ChangeZoneAi extends SpellAbilityAi {
final List objects = ComputerUtil
.predictThreatenedObjects(ai, sa);
- final List threatenedTargets = new ArrayList();
+ final List threatenedTargets = new ArrayList<>();
for (final Card c : aiPermanents) {
if (objects.contains(c)) {
@@ -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..1f4e627bda8 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ChooseGenericEffectAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ChooseGenericEffectAi.java
@@ -103,7 +103,7 @@ public class ChooseGenericEffectAi extends SpellAbilityAi {
Cost unless = new Cost(unlessCost, false);
SpellAbility paycost = new SpellAbility.EmptySa(sa.getHostCard(), player);
paycost.setPayCosts(unless);
- if (ComputerUtilCost.willPayUnlessCost(sp, player, unless, false, new FCollection(player))
+ if (ComputerUtilCost.willPayUnlessCost(sp, player, unless, false, new FCollection<>(player))
&& ComputerUtilCost.canPayCost(paycost, player)) {
return sp;
}
@@ -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/DamagePreventAi.java b/forge-ai/src/main/java/forge/ai/ability/DamagePreventAi.java
index 84bb04ebaf9..c80e1db1a93 100644
--- a/forge-ai/src/main/java/forge/ai/ability/DamagePreventAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/DamagePreventAi.java
@@ -100,7 +100,7 @@ public class DamagePreventAi extends SpellAbilityAi {
tcs.add(ai);
chance = true;
}
- final List threatenedTargets = new ArrayList();
+ final List threatenedTargets = new ArrayList<>();
// filter AIs battlefield by what I can target
List targetables = CardLists.getValidCards(ai.getCardsIn(ZoneType.Battlefield), tgt.getValidTgts(), ai, hostCard, sa);
targetables = CardLists.getTargetableCards(targetables, sa);
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..37bcd7c2d33 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:
@@ -266,7 +266,7 @@ public class DebuffAi extends SpellAbilityAi {
@Override
protected boolean doTriggerAINoCost(Player ai, SpellAbility sa, boolean mandatory) {
- final List kws = sa.hasParam("Keywords") ? Arrays.asList(sa.getParam("Keywords").split(" & ")) : new ArrayList();
+ final List kws = sa.hasParam("Keywords") ? Arrays.asList(sa.getParam("Keywords").split(" & ")) : new ArrayList<>();
if (sa.getTargetRestrictions() == null) {
if (mandatory) {
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..23ee0a4712e 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;
@@ -184,15 +182,11 @@ public class LifeLoseAi extends SpellAbilityAi {
}
final List tgtPlayers = sa.usesTargeting() && !sa.hasParam("Defined")
- ? new FCollection(sa.getTargets().getTargetPlayers())
+ ? 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..51db49de678 100644
--- a/forge-ai/src/main/java/forge/ai/ability/ProtectAi.java
+++ b/forge-ai/src/main/java/forge/ai/ability/ProtectAi.java
@@ -30,7 +30,7 @@ import forge.util.MyRandom;
public class ProtectAi extends SpellAbilityAi {
private static boolean hasProtectionFrom(final Card card, final String color) {
- final List onlyColors = new ArrayList