質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
87.20%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

解決済

ババ抜きで手札から同じカードを2枚捨てるメソッドが作れないです

itchee92
itchee92

総合スコア7

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

3回答

0評価

0クリップ

708閲覧

投稿2018/01/30 10:55

編集2018/02/01 03:57

前提・実現したいこと

Javaでトランプのババ抜きを作っていてその中でarrayクラスの配列を参照して同じカードが2枚あったら捨てる(削除する)メソッドを作りたいのですが難しいです。
今はfor文2回で繰り返しているのですがおそらく配列の要素数が変わるためエラーが起きて実行できません。

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

Exception in thread "main" java.util.ConcurrentModificationException

該当のソースコード

lang

public ArrayList<Card> checkSameCards(Player p) { ArrayList<Card> cards = p.getCardHand().getCards(); ArrayList<Card> discards = p.getCardHand().getCards(); for(Card c1: cards) { for(Card c2: cards) {//エラーはこの行に出てます if(c1.getNumber() == c2.getNumber()) { if(c1 != c2) { discards.add(c1); discards.add(c2); } } } } return discards; }

以下がカードが同じかをチェックして捨てるプログラムです

lang

for(Player p: playerList) { //実行しているgetGameメソッドの一部 if(p instanceof User) { p.getCardHand().showAllCards(); } ArrayList<Card> cards = judge.checkSameCards(p); for(Card c: cards) { p.discardCard(c); } } for(Player p: playerList) { if(p instanceof User) { p.getCardHand().showAllCards(); } }

参考までにプレーヤクラスも載せときます

lang

public abstract class Player { //プレーヤクラス /** プレーヤー名 */ private String name; /** 手札 */ private CardHand hand = new CardHand(); /** * 名前を指定して、じゃんけんのプレイヤーを作る * @param name プレイヤー名 */ public Player(String name) { super(); this.name = name; } /** * 自分の名前を外部に取得させる * @return 名前 */ public String getName() { return name; } /** * 自分の手札を外部に取得させる */ public CardHand getCardHand() { return hand; } /** * 自分の名前をセットする * @return セットする名前 */ public void setName(String name) { this.name = name; } /** * 相手の手札から手札を一枚引く * @return 引いたカード */ public void getCardHand(Player p) { Card c = p.hand.getCardAtRandom(); hand.addCard(c); } public void addCardHand(Card c) { hand.addCard(c); } public void discardCard(Card c) { hand.getCards().remove(c); } /** * 手札の枚数を表示する */ public void showNumberOfCards() { System.out.println(name+"さんの手札は"+hand.getNumberOfCards()+"枚です."); } }

良い質問の評価を上げる

以下のような質問は評価を上げましょう

  • 質問内容が明確
  • 自分も答えを知りたい
  • 質問者以外のユーザにも役立つ

評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

気になる質問をクリップする

クリップした質問は、後からいつでもマイページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

  • プログラミングに関係のない質問
  • やってほしいことだけを記載した丸投げの質問
  • 問題・課題が含まれていない質問
  • 意図的に内容が抹消された質問
  • 過去に投稿した質問と同じ内容の質問
  • 広告と受け取られるような投稿

評価を下げると、トップページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

まだ回答がついていません

会員登録して回答してみよう

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
87.20%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問

同じタグがついた質問を見る

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。