Moved helper function's ownership

1 files changed, 7 insertions, 28 deletions

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;
     }