Commit 167320a

mo khan <mo@mokhan.ca>
2019-08-04 22:10:07
ensure max continuous points calc works for player 1
1 parent 286134f
src/Q8/BadmintonScoringWithStroke.java
@@ -72,14 +72,16 @@ public class BadmintonScoringWithStroke extends BadmintonScoring {
   }
 
   private int[][] to2DArray(ArrayList<Point> points) {
-    int[][] scores = new int[points.size()][2];
+    int[][] scores = new int[points.size() + 1][2];
+    scores[0][Point.PLAYER1] = 0;
+    scores[0][Point.PLAYER2] = 0;
 
     for (int i = 0; i < points.size(); i++) {
       Point point = points.get(i);
 
-      scores[i][Point.PLAYER1] = i == 0 ? 0 : scores[i - 1][Point.PLAYER1];
-      scores[i][Point.PLAYER2] = i == 0 ? 0 : scores[i - 1][Point.PLAYER2];
-      scores[i][point.getPlayer()] = point.getScore();
+      scores[i + 1][Point.PLAYER1] = i == 0 ? 0 : scores[i - 1][Point.PLAYER1];
+      scores[i + 1][Point.PLAYER2] = i == 0 ? 0 : scores[i - 1][Point.PLAYER2];
+      scores[i + 1][point.getPlayer()] = point.getScore();
     }
 
     return scores;
src/Q8/BadmintonScoringWithStrokeTest.java
@@ -67,9 +67,9 @@ public class BadmintonScoringWithStrokeTest extends TestCase {
     assertEquals(21, this.subject.getPlayer2Points());
   }
 
-  // public void test_getContinuousPointsPlayer1() {
-  // assertEquals(2, this.subject.getContinuousPointsPlayer1());
-  // }
+  public void test_getContinuousPointsPlayer1() {
+    assertEquals(2, this.subject.getContinuousPointsPlayer1());
+  }
 
   // public void test_getContinuousPointsPlayer2() {
   // assertEquals(9, this.subject.getContinuousPointsPlayer2());