NameChanger */
private static final NameChanger NameChanger = new NameChanger();
+ /** Constant ColorChanger */
+ private static final ColorChanger colorChanger = new ColorChanger();
/** Constant EndOfTurn */
private static EndOfTurn EndOfTurn = new EndOfTurn();
@@ -62,8 +66,11 @@ public class AllZone implements NewConstants {
private static final GameAction GameAction = new GameAction();
/** Constant StaticEffects */
private static final StaticEffects StaticEffects = new StaticEffects();
- /** Constant GameInfo */
- private static final GameInfo GameInfo = new GameInfo();
+
+ /** Game state observer GameSummary collects statistics and players' performance*/
+ private static GameSummary gameInfo = new GameSummary();
+ /** Match State for quests are stored in a QuestMatchState class instance*/
+ public static QuestMatchState matchState = new QuestMatchState();
/** Constant TriggerHandler */
private static final TriggerHandler TriggerHandler = new TriggerHandler();
@@ -308,11 +315,11 @@ public class AllZone implements NewConstants {
/**
* getGameInfo.
* - * @return a {@link forge.GameInfo} object. + * @return a {@link forge.GameSummary} object. * @since 1.0.15 */ - public static GameInfo getGameInfo() { - return GameInfo; + public static GameSummary getGameInfo() { + return gameInfo; } /** @@ -606,4 +613,54 @@ public class AllZone implements NewConstants { timestamp++; return timestamp; } + + /** + *Resets everything possible to set a new game
+ */ + public static void newGameCleanup() { + + gameInfo = new GameSummary(); + + getHumanPlayer().reset(); + getComputerPlayer().reset(); + + getPhase().reset(); + getStack().reset(); + getCombat().reset(); + getDisplay().showCombat(""); + getDisplay().loadPrefs(); + + getHumanGraveyard().reset(); + getHumanHand().reset(); + getHumanLibrary().reset(); + getHumanBattlefield().reset(); + getHumanExile().reset(); + + getComputerGraveyard().reset(); + getComputerHand().reset(); + getComputerLibrary().reset(); + getComputerBattlefield().reset(); + getComputerExile().reset(); + + getInputControl().clearInput(); + + getStaticEffects().reset(); + getColorChanger().reset(); + + // player.reset() now handles this + //AllZone.getHumanPlayer().clearHandSizeOperations(); + //AllZone.getComputerPlayer().clearHandSizeOperations(); + + getTriggerHandler().clearRegistered(); + + } + + public static QuestMatchState getMatchState() { + return matchState; + } + + public static ColorChanger getColorChanger() { + // TODO Auto-generated method stub + return colorChanger; + } }//AllZone diff --git a/src/main/java/forge/Card.java b/src/main/java/forge/Card.java index dcc9aaa62ea..b99495e59eb 100644 --- a/src/main/java/forge/Card.java +++ b/src/main/java/forge/Card.java @@ -1200,7 +1200,7 @@ public class Card extends MyObservable implements ComparableaddColorChanges.
+ * + * @param s a {@link java.lang.String} object. + * @param c a {@link forge.Card} object. + * @param addToColors a boolean. + * @param bIncrease a boolean. + * @return a long. + */ + public long addColorChanges(String s, Card c, boolean addToColors, boolean bIncrease) { + if (bIncrease) { + Card_Color.increaseTimestamp(); + } + globalColorChanges.add(new Card_Color(new ManaCost(s), c, addToColors, false)); + return Card_Color.getTimestamp(); + } + + /** + *removeColorChanges.
+ * + * @param s a {@link java.lang.String} object. + * @param c a {@link forge.Card} object. + * @param addTo a boolean. + * @param timestamp a long. + */ + public final void removeColorChanges(String s, Card c, boolean addTo, long timestamp) { + Card_Color removeCol = null; + for (Card_Color cc : globalColorChanges) { + if (cc.equals(s, c, addTo, timestamp)) { + removeCol = cc; + } + } + + if (removeCol != null) { + globalColorChanges.remove(removeCol); + } + } + + /** + *reset = clearColorChanges.
+ */ + public final void reset() { + clearColorChanges(); + } + + /** + *clearColorChanges.
+ */ + public final void clearColorChanges() { + // clear the global color changes at end of each game + globalColorChanges.clear(); + } + + /** + *getColorChanges.
+ * + * @return a {@link java.util.ArrayList} object. + */ + public final ArrayListGameInfo class.
- * - * @author Forge - * @version $Id$ - */ -public class GameInfo { - private boolean computerStartedThisGame = false; - - private int humanNumberOfTimesMulliganed; - private boolean humanMulliganedToZero; - - private boolean preventCombatDamageThisTurn; - private boolean assignedFirstStrikeDamageThisCombat; - private boolean resolvedFirstStrikeDamageThisCombat; - - private ArrayListGetter for the field humanNumberOfTimesMulliganed.
addHumanNumberOfTimesMulliganed.
- * - * @param n a int. - */ - public void addHumanNumberOfTimesMulliganed(int n) { - humanNumberOfTimesMulliganed += n; - } - - /** - *Setter for the field humanNumberOfTimesMulliganed.
Getter for the field humanMulliganedToZero.
Setter for the field humanMulliganedToZero.
Setter for the field preventCombatDamageThisTurn.
isPreventCombatDamageThisTurn.
- * - * @return a boolean. - */ - public boolean isPreventCombatDamageThisTurn() { - return preventCombatDamageThisTurn; - } - - /** - *Setter for the field assignedFirstStrikeDamageThisCombat.
Getter for the field assignedFirstStrikeDamageThisCombat.
Setter for the field resolvedFirstStrikeDamageThisCombat.
Getter for the field resolvedFirstStrikeDamageThisCombat.
Setter for the field computerStartedThisGame.
isComputerStartedThisGame.
- * - * @return a boolean. - */ - public boolean isComputerStartedThisGame() { - return computerStartedThisGame; - } - - /** - *addColorChanges.
- * - * @param s a {@link java.lang.String} object. - * @param c a {@link forge.Card} object. - * @param addToColors a boolean. - * @param bIncrease a boolean. - * @return a long. - */ - public long addColorChanges(String s, Card c, boolean addToColors, boolean bIncrease) { - if (bIncrease) - Card_Color.increaseTimestamp(); - globalColorChanges.add(new Card_Color(new ManaCost(s), c, addToColors, false)); - return Card_Color.getTimestamp(); - } - - /** - *removeColorChanges.
- * - * @param s a {@link java.lang.String} object. - * @param c a {@link forge.Card} object. - * @param addTo a boolean. - * @param timestamp a long. - */ - public void removeColorChanges(String s, Card c, boolean addTo, long timestamp) { - Card_Color removeCol = null; - for (Card_Color cc : globalColorChanges) - if (cc.equals(s, c, addTo, timestamp)) - removeCol = cc; - - if (removeCol != null) - globalColorChanges.remove(removeCol); - } - - /** - *clearColorChanges.
- */ - public void clearColorChanges() { - // clear the global color changes at end of each game - globalColorChanges.clear(); - } - - /** - *getColorChanges.
- * - * @return a {@link java.util.ArrayList} object. - */ - public ArrayListgreenColor */
public static Color greenColor = new Color(0, 164, 0);
@@ -1134,10 +1134,8 @@ public class GuiDisplay4 extends JFrame implements CardContainer, Display, NewCo
* Exit the Application
*/
private void concede() {
- savePrefs();
- dispose();
- Constant.Runtime.matchState.addLose();
- new Gui_WinLose();
+ AllZone.getHumanPlayer().loseConditionMet(GameLossReason.Conceded, null );
+ AllZone.getGameAction().checkStateEffects();
}
// ********** Phase stuff in Display ******************
diff --git a/src/main/java/forge/Gui_WinLose.java b/src/main/java/forge/Gui_WinLose.java
index 9b12e97ef13..c9b1d2d88fa 100644
--- a/src/main/java/forge/Gui_WinLose.java
+++ b/src/main/java/forge/Gui_WinLose.java
@@ -1,25 +1,38 @@
package forge;
-
-import forge.error.ErrorViewer;
+import forge.error.E
+import forge.game.GameEndReason;
+rrorViewer;
+import forge.game.GameLossReason;
+import forge.game.GamePlayerRating;
+import forge.game.GameSummary;
+import forge.game.PlayerIndex;
import forge.properties.ForgeProps;
import forge.properties.NewConstants;
import forge.properties.NewConstants.LANG.Gui_WinLose.WINLOSE_TEXT;
+import forge.quest.data.QuestData;
import forge.quest.data.QuestMatchState;
import forge.quest.data.QuestPreferences;
import forge.quest.gui.QuestFrame;
import forge.view.swing.OldGuiNewGame;
import net.miginfocom.swing.MigLayout;
-import javax.swing.*;
+import javax.swing.BorderFactory;
+import javax.swing.ImageIcon;
+import javax.swing.JButton;
+import javax.swing.JFrame;
+import javax.swing.JLabel;
+import javax.swing.JOptionPane;
+import javax.swing.JPanel;
+import javax.swing.SwingConstants;
import javax.swing.border.Border;
import javax.swing.border.TitledBorder;
import java.awt.Color;
-import java.awt.*;
+import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.WindowEvent;
import java.io.File;
-
+import java.util.ArrayList;
/**
* Gui_WinLose class.
@@ -31,14 +44,6 @@ public class Gui_WinLose extends JFrame implements NewConstants { /** ConstantserialVersionUID=-5800412940994975483L */
private static final long serialVersionUID = -5800412940994975483L;
- //private CardList humanList;
- //private CardList computerList;
-
- //private int humanLife;
- //private int computerLife;
-
- //private boolean fantasyQuest = false;
-
private JLabel titleLabel = new JLabel();
private JButton continueButton = new JButton();
private JButton restartButton = new JButton();
@@ -52,51 +57,22 @@ public class Gui_WinLose extends JFrame implements NewConstants {
// border1
private Border border1;
- /**
- * Constructor for Gui_WinLose.
- * - * @param human a {@link forge.CardList} object. - * @param computer a {@link forge.CardList} object. - * @param hLife a int. - * @param cLife a int. - */ - public Gui_WinLose(CardList human, CardList computer, int hLife, int cLife) { - /* - fantasyQuest = true; - - humanList = human; - computerList = computer; - - humanLife = hLife; - computerLife= cLife; - */ - try { - jbInit(); - } catch (Exception ex) { - ErrorViewer.showError(ex); - } - - setup(); - - Dimension screen = this.getToolkit().getScreenSize(); - setBounds(screen.width / 3, 100, //position - 215, 370); //size - setVisible(true); + private class WinLoseModel { + public QuestMatchState match; + public QuestData quest; + public Quest_Assignment qa; } + private WinLoseModel model; + /** *Constructor for Gui_WinLose.
*/ - public Gui_WinLose() { - /* - fantasyQuest = false; - - humanList = new CardList(); - computerList = new CardList(); - - humanLife = 20; - computerLife= 20; - */ + public Gui_WinLose(final QuestMatchState matchState, final QuestData quest, final Quest_Assignment qa) { + model = new WinLoseModel(); + model.match = matchState; + model.quest = quest; + model.qa = qa; try { jbInit(); @@ -107,8 +83,7 @@ public class Gui_WinLose extends JFrame implements NewConstants { setup(); Dimension screen = this.getToolkit().getScreenSize(); - setBounds(screen.width / 3, 100, //position - 215, 370); //size + setBounds(screen.width / 3, 100 /*position*/, 215, 370 /*size*/); setVisible(true); } @@ -116,47 +91,28 @@ public class Gui_WinLose extends JFrame implements NewConstants { *setup.
*/ private void setup() { - AllZone.getGameInfo().clearColorChanges(); - QuestMatchState winLose = Constant.Runtime.matchState; Phase.setGameBegins(0); - //3 is the match length, 3 is the number of games - //disable buttons if match is up, or human player won 2 or lost 2 games already - if ((winLose.countWinLose() == 3) || (winLose.getWin() == 2) || (winLose.getLose() == 2)) { + + if (model.match.isMatchOver()) { // editDeckButton.setEnabled(false); continueButton.setEnabled(false); quitButton.grabFocus(); } - if (winLose.getWin() == 2) - restartButton.setEnabled(false); + restartButton.setEnabled(!model.match.isMatchWonByHuman()); //show Wins and Loses - statsLabel.setText(ForgeProps.getLocalized(WINLOSE_TEXT.WON) + winLose.getWin() + ForgeProps.getLocalized(WINLOSE_TEXT.LOST) + winLose.getLose()); + int humanWins = model.match.getGamesCountWonByHuman(); + int humanLosses = model.match.getGamesCountLostByHuman(); + statsLabel.setText(ForgeProps.getLocalized(WINLOSE_TEXT.WON) + humanWins + + ForgeProps.getLocalized(WINLOSE_TEXT.LOST) + humanLosses); + //show "You Won" or "You Lost" - if (winLose.didWinRecently()) { + if (model.match.hasHumanWonLastGame()) { titleLabel.setText(ForgeProps.getLocalized(WINLOSE_TEXT.WIN)); - - int game = 0; - if (winLose.getWinTurns()[0] != 0) - game = 1; - int turn = AllZone.getPhase().getTurn(); - if (AllZone.getGameInfo().isComputerStartedThisGame()) - turn--; - - if (turn < 1) - turn = 1; - - winLose.setWinTurn(game, turn); - winLose.setMulliganedToZero(game, AllZone.getGameInfo().getHumanMulliganedToZero()); - - //winLose.setWinTurn(winLose.countWinLose()-1, AllZone.getPhase().getTurn()); - - //System.out.println("CountwinLose:" + winLose.countWinLose()); - //System.out.println("You won by turn: " + AllZone.getPhase().getTurn()); } else { titleLabel.setText(ForgeProps.getLocalized(WINLOSE_TEXT.LOSE)); - //System.out.println("You lost by turn: " + AllZone.getPhase().getTurn()); } }//setup(); @@ -174,21 +130,15 @@ public class Gui_WinLose extends JFrame implements NewConstants { this.getContentPane().setLayout(new MigLayout("fill")); continueButton.setText(ForgeProps.getLocalized(WINLOSE_TEXT.CONTINUE)); continueButton.addActionListener(new java.awt.event.ActionListener() { - public void actionPerformed(ActionEvent e) { - continueButton_actionPerformed(e); - } + public void actionPerformed(ActionEvent e) { continueButton_actionPerformed(e); } }); restartButton.setText(ForgeProps.getLocalized(WINLOSE_TEXT.RESTART)); restartButton.addActionListener(new java.awt.event.ActionListener() { - public void actionPerformed(ActionEvent e) { - restartButton_actionPerformed(e); - } + public void actionPerformed(ActionEvent e) { restartButton_actionPerformed(e); } }); quitButton.setText(ForgeProps.getLocalized(WINLOSE_TEXT.QUIT)); quitButton.addActionListener(new java.awt.event.ActionListener() { - public void actionPerformed(ActionEvent e) { - quitButton_actionPerformed(e); - } + public void actionPerformed(ActionEvent e) { quitButton_actionPerformed(e); } }); statsLabel.setFont(new java.awt.Font("Dialog", 0, 16)); statsLabel.setHorizontalAlignment(SwingConstants.CENTER); @@ -196,9 +146,7 @@ public class Gui_WinLose extends JFrame implements NewConstants { jPanel2.setLayout(new MigLayout("align center")); this.addWindowListener(new java.awt.event.WindowAdapter() { @Override - public void windowClosing(WindowEvent e) { - this_windowClosing(e); - } + public void windowClosing(WindowEvent e) { this_windowClosing(e); } }); this.getContentPane().add(titleLabel, "align center, grow, wrap"); this.getContentPane().add(statsLabel, "align center, grow, wrap"); @@ -209,26 +157,6 @@ public class Gui_WinLose extends JFrame implements NewConstants { } - /** - *editDeckButton_actionPerformed.
- * - * @param e a {@link java.awt.event.ActionEvent} object. - */ - void editDeckButton_actionPerformed(ActionEvent e) { - Command exit = new Command() { - private static final long serialVersionUID = 4735992294414389187L; - - public void execute() { - new Gui_WinLose(); - } - }; - Gui_DeckEditor editor = new Gui_DeckEditor(); - - editor.show(exit); - - dispose(); - }//editDeckButton_actionPerformed() - /** *continueButton_actionPerformed.
* @@ -236,32 +164,36 @@ public class Gui_WinLose extends JFrame implements NewConstants { */ void continueButton_actionPerformed(ActionEvent e) { //open up "Game" screen - //AllZone.getComputerPlay().reset();//sometimes computer has creature in play in the 2nd game of the match - - if (!Constant.Quest.fantasyQuest[0]) - AllZone.getGameAction().newGame(Constant.Runtime.HumanDeck[0], Constant.Runtime.ComputerDeck[0]); - else { + + PrepareForNextRound(); + AllZone.getDisplay().setVisible(true); + dispose(); + } + + void PrepareForNextRound() + { + if (Constant.Quest.fantasyQuest[0]) { int extraLife = 0; - if (AllZone.getQuestAssignment() != null) { - forge.quest.data.QuestUtil.setupQuest(AllZone.getQuestAssignment()); - if (AllZone.getQuestData().getInventory().hasItem("Zeppelin")) + if (model.qa != null) { + forge.quest.data.QuestUtil.setupQuest(model.qa); + if (model.quest.getInventory().hasItem("Zeppelin")) extraLife = 3; } //AllZone.getGameAction().newGame(Constant.Runtime.HumanDeck[0], Constant.Runtime.ComputerDeck[0], humanList, computerList, humanLife, computerLife); - CardList humanList = forge.quest.data.QuestUtil.getHumanPlantAndPet(AllZone.getQuestData(), AllZone.getQuestAssignment()); + CardList humanList = forge.quest.data.QuestUtil.getHumanPlantAndPet(model.quest, model.qa); CardList computerList = new CardList(); - int humanLife = AllZone.getQuestData().getLife() + extraLife; + int humanLife = model.quest.getLife() + extraLife; int computerLife = 20; - if (AllZone.getQuestAssignment() != null) - computerLife = AllZone.getQuestAssignment().getComputerLife(); + if ( model.qa != null ) + computerLife = model.qa.getComputerLife(); - AllZone.getGameAction().newGame(Constant.Runtime.HumanDeck[0], Constant.Runtime.ComputerDeck[0], humanList, computerList, humanLife, computerLife, AllZone.getQuestAssignment()); + AllZone.getGameAction().newGame(Constant.Runtime.HumanDeck[0], Constant.Runtime.ComputerDeck[0], humanList, computerList, humanLife, computerLife, model.qa ); + } else { + AllZone.getGameAction().newGame(Constant.Runtime.HumanDeck[0], Constant.Runtime.ComputerDeck[0]); } - AllZone.getDisplay().setVisible(true); - dispose(); } /** @@ -270,33 +202,9 @@ public class Gui_WinLose extends JFrame implements NewConstants { * @param e a {@link java.awt.event.ActionEvent} object. */ void restartButton_actionPerformed(ActionEvent e) { - Constant.Runtime.matchState.reset(); - - if (!Constant.Quest.fantasyQuest[0]) - AllZone.getGameAction().newGame(Constant.Runtime.HumanDeck[0], Constant.Runtime.ComputerDeck[0]); - else { - int extraLife = 0; - //AllZone.getGameAction().newGame(Constant.Runtime.HumanDeck[0], Constant.Runtime.ComputerDeck[0], humanList, computerList, humanLife, computerLife); - if (AllZone.getQuestAssignment() != null) { - forge.quest.data.QuestUtil.setupQuest(AllZone.getQuestAssignment()); - if (AllZone.getQuestData().getInventory().hasItem("Zeppelin")) - extraLife = 3; - } - - CardList humanList = forge.quest.data.QuestUtil.getHumanPlantAndPet(AllZone.getQuestData(), AllZone.getQuestAssignment()); - //CardList computerList = QuestUtil.getComputerCreatures(AllZone.getQuestData(), AllZone.getQuestAssignment()); - CardList computerList = new CardList(); - - int humanLife = AllZone.getQuestData().getLife() + extraLife; - int computerLife = 20; - - if (AllZone.getQuestAssignment() != null) - computerLife = AllZone.getQuestAssignment().getComputerLife(); - - AllZone.getGameAction().newGame(Constant.Runtime.HumanDeck[0], Constant.Runtime.ComputerDeck[0], humanList, computerList, humanLife, computerLife, AllZone.getQuestAssignment()); - } + model.match.reset(); + PrepareForNextRound(); AllZone.getDisplay().setVisible(true); - dispose(); } @@ -304,88 +212,82 @@ public class Gui_WinLose extends JFrame implements NewConstants { *getWinText.
* * @param creds a long. - * @param winLose a {@link forge.quest.data.QuestMatchState} object. + * @param matchState a {@link forge.quest.data.QuestMatchState} object. * @param q a {@link forge.quest.data.QuestData} object. * @return a {@link java.lang.String} object. */ - private String getWinText(long creds, QuestMatchState winLose, forge.quest.data.QuestData q) { + private String getCreditsAwardedText(final long creds, final QuestMatchState matchState, final forge.quest.data.QuestData q) { // TODO use q.qdPrefs to write bonus credits in prefs file - StringBuilder sb = new StringBuilder(); - String[] wins = winLose.getWinMethods(); + StringBuilder sb = new StringBuilder(""); - sb.append(""); + boolean hasNeverLost = true; + for (GameSummary game : matchState.getGamesPlayed()) { + if (game.isAIWinner()) { + hasNeverLost = true; + continue; // no rewards for losing a game + } - for (String s : wins) { - if (s != null) { - sb.append("Alternate win condition: "); - sb.append(""); - sb.append(s); - sb.append(""); - sb.append("! Bonus: +"); + GamePlayerRating aiRating = game.getPlayerRating(PlayerIndex.AI); + GamePlayerRating humanRating = game.getPlayerRating(PlayerIndex.HUMAN); + GameLossReason whyAiLost = aiRating.getLossReason(); - if (s.equals("Poison Counters")) - sb.append(QuestPreferences.getMatchRewardPoisonWinBonus()); - else if (s.equals("Milled")) - sb.append(QuestPreferences.getMatchRewardMilledWinBonus()); - else if (s.equals("Battle of Wits") || - s.equals("Felidar Sovereign") || s.equals("Helix Pinnacle") || s.equals("Epic Struggle") || - s.equals("Door to Nothingness") || s.equals("Barren Glory") || s.equals("Near-Death Experience") || - s.equals("Mortal Combat") || s.equals("Test of Endurance")) { - - sb.append(QuestPreferences.getMatchRewardAltWinBonus()); + int rewardAltWinCondition = q.getCreditsRewardForAltWin(whyAiLost); + if (rewardAltWinCondition > 0) { + String winConditionName = "Unknown (bug)"; + if ( game.getWinCondition() == GameEndReason.WinsGameSpellEffect ) { + winConditionName = game.getWinSpellEffect(); + } else { + switch( whyAiLost ) { + case Poisoned: winConditionName = "Poison"; break; + case Milled: winConditionName = "Milled"; break; + default: break; + } } - sb.append(" credits.nextPhase.
*/ @@ -420,9 +426,11 @@ public class Phase extends MyObservable implements java.io.Serializable { this.bCombat = false; } - if (phaseOrder[phaseIndex].equals(Constant.Phase.Cleanup)) + if (phaseOrder[phaseIndex].equals(Constant.Phase.Cleanup)) { + bPreventCombatDamageThisTurn = false; if (!bRepeat) AllZone.getPhase().setPlayerTurn(handleNextTurn()); + } if (is(Constant.Phase.Combat_Declare_Blockers)) { AllZone.getStack().unfreezeStack(); @@ -666,9 +674,6 @@ public class Phase extends MyObservable implements java.io.Serializable { if (c.getCreatureBlockedThisCombat()) c.setCreatureBlockedThisCombat(false); if (c.getCreatureGotBlockedThisCombat()) c.setCreatureGotBlockedThisCombat(false); - - AllZone.getGameInfo().setAssignedFirstStrikeDamageThisCombat(false); - AllZone.getGameInfo().setResolvedFirstStrikeDamageThisCombat(false); } } @@ -1026,4 +1031,8 @@ public class Phase extends MyObservable implements java.io.Serializable { static void setComputerInstantSpellCount(int i) { ComputerInstantSpellCount = (i); } + + public void setPreventCombatDamageThisTurn(boolean b) { + bPreventCombatDamageThisTurn = true; + } } diff --git a/src/main/java/forge/PhaseUtil.java b/src/main/java/forge/PhaseUtil.java index afcdbb7d768..ab6088e5458 100644 --- a/src/main/java/forge/PhaseUtil.java +++ b/src/main/java/forge/PhaseUtil.java @@ -38,6 +38,7 @@ public class PhaseUtil { Player turn = AllZone.getPhase().getPlayerTurn(); AllZone.getPhase().turnReset(); + AllZone.getGameInfo().notifyNextTurn(); AllZone.getCombat().reset(); AllZone.getCombat().setAttackingPlayer(turn); diff --git a/src/main/java/forge/Player.java b/src/main/java/forge/Player.java index a4da061ad16..b134810a0fd 100644 --- a/src/main/java/forge/Player.java +++ b/src/main/java/forge/Player.java @@ -1,5 +1,6 @@ package forge; +import forge.game.GameLossReason; import forge.card.cardFactory.CardFactoryUtil; import forge.card.mana.ManaPool; import forge.card.spellability.Ability; @@ -24,9 +25,10 @@ public abstract class Player extends MyObservable { protected int numPowerSurgeLands; protected boolean altWin = false; - protected String winCondition = ""; + protected String altWinSourceName; protected boolean altLose = false; - protected String loseCondition = ""; + protected GameLossReason lossState = GameLossReason.DidNotLoseYet; + protected String loseConditionSpell; protected int nTurns = 0; protected boolean skipNextUntap = false; @@ -43,6 +45,7 @@ public abstract class Player extends MyObservable { protected ManaPool manaPool = null; protected Object mustAttackEntity = null; + /** *Constructor for Player.
@@ -81,9 +84,10 @@ public abstract class Player extends MyObservable { slowtripList = new CardList(); nTurns = 0; altWin = false; + altWinSourceName = null; altLose = false; - winCondition = ""; - loseCondition = ""; + lossState = GameLossReason.DidNotLoseYet; + loseConditionSpell = null; maxLandsToPlay = 1; numLandsPlayed = 0; @@ -907,7 +911,8 @@ public abstract class Player extends MyObservable { //lose: else if (!Constant.Runtime.DevMode[0] || AllZone.getDisplay().canLoseByDecking()) { // if devMode is off, or canLoseByDecking is Enabled, run Lose Condition - if (altLoseConditionMet("Milled")) { + if (!cantLose()) { + loseConditionMet(GameLossReason.Milled, null); AllZone.getGameAction().checkStateEffects(); } } @@ -1424,22 +1429,14 @@ public abstract class Player extends MyObservable { return altWin; } - /** - *Getter for the field altLose.
Getter for the field winCondition.
cantLose.
@@ -1539,12 +1538,12 @@ public abstract class Player extends MyObservable { if (cantLose()) return false; - if (altLose) { + if (lossState != GameLossReason.DidNotLoseYet) { return true; } if (poisonCounters >= 10) { - altLoseConditionMet("Poison Counters"); + loseConditionMet( GameLossReason.Poisoned, null ); return true; } @@ -1552,7 +1551,13 @@ public abstract class Player extends MyObservable { return false; } - return getLife() <= 0; + boolean hasNoLife = getLife() <= 0; + if (hasNoLife) { + loseConditionMet( GameLossReason.LifeReachedZero, null); + return true; + } + + return false; } diff --git a/src/main/java/forge/Upkeep.java b/src/main/java/forge/Upkeep.java index bec2c6db9ca..3cf9aa33e20 100644 --- a/src/main/java/forge/Upkeep.java +++ b/src/main/java/forge/Upkeep.java @@ -2345,7 +2345,7 @@ public class Upkeep implements java.io.Serializable { @Override public void resolve() { if (player.getLife() >= 40) - player.altWinConditionMet(source.getName()); + player.altWinBySpellEffect(source.getName()); } };// Ability @@ -2377,7 +2377,7 @@ public class Upkeep implements java.io.Serializable { CardList grave = AllZoneUtil.getPlayerGraveyard(player); grave = grave.filter(AllZoneUtil.creatures); if (grave.size() >= 20) - player.altWinConditionMet(source.getName()); + player.altWinBySpellEffect(source.getName()); } };// Ability @@ -2405,7 +2405,7 @@ public class Upkeep implements java.io.Serializable { @Override public void resolve() { if (c.getCounters(Counters.TOWER) >= 100) - player.altWinConditionMet(c.getName()); + player.altWinBySpellEffect(c.getName()); } };// Ability @@ -2435,7 +2435,7 @@ public class Upkeep implements java.io.Serializable { @Override public void resolve() { if (player.getLife() == 1) - player.altWinConditionMet(source.getName()); + player.altWinBySpellEffect(source.getName()); } };// Ability @@ -2465,7 +2465,7 @@ public class Upkeep implements java.io.Serializable { @Override public void resolve() { if (player.getLife() >= 50) - player.altWinConditionMet(source.getName()); + player.altWinBySpellEffect(source.getName()); } };// Ability @@ -2507,7 +2507,7 @@ public class Upkeep implements java.io.Serializable { playList.remove(source); if (playList.size() == 0 && handList.size() == 0) - player.altWinConditionMet(source.getName()); + player.altWinBySpellEffect(source.getName()); } };// Ability diff --git a/src/main/java/forge/card/abilityFactory/AbilityFactory_Combat.java b/src/main/java/forge/card/abilityFactory/AbilityFactory_Combat.java index 1e837e2ecdc..9b4b0b39253 100644 --- a/src/main/java/forge/card/abilityFactory/AbilityFactory_Combat.java +++ b/src/main/java/forge/card/abilityFactory/AbilityFactory_Combat.java @@ -166,7 +166,7 @@ public class AbilityFactory_Combat { if (AllZone.getStack().size() != 0) return false; // Don't cast it, if the effect is already in place - if (AllZone.getGameInfo().isPreventCombatDamageThisTurn()) return false; + if (AllZone.getPhase().isPreventCombatDamageThisTurn()) return false; Ability_Sub subAb = sa.getSubAbility(); if (subAb != null) @@ -234,7 +234,7 @@ public class AbilityFactory_Combat { public static void fogResolve(final AbilityFactory af, final SpellAbility sa) { // Expand Fog keyword here depending on what we need out of it. - AllZone.getGameInfo().setPreventCombatDamageThisTurn(true); + AllZone.getPhase().setPreventCombatDamageThisTurn(true); } //************************************************************** diff --git a/src/main/java/forge/card/abilityFactory/AbilityFactory_EndGameCondition.java b/src/main/java/forge/card/abilityFactory/AbilityFactory_EndGameCondition.java index 66260e34be9..ff4fddfe713 100644 --- a/src/main/java/forge/card/abilityFactory/AbilityFactory_EndGameCondition.java +++ b/src/main/java/forge/card/abilityFactory/AbilityFactory_EndGameCondition.java @@ -5,6 +5,7 @@ import forge.Card; import forge.ComputerUtil; import forge.Player; import forge.card.spellability.*; +import forge.game.GameLossReason; import java.util.ArrayList; @@ -213,7 +214,7 @@ public class AbilityFactory_EndGameCondition { ArrayListGameInfo class.
+ * + * @author Forge + * @version $Id$ + */ + +// This should be divided into two: the very summary (immutable with only getters) and +// GameObserver class - who should be notified of any considerable ingame event +public class GameSummary { + protected int playerWinner = PlayerIndex.UNDEFINED; + protected int playerGotFirstTurn = PlayerIndex.UNDEFINED; + protected int lastTurnNumber = 0; + + protected GameEndReason winCondition; + protected String spellEffectWin; + protected GamePlayerRating playerRating[] = new GamePlayerRating[2/*players*/]; + + public GameSummary() + { + playerRating[PlayerIndex.AI] = new GamePlayerRating(); + playerRating[PlayerIndex.HUMAN] = new GamePlayerRating(); + } + + public final void end( final GameEndReason condition, int iPlayerWinner, String spellEffectWin ) + { + winCondition = condition; + playerWinner = iPlayerWinner; + this.spellEffectWin = spellEffectWin; + } + + public final boolean isHumanWinner() { return PlayerIndex.HUMAN == playerWinner; } + public final boolean isAIWinner() { return PlayerIndex.AI == playerWinner; } + public final boolean isDraw() { return PlayerIndex.DRAW == playerWinner; } + + public GameEndReason getWinCondition() { return winCondition; } + public GamePlayerRating getPlayerRating(int iPlayer) { + if (iPlayer >= 0 && iPlayer < playerRating.length) { + return playerRating[iPlayer]; + } + return null; + } + + public int getTurnGameEnded() { + return lastTurnNumber; + } + + public final void setPlayerWhoGotFirstTurn(int iPlayer) + { + playerGotFirstTurn = iPlayer; + } + + public void notifyNextTurn() { + lastTurnNumber++; + } + + public String getWinSpellEffect() { + return spellEffectWin; + } + + + + +} diff --git a/src/main/java/forge/game/PlayerIndex.java b/src/main/java/forge/game/PlayerIndex.java new file mode 100644 index 00000000000..8a7b054bb68 --- /dev/null +++ b/src/main/java/forge/game/PlayerIndex.java @@ -0,0 +1,8 @@ +package forge.game; + +public interface PlayerIndex { + int UNDEFINED = Integer.MIN_VALUE; + int DRAW = -1; + int HUMAN = 0; + int AI = 1; +} diff --git a/src/main/java/forge/game/package-info.java b/src/main/java/forge/game/package-info.java new file mode 100644 index 00000000000..abf03bdb001 --- /dev/null +++ b/src/main/java/forge/game/package-info.java @@ -0,0 +1,2 @@ +/** Forge Card Game */ +package forge.game; diff --git a/src/main/java/forge/gui/input/Input_Mulligan.java b/src/main/java/forge/gui/input/Input_Mulligan.java index f02a1dfe611..21f9278864d 100644 --- a/src/main/java/forge/gui/input/Input_Mulligan.java +++ b/src/main/java/forge/gui/input/Input_Mulligan.java @@ -1,6 +1,9 @@ package forge.gui.input; import forge.*; +import forge.game.GamePlayerRating; +import forge.game.PlayerIndex; +import forge.quest.data.QuestData; /** *Input_Mulligan class.
@@ -11,6 +14,9 @@ import forge.*; public class Input_Mulligan extends Input { /** ConstantserialVersionUID=-8112954303001155622L */
private static final long serialVersionUID = -8112954303001155622L;
+
+ private static final int MAGIC_NUMBER_OF_SHUFFLES = 100;
+ private static final int AI_MULLIGAN_THRESHOLD = 5;
/** {@inheritDoc} */
@Override
@@ -27,31 +33,32 @@ public class Input_Mulligan extends Input {
end();
}
+ public int doMulligan( Player player, GamePlayerRating playerRating ) {
+ CardList hand = AllZoneUtil.getPlayerHand(player);
+ for (Card c : hand) { AllZone.getGameAction().moveToLibrary(c); }
+ for (int i = 0; i < MAGIC_NUMBER_OF_SHUFFLES; i++) { player.shuffle(); }
+ int newHand = hand.size() - 1;
+ for (int i = 0; i < newHand; i++) { player.drawCard(); }
+ playerRating.notifyHasMulliganed();
+ playerRating.notifyOpeningHandSize(newHand);
+ return newHand;
+ }
+
/** {@inheritDoc} */
@Override
public void selectButtonCancel() {
- AllZone.getGameInfo().setHumanMulliganedToZero(false);
+ GamePlayerRating humanRating = AllZone.getGameInfo().getPlayerRating(PlayerIndex.HUMAN);
+ Player humanPlayer = AllZone.getHumanPlayer();
- CardList hand = AllZoneUtil.getPlayerHand(AllZone.getHumanPlayer());
- for (Card c : hand)
- AllZone.getGameAction().moveToLibrary(c);
+ int newHand = doMulligan(humanPlayer, humanRating);
- for (int i = 0; i < 100; i++)
- AllZone.getHumanPlayer().shuffle();
-
- int newHand = hand.size() - 1;
-
- AllZone.getGameInfo().addHumanNumberOfTimesMulliganed(1);
-
- if (AllZone.getQuestData() != null) {
- if (AllZone.getQuestData().getInventory().hasItem("Sleight") && AllZone.getGameInfo().getHumanNumberOfTimesMulliganed() == 1)
- newHand++;
- }
- for (int i = 0; i < newHand; i++)
+ QuestData quest = AllZone.getQuestData();
+ if (quest != null && quest.getInventory().hasItem("Sleight") && humanRating.getMulliganCount() == 1) {
AllZone.getHumanPlayer().drawCard();
+ humanRating.notifyOpeningHandSize(newHand + 1);
+ }
if (newHand == 0) {
- AllZone.getGameInfo().setHumanMulliganedToZero(true);
end();
}
}//selectButtonOK()
@@ -61,29 +68,27 @@ public class Input_Mulligan extends Input {
*/
void end() {
//Computer mulligan
- CardList CHandList = AllZoneUtil.getPlayerHand(AllZone.getComputerPlayer());
-
- Card dummy = CHandList.get(0);
+ Player aiPlayer = AllZone.getComputerPlayer();
+ GamePlayerRating aiRating = AllZone.getGameInfo().getPlayerRating(PlayerIndex.AI);
+ boolean aiTakesMulligan = true;
+
//Computer mulligans if there are no cards with converted mana cost of 0 in its hand
- while (CHandList.size() > 5 && CHandList.getValidCards("Card.cmcEQ0", AllZone.getComputerPlayer(), dummy).size() < 2) {
- for (Card c : CHandList)
- AllZone.getGameAction().moveToLibrary(c);
+ while (aiTakesMulligan) {
- for (int i = 0; i < 100; i++)
- AllZone.getComputerPlayer().shuffle();
+ CardList handList = AllZoneUtil.getPlayerHand(aiPlayer);
+ boolean hasLittleCmc0Cards = handList.getValidCards("Card.cmcEQ0", aiPlayer, null).size() < 2;
+ aiTakesMulligan = handList.size() > AI_MULLIGAN_THRESHOLD && hasLittleCmc0Cards;
- int newHand = CHandList.size() - 1;
- for (int i = 0; i < newHand; i++)
- AllZone.getComputerPlayer().drawCard();
-
- CHandList = AllZoneUtil.getPlayerHand(AllZone.getComputerPlayer());
+ if (aiTakesMulligan) {
+ doMulligan(aiPlayer, aiRating);
+ }
}
//Human Leylines
ButtonUtil.reset();
- CardList HHandList = AllZoneUtil.getPlayerHand(AllZone.getHumanPlayer());
+ CardList humanOpeningHand = AllZoneUtil.getPlayerHand(AllZone.getHumanPlayer());
- for (Card c : HHandList) {
+ for (Card c : humanOpeningHand) {
if (c.getName().startsWith("Leyline")) {
if (GameActionUtil.showYesNoDialog(c, "Put onto Battlefield?"))
AllZone.getGameAction().moveToPlay(c);
@@ -91,7 +96,8 @@ public class Input_Mulligan extends Input {
}
//Computer Leylines
- for (Card c : CHandList) {
+ CardList aiOpeningHand = AllZoneUtil.getPlayerHand(AllZone.getComputerPlayer());
+ for (Card c : aiOpeningHand) {
if (c.getName().startsWith("Leyline") && !(c.getName().startsWith("Leyline of Singularity")
&& AllZoneUtil.getCardsInPlay("Leyline of Singularity").size() > 0)) {
AllZone.getGameAction().moveToPlay(c);
@@ -99,8 +105,8 @@ public class Input_Mulligan extends Input {
}
}
- if (AllZone.getGameAction().isStartCut() && !(HHandList.contains(AllZone.getGameAction().getHumanCut())
- || CHandList.contains(AllZone.getGameAction().getComputerCut()))) {
+ if (AllZone.getGameAction().isStartCut() && !(humanOpeningHand.contains(AllZone.getGameAction().getHumanCut())
+ || aiOpeningHand.contains(AllZone.getGameAction().getComputerCut()))) {
AllZone.getGameAction().moveTo(AllZone.getZone(Constant.Zone.Library, AllZone.getHumanPlayer()), AllZone.getGameAction().getHumanCut());
AllZone.getGameAction().moveTo(AllZone.getZone(Constant.Zone.Library, AllZone.getComputerPlayer()), AllZone.getGameAction().getComputerCut());
}
diff --git a/src/main/java/forge/quest/data/QuestData.java b/src/main/java/forge/quest/data/QuestData.java
index c3522f156dd..51f1b50a4c9 100644
--- a/src/main/java/forge/quest/data/QuestData.java
+++ b/src/main/java/forge/quest/data/QuestData.java
@@ -3,6 +3,10 @@ package forge.quest.data;
import forge.*;
import forge.deck.Deck;
import forge.error.ErrorViewer;
+import forge.game.GameLossReason;
+import forge.game.GamePlayerRating;
+import forge.game.GameSummary;
+import forge.game.PlayerIndex;
import forge.properties.ForgeProps;
import forge.properties.NewConstants;
import forge.quest.data.item.QuestInventory;
@@ -482,63 +486,80 @@ public class QuestData {
}
}
+
+ public final int getCreditsRewardForAltWin(final GameLossReason whyAiLost) {
+ int rewardAltWinCondition = 0;
+ switch (whyAiLost) {
+ case LifeReachedZero:
+ break; // nothing special here, ordinary kill
+ case Milled:
+ rewardAltWinCondition = QuestPreferences.getMatchRewardMilledWinBonus();
+ break;
+ case Poisoned:
+ rewardAltWinCondition = QuestPreferences.getMatchRewardPoisonWinBonus();
+ break;
+ case DidNotLoseYet: // must be player's alternate win condition: felidar, helix pinnacle and like this
+ rewardAltWinCondition = QuestPreferences.getMatchRewardAltWinBonus();
+ break;
+ default: // this .checkstyle forces us to write some idiotic code
+ rewardAltWinCondition = 0;
+ }
+ return rewardAltWinCondition;
+ }
+
+ public final int getCreditsRewardForWinByTurn(final int iTurn) {
+ if (iTurn == 1) {
+ return QuestPreferences.getMatchRewardWinFirst();
+ } else if (iTurn <= 5) {
+ return QuestPreferences.getMatchRewardWinByFifth();
+ } else if (iTurn <= 10) {
+ return QuestPreferences.getMatchRewardWinByTen();
+ } else if (iTurn <= 15) {
+ return QuestPreferences.getMatchRewardWinByFifteen();
+ }
+ return 0;
+ }
+
/**
* getCreditsToAdd.
* * @param matchState a {@link forge.quest.data.QuestMatchState} object. * @return a long. */ - public long getCreditsToAdd(QuestMatchState matchState) { - long creds = (long) (QuestPreferences.getMatchRewardBase() + (QuestPreferences.getMatchRewardTotalWins() * win)); - String[] wins = matchState.getWinMethods(); - int[] winTurns = matchState.getWinTurns(); - boolean[] mulliganedToZero = matchState.getMulliganedToZero(); + public final long getCreditsToAdd(final QuestMatchState matchState) { + long creds = (long) (QuestPreferences.getMatchRewardBase() + + (QuestPreferences.getMatchRewardTotalWins() * win)); - if (matchState.getLose() == 0) { - creds += QuestPreferences.getMatchRewardNoLosses(); - } - - for (String s : wins) { - if (s != null) { - if (s.equals("Poison Counters")) { - creds += QuestPreferences.getMatchRewardPoisonWinBonus(); - } else if (s.equals("Milled")) { - creds += QuestPreferences.getMatchRewardMilledWinBonus(); - } else if (s.equals("Battle of Wits") || s.equals("Felidar Sovereign") || s.equals("Helix Pinnacle") || - s.equals("Epic Struggle") || s.equals("Door to Nothingness") || s.equals("Barren Glory") || - s.equals("Near-Death Experience") || s.equals("Mortal Combat") || s.equals("Test of Endurance")) { - creds += QuestPreferences.getMatchRewardAltWinBonus(); - } + boolean hasNeverLost = true; + for (GameSummary game : matchState.getGamesPlayed()) { + if (game.isAIWinner()) { + hasNeverLost = true; + continue; // no rewards for losing a game } - } - for (int i : winTurns) { - if (i == 1) { - creds += QuestPreferences.getMatchRewardWinFirst(); - } else if (i <= 5) { - creds += QuestPreferences.getMatchRewardWinByFifth(); - } else if (i <= 10) { - creds += QuestPreferences.getMatchRewardWinByTen(); - } else if (i <= 15) { - creds += QuestPreferences.getMatchRewardWinByFifteen(); - } - } + GamePlayerRating aiRating = game.getPlayerRating(PlayerIndex.AI); + GamePlayerRating humanRating = game.getPlayerRating(PlayerIndex.HUMAN); + GameLossReason whyAiLost = aiRating.getLossReason(); - for (boolean b : mulliganedToZero) { - if (b == true) { + creds += getCreditsRewardForAltWin(whyAiLost); + creds += getCreditsRewardForWinByTurn(game.getTurnGameEnded()); + + int cntCardsHumanStartedWith = humanRating.getOpeningHandSize(); + if (0 == cntCardsHumanStartedWith) { creds += QuestPreferences.getMatchMullToZero(); } } - if (inventory.getItemLevel("Estates") == 1) { - creds *= 1.1; - } else if (inventory.getItemLevel("Estates") == 2) { - creds *= 1.15; - } else if (inventory.getItemLevel("Estates") == 3) { - creds *= 1.2; + if (hasNeverLost) { + creds += QuestPreferences.getMatchRewardNoLosses(); } - this.addCredits(creds); + switch(inventory.getItemLevel("Estates")) { + case 1: creds *= 1.1; break; + case 2: creds *= 1.15; break; + case 3: creds *= 1.2; break; + default: break; + } return creds; } diff --git a/src/main/java/forge/quest/data/QuestMatchState.java b/src/main/java/forge/quest/data/QuestMatchState.java index 37204ccd118..e94366cdb34 100644 --- a/src/main/java/forge/quest/data/QuestMatchState.java +++ b/src/main/java/forge/quest/data/QuestMatchState.java @@ -1,163 +1,79 @@ package forge.quest.data; +import java.util.ArrayList; +import java.util.List; + +import forge.game.GameSummary; + + /** *QuestMatchState class.
* * @author Forge * @version $Id$ */ + + public class QuestMatchState { - //the way wins were achieved: - //Damage - //Poison Counters - //Battle of Wits - //Mortal Combat - //Milled - //Felidar Sovereign - //... - // - private String[] winMethods = new String[2]; - private int[] winTurns = new int[2]; + public static final int GAMES_PER_MATCH = 3; + public static final int MIN_GAMES_TO_WIN_MATCH = 2; - private boolean[] mulliganedToZero = new boolean[2]; + private Listreset.
- */ - public void reset() { - win = 0; - lose = 0; - winMethods = new String[2]; + public final void addGamePlayed(final GameSummary completedGame) { + gamesPlayed.add(completedGame); } - /** - *addWin.
- */ - public void addWin() { - win++; - winRecently = true; + public final GameSummary[] getGamesPlayed() { + return gamesPlayed.toArray(new GameSummary[gamesPlayed.size()]); } - /** - *addLose.
- */ - public void addLose() { - lose++; - winRecently = false; + public final int getGamesPlayedCount() { + return gamesPlayed.size(); } - /** - *Getter for the field win.
Getter for the field lose.
countWinLose.
- * - * @return a int. - */ - public int countWinLose() { - return win + lose; + public final int getGamesCountWonByHuman() { + int winsHuman = 0; + for (GameSummary game : gamesPlayed) { + if (game.isHumanWinner()) { winsHuman++; } + } + return winsHuman; } - /** - *setWinMethod.
- * - * @param gameNumber a int. - * @param method a {@link java.lang.String} object. - */ - public void setWinMethod(int gameNumber, String method) { - winMethods[gameNumber] = method; + public final int getGamesCountLostByHuman() { + int lossHuman = 0; + for (GameSummary game : gamesPlayed) { + if (!game.isHumanWinner()) { lossHuman++; } + } + return lossHuman; } - /** - *Getter for the field winMethods.
setWinTurn.
- * - * @param gameNumber a int. - * @param turns a int. - */ - public void setWinTurn(int gameNumber, int turns) { - winTurns[gameNumber] = turns; + public final void reset() { + gamesPlayed.clear(); } - - /** - *Getter for the field winTurns.
Getter for the field mulliganedToZero.
Setter for the field mulliganedToZero.
didWinRecently.
- * - * @return a boolean. - */ - public boolean didWinRecently() { - return winRecently; - } - - /** - *isMatchOver.
- * @return whether match is complete - */ - public boolean isMatchOver() - { - return win == 2 || lose == 2; - } - /** - *isMatchOver.
- * @return whether match is won - */ - public boolean isMatchWon() - { - return win == 2; - } } diff --git a/src/main/java/forge/quest/gui/main/QuestMainPanel.java b/src/main/java/forge/quest/gui/main/QuestMainPanel.java index ffcb0ad1d86..14a77a4928a 100644 --- a/src/main/java/forge/quest/gui/main/QuestMainPanel.java +++ b/src/main/java/forge/quest/gui/main/QuestMainPanel.java @@ -149,7 +149,7 @@ public class QuestMainPanel extends QuestAbstractPanel { */ private JPanel createSidePanel() { JPanel panel = new JPanel(); - JPanel optionsPanel;//Create options checkbox list + JPanel optionsPanel; //Create options checkbox list optionsPanel = createOptionsPanel(); List