java 多次元配列について

Question

javaの勉強をしています。
多次元配列がうまくいかずに困っています。

ポーカーを作ろうと思っています。
手札が
スペード A, ダイヤ 2, クラブ 3, ハート 4, スペード 5
のときに多次元配列の中身が,
x = {{"スペード", "A"}, {"ダイヤ", "2"}, {"クラブ", "3"}, {"ハート", "4"}, {"スペード", "5"}}
になるようにしたいのですが{"スペード", "A"}のところが{"スペード", "5"}と表示されてしまいます。

Mainのfor文のところで、模様と数字を表示させるようにしてあります。

java初心者ですので、こうするとソース短くなるよ。
などのアドバイスも頂けると幸いです。

package game;

public class Main {

 public static void main(String[]args){
  Card_index ci = new Card_index();
  Keyin k = new Keyin();

  int hand = 5;
  int joker = 2;

  String[][] x;

  k.println("自分");
  x = ci.drow(hand, joker);
  k.println("----------------");

  /*
  k.println("相手");
  ci.drow(hand, joker);
  k.println("----------------");
  //ci.debug();

  */
  for(int i=0; i<x.length; i++){
   for(int j=0; j<x[i].length; j++){
    k.print(x[i][j]);
   }
   k.println("");
  }
  
  ci.debug();
 }
}

package game;

import java.util.ArrayList;

public class Card_index {

 Keyin k = new Keyin();
 Card[] card = new Card[5];
 private int i;
 private String[][] x;
 private int r;
 private int d;

 final String[] Black = {"スペード", "クラブ", "ジョーカー"};
 final String[] Red = {"ハート", "ダイヤ", "ジョーカー"};

 final String[][] Kind = {Black, Red};
 final int[] Sheet = {13, 2};
 final String[] Color = {"黒", "赤"};
 ArrayList<Integer> deck = new ArrayList<Integer>();

 Card_index(){
  reset();
 }

 void reset(){
  deck.clear();

  for(i=1; i<=(Sheet[0]*4+2); i++){
   deck.add(i);
   //System.out.println(deck.get(i-1));
  }

  // カード4種設定
  for(i=0; i<card.length-1; i++){
   if(i >= 2){
    // red
    card[i] = new Card(Kind[1][i-2], Sheet[0], Color[1]);
   }else{
    //black
    card[i] = new Card(Kind[0][i], Sheet[0], Color[0]);
   }
   //card[i].status();
  }
  // joker
  card[4] = new Card(Kind[1][2], Sheet[1], Color[0], Color[1]);
  //card[4].status();

 }
 String[][] drow(int hand, int joker){
  x = new String[hand][2];
  /*
   * joker 0 = -2
   * joker 1 = -1
   * joker 2 = 0
   */
  switch(joker){
   case 0: joker = 2; break;
   case 1: joker = 1; break;
   case 2: joker = 0; break;
   default: joker = 2; break;
  }
  for(i=0; i<hand; i++){
   r = (int)(Math.random()*(deck.size()-joker))+1;
   d = deck.get(r-1);
   if(d <= 13){
    x[i] = card[0].drow();
    deck.remove(r-1);
   }else if(d <= (13*2)){
    x[i] = card[1].drow();
    deck.remove(r-1);
   }else if(d <= (13*3)){
    x[i] = card[2].drow();
    deck.remove(r-1);
   }else if(d <= (13*4)){
    x[i] = card[3].drow();
    deck.remove(r-1);
   }else{
    x[i] = card[4].drow("joker");
    deck.remove(r-1);
   }

   /*
   // 問題なし
   k.print(x[i][0]);
   k.println(x[i][1]);
   */
  }
  return x;
  //card[4].drow("joker");// debug
 }

 

 void debug(){
  String[] y = new String[2];
  x = new String[2][2];
  
  y = card[0].drow();
  x[0] = y;
  k.print(x[0][0]);
  k.println(x[0][1]);
  
  y = card[0].drow();
  x[1] = y;
  k.print(x[1][0]);
  k.println(x[1][1]);
  
  k.print(x[0][0]);
  k.println(x[0][1]);
  
 }
}

package game;

import java.util.ArrayList;

public class Card {

 Keyin k = new Keyin();
 Judge j = new Judge();
 private String kind;
 private String[] color;
 private int sheet;

 private int r;
 private int d;
 private String[] x = new String[2];
 private String r_convert;
 private String k_convert;
 ArrayList<Integer> kind_deck = new ArrayList<Integer>();
 ArrayList<Integer> joker_deck = new ArrayList<Integer>();

 

 Card(String kind, int sheet, String... color){
  this.kind = kind;
  this.sheet = sheet;
  this.color = color;

  reset();
 }

 void reset(){
  kind_deck.clear();
  joker_deck.clear();

  for(int i=1; i<=13; i++){
   kind_deck.add(i);
  }
  joker_deck.add(0);
  joker_deck.add(1);
 }

 void status(){
  k.println(kind);
  k.println(sheet);
  for(int i=0; i<color.length; i++){
   k.println(color[i]);
  }

 }

 String[] drow(){
  r = (int)(Math.random()*kind_deck.size())+1;
  d = kind_deck.get(r-1);
  kind_deck.remove(r-1);

  k_convert = j.convert(kind);
  if(d >= 11 | d == 1){
   r_convert = j.convert(d);
   k.println(k_convert+" ："+r_convert);
  }else{
   k.println(k_convert+" ："+d);
  }

  x[0] = kind;
  x[1] = ""+d;
  return x;
 }

 String[] drow(String joker){
  r = (int)(Math.random()*joker_deck.size())+1;
  d = joker_deck.get(r-1);
  joker_deck.remove(r-1);

  k_convert = j.convert(kind);
  k.println(k_convert+" ："+color[d]);

  x[0] = kind;
  x[1] = ""+d;
  return x;
 }

 void debug(){
  // joker list の要素数を取得
  for(int i=0; i<joker_deck.size(); i++){
   System.out.println(joker_deck.get(i));
  }

 }
}

Comments

[m.ts10806]

どういうルールのゲームを作ろうとしているのでしょうか。
仕様をきちんとご提示ください。

[tera12]

ポーカーを作ろうと思っています。
手札が
スペード A, ダイヤ 2, クラブ 3, ハート 4, スペード 5
のときに多次元配列の中身が,
x = {{"スペード", "A"}, {"ダイヤ", "2"}, {"クラブ", "3"}, {"ハート", "4"}, {"スペード", "5"}}
になるようにしたいのですが{"スペード", "A"}のところが{"スペード", "5"}と表示されてしまいます。

Answer 1

原因だけ端的に申しますと、class Cardのprivate String[] x = new String[2];です。

String[] x = new String[2];
x[0] = kind;
x[1] = "" + d;
return x;

のように新しい配列を返す必要があります。

---

以下アドバイス

• i,r,d等メンバー変数にする必要がないものが多い
  不必要に状態を持ってしまうので事故の元。
• メンバー変数に1文字はNG
  ローカル変数と名前がかぶって事故の元。
• メンバー変数にはできるだけprivate finalをつける
  予想外の場所で変更され事故の元。
• ごちゃごちゃとトリッキーなことをしだしたときは要注意
  1か月後見て意味が分からなくなる。発想を変えてシンプルにならないか検討。

ご破算にするようですが^^;
折角Cardクラスがあるのに、引いた後は文字列配列になってしまうのはもったいないなと感じました。
参考コードが出せればいいのですが、こういったものはあまりやらないのでどういう作りが便利なのかよくわかりません^^;

Comments

[tera12]

新規配列を返す必要があるのを知りませんでした。
他のアドバイスも今後のプログラミングで気を付けて作成します。

Answer 2

ソースは全部見てませんが、mainのforのところだけ実行しましたが普通に出力できましたので、
問題があるとすれば、多次元配列の作成を失敗しているか(j.convert？)、表示の変換に失敗しているかだと思います。

String[][] x =
{
	{"スペード", "A"},
	{"ダイヤ", "2"},
	{"クラブ", "3"},
	{"ハート", "4"},
	{"スペード", "5"}
};

for(int i=0; i<x.length; i++){
    for(int j=0; j<x[i].length; j++){
	    System.out.print(x[i][j]);
	    //k.print(x[i][j]);
    }
    System.out.println();
    //k.print("");
}

まずは配列が想定通りに作成できているかの確認。
そのあと、k.printの実装を確認してみてはどうでしょうか。

Comments

[tera12]

なるほど。配列が想定通りに動いているか確認せずに、他メソッドと組み合わせていました。
今後プログラム作成する際確認しながら作るようにします。