From 5ed3c8bda3dee9971000398ba3caac34326f52a2 Mon Sep 17 00:00:00 2001
From: ericmaxwell2003 <ericmaxwell2003@yahoo.com>
Date: Thu, 19 Jan 2017 14:38:00 -0800
Subject: [PATCH 1/5] View complete

---
 app/src/main/AndroidManifest.xml              |  3 +-
 .../controller/TicTacToeActivity.java         | 17 ++++
 app/src/main/res/layout/tictactoe.xml         | 93 ++++++++++++++++++-
 app/src/main/res/values/dimens.xml            |  2 +-
 app/src/main/res/values/strings.xml           |  1 +
 app/src/main/res/values/styles.xml            |  6 ++
 6 files changed, 115 insertions(+), 7 deletions(-)

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 6ae66cc..0c44595 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -8,10 +8,9 @@
         android:label="@string/app_name"
         android:supportsRtl="true"
         android:theme="@style/AppTheme">
-        <activity android:name=".controller.TicTacToeActivity">
+        <activity android:name=".controller.TicTacToeActivity" android:screenOrientation="portrait">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN" />
-
                 <category android:name="android.intent.category.LAUNCHER" />
             </intent-filter>
         </activity>
diff --git a/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java b/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
index 84ba558..53f7988 100644
--- a/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
+++ b/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
@@ -2,14 +2,31 @@
 
 import android.support.v7.app.AppCompatActivity;
 import android.os.Bundle;
+import android.util.Log;
+import android.view.View;
 
 import com.acme.tictactoe.R;
+import com.acme.tictactoe.model.Board;
 
 public class TicTacToeActivity extends AppCompatActivity {
 
+    private static String TAG = TicTacToeActivity.class.getName();
+
+    private Board gameBoard;
+
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
         setContentView(R.layout.tictactoe);
+        gameBoard = new Board();
+    }
+
+    public void onCellClicked(View button) {
+
+        String tag = button.getTag().toString();
+        int row = Integer.valueOf(tag.substring(0,1));
+        int col = Integer.valueOf(tag.substring(1,2));
+        Log.i(TAG, "Click Row: [" + row + "," + col + "]");
+
     }
 }
diff --git a/app/src/main/res/layout/tictactoe.xml b/app/src/main/res/layout/tictactoe.xml
index d97f0f8..03aaee1 100644
--- a/app/src/main/res/layout/tictactoe.xml
+++ b/app/src/main/res/layout/tictactoe.xml
@@ -1,17 +1,102 @@
 <?xml version="1.0" encoding="utf-8"?>
-<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:tools="http://schemas.android.com/tools"
     android:id="@+id/tictactoe"
+    android:orientation="vertical"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
     android:paddingBottom="@dimen/activity_vertical_margin"
     android:paddingLeft="@dimen/activity_horizontal_margin"
     android:paddingRight="@dimen/activity_horizontal_margin"
     android:paddingTop="@dimen/activity_vertical_margin"
+    android:gravity="center_horizontal"
     tools:context="com.acme.tictactoe.controller.TicTacToeActivity">
 
-    <TextView
+    <GridLayout
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
-        android:text="Hello World!" />
-</RelativeLayout>
+        android:columnCount="3" android:rowCount="3">
+
+        <Button
+            android:tag="00"
+            android:onClick="onCellClicked"
+            style="@style/tictactoebutton"
+            />
+
+        <Button
+            android:tag="01"
+            android:onClick="onCellClicked"
+            style="@style/tictactoebutton"
+            />
+
+        <Button
+            android:tag="02"
+            android:onClick="onCellClicked"
+            style="@style/tictactoebutton"
+            />
+
+        <Button
+            android:tag="10"
+            android:onClick="onCellClicked"
+            style="@style/tictactoebutton"
+            />
+
+        <Button
+            android:tag="11"
+            android:onClick="onCellClicked"
+            style="@style/tictactoebutton"
+            />
+
+        <Button
+            android:tag="12"
+            android:onClick="onCellClicked"
+            style="@style/tictactoebutton"
+            />
+
+        <Button
+            android:tag="20"
+            android:onClick="onCellClicked"
+            style="@style/tictactoebutton"
+            />
+
+        <Button
+            android:tag="21"
+            android:onClick="onCellClicked"
+            style="@style/tictactoebutton"
+            />
+
+        <Button
+            android:tag="22"
+            android:onClick="onCellClicked"
+            style="@style/tictactoebutton"
+            />
+
+    </GridLayout>
+
+    <LinearLayout
+        android:id="@+id/winnerLabels"
+        android:layout_width="match_parent"
+        android:layout_height="match_parent"
+        android:gravity="center"
+        android:orientation="vertical"
+        android:visibility="gone"
+        tools:visibility="visible"
+        >
+
+        <TextView
+            android:id="@+id/winnerPlayerLabel"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:textSize="40sp"
+            android:layout_margin="20dp"
+            tools:text="X" />
+
+        <TextView
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:textSize="30sp"
+            android:text="@string/winner" />
+
+    </LinearLayout>
+
+</LinearLayout>
diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
index 47c8224..277ea4c 100644
--- a/app/src/main/res/values/dimens.xml
+++ b/app/src/main/res/values/dimens.xml
@@ -1,5 +1,5 @@
 <resources>
     <!-- Default screen margins, per the Android Design guidelines. -->
     <dimen name="activity_horizontal_margin">16dp</dimen>
-    <dimen name="activity_vertical_margin">16dp</dimen>
+    <dimen name="activity_vertical_margin">44dp</dimen>
 </resources>
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 40138db..6ce0211 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -1,3 +1,4 @@
 <resources>
     <string name="app_name">TicTacToe</string>
+    <string name="winner">Winner</string>
 </resources>
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index 5885930..3023473 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -8,4 +8,10 @@
         <item name="colorAccent">@color/colorAccent</item>
     </style>
 
+    <style name="tictactoebutton">
+        <item name="android:layout_width">100dp</item>
+        <item name="android:layout_height">100dp</item>
+        <item name="android:textSize">30sp</item>
+    </style>
+
 </resources>

From d365f371a70b43c424d31204da8a73ef2b6c6f96 Mon Sep 17 00:00:00 2001
From: ericmaxwell2003 <ericmaxwell2003@yahoo.com>
Date: Thu, 19 Jan 2017 15:52:51 -0800
Subject: [PATCH 2/5] Finished working MVC

---
 .../controller/TicTacToeActivity.java         | 61 ++++++++++++++++++-
 .../java/com/acme/tictactoe/model/Board.java  | 11 +++-
 .../java/com/acme/tictactoe/model/Player.java |  2 +-
 app/src/main/res/layout/tictactoe.xml         |  8 ++-
 app/src/main/res/menu/menu_tictactoe.xml      | 11 ++++
 app/src/main/res/values/strings.xml           |  1 +
 6 files changed, 88 insertions(+), 6 deletions(-)
 create mode 100644 app/src/main/res/menu/menu_tictactoe.xml

diff --git a/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java b/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
index 53f7988..0fdec14 100644
--- a/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
+++ b/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
@@ -1,12 +1,23 @@
 package com.acme.tictactoe.controller;
 
+import android.content.Intent;
 import android.support.v7.app.AppCompatActivity;
 import android.os.Bundle;
 import android.util.Log;
+import android.view.Menu;
+import android.view.MenuInflater;
+import android.view.MenuItem;
 import android.view.View;
+import android.view.ViewGroup;
+import android.widget.Button;
+import android.widget.LinearLayout;
+import android.widget.TextView;
 
 import com.acme.tictactoe.R;
 import com.acme.tictactoe.model.Board;
+import com.acme.tictactoe.model.Player;
+
+import org.w3c.dom.Text;
 
 public class TicTacToeActivity extends AppCompatActivity {
 
@@ -14,19 +25,67 @@ public class TicTacToeActivity extends AppCompatActivity {
 
     private Board gameBoard;
 
+    private ViewGroup buttonGrid;
+    private View winnerPlayerViewGroup;
+    private TextView winnerPlayerLabel;
+
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
         setContentView(R.layout.tictactoe);
+        winnerPlayerLabel = (TextView) findViewById(R.id.winnerPlayerLabel);
+        winnerPlayerViewGroup = findViewById(R.id.winnerPlayerViewGroup);
+        buttonGrid = (ViewGroup) findViewById(R.id.buttonGrid);
+
         gameBoard = new Board();
     }
 
-    public void onCellClicked(View button) {
+    @Override
+    public boolean onCreateOptionsMenu(Menu menu) {
+        MenuInflater inflater = getMenuInflater();
+        inflater.inflate(R.menu.menu_tictactoe, menu);
+        return true;
+    }
+    @Override
+    public boolean onOptionsItemSelected(MenuItem item) {
+        switch (item.getItemId()) {
+            case R.id.action_reset:
+                reset();
+                return true;
+            default:
+                return super.onOptionsItemSelected(item);
+        }
+    }
+
+    public void reset() {
+        winnerPlayerViewGroup.setVisibility(View.GONE);
+        winnerPlayerLabel.setText("");
+
+        gameBoard.restart();
+
+        for( int i = 0; i < buttonGrid.getChildCount(); i++ ) {
+            ((Button) buttonGrid.getChildAt(i)).setText("");
+        }
+    }
+
+    public void onCellClicked(View v) {
+
+        Button button = (Button) v;
 
         String tag = button.getTag().toString();
         int row = Integer.valueOf(tag.substring(0,1));
         int col = Integer.valueOf(tag.substring(1,2));
         Log.i(TAG, "Click Row: [" + row + "," + col + "]");
 
+        Player playerThatMoved = gameBoard.mark(row, col);
+
+        if(playerThatMoved != null) {
+            button.setText(playerThatMoved.toString());
+            if (gameBoard.getWinner() != null) {
+                winnerPlayerLabel.setText(playerThatMoved.toString());
+                winnerPlayerViewGroup.setVisibility(View.VISIBLE);
+            }
+        }
+
     }
 }
diff --git a/app/src/main/java/com/acme/tictactoe/model/Board.java b/app/src/main/java/com/acme/tictactoe/model/Board.java
index e65bab0..c87a16e 100644
--- a/app/src/main/java/com/acme/tictactoe/model/Board.java
+++ b/app/src/main/java/com/acme/tictactoe/model/Board.java
@@ -22,6 +22,7 @@ public Board() {
      */
     public void restart() {
         clearCells();
+        winner = null;
         currentTurn = Player.X;
         state = GameState.IN_PROGRESS;
     }
@@ -33,11 +34,17 @@ public void restart() {
      *
      * @param row 0..2
      * @param col 0..2
+     * @return the player that moved or null if we did not move anything.
      *
      */
-    public void mark( int row, int col ) {
+    public Player mark( int row, int col ) {
+
+        Player playerThatMoved = null;
+
         if(isValid(row, col)) {
+
             cells[row][col].setValue(currentTurn);
+            playerThatMoved = currentTurn;
 
             if(isWinningMoveByPlayer(currentTurn, row, col)) {
                 state = GameState.FINISHED;
@@ -48,6 +55,8 @@ public void mark( int row, int col ) {
                 flipCurrentTurn();
             }
         }
+
+        return playerThatMoved;
     }
 
     public Player getWinner() {
diff --git a/app/src/main/java/com/acme/tictactoe/model/Player.java b/app/src/main/java/com/acme/tictactoe/model/Player.java
index 73130f8..3be3b79 100644
--- a/app/src/main/java/com/acme/tictactoe/model/Player.java
+++ b/app/src/main/java/com/acme/tictactoe/model/Player.java
@@ -1,3 +1,3 @@
 package com.acme.tictactoe.model;
 
-enum Player { X , O }
+public enum Player { X , O }
diff --git a/app/src/main/res/layout/tictactoe.xml b/app/src/main/res/layout/tictactoe.xml
index 03aaee1..e90ef83 100644
--- a/app/src/main/res/layout/tictactoe.xml
+++ b/app/src/main/res/layout/tictactoe.xml
@@ -13,9 +13,11 @@
     tools:context="com.acme.tictactoe.controller.TicTacToeActivity">
 
     <GridLayout
+        android:id="@+id/buttonGrid"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
-        android:columnCount="3" android:rowCount="3">
+        android:columnCount="3"
+        android:rowCount="3">
 
         <Button
             android:tag="00"
@@ -74,8 +76,8 @@
     </GridLayout>
 
     <LinearLayout
-        android:id="@+id/winnerLabels"
-        android:layout_width="match_parent"
+        android:id="@+id/winnerPlayerViewGroup"
+        android:layout_width="wrap_content"
         android:layout_height="match_parent"
         android:gravity="center"
         android:orientation="vertical"
diff --git a/app/src/main/res/menu/menu_tictactoe.xml b/app/src/main/res/menu/menu_tictactoe.xml
new file mode 100644
index 0000000..f2d442f
--- /dev/null
+++ b/app/src/main/res/menu/menu_tictactoe.xml
@@ -0,0 +1,11 @@
+<menu xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    tools:context=".controller.TicTacToeActivity">
+
+    <item android:id="@+id/action_reset"
+        android:title="@string/action_reset"
+        android:orderInCategory="100"
+        app:showAsAction="ifRoom" />
+
+</menu>
\ No newline at end of file
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 6ce0211..afa6d27 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -1,4 +1,5 @@
 <resources>
     <string name="app_name">TicTacToe</string>
     <string name="winner">Winner</string>
+    <string name="action_reset">Reset</string>
 </resources>

From 2d24a4c2ecb2a09729a803f364d752de205662ea Mon Sep 17 00:00:00 2001
From: ericmaxwell2003 <ericmaxwell2003@yahoo.com>
Date: Thu, 19 Jan 2017 15:54:41 -0800
Subject: [PATCH 3/5] Re arrange method name.

---
 .../controller/TicTacToeActivity.java         | 23 ++++++++++---------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java b/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
index 0fdec14..cae94e8 100644
--- a/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
+++ b/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
@@ -57,17 +57,6 @@ public boolean onOptionsItemSelected(MenuItem item) {
         }
     }
 
-    public void reset() {
-        winnerPlayerViewGroup.setVisibility(View.GONE);
-        winnerPlayerLabel.setText("");
-
-        gameBoard.restart();
-
-        for( int i = 0; i < buttonGrid.getChildCount(); i++ ) {
-            ((Button) buttonGrid.getChildAt(i)).setText("");
-        }
-    }
-
     public void onCellClicked(View v) {
 
         Button button = (Button) v;
@@ -88,4 +77,16 @@ public void onCellClicked(View v) {
         }
 
     }
+
+    private void reset() {
+        winnerPlayerViewGroup.setVisibility(View.GONE);
+        winnerPlayerLabel.setText("");
+
+        gameBoard.restart();
+
+        for( int i = 0; i < buttonGrid.getChildCount(); i++ ) {
+            ((Button) buttonGrid.getChildAt(i)).setText("");
+        }
+    }
+
 }

From 76eafd357ce4e5187124315ce5d63f03058995c9 Mon Sep 17 00:00:00 2001
From: ericmaxwell2003 <ericmaxwell2003@yahoo.com>
Date: Thu, 19 Jan 2017 17:52:07 -0800
Subject: [PATCH 4/5] Updated the name of the gameboard to model.

---
 .../tictactoe/controller/TicTacToeActivity.java    | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java b/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
index cae94e8..a6c843b 100644
--- a/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
+++ b/app/src/main/java/com/acme/tictactoe/controller/TicTacToeActivity.java
@@ -1,6 +1,5 @@
 package com.acme.tictactoe.controller;
 
-import android.content.Intent;
 import android.support.v7.app.AppCompatActivity;
 import android.os.Bundle;
 import android.util.Log;
@@ -10,20 +9,17 @@
 import android.view.View;
 import android.view.ViewGroup;
 import android.widget.Button;
-import android.widget.LinearLayout;
 import android.widget.TextView;
 
 import com.acme.tictactoe.R;
 import com.acme.tictactoe.model.Board;
 import com.acme.tictactoe.model.Player;
 
-import org.w3c.dom.Text;
-
 public class TicTacToeActivity extends AppCompatActivity {
 
     private static String TAG = TicTacToeActivity.class.getName();
 
-    private Board gameBoard;
+    private Board model;
 
     private ViewGroup buttonGrid;
     private View winnerPlayerViewGroup;
@@ -37,7 +33,7 @@ protected void onCreate(Bundle savedInstanceState) {
         winnerPlayerViewGroup = findViewById(R.id.winnerPlayerViewGroup);
         buttonGrid = (ViewGroup) findViewById(R.id.buttonGrid);
 
-        gameBoard = new Board();
+        model = new Board();
     }
 
     @Override
@@ -66,11 +62,11 @@ public void onCellClicked(View v) {
         int col = Integer.valueOf(tag.substring(1,2));
         Log.i(TAG, "Click Row: [" + row + "," + col + "]");
 
-        Player playerThatMoved = gameBoard.mark(row, col);
+        Player playerThatMoved = model.mark(row, col);
 
         if(playerThatMoved != null) {
             button.setText(playerThatMoved.toString());
-            if (gameBoard.getWinner() != null) {
+            if (model.getWinner() != null) {
                 winnerPlayerLabel.setText(playerThatMoved.toString());
                 winnerPlayerViewGroup.setVisibility(View.VISIBLE);
             }
@@ -82,7 +78,7 @@ private void reset() {
         winnerPlayerViewGroup.setVisibility(View.GONE);
         winnerPlayerLabel.setText("");
 
-        gameBoard.restart();
+        model.restart();
 
         for( int i = 0; i < buttonGrid.getChildCount(); i++ ) {
             ((Button) buttonGrid.getChildAt(i)).setText("");

From e42d4f7ee30ab57e81aab0b3baf92c2c79cc3fb5 Mon Sep 17 00:00:00 2001
From: ericmaxwell2003 <ericmaxwell2003@yahoo.com>
Date: Thu, 19 Jan 2017 23:21:59 -0800
Subject: [PATCH 5/5] Update second test desc

---
 app/src/test/java/com/acme/tictactoe/model/TicTacToeTests.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/src/test/java/com/acme/tictactoe/model/TicTacToeTests.java b/app/src/test/java/com/acme/tictactoe/model/TicTacToeTests.java
index 27c4268..b33a136 100644
--- a/app/src/test/java/com/acme/tictactoe/model/TicTacToeTests.java
+++ b/app/src/test/java/com/acme/tictactoe/model/TicTacToeTests.java
@@ -48,7 +48,7 @@ public void test3inRowAcrossTopForX() {
 
 
     /**
-     * This test will simulate and verify x is the winner.
+     * This test will simulate and verify o is the winner.
      *
      *    O | X | X
      *      | O |