ArrayListの重複要素の取り出しについて

Question

前提・実現したいこと

Java初心者です
お金というトランプの遊びをjavaで現在書いています

①重複した要素のあるdeckリストをシャッフルしたのちにplayerがdeckリストの一番上からdeck.get(0);をして要素を取り出したとき、例えばその要素が１００ならplayerリストに追加してまた別の１００の要素をopponentから１枚受け取ってopponentの１００の要素を１枚削除という条件分岐の書き方。５００、１０００、２０００の場合を含む

②例えば１０００の要素をplayerが山札から引いたとき、opponentの手札リストに[100,100,100,100,100]しかない場合などがあると思います。その時にopponentの全要素をplayerにaddしてopponentの全要素を削除する条件分岐の書き方

③例えばplayerがdeckから２０００の要素を引いたとき、opponentのリストの要素は[1000,1000,500,500,500,500]だったとする。この時、２０００の要素はないので①の条件分岐は使えない。必要最小限の組み合わせを用いて（ここでは１０００を２枚）playerへaddしてopponentから要素を削除する条件分岐の書き方

①はなんとか発生している問題にあるようなソースコードを書きましたが、エラーが出ました。IndexOutOfBoundsExceptionindexで（１００）の９９番目にはないよ！というエラーだと思いますがここをどうにかして重複して存在する１００の要素のどれか・・・というソースコードを書きたいです。②と③は１週間くらい考えましたが全く思いつきません。どなたか教えてください
ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

// 引いた手札と同額のカードを相手から貰うメソッド

public void takeOpponentCard(int num) {
	if(num == 100) {
		player.add(opponent.get(100));
		opponent.remove(100);
	} else if(num == 500) {
		player.add(opponent.get(500));
		opponent.remove(500);
	} else if(num == 1000) {
		player.add(opponent.get(1000));
		opponent.remove(1000);
	} else if(num == 2000) {
		player.add(opponent.get(2000));
		opponent.remove(2000);
	}
}

// 引いた手札と同額のカードを相手がプレイヤーから貰うメソッド
public void takePlayerCard(int num) {
if(num == 100) {
opponent.add(player.get(100));
player.remove(100);
} else if(num == 500) {
opponent.add(player.get(500));
player.remove(500);
} else if(num == 1000) {
opponent.add(player.get(1000));
player.remove(1000);
} else if(num == 2000) {
opponent.add(player.get(2000));
player.remove(2000);
}

エラーメッセージ

該当のソースコード

Java
ソースコード
public class Deck {
// 山札の配列メソッド
public void createList() {
deck = new ArrayList<>();
int oneHundredCount = 32;
int fiveHundredCount = 16;
int thousandCount = 4;
int twoThousandCount = 2;

	for(i = 1; i <= oneHundredCount; i++) {
		deck.add(100);
	}
	for(i = 1; i <= fiveHundredCount; i++) {
		deck.add(500);
	}
	for(i = 1; i <= thousandCount; i++) {
		deck.add(1000);
	}
	for(i = 1; i <= twoThousandCount; i++) {
		deck.add(2000);
	}
	Collections.shuffle(deck);
/*	プレイヤー・相手の手札リストと５枚引く行為のメソッド*/

public void playerOpponentDeck(){
player = new ArrayList<>();

	player.add(deck.get(0));
	player.add(deck.get(1));
	player.add(deck.get(2));
	player.add(deck.get(3));
	player.add(deck.get(4));
	
	opponent = new ArrayList<>();
	
	opponent.add(deck.get(5));
	opponent.add(deck.get(6));
	opponent.add(deck.get(7));
	opponent.add(deck.get(8));
	opponent.add(deck.get(9));

// index番号１～１０を削除する(subListメソッド)
deck.subList(0, 10).clear();
}
// カードを交互に引いていくためのメソッド
public void playerDraw() {

	if(drawCount % 2 == 0) {
		player.add(deck.get(i));
		drawCount++;
	} else {
		opponent.add(deck.get(i));
		drawCount++;
	}
}	

// 引いた手札と同額の手札がなかった場合の釣銭メソッド
public void ChargeCard(List<Integer> list) {
charge = new ArrayList<>();
for(i = 1; i <=50; i++) {
charge.add(100);
}
for(i = 1; i <=50; i++) {
charge.add(500);
}
for(i = 1; i <=50; i++) {
charge.add(1000);
}
for(i = 1; i <= 50; i++) {
charge.add(2000);
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

「お金というトランプの遊び」を知らなかったので調べると, プレイヤーの技量は全く関係無いのですね.
そしてどうも, カードは一度引いたら後は合計金額しか意味が無さそうです.
ということで, 直接回答にはならないのですが, 構造を変えることで解決(?)もありえるかと.

MoneyGame.java

import java.util.*;

class Card {
  private int money;
  Card(int money) { this.money = money;  }
  int getMoney() { return money; }
  public String toString() { return ""+money; }
}

class Deck {
  final static private int[][] values = {
      {1*4,1000}, {8*4,100}, {4*4,500}, {1,2000} //A,2-9,10-K,Joker
  };
  private List<Card> cardList = new ArrayList<>();
  Deck() {
    for(int i=0;i<values.length; i++) {
      for(int j=0; j<values[i][0]; j++) cardList.add(new Card(values[i][1]));
    }
    Collections.shuffle(cardList);
  }
  Card get() { return cardList.remove(0); }
  boolean isEmpty() { return cardList.isEmpty(); }
}

class Player {
  private String name;
  private int hand;
  Player(String name) { this.name = name; }
  void add(Card card) { hand += card.getMoney(); }
  void add(int money) { hand += money; }
  int sub(Card card) {
    int payment = Math.min(card.getMoney(), hand);
    hand -= card.getMoney();
    return payment;
  }
  boolean isBankrupt() { return hand < 0; }
  int getHand() { return hand; }
  String getName() { return name; }
  public String toString() { return name+"="+(isBankrupt()?"Bankrupt":""+getHand()); }
}

public class MoneyGame {
  private static final int PLAYERS = 3;
  public static void main(String args[]) {
    Deck deck = new Deck();
    List<Player> playerList = new ArrayList<>(PLAYERS);
    for(int i=0; i<PLAYERS; i++) {
      Player newPlayer = new Player("Player"+(i+1));
      for(int j=0; j<5; j++) newPlayer.add(deck.get());
      playerList.add(newPlayer);
    }
    for(Player player : playerList) System.out.println(player);
    System.out.println("----");

    for(int i=0, left=PLAYERS; !deck.isEmpty() && left>1; i=(i+1)%PLAYERS) {
      Player player = playerList.get(i);
      if(player.isBankrupt()) continue;
      Card draw = deck.get();
      System.out.println(player.getName()+" draw: "+draw);
      player.add(draw);
      for(Player opponent : playerList) {
        if(player != opponent && !opponent.isBankrupt()) {
          player.add(opponent.sub(draw));
          if(opponent.isBankrupt()) left--;
        }
      }
      System.out.println(" > Status "+playerList);
    }

    System.out.println("----");
    for(Player player : playerList) System.out.println(player);
    Collections.sort(playerList, (o1,o2)->{ return o2.getHand()-o1.getHand(); });
    Player top = playerList.get(0);
    StringBuilder sb = new StringBuilder(top.getName());
    for(int i=1; i<playerList.size(); i++) {
      if(playerList.get(i).getHand() < top.getHand()) break;
      sb.append(" and "+playerList.get(i).getName());
    }
    System.out.println("** Winner, "+sb+" !! **");
  }
}

Player1=500
Player2=1700
Player3=2600
----
Player1 draw: 500
 > Status [Player1=2000, Player2=1200, Player3=2100]
Player2 draw: 100
 > Status [Player1=1900, Player2=1500, Player3=2000]
Player3 draw: 100
 > Status [Player1=1800, Player2=1400, Player3=2300]
Player1 draw: 100
 > Status [Player1=2100, Player2=1300, Player3=2200]
Player2 draw: 100
 > Status [Player1=2000, Player2=1600, Player3=2100]
Player3 draw: 500
 > Status [Player1=1500, Player2=1100, Player3=3600]
Player1 draw: 100
 > Status [Player1=1800, Player2=1000, Player3=3500]
Player2 draw: 100
 > Status [Player1=1700, Player2=1300, Player3=3400]
Player3 draw: 100
 > Status [Player1=1600, Player2=1200, Player3=3700]
Player1 draw: 500
 > Status [Player1=3100, Player2=700, Player3=3200]
Player2 draw: 100
 > Status [Player1=3000, Player2=1000, Player3=3100]
Player3 draw: 500
 > Status [Player1=2500, Player2=500, Player3=4600]
Player1 draw: 100
 > Status [Player1=2800, Player2=400, Player3=4500]
Player2 draw: 100
 > Status [Player1=2700, Player2=700, Player3=4400]
Player3 draw: 100
 > Status [Player1=2600, Player2=600, Player3=4700]
Player1 draw: 1000
 > Status [Player1=5200, Player2=Bankrupt, Player3=3700]
Player3 draw: 100
 > Status [Player1=5100, Player2=Bankrupt, Player3=3900]
Player1 draw: 100
 > Status [Player1=5300, Player2=Bankrupt, Player3=3800]
Player3 draw: 100
 > Status [Player1=5200, Player2=Bankrupt, Player3=4000]
Player1 draw: 100
 > Status [Player1=5400, Player2=Bankrupt, Player3=3900]
Player3 draw: 1000
 > Status [Player1=4400, Player2=Bankrupt, Player3=5900]
Player1 draw: 500
 > Status [Player1=5400, Player2=Bankrupt, Player3=5400]
Player3 draw: 1000
 > Status [Player1=4400, Player2=Bankrupt, Player3=7400]
Player1 draw: 2000
 > Status [Player1=8400, Player2=Bankrupt, Player3=5400]
Player3 draw: 100
 > Status [Player1=8300, Player2=Bankrupt, Player3=5600]
Player1 draw: 500
 > Status [Player1=9300, Player2=Bankrupt, Player3=5100]
Player3 draw: 100
 > Status [Player1=9200, Player2=Bankrupt, Player3=5300]
Player1 draw: 100
 > Status [Player1=9400, Player2=Bankrupt, Player3=5200]
Player3 draw: 100
 > Status [Player1=9300, Player2=Bankrupt, Player3=5400]
Player1 draw: 500
 > Status [Player1=10300, Player2=Bankrupt, Player3=4900]
Player3 draw: 100
 > Status [Player1=10200, Player2=Bankrupt, Player3=5100]
Player1 draw: 500
 > Status [Player1=11200, Player2=Bankrupt, Player3=4600]
Player3 draw: 500
 > Status [Player1=10700, Player2=Bankrupt, Player3=5600]
Player1 draw: 100
 > Status [Player1=10900, Player2=Bankrupt, Player3=5500]
Player3 draw: 100
 > Status [Player1=10800, Player2=Bankrupt, Player3=5700]
Player1 draw: 500
 > Status [Player1=11800, Player2=Bankrupt, Player3=5200]
Player3 draw: 100
 > Status [Player1=11700, Player2=Bankrupt, Player3=5400]
Player1 draw: 100
 > Status [Player1=11900, Player2=Bankrupt, Player3=5300]
----
Player1=11900
Player2=Bankrupt
Player3=5300
** Winner, Player1 !! **

Comments

[sirokuma123]

私のコードと比較して見やすくスッキリしており学ぶ点が多々あって、本当に参考になりました。ソースコードを組んでいただき本当にありがとうございました。

Answer 2

こういうこと？
どれを取っても100は100なので、getする意味は全くない。
removeもintのままだと、インデックスを探して削除しようとするため、同じエラーになる。
回避するためにはInteger.valueOfを使ってIntegerに変換する必要がある。

public void takeOpponentCard(int num) {
    if(num == 100) {
        player.add(100);
        opponent.remove(Integer.valueOf(100));
    } else if(num == 500) {
        player.add(500);
        opponent.remove(Integer.valueOf(500));
    } else if(num == 1000) {
        player.add(1000);
        opponent.remove(Integer.valueOf(1000));
    } else if(num == 2000) {
        player.add(2000);
        opponent.remove(Integer.valueOf(2000));
    }
}

あとこれなら、こんなにifを書かなくてもこれでいいのでは？

public void takeOpponentCard(int num) {
    switch(num) {
        case 100:
        case 500:
        case 1000:
        case 2000:
            player.add(num);
            opponent.remove(Integer.valueOf(num));
    }
}

あとこれならplayerとかopponentとかをPlayerクラスみたいなののインスタンスとして扱いたい。

Comments

[sirokuma123]

switch文の書き方は盲点でした。これからの学習の参考にします。ありがとうございました。