1

私の主な方法では、新しいデッキを作成しようとすると、Eclipseが「DeckOfCardsを変数に解決できません」と表示されます

これは、移植性を高めるために複数の .java ファイルを 1 つに結合していたときに発生し始めました。

これを修正しようとして眠れなくなっています...助けてください

import java.util.Random;

public class PokerGame {

    class Card {

        private String face; // face of card ("Ace", "Deuce", ...)
        private String suit; // suit of card ("Hearts", "Diamonds", ...)

        // two-argument constructor initializes card's face and suit
        public Card(String cardFace, String cardSuit) {

            face = cardFace; // initialize face of card
            suit = cardSuit; // initialize suit of card
        } // end two-argument Card constructor

        // return String representation of Card
        public String toString() {
            return face + " of " + suit;
        } // end method toString
    }

    class DeckOfCards {

        private Card[] deck; // arrays of Card objects
        private int currentCard; // index of next Card to be dealt (0-51)
        private static final int NUMBER_OF_CARDS = 52; // constant # of Cards
        // random number generator
        private final Random randomNumbers = new Random();

        // constructor fills deck of Cards
        public DeckOfCards() {
            String[] faces = {
                "Ace", "Deuce", "Three", "Four", "Five", "Six", "Seven", "Eight", "Nine", "Ten", "Jack", "Queen", "King"
            };
            String[] suits = {
                "Hearts", "Diamonds", "Clubs", "Spades"
            };

            deck = new Card[NUMBER_OF_CARDS]; // create array of Card objects
            currentCard = 0; // set currentCard so first Card dealt is deck[ 0 ]

            // populate deck with Card objects
            for (int count = 0; count < deck.length; count++)
            deck[count] = new Card(faces[count % 13], suits[count / 13]);
        } // end DeckOfCards constructor

        // shuffle deck of Cards with one-pass algorithm
        public void shuffle() {
            // after shuffling, dealing should start at deck[ 0 ] again
            currentCard = 0; // reinitialize currentCard

            // for each Card, pick another random Card (0-51) and swap them
            for (int first = 0; first < deck.length; first++) {
                // select a random number between 0 and 51 
                int second = randomNumbers.nextInt(NUMBER_OF_CARDS);

                // swap current Card with randomly selected Card
                Card temp = deck[first];
                deck[first] = deck[second];
                deck[second] = temp;
            } // end for
        } // end method shuffle

        // deal one Card
        public Card dealCard() {
            // determine whether Cards remain to be dealt
            if (currentCard < deck.length) return deck[currentCard++]; // return current Card in array
            else return null; // return null to indicate that all Cards were dealt
        } // end method dealCard

    }
    public static void main(String[] args) {
        Card[] hand1, hand2;
        PokerGame.DeckOfCards myDeckOfCards = DeckOfCards.new DeckOfCards();
        myDeckOfCards.shuffle(); // place Cards in random order

        hand1 = new Card[5];
        hand2 = new Card[5];

        // print 5 Cards in the order in which they are dealt
        for (int i = 0; i < 10; i++) {

            if (i == 0) //display hand labels
            System.out.printf("Hand 1: \t   Hand 2: \n");

            if (i < 5) {
                hand1[i] = myDeckOfCards.dealCard();
                System.out.printf("%-19s", hand1[i]);
            }
            if (i >= 5) {
                hand2[i - 5] = myDeckOfCards.dealCard();
                System.out.printf("%-19s", hand2[i - 5]);
            }

            if ((i + 1) % 2 == 0) // output a newline after every other card
            System.out.println();
        } // end for
    }

}
4

5 に答える 5

2

変化する

PokerGame.DeckOfCards myDeckOfCards = DeckOfCards.new DeckOfCards();

に

PokerGame.DeckOfCards myDeckOfCards = new PokerGame.DeckOfCards();
于 2013-02-26T05:22:46.867 に答える
1

あなたの2つのクラスCardとDeckOfCardsはstatic、次のようになります。

static class Card {
    ...
}

static class DeckOfCards {
   ...
}

それ以外の場合は、それらを作成するためにクラスのインスタンスが必要です。これは、 「状態」がない (フィールド/変数がないPokerGame) ため、設計上の欠陥です。PokerGame

実際には、それらは独自の最上位クラス (独自の Java ファイル内) である必要があります。

于 2013-02-26T05:24:52.890 に答える
0

このように、CardとDeckOfCardsの定義をPokerGameクラスの定義の外に移動します

public class PokerGame {
   //your code goes here
}

class Card {
   //your code goes here
}

class DeckOfCards{
   //your code goes here
}

これにより、内部クラスのインスタンス化の問題は解決されますが、適切な設計ではありません。クラス定義はそれぞれのファイルに含める必要があります。これにより、後で変更を簡単に組み込むことができます。

于 2013-02-26T05:38:57.383 に答える
0 
DeckOfCards.new DeckOfCards();

これは間違っています。試してみるnew PokerGame.DeckOfCards()か、ただnew DeckOfCards();

于 2013-02-26T05:21:10.647 に答える
0

それは内部クラスがどのように機能するかではありません。

最初にインスタンスのインスタンスを作成する必要がありPokerGameます:

PokerGame game = new PokerGame();
DeckOfCards myDeckOfCards = game.new DeckOfCards();
...
于 2013-02-26T05:24:47.910 に答える