Checkstyle fixes in AF_Cleanup

This commit is contained in:
slapshot5
2011-08-30 07:53:59 +00:00
parent 4f9d9a5e59
commit 9d3beceeed

View File

@@ -6,7 +6,8 @@ import forge.card.spellability.SpellAbility;
import java.util.HashMap; import java.util.HashMap;
// Cleanup is not the same as other AFs, it is only used as a Drawback, and only used to Cleanup particular card states // Cleanup is not the same as other AFs, it is only used as a Drawback, and only used to Cleanup particular card states
// That need to be reset. I'm creating this to clear Remembered Cards at the end of an Effect so they don't get shown on a card // That need to be reset. I'm creating this to clear Remembered Cards at the
// end of an Effect so they don't get shown on a card
// After the effect finishes resolving. // After the effect finishes resolving.
/** /**
* <p>AbilityFactory_Cleanup class.</p> * <p>AbilityFactory_Cleanup class.</p>
@@ -14,16 +15,20 @@ import java.util.HashMap;
* @author Forge * @author Forge
* @version $Id$ * @version $Id$
*/ */
public class AbilityFactory_Cleanup { public final class AbilityFactory_Cleanup {
private AbilityFactory_Cleanup() {
throw new AssertionError();
}
/** /**
* <p>getDrawback.</p> * <p>getDrawback.</p>
* *
* @param AF a {@link forge.card.abilityFactory.AbilityFactory} object. * @param af a {@link forge.card.abilityFactory.AbilityFactory} object.
* @return a {@link forge.card.spellability.Ability_Sub} object. * @return a {@link forge.card.spellability.Ability_Sub} object.
*/ */
public static Ability_Sub getDrawback(final AbilityFactory AF) { public static Ability_Sub getDrawback(final AbilityFactory af) {
final Ability_Sub drawback = new Ability_Sub(AF.getHostCard(), AF.getAbTgt()) { final Ability_Sub drawback = new Ability_Sub(af.getHostCard(), af.getAbTgt()) {
private static final long serialVersionUID = 6192972525033429820L; private static final long serialVersionUID = 6192972525033429820L;
@Override @Override
@@ -32,13 +37,13 @@ public class AbilityFactory_Cleanup {
} }
@Override @Override
public boolean doTrigger(boolean mandatory) { public boolean doTrigger(final boolean mandatory) {
return true; return true;
} }
@Override @Override
public void resolve() { public void resolve() {
doResolve(AF, this); doResolve(af, this);
} }
}; };
@@ -51,12 +56,15 @@ public class AbilityFactory_Cleanup {
* @param AF a {@link forge.card.abilityFactory.AbilityFactory} object. * @param AF a {@link forge.card.abilityFactory.AbilityFactory} object.
* @param sa a {@link forge.card.spellability.SpellAbility} object. * @param sa a {@link forge.card.spellability.SpellAbility} object.
*/ */
private static void doResolve(AbilityFactory AF, SpellAbility sa) { private static void doResolve(final AbilityFactory af, final SpellAbility sa) {
HashMap<String, String> params = AF.getMapParams(); HashMap<String, String> params = af.getMapParams();
if (params.containsKey("ClearRemembered")) if (params.containsKey("ClearRemembered")) {
sa.getSourceCard().clearRemembered(); sa.getSourceCard().clearRemembered();
if (params.containsKey("ClearImprinted")) }
if (params.containsKey("ClearImprinted")) {
sa.getSourceCard().clearImprinted(); sa.getSourceCard().clearImprinted();
} }
} }
} //end class AbilityFactory_Cleanup