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

- Simplified the code found in forge.card.cardFactory.CardFactory_Soceries and forge.card.cardFactory.CardFactory. Updating c.getType().contains(string) to c.isType(string).

Browse Source
This commit is contained in:
jendave
2011-08-07 01:08:09 +00:00
parent 48ad24d0e9
commit ad3fb8a895
2 changed files with 44 additions and 42 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
src/forge/card/cardFactory/CardFactory.java
View File

@@ -139,27 +139,27 @@ public class CardFactory implements NewConstants {
final public Card dynamicCopyCard(Card in)
{
if(in.getType().contains("Creature")) {
if(in.isType("Creature")) {
Card card2 = new Card();
card2 = CardFactory_Creatures.getCard(in, in.getName(), in.getOwner(), this);
return card2;
} else if(in.getType().contains("Aura")) {
} else if(in.isType("Aura")) {
Card card2 = new Card();
card2 = CardFactory_Auras.getCard(in, in.getName(), in.getOwner());
return card2;
} else if(in.getType().contains("Equipment")) {
} else if(in.isType("Equipment")) {
Card card2 = new Card();
card2 = CardFactory_Equipment.getCard(in, in.getName(), in.getOwner());
return card2;
} else if(in.getType().contains("Planeswalker")) {
} else if(in.isType("Planeswalker")) {
Card card2 = new Card();
card2 = CardFactory_Planeswalkers.getCard(in, in.getName(), in.getOwner());
return card2;
} else if(in.getType().contains("Land")) {
} else if(in.isType("Land")) {
Card card2 = new Card();
card2 = CardFactory_Lands.getCard(in, in.getName(), in.getOwner());
@@ -416,7 +416,7 @@ public class CardFactory implements NewConstants {
{
Card c = clICtrl.get(i);
for (int j = 0; j < types.length; j++)
if (c.getType().contains(types[j].trim()))
if (c.isType(types[j].trim()))
fnd = true;
}
@@ -819,19 +819,19 @@ public class CardFactory implements NewConstants {
//******************************************************************
//************** Link to different CardFactories *******************
Card card2 = null;
if(card.getType().contains("Creature")) {
if(card.isType("Creature")) {
card2 = CardFactory_Creatures.getCard(card, cardName, owner, this);
} else if(card.getType().contains("Aura")) {
} else if(card.isType("Aura")) {
card2 = CardFactory_Auras.getCard(card, cardName, owner);
} else if(card.getType().contains("Equipment")) {
} else if(card.isType("Equipment")) {
card2 = CardFactory_Equipment.getCard(card, cardName, owner);
} else if(card.getType().contains("Planeswalker")) {
} else if(card.isType("Planeswalker")) {
card2 = CardFactory_Planeswalkers.getCard(card, cardName, owner);
} else if(card.getType().contains("Land")) {
} else if(card.isType("Land")) {
card2 = CardFactory_Lands.getCard(card, cardName, owner);
} else if (card.getType().contains("Instant")) {
} else if (card.isType("Instant")) {
card2 = CardFactory_Instants.getCard(card, cardName, owner);
} else if (card.getType().contains("Sorcery")) {
} else if (card.isType("Sorcery")) {
card2 = CardFactory_Sorceries.getCard(card, cardName, owner);
}
@@ -1443,7 +1443,7 @@ public class CardFactory implements NewConstants {
CardList creatures = new CardList();
for(int i = 0; i < hand.size(); i++) {
if(hand.get(i).getType().contains("Creature")
if (hand.get(i).isType("Creature")
&& CardUtil.getConvertedManaCost(hand.get(i).getManaCost()) <= 3) creatures.add(hand.get(i));
}

42
src/forge/card/cardFactory/CardFactory_Sorceries.java
View File

@@ -1097,7 +1097,9 @@ public class CardFactory_Sorceries {
@Override
public void selectCard(Card c, PlayerZone zone) {
if(c.isLand() && zone.is(Constant.Zone.Battlefield) && !c.getType().contains("Basic")) {
if (c.isLand()
&& zone.is(Constant.Zone.Battlefield)
&& !c.isType("Basic")) {
if (card.isCopiedSpell()) card.getChoiceTargets().remove(0);
m_land[0] = c;
spell.setTargetCard(c);
@@ -1330,11 +1332,11 @@ public class CardFactory_Sorceries {
land = land.filter(new CardListFilter() {
public boolean addCard(Card c){
if (c.getName().contains("Dryad Arbor")) return true;
else if (!(c.getType().contains("Forest")
|| c.getType().contains("Plains")
|| c.getType().contains("Mountain")
|| c.getType().contains("Island")
|| c.getType().contains("Swamp"))) return true;
else if (!(c.isType("Forest")
|| c.isType("Plains")
|| c.isType("Mountain")
|| c.isType("Island")
|| c.isType("Swamp"))) return true;
else return false;
}
});
@@ -1380,7 +1382,7 @@ public class CardFactory_Sorceries {
if(c.isLand() && zone.is(Constant.Zone.Battlefield)
&& c.getController().isHuman()
/*&& c.getName().equals(humanBasic.get(count))*/
&& c.getType().contains(humanBasic.get(count))
&& c.isType(humanBasic.get(count))
/*&& !saveList.contains(c) */) {
//get all other basic[count] lands human player controls and add them to target
CardList land = AllZoneUtil.getPlayerLandsInPlay(AllZone.HumanPlayer);
@@ -1408,11 +1410,11 @@ public class CardFactory_Sorceries {
land = land.filter(new CardListFilter() {
public boolean addCard(Card c){
if (c.getName().contains("Dryad Arbor")) return true;
else if (!(c.getType().contains("Forest")
|| c.getType().contains("Plains")
|| c.getType().contains("Mountain")
|| c.getType().contains("Island")
|| c.getType().contains("Swamp"))) return true;
else if (!(c.isType("Forest")
|| c.isType("Plains")
|| c.isType("Mountain")
|| c.isType("Island")
|| c.isType("Swamp"))) return true;
else return false;
}
});
@@ -1495,11 +1497,11 @@ public class CardFactory_Sorceries {
c2.getController().discard(c2, null);
if(c1.getType().contains("Land")) {
if(c1.isType("Land")) {
AllZone.HumanPlayer.gainLife(3, card);
}
if(c2.getType().contains("Land")) {
if(c2.isType("Land")) {
AllZone.HumanPlayer.gainLife(3, card);
}
@@ -1518,7 +1520,7 @@ public class CardFactory_Sorceries {
c.getController().discard(c, null);
if(c.getType().contains("Land")) {
if (c.isType("Land")) {
AllZone.ComputerPlayer.gainLife(3, card);
}
@@ -1530,7 +1532,7 @@ public class CardFactory_Sorceries {
c2.getController().discard(c2, null);
if(c2.getType().contains("Land")) {
if (c2.isType("Land")) {
AllZone.ComputerPlayer.gainLife(3, card);
}
}
@@ -1594,7 +1596,7 @@ public class CardFactory_Sorceries {
CardList creature = AllZoneUtil.getCreaturesInPlay(card.getController());
creature = creature.filter(new CardListFilter() {
public boolean addCard(Card card) {
return (!card.getType().contains("Legendary"));
return (!card.isType("Legendary"));
}
});
if(creature.size() == 0) return false;
@@ -1611,7 +1613,7 @@ public class CardFactory_Sorceries {
CardList creature = AllZoneUtil.getCreaturesInPlay();
creature = creature.filter(new CardListFilter() {
public boolean addCard(Card card) {
return (!card.getType().contains("Legendary"));
return (!card.isType("Legendary"));
}
});
if(creature.size() > 0) {
@@ -1673,7 +1675,7 @@ public class CardFactory_Sorceries {
CardList creature = AllZoneUtil.getCreaturesInPlay(card.getController());
creature = creature.filter(new CardListFilter() {
public boolean addCard(Card card) {
return (!card.getType().contains("Legendary"));
return (!card.isType("Legendary"));
}
});
if(creature.size() == 0) return false;
@@ -1690,7 +1692,7 @@ public class CardFactory_Sorceries {
CardList creature = AllZoneUtil.getCreaturesInPlay(card.getController());
creature = creature.filter(new CardListFilter() {
public boolean addCard(Card card) {
return (!card.getType().contains("Legendary"));
return (!card.isType("Legendary"));
}
});
if (creature.size() > 0) {