external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-18 03:38:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

ControlGainAi: some fixes about using LoseControl parameter

Browse Source
This commit is contained in:
Hanmac
2016-11-08 06:15:30 +00:00
parent 9ac28f4bc8
commit 16d8233560
1 changed files with 21 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
forge-ai/src/main/java/forge/ai/ability/ControlGainAi.java
View File

@@ -17,6 +17,8 @@
*/ */
package forge.ai.ability; package forge.ai.ability;
import java.util.List;
import com.google.common.base.Predicate; import com.google.common.base.Predicate;
import com.google.common.collect.Iterables; import com.google.common.collect.Iterables;
import com.google.common.collect.Lists; import com.google.common.collect.Lists;
@@ -32,15 +34,13 @@ import forge.game.card.CardCollectionView;
import forge.game.card.CardLists; import forge.game.card.CardLists;
import forge.game.phase.PhaseType; import forge.game.phase.PhaseType;
import forge.game.player.Player; import forge.game.player.Player;
import forge.game.player.PlayerPredicates;
import forge.game.spellability.SpellAbility; import forge.game.spellability.SpellAbility;
import forge.game.spellability.TargetRestrictions; import forge.game.spellability.TargetRestrictions;
import forge.game.zone.ZoneType; import forge.game.zone.ZoneType;
import forge.util.Aggregates; import forge.util.Aggregates;
import forge.util.collect.FCollectionView; import forge.util.collect.FCollectionView;
import java.util.Arrays;
import java.util.List;
//AB:GainControl|ValidTgts$Creature|TgtPrompt$Select target legendary creature|LoseControl$Untap,LoseControl|SpellDescription$Gain control of target xxxxxxx //AB:GainControl|ValidTgts$Creature|TgtPrompt$Select target legendary creature|LoseControl$Untap,LoseControl|SpellDescription$Gain control of target xxxxxxx
@@ -69,7 +69,11 @@ public class ControlGainAi extends SpellAbilityAi {
@Override @Override
protected boolean canPlayAI(final Player ai, final SpellAbility sa) { protected boolean canPlayAI(final Player ai, final SpellAbility sa) {
final List<String> lose = sa.hasParam("LoseControl") ? Arrays.asList(sa.getParam("LoseControl").split(",")) : null; final List<String> lose = Lists.newArrayList();
if (sa.hasParam("LoseControl")) {
lose.addAll(Lists.newArrayList(sa.getParam("LoseControl").split(",")));
}
final TargetRestrictions tgt = sa.getTargetRestrictions(); final TargetRestrictions tgt = sa.getTargetRestrictions();
final Game game = ai.getGame(); final Game game = ai.getGame();
@@ -96,22 +100,20 @@ public class ControlGainAi extends SpellAbilityAi {
sa.getTargets().add(Aggregates.random(tgt.getAllCandidates(sa, false))); sa.getTargets().add(Aggregates.random(tgt.getAllCandidates(sa, false)));
} }
if (tgt.canOnlyTgtOpponent()) { if (tgt.canOnlyTgtOpponent()) {
boolean found = false; List<Player> oppList = Lists
for (final Player opp : opponents) { .newArrayList(Iterables.filter(opponents, PlayerPredicates.isTargetableBy(sa)));
if (opp.canBeTargetedBy(sa)) {
sa.getTargets().add(opp); if (oppList.isEmpty()) {
break;
}
}
if (!found) {
return false; return false;
} }
sa.getTargets().add(oppList.get(0));
} }
} }
// Don't steal something if I can't Attack without, or prevent it from // Don't steal something if I can't Attack without, or prevent it from
// blocking at least // blocking at least
if (lose != null && lose.contains("EOT") if (lose.contains("EOT")
&& ai.getGame().getPhaseHandler().getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS) && ai.getGame().getPhaseHandler().getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)
&& !sa.isTrigger()) { && !sa.isTrigger()) {
return false; return false;
@@ -276,9 +278,13 @@ public class ControlGainAi extends SpellAbilityAi {
return false; return false;
} }
} }
final List<String> lose = Lists.newArrayList();
final List<String> lose = sa.hasParam("LoseControl") ? Arrays.asList(sa.getParam("LoseControl").split(",")) : null; if (sa.hasParam("LoseControl")) {
if ((lose != null) && lose.contains("EOT") lose.addAll(Lists.newArrayList(sa.getParam("LoseControl").split(",")));
}
if (lose.contains("EOT")
&& game.getPhaseHandler().getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)) { && game.getPhaseHandler().getPhase().isAfter(PhaseType.COMBAT_DECLARE_ATTACKERS)) {
return false; return false;
} }