Feedback

git-svn-id: svn://sunsvr01.isp.uni-luebeck.de/swproj13/trunk@461 72836036-5685-4462-b002-a69064685172

6 files changed, 108 insertions, 33 deletions

diff --git a/src/jrummikub/view/IStoneCollectionPanel.java b/src/jrummikub/view/IStoneCollectionPanel.java
index 5fa234c..fa3ff73 100644
--- a/src/jrummikub/view/IStoneCollectionPanel.java
+++ b/src/jrummikub/view/IStoneCollectionPanel.java
@@ -4,4 +4,6 @@ package jrummikub.view;
  * The view of the collection that shows the stones a player has selected
  */
 public interface IStoneCollectionPanel extends IStonePanel {
+
+	void setHidden(boolean enable);
 }
diff --git a/src/jrummikub/view/IView.java b/src/jrummikub/view/IView.java
index 33f6284..61925fe 100644
--- a/src/jrummikub/view/IView.java
+++ b/src/jrummikub/view/IView.java
@@ -69,6 +69,14 @@ public interface IView {
 	public IEvent getStartTurnEvent();
 
 	/**
+	 * The start turn event is emitted when the player knows what invalid stones
+	 * he played
+	 * 
+	 * @return the event
+	 */
+	IEvent getAcknowledgeInvalidEvent();
+
+	/**
 	 * The quit event is emitted when the player wants to quit the game
 	 * 
 	 * @return the event
@@ -237,6 +245,8 @@ public interface IView {
 		/** */
 		START_TURN_PANEL,
 		/** */
+		INVALID_TURN_PANEL,
+		/** */
 		HUMAN_HAND_PANEL,
 		/** */
 		COMPUTER_HAND_PANEL,
@@ -244,4 +254,10 @@ public interface IView {
 		WIN_PANEL
 	}
 
+	void setInitialMeldError(int points);
+
+	void setStoneCollectionHidden(boolean enable);
+
+	void setInitialMeldFirstError();
+
 }
diff --git a/src/jrummikub/view/impl/StartTurnPanel.java b/src/jrummikub/view/impl/StartTurnPanel.java
index 6d6e1b5..9418a3b 100644
--- a/src/jrummikub/view/impl/StartTurnPanel.java
+++ b/src/jrummikub/view/impl/StartTurnPanel.java
@@ -13,6 +13,7 @@ import javax.swing.border.EmptyBorder;
 

 import jrummikub.util.Event;

 import jrummikub.util.IEvent;

+import jrummikub.view.IView.BottomPanelType;

 

 /**

  * A panel that is displayed before a player's turn

@@ -29,6 +30,9 @@ class StartTurnPanel extends JPanel {
 	private JButton startTurnButton;

 

 	private Event startTurnEvent = new Event();

+	private Event acknowledgeInvalidEvent = new Event();

+	

+	private Event buttonEvent = startTurnEvent;

 

 	/**

 	 * Creates a new StartTurnPanel

@@ -50,7 +54,7 @@ class StartTurnPanel extends JPanel {
 		startTurnButton.addActionListener(new ActionListener() {

 			@Override

 			public void actionPerformed(ActionEvent arg0) {

-				startTurnEvent.emit();

+				buttonEvent.emit();

 			}

 		});

 		add(startTurnButton);

@@ -66,10 +70,22 @@ class StartTurnPanel extends JPanel {
 	void setCurrentPlayerName(String playerName) {

 		startTurnLabel.setText(playerName + " ist jetzt an der Reihe.");

 	}

+	

+	void setInitialMeldError(int points) {

+		startTurnLabel.setText("Es wurden weniger als " + points + " Punkte ausgelegt.");

+	}

+	

+	void setInitialMeldFirstError() {

+		startTurnLabel.setText("Vor dem Rauskommen darf nicht angelegt werden.");

+	}

 

 	IEvent getStartTurnEvent() {

 		return startTurnEvent;

 	}

+	

+	IEvent getAcknowledgeInvalidEvent() {

+		return acknowledgeInvalidEvent;

+	}

 

 	private void rescale() {

 		Insets insets = getInsets();

@@ -93,4 +109,19 @@ class StartTurnPanel extends JPanel {
 				buttonWidth, buttonHeight);

 		startTurnButton.setFont(startTurnButton.getFont().deriveFont(fontSize));

 	}

+

+	public void setType(BottomPanelType type) {

+		switch (type) {

+		case START_TURN_PANEL:

+			startTurnButton.setText("Zug beginnen");

+			buttonEvent = startTurnEvent;

+			break;

+		case INVALID_TURN_PANEL:

+			startTurnLabel.setText("Es liegen ung\u00FCltige Serien auf dem Tisch.");

+			startTurnButton.setText("N\u00E4chster Spieler");

+			buttonEvent = acknowledgeInvalidEvent;

+			break;

+		}

+		

+	}

 }

diff --git a/src/jrummikub/view/impl/StoneCollectionPanel.java b/src/jrummikub/view/impl/StoneCollectionPanel.java
index c742522..5b08f51 100644
--- a/src/jrummikub/view/impl/StoneCollectionPanel.java
+++ b/src/jrummikub/view/impl/StoneCollectionPanel.java
@@ -37,7 +37,7 @@ class StoneCollectionPanel extends AbstractStonePanel implements
 
 	private Event1<Point> otherClickEvent = new Event1<Point>();
 
-	private boolean pauseMode = false;
+	private boolean hidden = false;
 
 	/**
 	 * Creates a new StoneCollection instance
@@ -123,7 +123,7 @@ class StoneCollectionPanel extends AbstractStonePanel implements
 			}
 		}
 
-		if (pauseMode) {
+		if (hidden) {
 			return;
 		}
 
@@ -155,8 +155,9 @@ class StoneCollectionPanel extends AbstractStonePanel implements
 		}
 	}
 
-	void enablePauseMode(boolean enable) {
-		pauseMode = enable;
+	@Override
+	public void setHidden(boolean enable) {
+		hidden = enable;
 		repaint();
 	}
 }
diff --git a/src/jrummikub/view/impl/TablePanel.java b/src/jrummikub/view/impl/TablePanel.java
index a06eae7..1e8a2b6 100644
--- a/src/jrummikub/view/impl/TablePanel.java
+++ b/src/jrummikub/view/impl/TablePanel.java
@@ -381,7 +381,7 @@ class TablePanel extends AbstractStonePanel implements ITablePanel {
 	}
 
 	void enablePauseMode(boolean enable) {
-		stoneCollection.enablePauseMode(enable);
+		stoneCollection.setHidden(enable);
 
 		pauseMode = enable;
 		setScale();
diff --git a/src/jrummikub/view/impl/View.java b/src/jrummikub/view/impl/View.java
index c18d3b3..83ca40b 100644
--- a/src/jrummikub/view/impl/View.java
+++ b/src/jrummikub/view/impl/View.java
@@ -154,7 +154,8 @@ public class View extends JFrame implements IView {
 		showSettingsPanel(false);
 		showLoginPanel(false);
 		showGameListPanel(false);
-		getHandPanel().setStones(Collections.<Pair<Stone, Position>> emptyList());
+		getHandPanel().setStones(
+				Collections.<Pair<Stone, Position>> emptyList());
 		getTablePanel().setStoneSets(
 				Collections.<Pair<StoneSet, Position>> emptyList());
 		setSelectedStones(Collections.<Stone> emptyList());
@@ -305,8 +306,8 @@ public class View extends JFrame implements IView {
 		mainLayer.add(table);
 
 		playerPanel = new PlayerPanel();
-		playerPanel.setBorder(new MatteBorder(PLAYER_PANEL_BORDER_WIDTH, 0, 0, 0,
-				Color.BLACK));
+		playerPanel.setBorder(new MatteBorder(PLAYER_PANEL_BORDER_WIDTH, 0, 0,
+				0, Color.BLACK));
 		mainLayer.add(playerPanel);
 
 		startTurnPanel = new StartTurnPanel();
@@ -397,6 +398,16 @@ public class View extends JFrame implements IView {
 		startTurnPanel.setCurrentPlayerName(playerName);
 		pausePanel.setCurrentPlayerName(playerName);
 	}
+	
+	@Override
+	public void setInitialMeldError(int points) {
+		startTurnPanel.setInitialMeldError(points);
+	}
+
+	@Override
+	public void setInitialMeldFirstError() {
+		startTurnPanel.setInitialMeldFirstError();
+	}
 
 	@Override
 	public void setCurrentPlayerColor(Color color) {
@@ -414,6 +425,11 @@ public class View extends JFrame implements IView {
 	}
 
 	@Override
+	public IEvent getAcknowledgeInvalidEvent() {
+		return startTurnPanel.getAcknowledgeInvalidEvent();
+	}
+	
+	@Override
 	public IEvent getNewRoundEvent() {
 		return winPanel.getNewRoundEvent();
 	}
@@ -430,24 +446,24 @@ public class View extends JFrame implements IView {
 
 	@SuppressWarnings("unchecked")
 	private List<Pair<Stone, Position>> createDecorationStones() {
-		Pair<Stone, Position> stoneJ = new Pair<Stone, Position>(new Stone(-'J',
-				StoneColor.BLACK), new Position(2.5f, 0));
-		Pair<Stone, Position> stoneR = new Pair<Stone, Position>(new Stone(-'R',
-				StoneColor.ORANGE), new Position(3.5f, 0));
-		Pair<Stone, Position> stoneu1 = new Pair<Stone, Position>(new Stone(-'u',
-				StoneColor.BLUE), new Position(4.5f, 0));
-		Pair<Stone, Position> stonem1 = new Pair<Stone, Position>(new Stone(-'m',
-				StoneColor.RED), new Position(5.5f, 0));
-		Pair<Stone, Position> stonem2 = new Pair<Stone, Position>(new Stone(-'m',
-				StoneColor.GREEN), new Position(6.5f, 0));
-		Pair<Stone, Position> stonei = new Pair<Stone, Position>(new Stone(-'i',
-				StoneColor.VIOLET), new Position(7.5f, 0));
-		Pair<Stone, Position> stonek = new Pair<Stone, Position>(new Stone(-'k',
-				StoneColor.AQUA), new Position(8.5f, 0));
-		Pair<Stone, Position> stoneu2 = new Pair<Stone, Position>(new Stone(-'u',
-				StoneColor.GRAY), new Position(9.5f, 0));
-		Pair<Stone, Position> stoneb = new Pair<Stone, Position>(new Stone(-'b',
-				StoneColor.BLACK), new Position(10.5f, 0));
+		Pair<Stone, Position> stoneJ = new Pair<Stone, Position>(new Stone(
+				-'J', StoneColor.BLACK), new Position(2.5f, 0));
+		Pair<Stone, Position> stoneR = new Pair<Stone, Position>(new Stone(
+				-'R', StoneColor.ORANGE), new Position(3.5f, 0));
+		Pair<Stone, Position> stoneu1 = new Pair<Stone, Position>(new Stone(
+				-'u', StoneColor.BLUE), new Position(4.5f, 0));
+		Pair<Stone, Position> stonem1 = new Pair<Stone, Position>(new Stone(
+				-'m', StoneColor.RED), new Position(5.5f, 0));
+		Pair<Stone, Position> stonem2 = new Pair<Stone, Position>(new Stone(
+				-'m', StoneColor.GREEN), new Position(6.5f, 0));
+		Pair<Stone, Position> stonei = new Pair<Stone, Position>(new Stone(
+				-'i', StoneColor.VIOLET), new Position(7.5f, 0));
+		Pair<Stone, Position> stonek = new Pair<Stone, Position>(new Stone(
+				-'k', StoneColor.AQUA), new Position(8.5f, 0));
+		Pair<Stone, Position> stoneu2 = new Pair<Stone, Position>(new Stone(
+				-'u', StoneColor.GRAY), new Position(9.5f, 0));
+		Pair<Stone, Position> stoneb = new Pair<Stone, Position>(new Stone(
+				-'b', StoneColor.BLACK), new Position(10.5f, 0));
 
 		Pair<Stone, Position> stone1 = new Pair<Stone, Position>(new Stone(
 				StoneColor.RED), new Position(2, 1));
@@ -462,9 +478,9 @@ public class View extends JFrame implements IView {
 		Pair<Stone, Position> stone6 = new Pair<Stone, Position>(new Stone(
 				StoneColor.BLACK), new Position(11, 1));
 
-		return Arrays
-				.asList(stoneJ, stoneR, stoneu1, stonem1, stonem2, stonei, stonek,
-						stoneu2, stoneb, stone1, stone2, stone3, stone4, stone5, stone6);
+		return Arrays.asList(stoneJ, stoneR, stoneu1, stonem1, stonem2, stonei,
+				stonek, stoneu2, stoneb, stone1, stone2, stone3, stone4,
+				stone5, stone6);
 	}
 
 	@Override
@@ -475,17 +491,26 @@ public class View extends JFrame implements IView {
 	}
 
 	private void doSetBottomPanel(BottomPanelType type) {
-		startTurnPanel.setVisible(type == BottomPanelType.START_TURN_PANEL);
+		boolean showStartTurnPanel = type == BottomPanelType.START_TURN_PANEL
+				|| type == BottomPanelType.INVALID_TURN_PANEL;
+		startTurnPanel.setVisible(showStartTurnPanel);
+		startTurnPanel.setType(type);
 		winPanel.setVisible(type == BottomPanelType.WIN_PANEL);
-		playerPanel.setVisible(type != BottomPanelType.START_TURN_PANEL
+		playerPanel.setVisible((!showStartTurnPanel)
 				&& type != BottomPanelType.WIN_PANEL && type != null);
 
 		if (type == BottomPanelType.START_GAME_PANEL) {
-			table.setStoneSets(Collections.<Pair<StoneSet, Position>> emptyList());
+			table.setStoneSets(Collections
+					.<Pair<StoneSet, Position>> emptyList());
 			playerPanel.getHandPanel().setStones(createDecorationStones());
 		}
 
 		playerPanel.showButtons(type != BottomPanelType.START_GAME_PANEL);
 		playerPanel.enableButtons(type != BottomPanelType.COMPUTER_HAND_PANEL);
 	}
+	
+	@Override
+	public void setStoneCollectionHidden(boolean enable) {
+		table.getStoneCollectionPanel().setHidden(enable);
+	}
 }