diff options
-rw-r--r-- | src/jrummikub/control/RoundControl.java | 16 | ||||
-rw-r--r-- | src/jrummikub/view/IHandPanel.java | 18 | ||||
-rw-r--r-- | src/jrummikub/view/impl/HandPanel.java | 59 | ||||
-rw-r--r-- | test/jrummikub/control/RoundControlTest.java | 30 |
4 files changed, 85 insertions, 38 deletions
diff --git a/src/jrummikub/control/RoundControl.java b/src/jrummikub/control/RoundControl.java index efa6f62..154fcb9 100644 --- a/src/jrummikub/control/RoundControl.java +++ b/src/jrummikub/control/RoundControl.java @@ -14,6 +14,9 @@ import jrummikub.util.Pair; import jrummikub.view.IView; public class RoundControl { + // TODO move to game control once existent + final static int HAND_HEIGHT = 2; + final static int HAND_WIDTH = 14; private IGameState gameState; private IView view; private ITable clonedTable; @@ -21,6 +24,8 @@ public class RoundControl { public RoundControl(IGameState gameState, IView view) { this.gameState = gameState; this.view = view; + view.getPlayerPanel().getHandPanel().setHandHeight(HAND_HEIGHT); + view.getPlayerPanel().getHandPanel().setHandWidth(HAND_WIDTH); } public void startRound() { @@ -68,14 +73,17 @@ public class RoundControl { for (int i = 0; i < gameState.getPlayerCount(); i++) { IHand hand = gameState.getNthNextPlayer(i).getHand(); for (int j = 0; j < 7; j++) { - hand.drop(gameState.getGameHeap().drawStone(), new Position(j, 0)); - hand.drop(gameState.getGameHeap().drawStone(), new Position(j, 1)); + hand.drop(gameState.getGameHeap().drawStone(), new Position(j, + 0)); + hand.drop(gameState.getGameHeap().drawStone(), new Position(j, + 1)); } } } private void endOfTurn() { - Set<Stone> tableDiff = tableDifference(gameState.getTable(), clonedTable); + Set<Stone> tableDiff = tableDifference(gameState.getTable(), + clonedTable); if (!tableDiff.isEmpty()) { // Player has made a move if (clonedTable.isValid()) { @@ -118,7 +126,7 @@ public class RoundControl { return ret; } - private void dealStone() { + void dealStone() { gameState .getActivePlayer() .getHand() diff --git a/src/jrummikub/view/IHandPanel.java b/src/jrummikub/view/IHandPanel.java index a58edab..3436062 100644 --- a/src/jrummikub/view/IHandPanel.java +++ b/src/jrummikub/view/IHandPanel.java @@ -8,11 +8,15 @@ import jrummikub.util.Pair; * The view for a player's hand that displays his stones */ public interface IHandPanel extends IStonePanel, IClickable { - /** - * Set the player's stones to display on the board - * - * @param stones - * the stones - */ - public void setStones(Iterable<Pair<Stone, Position>> stones); + /** + * Set the player's stones to display on the board + * + * @param stones + * the stones + */ + public void setStones(Iterable<Pair<Stone, Position>> stones); + + public void setHandWidth(int width); + + public void setHandHeight(int height); } diff --git a/src/jrummikub/view/impl/HandPanel.java b/src/jrummikub/view/impl/HandPanel.java index 38a8440..6ad6270 100644 --- a/src/jrummikub/view/impl/HandPanel.java +++ b/src/jrummikub/view/impl/HandPanel.java @@ -25,15 +25,15 @@ import jrummikub.view.IHandPanel; */ @SuppressWarnings("serial") class HandPanel extends AbstractStonePanel implements IHandPanel { - private final static int HAND_HEIGHT = 2; - private final static int HAND_WIDTH = 14; + private int handWidth = 1; + private int handHeight = 1; private final static BufferedImage BACKGROUND; static { ImageIcon image = new ImageIcon( HandPanel.class.getResource("/jrummikub/resource/wood.png")); - BACKGROUND = new BufferedImage(image.getIconWidth(), image.getIconHeight(), - BufferedImage.TYPE_INT_RGB); + BACKGROUND = new BufferedImage(image.getIconWidth(), + image.getIconHeight(), BufferedImage.TYPE_INT_RGB); image.paintIcon(null, BACKGROUND.createGraphics(), 0, 0); } @@ -42,6 +42,20 @@ class HandPanel extends AbstractStonePanel implements IHandPanel { private boolean repaintAll = true; private Collection<Stone> selectedStones = Collections.emptyList(); + + @Override + public void setHandWidth(int width) { + handWidth = width; + rescale(); + repaint(); + } + + @Override + public void setHandHeight(int height) { + handHeight = height; + rescale(); + repaint(); + } /** * Creates a new Board instance @@ -53,15 +67,7 @@ class HandPanel extends AbstractStonePanel implements IHandPanel { @Override public void componentResized(ComponentEvent e) { - Insets insets = getInsets(); - int size = (getHeight() - insets.top - insets.bottom) / HAND_HEIGHT; - - getStonePainter().setScale(size * StonePainter.HEIGHT_SCALE); - - setSize(new Dimension(HAND_WIDTH * getStonePainter().getStoneWidth() - + insets.left + insets.right, getHeight())); - - repaintAll = true; + rescale(); } }); } @@ -82,16 +88,17 @@ class HandPanel extends AbstractStonePanel implements IHandPanel { protected void paintComponent(Graphics g1) { Insets insets = getInsets(); int x = insets.left, y = insets.top, width = getWidth() - insets.left - - insets.right, height = getHeight() - insets.top - insets.bottom; + - insets.right, height = getHeight() - insets.top + - insets.bottom; Graphics2D g = (Graphics2D) g1.create(x, y, width, height); - int size = height / HAND_HEIGHT; + int size = height / handHeight; if (repaintAll) { if (scaledBackground.getHeight() != size) scaledBackground = getScaledBackground(size); - for (int i = 0; i < HAND_HEIGHT; ++i) { + for (int i = 0; i < handHeight; ++i) { for (int xpos = -size * i / 3; xpos < width; xpos += size) { g.drawImage(scaledBackground, xpos, size * i, null); } @@ -99,8 +106,10 @@ class HandPanel extends AbstractStonePanel implements IHandPanel { } for (Pair<Stone, Position> entry : getStones()) { - getStonePainter().paintStone(g, entry.getFirst(), entry.getSecond(), - selectedStones.contains(entry.getFirst()), entry.getFirst() == getHoveredStone()); + getStonePainter().paintStone(g, entry.getFirst(), + entry.getSecond(), + selectedStones.contains(entry.getFirst()), + entry.getFirst() == getHoveredStone()); } } @@ -115,10 +124,22 @@ class HandPanel extends AbstractStonePanel implements IHandPanel { * Sets the stones that are to be painted selected * * @param stones - * the selected stones + * the selected stones */ void setSelectedStones(Collection<Stone> stones) { selectedStones = stones; repaint(); } + + private void rescale() { + Insets insets = getInsets(); + int size = (getHeight() - insets.top - insets.bottom) / handHeight; + + getStonePainter().setScale(size * StonePainter.HEIGHT_SCALE); + + setSize(new Dimension(handWidth * getStonePainter().getStoneWidth() + + insets.left + insets.right, getHeight())); + + repaintAll = true; + } } diff --git a/test/jrummikub/control/RoundControlTest.java b/test/jrummikub/control/RoundControlTest.java index 38060c0..f183505 100644 --- a/test/jrummikub/control/RoundControlTest.java +++ b/test/jrummikub/control/RoundControlTest.java @@ -49,7 +49,8 @@ public class RoundControlTest { - testGameState.table.getSize(), testGameState.getGameHeap() .getSize()); for (int i = 0; i < testGameState.getPlayerCount(); i++) { - assertEquals(14, testGameState.getNthNextPlayer(i).getHand().getSize()); + assertEquals(14, testGameState.getNthNextPlayer(i).getHand() + .getSize()); } } @@ -67,8 +68,8 @@ public class RoundControlTest { private void checkTableDisplay() { Iterator<Pair<StoneSet, Position>> stoneSetsView = view.tablePanel.stoneSets .iterator(); - Iterator<Pair<StoneSet, Position>> stoneSetsModel = testGameState.table.clone().sets - .iterator(); + Iterator<Pair<StoneSet, Position>> stoneSetsModel = testGameState.table + .clone().sets.iterator(); while (stoneSetsView.hasNext()) { assertTrue(stoneSetsModel.hasNext()); @@ -85,6 +86,17 @@ public class RoundControlTest { view.displayStartTurnPanel = false; } + // TODO hier weitermachen + @Test + public void testDealStone() { + testRound.deal(); + checkCorrectlyDealed(); + for (int i = 0; i < 28 - 14; i++) { + testRound.dealStone(); + } + assertEquals(28, testGameState.getActivePlayer().getHand().getSize()); + } + @Test public void testDeal() { testRound.deal(); @@ -136,7 +148,8 @@ public class RoundControlTest { view.startTurnEvent.emit(); assertFalse(view.displayStartTurnPanel); - Stone stone = testGameState.players.get(0).hand.stones.remove(0).getFirst(); + Stone stone = testGameState.players.get(0).hand.stones.remove(0) + .getFirst(); newTable.drop(new StoneSet(stone), new Position(0, 0)); resetTurnStart(); view.getPlayerPanel().endTurnEvent.emit(); @@ -194,7 +207,8 @@ public class RoundControlTest { view.startTurnEvent.emit(); assertFalse(view.displayStartTurnPanel); - Stone stone = testGameState.players.get(0).hand.stones.remove(0).getFirst(); + Stone stone = testGameState.players.get(0).hand.stones.remove(0) + .getFirst(); newTable.drop(new StoneSet(stone), new Position(0, 0)); testGameState.players.get(0).hand.stones.clear(); resetTurnStart(); @@ -224,13 +238,13 @@ public class RoundControlTest { blueThree)); newTable.drop(newSet1, new Position(0, 0)); newTable.drop(newSet2, new Position(0, 0)); - + Set<Stone> expectedStones = new HashSet<Stone>(); expectedStones.add(blueThree); expectedStones.add(blueFour); - + Set<Stone> stones = RoundControl.tableDifference(oldTable, newTable); - + assertTrue(expectedStones.containsAll(stones)); assertTrue(stones.containsAll(expectedStones)); } |