Added Flip Coin effect.

Mage.Sets/src/mage/cards/s/SorcerersStrongbox.java

@@ -25,23 +25,21 @@
  *  authors and should not be interpreted as representing official policies, either expressed
  *  or implied, of BetaSteward_at_googlemail.com.
  */
-
 package mage.cards.s;
 
 import java.util.UUID;
 import mage.constants.CardType;
-import mage.constants.Outcome;
 import mage.constants.Zone;
 import mage.abilities.Ability;
 import mage.abilities.common.SimpleActivatedAbility;
 import mage.abilities.costs.common.TapSourceCost;
 import mage.abilities.costs.mana.GenericManaCost;
-import mage.abilities.effects.OneShotEffect;
+import mage.abilities.effects.Effect;
+import mage.abilities.effects.common.DrawCardSourceControllerEffect;
+import mage.abilities.effects.common.FlipCoinEffect;
+import mage.abilities.effects.common.SacrificeSourceEffect;
 import mage.cards.CardImpl;
 import mage.cards.CardSetInfo;
-import mage.game.Game;
-import mage.game.permanent.Permanent;
-import mage.players.Player;
 
 /**
  *
@@ -50,8 +48,14 @@ import mage.players.Player;
 public class SorcerersStrongbox extends CardImpl {
 
     public SorcerersStrongbox(UUID ownerId, CardSetInfo setInfo) {
-        super(ownerId,setInfo,new CardType[]{CardType.ARTIFACT},"{4}");
+        super(ownerId, setInfo, new CardType[]{CardType.ARTIFACT}, "{4}");
-        Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new SorcerersStrongboxEffect(), new TapSourceCost());
+
+        // {2}, {T} : Flip a coin. If you win the flip, sacrifice Sorcerer's Strongbox and draw three cards.
+        FlipCoinEffect flipCoinEffect = new FlipCoinEffect(new SacrificeSourceEffect());
+        Effect effect = new DrawCardSourceControllerEffect(3);
+        effect.setText("and draw three cards");
+        flipCoinEffect.addEffectWon(effect);
+        Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, flipCoinEffect, new TapSourceCost());
         ability.addManaCost(new GenericManaCost(2));
         this.addAbility(ability);
     }
@@ -66,37 +70,3 @@ public class SorcerersStrongbox extends CardImpl {
     }
 
 }
-
-class SorcerersStrongboxEffect extends OneShotEffect {
-
-    public SorcerersStrongboxEffect() {
-        super(Outcome.DrawCard);
-        staticText = "Flip a coin. If you win the flip, sacrifice {this} and draw three cards";
-    }
-
-    public SorcerersStrongboxEffect(final SorcerersStrongboxEffect effect) {
-        super(effect);
-    }
-
-    @Override
-    public boolean apply(Game game, Ability source) {
-        Player player = game.getPlayer(source.getControllerId());
-        if (player != null) {
-            if (player.flipCoin(game)) {
-                Permanent perm = game.getPermanent(source.getSourceId());
-                if (perm != null) {
-                    perm.sacrifice(source.getSourceId(), game);
-                }
-                player.drawCards(3, game);
-                return true;
-            }
-        }
-        return false;
-    }
-
-    @Override
-    public SorcerersStrongboxEffect copy() {
-        return new SorcerersStrongboxEffect(this);
-    }
-
-}

Mage.Sets/src/mage/cards/w/WireflyHive.java

@@ -32,19 +32,16 @@ import mage.abilities.Ability;
 import mage.abilities.common.SimpleActivatedAbility;
 import mage.abilities.costs.common.TapSourceCost;
 import mage.abilities.costs.mana.GenericManaCost;
-import mage.abilities.effects.OneShotEffect;
 import mage.abilities.effects.common.CreateTokenEffect;
 import mage.abilities.effects.common.DestroyAllEffect;
+import mage.abilities.effects.common.FlipCoinEffect;
 import mage.cards.CardImpl;
 import mage.cards.CardSetInfo;
 import mage.constants.CardType;
-import mage.constants.Outcome;
 import mage.constants.Zone;
 import mage.filter.FilterPermanent;
 import mage.filter.predicate.mageobject.NamePredicate;
-import mage.game.Game;
 import mage.game.permanent.token.Token;
-import mage.players.Player;
 
 /**
  *
@@ -53,11 +50,13 @@ import mage.players.Player;
 public class WireflyHive extends CardImpl {
 
     public WireflyHive(UUID ownerId, CardSetInfo setInfo) {
-        super(ownerId,setInfo,new CardType[]{CardType.ARTIFACT},"{3}");
+        super(ownerId, setInfo, new CardType[]{CardType.ARTIFACT}, "{3}");
-
         // {3}, {tap}: Flip a coin. If you win the flip, put a 2/2 colorless Insect artifact creature token with flying named Wirefly onto the battlefield.
         // If you lose the flip, destroy all permanents named Wirefly.
-        Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new WireflyHiveEffect(), new GenericManaCost(3));
+        FilterPermanent filter = new FilterPermanent("permanents named Wirefly");
+        filter.add(new NamePredicate("Wirefly"));
+        Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD,
+                new FlipCoinEffect(new CreateTokenEffect(new WireflyToken()), new DestroyAllEffect(filter)), new GenericManaCost(3));
         ability.addCost(new TapSourceCost());
         this.addAbility(ability);
     }
@@ -72,41 +71,6 @@ public class WireflyHive extends CardImpl {
     }
 }
 
-class WireflyHiveEffect extends OneShotEffect {
-
-    public WireflyHiveEffect() {
-        super(Outcome.Damage);
-        staticText = "Flip a coin. If you win the flip, put a 2/2 colorless Insect artifact creature token with flying named Wirefly onto the battlefield."
-                + "If you lose the flip, destroy all permanents named Wirefly";
-    }
-
-    public WireflyHiveEffect(WireflyHiveEffect effect) {
-        super(effect);
-    }
-
-    @Override
-    public boolean apply(Game game, Ability source) {
-        Player controller = game.getPlayer(source.getControllerId());
-        if (controller != null) {
-            if (controller.flipCoin(game)) {
-                new CreateTokenEffect(new WireflyToken()).apply(game, source);
-                return true;
-            } else {
-                FilterPermanent filter = new FilterPermanent("permanents named Wirefly");
-                filter.add(new NamePredicate("Wirefly"));
-                new DestroyAllEffect(filter).apply(game, source);
-                return true;
-            }
-        }
-        return false;
-    }
-
-    @Override
-    public WireflyHiveEffect copy() {
-        return new WireflyHiveEffect(this);
-    }
-}
-
 class WireflyToken extends Token {
 
     WireflyToken() {

Mage/src/main/java/mage/abilities/effects/common/FlipCoinEffect.java

@@ -0,0 +1,118 @@
+/*
+ *  Copyright 2010 BetaSteward_at_googlemail.com. All rights reserved.
+ *
+ *  Redistribution and use in source and binary forms, with or without modification, are
+ *  permitted provided that the following conditions are met:
+ *
+ *     1. Redistributions of source code must retain the above copyright notice, this list of
+ *        conditions and the following disclaimer.
+ *
+ *     2. Redistributions in binary form must reproduce the above copyright notice, this list
+ *        of conditions and the following disclaimer in the documentation and/or other materials
+ *        provided with the distribution.
+ *
+ *  THIS SOFTWARE IS PROVIDED BY BetaSteward_at_googlemail.com ``AS IS'' AND ANY EXPRESS OR IMPLIED
+ *  WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND
+ *  FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL BetaSteward_at_googlemail.com OR
+ *  CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ *  CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ *  SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON
+ *  ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+ *  NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
+ *  ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ *  The views and conclusions contained in the software and documentation are those of the
+ *  authors and should not be interpreted as representing official policies, either expressed
+ *  or implied, of BetaSteward_at_googlemail.com.
+ */
+package mage.abilities.effects.common;
+
+import mage.MageObject;
+import mage.abilities.Ability;
+import mage.abilities.Mode;
+import mage.abilities.effects.ContinuousEffect;
+import mage.abilities.effects.Effect;
+import mage.abilities.effects.Effects;
+import mage.abilities.effects.OneShotEffect;
+import mage.constants.Outcome;
+import mage.game.Game;
+import mage.players.Player;
+
+/**
+ *
+ * @author LevelX2
+ */
+public class FlipCoinEffect extends OneShotEffect {
+
+    protected Effects executingEffectsWon = new Effects();
+    protected Effects executingEffectsLost = new Effects();
+
+    public FlipCoinEffect(Effect effectWon) {
+        this(effectWon, null);
+    }
+
+    public FlipCoinEffect(Effect effectWon, Effect effectLost) {
+        this(effectWon, effectLost, Outcome.Benefit);
+
+    }
+
+    public FlipCoinEffect(Effect effectWon, Effect effectLost, Outcome outcome) {
+        super(outcome);
+        addEffectWon(effectWon);
+        addEffectLost(effectLost);
+    }
+
+    public FlipCoinEffect(final FlipCoinEffect effect) {
+        super(effect);
+        this.executingEffectsWon = effect.executingEffectsWon.copy();
+        this.executingEffectsLost = effect.executingEffectsLost.copy();
+    }
+
+    public void addEffectWon(Effect effect) {
+        if (effect != null) {
+            executingEffectsWon.add(effect);
+        }
+    }
+
+    public void addEffectLost(Effect effect) {
+        if (effect != null) {
+            executingEffectsLost.add(effect);
+        }
+    }
+
+    @Override
+    public boolean apply(Game game, Ability source) {
+        Player controller = game.getPlayer(source.getControllerId());
+        MageObject mageObject = game.getObject(source.getSourceId());
+        if (controller != null && mageObject != null) {
+            boolean result = true;
+            for (Effect effect : controller.flipCoin(game) ? executingEffectsWon : executingEffectsLost) {
+                effect.setTargetPointer(this.targetPointer);
+                if (effect instanceof OneShotEffect) {
+                    result &= effect.apply(game, source);
+                } else {
+                    game.addEffect((ContinuousEffect) effect, source);
+                }
+            }
+            return result;
+        }
+        return false;
+    }
+
+    @Override
+    public String getText(Mode mode) {
+        if (!staticText.isEmpty()) {
+            return staticText;
+        }
+        StringBuilder sb = new StringBuilder("Flip a coin. If you win the flip, ").append(executingEffectsWon.getText(mode));
+        if (!executingEffectsLost.isEmpty()) {
+            sb.append(". If you lose the flip, ").append(executingEffectsLost.getText(mode));
+        }
+        return sb.toString();
+    }
+
+    @Override
+    public FlipCoinEffect copy() {
+        return new FlipCoinEffect(this);
+    }
+}