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

- A [hacky] fix to avoid the AI cheating with Thought Lash (otherwise it keeps exiling 1 card to it, no matter how many age counters are on it, essentially by "exiling" (not) the same card multiple times). A better solution is needed, feel free to improve.

Browse Source
This commit is contained in:
Agetian
2017-10-01 18:10:12 +00:00
parent dab74b7bba
commit c3f7640c4d
2 changed files with 49 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
forge-ai/src/main/java/forge/ai/PlayerControllerAi.java
View File

@@ -25,9 +25,7 @@ import forge.game.ability.ApiType;
import forge.game.card.*;
import forge.game.card.CardPredicates.Presets;
import forge.game.combat.Combat;
import forge.game.cost.Cost;
import forge.game.cost.CostPart;
import forge.game.cost.CostPartMana;
import forge.game.cost.*;
import forge.game.mana.Mana;
import forge.game.mana.ManaCostBeingPaid;
import forge.game.phase.PhaseHandler;
@@ -466,6 +464,24 @@ public class PlayerControllerAi extends PlayerController {
final Ability ability = new AbilityStatic(c, cost, null) { @Override public void resolve() {} };
ability.setActivatingPlayer(c.getController());
// FIXME: This is a hack to check if the AI can play the "exile from library" pay costs (Cumulative Upkeep,
// e.g. Thought Lash). We have to do it early since the cost will later be reconsolidated into a single
// payment for the AI, to avoid cheating (otherwise the AI repeatedly "exiles" the same card multiple times)
int nExileLib = 0;
List<CostPart> parts = CostAdjustment.adjust(cost, sa).getCostParts();
for (final CostPart part : parts) {
if (part instanceof CostExile) {
CostExile exile = (CostExile) part;
if (exile.from == ZoneType.Library) {
nExileLib += exile.convertAmount();
}
}
}
if (nExileLib > c.getController().getCardsIn(ZoneType.Library).size()) {
return false;
}
// - End of hack for Exile a card from library Cumulative Upkeep -
if (ComputerUtilCost.canPayCost(ability, c.getController())) {
ComputerUtil.playNoStack(c.getController(), ability, game);
return true;

33
forge-game/src/main/java/forge/game/cost/CostPayment.java
View File

@@ -17,16 +17,16 @@
*/
package forge.game.cost;
import com.google.common.collect.Lists;
import com.google.common.collect.Maps;
import forge.game.Game;
import forge.game.card.Card;
import forge.game.spellability.SpellAbility;
import forge.game.zone.ZoneType;
import java.util.List;
import java.util.Map;
import com.google.common.collect.Lists;
import com.google.common.collect.Maps;
/**
* <p>
* Cost_Payment class.
@@ -169,6 +169,33 @@ public class CostPayment {
// Set all of the decisions before attempting to pay anything
final Game game = decisionMaker.getPlayer().getGame();
// FIXME: This is a hack which is necessary to make the AI pay the Cumulative Upkeep with "Exile a card from
// the top of your library" (Thought Lash) without cheating. Otherwise the AI pays only one card, no matter
// the age counters. We need a better solution here.
int nExileLib = 0;
List<CostPart> partsToChange = Lists.newArrayList();
CostExile primeExile = null;
for (final CostPart part : parts) {
if (part instanceof CostExile) {
CostExile exile = (CostExile) part;
if (exile.from == ZoneType.Library) {
nExileLib += exile.convertAmount();
if (nExileLib == 1) {
primeExile = exile;
} else {
partsToChange.add(exile);
}
}
}
}
if (nExileLib > 1 && primeExile != null) {
parts.removeAll(partsToChange);
primeExile.setAmount(String.valueOf(nExileLib));
}
// - End of hack for Cumulative Upkeep: Exile a card from the top of your library -
for (final CostPart part : parts) {
PaymentDecision decision = part.accept(decisionMaker);
if (null == decision) return false;