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

Optimization to avoid excessive allocations.

Browse Source
This commit is contained in:
Myrd
2014-12-30 21:30:25 +00:00
parent 1a8ab8dfd0
commit 4096643769
1 changed files with 11 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
forge-game/src/main/java/forge/game/GameActionUtil.java
View File

@@ -54,7 +54,16 @@ import java.util.Map;
* @version $Id$
*/
public final class GameActionUtil {
// Cache these instead of generating them on the fly, to avoid excessive allocations every time
// static abilities are checked.
private static final String[] BASIC_LAND_ABILITIES = new String[MagicColor.WUBRG.length];
static {
for (int i = 0; i < MagicColor.WUBRG.length; i++ ) {
String color = MagicColor.toShortString(MagicColor.WUBRG[i]);
BASIC_LAND_ABILITIES[i] = "AB$ Mana | Cost$ T | Produced$ " + color +
" | SpellDescription$ Add {" + color + "} to your mana pool.";
}
}
private GameActionUtil() {
throw new AssertionError();
@@ -130,8 +139,7 @@ public final class GameActionUtil {
// add all appropriate mana abilities based on current types
for (int i = 0; i < MagicColor.WUBRG.length; i++ ) {
String landType = MagicColor.Constant.BASIC_LANDS.get(i);
String color = MagicColor.toShortString(MagicColor.WUBRG[i]);
String abString = "AB$ Mana | Cost$ T | Produced$ " + color + " | SpellDescription$ Add {" + color + "} to your mana pool.";
String abString = BASIC_LAND_ABILITIES[i];
for (final Card land : lands) {
if (land.getType().hasSubtype(landType)) {
final SpellAbility sa = AbilityFactory.getAbility(abString, land);