2

私はこの方法でデッキを処理しようと試みてきましたが、52枚のカードを処理すると、無限ループになります。

これは、設定されていないカードを取得するまで乱数を生成する必要があるためですが、すべてのカードが設定されると、条件が真になることはなく、無限ループになります。

私は自分の問題を知っていますが、それを修正する方法がわかりません。私は何時間も試してきました。52枚のカードに達したら免除をスローしたいのですが、無限ループに入るとifステートメントに到達することはありません。

public PlayingCard deal() {

    Random swift = new Random();
    int index = swift.nextInt(DECK_SIZE);
    cardsInDeck = DECK_SIZE;
    int i = 51;

    while (this.deck[index] == false||i==cardsInDeck) {index = swift.nextInt(DECK_SIZE);}
    if(i==cardsInDeck) { throw new RuntimeException("Empty Deck");}

    PlayingCard.CardRank[] Ranking = PlayingCard.CardRank.values();
    PlayingCard.CardSuit[] Suiting = PlayingCard.CardSuit.values();

    PlayingCard.CardRank Rank = Ranking[index % 13];
    PlayingCard.CardSuit Suit = Suiting[index % 4];

    PlayingCard selected = new PlayingCard(Suit, Rank);
    this.deck[index] = false;
    i++;
    cardsInDeck--;

    return selected;

}

================================================== ===========================

コード全体

import java.util.Random;

パブリッククラスDeckOfCards{

public static final int DECK_SIZE = 52;
//Instance Variables
private boolean[] deck; //An implicit set of 52 Playing-Cards
private int cardsInDeck;//Number of cards currently in the deck
private Random dealer;  //Used to rendomly select a card to be dealt

//Constructor
public DeckOfCards() {
    this.deck = new boolean[DECK_SIZE];
    int index = 0;
    for (PlayingCard.CardSuit Suit : PlayingCard.CardSuit.values()) {
        for (PlayingCard.CardRank Rank : PlayingCard.CardRank.values()) {
            PlayingCard card = new PlayingCard(Suit, Rank);
            deck[index] = true;
            index++;
        }
    }
}

//Collect all 52 Playing-Cards into the deck
public void shuffle() {
    /*Random shuffle = new Random();
    for (int j = 0; j < this.deck.length; j++) {
    int k = shuffle.nextInt(this.deck.length);
    boolean temp = this.deck[j];
    this.deck[j] = this.deck[k];
    this.deck[k] = temp;*/
    int index = 0;
    for (PlayingCard.CardSuit Suit : PlayingCard.CardSuit.values()) {
        for (PlayingCard.CardRank Rank : PlayingCard.CardRank.values()) {
            PlayingCard card = new PlayingCard(Suit, Rank);
            deck[index] = true;
            index++;
        }

    }
}

//Simulate dealing a randomly selected card from the deck
//Dealing from an empty deck results in a RuntimeException
public PlayingCard deal() {

    Random swift = new Random();
    int index = swift.nextInt(DECK_SIZE);
    cardsInDeck = DECK_SIZE;
    int i = 0;

    while (this.deck[index] == false&&i>0) {index = swift.nextInt(DECK_SIZE);}
    if(i>cardsInDeck) { throw new RuntimeException("Empty Deck");}

    PlayingCard.CardRank[] Ranking = PlayingCard.CardRank.values();
    PlayingCard.CardSuit[] Suiting = PlayingCard.CardSuit.values();

    PlayingCard.CardRank Rank = Ranking[index % 13];
    PlayingCard.CardSuit Suit = Suiting[index % 4];

    PlayingCard selected = new PlayingCard(Suit, Rank);
    this.deck[index] = false;
    i++;
    cardsInDeck--;

    return selected;

}

================================================== ================

public static void main(String[] args) {
    DeckOfCards myDeck = new DeckOfCards();
    myDeck.shuffle();

    for (int p = 1; p <= 4; p++) {
        for (int c = 1; c <= 13; c++) {
            System.out.print(myDeck.deal() + "  ");
        }
        System.out.println();
    }

    try {
        System.out.println(myDeck.deal());
    } catch (RuntimeException rte) {
        System.out.println(rte.getMessage());
    }
}

}

4

3 に答える 3

4

この方法でデッキをディールしようとしてきたのですが、52枚ディールすると無限ループに陥ってしまいます。

カードのデッキを配る簡単な方法は次のとおりです。

  1. すべてのカードを に配置しますArrayList。
  2. Collections.shuffle()そのリストを呼び出します。
  3. シャッフルされたリストに表示される順序でカードを配ります。

コードは現在のものよりもはるかに単純になり、デバッグがはるかに簡単になります。

于 2012-05-19T06:33:31.430 に答える
1

ランダムなインデックスを選択し始めてからでは手遅れです。最初にデッキにカードがあるかどうかをテストする必要があります。

if (cardsInDeck <= 0) ...throw an exception?...
while (this.deck[index] == false) {index = swift.nextInt(DECK_SIZE);}

cardsInDeck正しく初期化する必要があります( DECK_SIZEnot in dealbut inshuffleとコンストラクターに設定)。cardsInDeckのインデックスの数に等しい不変量が必要ですdeck[index]==true。

于 2012-05-19T06:12:29.727 に答える
1

その間、このようなことを試しましたか

    while ((this.deck[index] == false||i==cardsInDeck)&&i<52)

?

于 2012-05-19T05:01:05.867 に答える