From f4a4f8a8cbd6a6277eff2cf9eb7cc48c35e99474 Mon Sep 17 00:00:00 2001
From: Joe Zhao <ztuowen@gmail.com>
Date: Wed, 3 Dec 2014 13:14:19 +0800
Subject: Moved helper function's ownership

---
 player/PlayerAction.java     |  7 -------
 player/human/LocalHuman.java | 35 +++++++----------------------------
 2 files changed, 7 insertions(+), 35 deletions(-)
 delete mode 100644 player/PlayerAction.java

(limited to 'player')

diff --git a/player/PlayerAction.java b/player/PlayerAction.java
deleted file mode 100644
index fb7b205..0000000
--- a/player/PlayerAction.java
+++ /dev/null
@@ -1,7 +0,0 @@
-package mahjong.player;
-
-/**
- * Created by joe on 12/3/14.
- */
-public class PlayerAction {
-}
diff --git a/player/human/LocalHuman.java b/player/human/LocalHuman.java
index a977177..dd47666 100644
--- a/player/human/LocalHuman.java
+++ b/player/human/LocalHuman.java
@@ -27,27 +27,6 @@ public class LocalHuman extends GenericPlayer {
             System.out.println(player.getName().concat(":\n\t").concat("Putdown:\t").concat(card.getNormalName()));
     }
 
-    public Card readCardFromInput()
-    {
-        BufferedReader br = new BufferedReader(new
-                InputStreamReader(System.in));
-        String cardStr="Yi Wan";
-        try {
-            cardStr = br.readLine();
-        } catch (IOException e) {
-            e.printStackTrace();
-        }
-        return new Card(cardStr);
-    }
-
-    public Card[] readMultipleCardFromInput(int num)
-    {
-        Card[] res=new Card[num];
-        for (int i=0;i<num;++i)
-            res[i]=readCardFromInput();
-        return res;
-    }
-
     public boolean genConfirm(String question)
     {
         System.out.println(question);
@@ -69,7 +48,7 @@ public class LocalHuman extends GenericPlayer {
     public Card putCard(){
         System.out.println("Your deck now:");
         hand.print();
-        return readCardFromInput();
+        return Card.readCardFromInput();
     }
 
     @Override
@@ -81,24 +60,24 @@ public class LocalHuman extends GenericPlayer {
     @Override
     public Aside confirmPeng(Card card, int from) {
         if (genConfirm("Peng?"))
-            return new Aside(readMultipleCardFromInput(3), Aside.Type.Peng,from);
+            return new Aside(Card.readMultipleCardFromInput(3), Aside.Type.Peng,from);
         return null;
     }
 
     @Override
     public Aside confirmChi(Card card, int from) {
         if (genConfirm("Chi?"))
-            return new Aside(readMultipleCardFromInput(3), Aside.Type.jGang, from);
+            return new Aside(Card.readMultipleCardFromInput(3), Aside.Type.jGang, from);
         return null;
     }
 
     @Override
     public Aside confirmJiaAnGang(Card card, int from) {
         if (genConfirm("Gang?")) {
-            if (readCardFromInput().getId() == card.getId())
-                return new Aside(readMultipleCardFromInput(4), Aside.Type.jGang, from);
+            if (Card.readCardFromInput().getId() == card.getId())
+                return new Aside(Card.readMultipleCardFromInput(4), Aside.Type.jGang, from);
             else
-                return new Aside(readMultipleCardFromInput(4), Aside.Type.aGang, from);
+                return new Aside(Card.readMultipleCardFromInput(4), Aside.Type.aGang, from);
         }
         return null;
     }
@@ -106,7 +85,7 @@ public class LocalHuman extends GenericPlayer {
     @Override
     public Aside confirmMingGang(Card card, int from) {
         if (genConfirm("Gang?"))
-            return new Aside(readMultipleCardFromInput(4), Aside.Type.mGang, from);
+            return new Aside(Card.readMultipleCardFromInput(4), Aside.Type.mGang, from);
         return null;
     }
 
-- 
cgit v1.2.3-70-g09d2