diff --git a/Mage/src/mage/Mana.java b/Mage/src/mage/Mana.java index a6455d3de4c..cff7381fbd6 100644 --- a/Mage/src/mage/Mana.java +++ b/Mage/src/mage/Mana.java @@ -160,9 +160,7 @@ public class Mana implements Comparable, Serializable { for (int i = 0; i < white; i++) sbMana.append("{W}"); if (colorless > 0) { - sbMana.append("{"); - sbMana.append(Integer.toString(colorless)); - sbMana.append("}"); + sbMana.append("{").append(Integer.toString(colorless)).append("}"); } return sbMana.toString(); } diff --git a/Mage/src/mage/abilities/AbilityImpl.java b/Mage/src/mage/abilities/AbilityImpl.java index 187fd4e5d2e..e0393bbc5fc 100644 --- a/Mage/src/mage/abilities/AbilityImpl.java +++ b/Mage/src/mage/abilities/AbilityImpl.java @@ -151,8 +151,7 @@ public abstract class AbilityImpl implements Ability, Serializable { if (!(this instanceof SpellAbility)) { if (!costs.getText().equals("")) { - sbRule.append(costs.getText()); - sbRule.append(": "); + sbRule.append(costs.getText()).append(": "); } } sbRule.append(effects.getText()); diff --git a/Mage/src/mage/abilities/costs/AbstractCosts.java b/Mage/src/mage/abilities/costs/AbstractCosts.java index 8da5dbb1254..6be8cf9078e 100644 --- a/Mage/src/mage/abilities/costs/AbstractCosts.java +++ b/Mage/src/mage/abilities/costs/AbstractCosts.java @@ -54,8 +54,7 @@ public abstract class AbstractCosts extends ArrayList impleme StringBuilder sbText = new StringBuilder(); for (T cost: this) { - sbText.append(cost.getText()); - sbText.append(", "); + sbText.append(cost.getText()).append(", "); } sbText.setLength(sbText.length() - 2); return sbText.toString(); diff --git a/Mage/src/mage/abilities/costs/CostsImpl.java b/Mage/src/mage/abilities/costs/CostsImpl.java index e468f9837df..5e23d078943 100644 --- a/Mage/src/mage/abilities/costs/CostsImpl.java +++ b/Mage/src/mage/abilities/costs/CostsImpl.java @@ -58,8 +58,7 @@ public class CostsImpl extends ArrayList implements Costs StringBuilder sbText = new StringBuilder(); for (T cost: this) { - sbText.append(cost.getText()); - sbText.append(", "); + sbText.append(cost.getText()).append(", "); } sbText.setLength(sbText.length() - 2); return sbText.toString(); diff --git a/Mage/src/mage/abilities/effects/ContinuousEffects.java b/Mage/src/mage/abilities/effects/ContinuousEffects.java index f77b76b452e..f7479418aab 100644 --- a/Mage/src/mage/abilities/effects/ContinuousEffects.java +++ b/Mage/src/mage/abilities/effects/ContinuousEffects.java @@ -272,7 +272,7 @@ public class ContinuousEffects implements Serializable { public String getText() { StringBuilder sbText = new StringBuilder(); for (ContinuousEffect effect: effects) { - sbText.append(effect.getText() + " "); + sbText.append(effect.getText()).append(" "); } return sbText.toString(); } diff --git a/Mage/src/mage/abilities/effects/Effects.java b/Mage/src/mage/abilities/effects/Effects.java index a3eb482454c..265936c67a0 100644 --- a/Mage/src/mage/abilities/effects/Effects.java +++ b/Mage/src/mage/abilities/effects/Effects.java @@ -47,7 +47,7 @@ public class Effects extends ArrayList { public String getText() { StringBuilder sbText = new StringBuilder(); for (Effect effect: this) { - sbText.append(effect.getText() + ". "); + sbText.append(effect.getText()).append(". "); } return sbText.toString(); } diff --git a/Mage/src/mage/abilities/effects/common/BoostControlledEffect.java b/Mage/src/mage/abilities/effects/common/BoostControlledEffect.java index 2ac9d003410..247b1784cab 100644 --- a/Mage/src/mage/abilities/effects/common/BoostControlledEffect.java +++ b/Mage/src/mage/abilities/effects/common/BoostControlledEffect.java @@ -85,8 +85,7 @@ public class BoostControlledEffect extends ContinuousEffectImpl { sb.append("Creatures"); else sb.append(filter.getMessage()); - sb.append(" you control get "); - sb.append(String.format("%1$+d/%2$+d", power, toughness)); + sb.append(" you control get ").append(String.format("%1$+d/%2$+d", power, toughness)); sb.append((duration==Duration.EndOfTurn?" until end of turn":"")); return sb.toString(); } diff --git a/Mage/src/mage/abilities/effects/common/SearchLibraryPutInPlayEffect.java b/Mage/src/mage/abilities/effects/common/SearchLibraryPutInPlayEffect.java index 04967b8ea4d..288c95a3572 100644 --- a/Mage/src/mage/abilities/effects/common/SearchLibraryPutInPlayEffect.java +++ b/Mage/src/mage/abilities/effects/common/SearchLibraryPutInPlayEffect.java @@ -88,16 +88,11 @@ public class SearchLibraryPutInPlayEffect extends OneShotEffect { StringBuilder sb = new StringBuilder(); sb.append("Search your library for "); if (target.getNumberOfTargets() == 0 && target.getMaxNumberOfTargets() > 0) { - sb.append("up to "); - sb.append(target.getMaxNumberOfTargets()); - sb.append(" "); - sb.append(target.getTargetName()); - sb.append(", and put them onto the battlefield"); + sb.append("up to ").append(target.getMaxNumberOfTargets()).append(" "); + sb.append(target.getTargetName()).append(", and put them onto the battlefield"); } else { - sb.append("a "); - sb.append(target.getTargetName()); - sb.append(", and put it onto the battlefield"); + sb.append("a ").append(target.getTargetName()).append(", and put it onto the battlefield"); } sb.append(". Then shuffle your library"); return sb.toString(); diff --git a/Mage/src/mage/abilities/effects/common/SearchLibraryRevealPutInHandEffect.java b/Mage/src/mage/abilities/effects/common/SearchLibraryRevealPutInHandEffect.java index b42b22bc733..4280fd8c0e6 100644 --- a/Mage/src/mage/abilities/effects/common/SearchLibraryRevealPutInHandEffect.java +++ b/Mage/src/mage/abilities/effects/common/SearchLibraryRevealPutInHandEffect.java @@ -76,16 +76,11 @@ public class SearchLibraryRevealPutInHandEffect extends OneShotEffect { StringBuilder sb = new StringBuilder(); sb.append("Search your library for "); if (target.getNumberOfTargets() == 0 && target.getMaxNumberOfTargets() > 0) { - sb.append("up to "); - sb.append(target.getMaxNumberOfTargets()); - sb.append(" "); - sb.append(target.getTargetName()); - sb.append(", reveal them, and put them into your hand"); + sb.append("up to ").append(target.getMaxNumberOfTargets()).append(" "); + sb.append(target.getTargetName()).append(", reveal them, and put them into your hand"); } else { - sb.append("a "); - sb.append(target.getTargetName()); - sb.append(", reveal that card, and put it into your hand"); + sb.append("a ").append(target.getTargetName()).append(", reveal that card, and put it into your hand"); } sb.append(". Then shuffle your library"); diff --git a/Mage/src/mage/target/TargetObject.java b/Mage/src/mage/target/TargetObject.java index 729c8c69862..411b39c8acf 100644 --- a/Mage/src/mage/target/TargetObject.java +++ b/Mage/src/mage/target/TargetObject.java @@ -61,8 +61,7 @@ public abstract class TargetObject extends TargetImpl { public String getTargetedName(Game game) { StringBuilder sb = new StringBuilder(); for (UUID targetId: getTargets()) { - sb.append(game.getObject(targetId).getName()); - sb.append(" "); + sb.append(game.getObject(targetId).getName()).append(" "); } return sb.toString(); } diff --git a/Mage/src/mage/target/TargetPlayer.java b/Mage/src/mage/target/TargetPlayer.java index 2616f378613..d9bc35b50eb 100644 --- a/Mage/src/mage/target/TargetPlayer.java +++ b/Mage/src/mage/target/TargetPlayer.java @@ -101,8 +101,7 @@ public class TargetPlayer extends TargetImpl { public String getTargetedName(Game game) { StringBuilder sb = new StringBuilder(); for (UUID targetId: getTargets()) { - sb.append(game.getPlayer(targetId).getName()); - sb.append(" "); + sb.append(game.getPlayer(targetId).getName()).append(" "); } return sb.toString(); } diff --git a/Mage/src/mage/target/common/TargetCreatureOrPlayer.java b/Mage/src/mage/target/common/TargetCreatureOrPlayer.java index 13718e01924..359d870a101 100644 --- a/Mage/src/mage/target/common/TargetCreatureOrPlayer.java +++ b/Mage/src/mage/target/common/TargetCreatureOrPlayer.java @@ -107,13 +107,11 @@ public class TargetCreatureOrPlayer extends TargetImpl { for (UUID targetId: getTargets()) { Permanent permanent = game.getPermanent(targetId); if (permanent != null) { - sb.append(permanent.getName()); - sb.append(" "); + sb.append(permanent.getName()).append(" "); } else { Player player = game.getPlayer(targetId); - sb.append(player.getName()); - sb.append(" "); + sb.append(player.getName()).append(" "); } } return sb.toString(); diff --git a/Mage/src/mage/target/common/TargetDefender.java b/Mage/src/mage/target/common/TargetDefender.java index 1d8fdd97de3..94e08d2219e 100644 --- a/Mage/src/mage/target/common/TargetDefender.java +++ b/Mage/src/mage/target/common/TargetDefender.java @@ -92,13 +92,11 @@ public class TargetDefender extends TargetImpl { for (UUID targetId: getTargets()) { Permanent permanent = game.getPermanent(targetId); if (permanent != null) { - sb.append(permanent.getName()); - sb.append(" "); + sb.append(permanent.getName()).append(" "); } else { Player player = game.getPlayer(targetId); - sb.append(player.getName()); - sb.append(" "); + sb.append(player.getName()).append(" "); } } return sb.toString();