Enchant land auras

[Chris H.]

I noticed something with the auras that enchant a land. The AI code will enchant the first land that came into play. As such, it had 3 Squirrel Nest auras in play but they all appeared on the same land.

I changed the AI code to only add non-enchanted lands to the list of lands and this resulted in the auras being attached to different lands. But in the various tests that I ran the stacking of lands into piles became out of sync. I seem to remember seeing this in one game. Not sure.

So what do people think? Does Dennis (MV) have any input … seeing that he is the one who designed the original aura system. There may be something that I am missing or this could be another case of us being human and we run out of steam. And then at a later time someone notices something strange and we scratch our heads as we ponder the greater meaning?

There currently are several aura cards and they all appear to enchant the same land rather than placing them on different lands.

[DennisBergkamp]

Hmm, yeah it's true I implemented this awhile back... I really can't remember the details of this though. Changing the AI for this to only consider lands that are not currently enchanted is a good enough solution, I think.
But why the stacking gets screwed up, I'm not sure? I'll see if I can reproduce this...

[Chris H.]

This is what I have at the moment for a new canPlayAI() method for Squirrel Nest and it works really well from what I can tell with my testing. I will go ahead and commit it.

Code: Select all

public boolean canPlayAI() {

    CardList list = new CardList(AllZone.Computer_Play.getCards());
    list = list.filter(new CardListFilter() {
        public boolean addCard(Card c) {
            return c.isLand() && !c.isEnchanted() && CardFactoryUtil.canTarget(card, c);
        }
    });

    if (list.isEmpty()) return false;
    else {
        list.shuffle();
        setTargetCard(list.get(0));
        return true;
    }
}//canPlayAI()

`
I almost added additional tests into this but at some point I tend to lose sight of what my code is trying to do. Old saying, keep it simple stupid.