2

我正在开发一个纸牌计数程序。我已经让主程序在运行，但是当我尝试实现自己的类时，我在第 19 行得到一个 NullPointerException 错误（只要它到达c.getRank）。

请注意，我首先在导入一个名为的类时创建了我的主程序，该类CardDeck具有我工作所需的所有功能，但现在我应该创建自己的类来完成完全相同的事情。（请注意，我无权访问导入的CardDeck类）。

这是主要代码：

import se.lth.cs.ptdc.cardGames.Card;

public class Patiens {
public static void main(String[] args) {
    double good = 0;
    double bad = 0;
    double result = 0;

    for (int a = 0; a < 1000000; a++) {
        CardDeck deck = new CardDeck();
        deck.shuffle();
        double fail = 0;
        while (deck.moreCards()) {

            for (int i = 1; i <= 3 && deck.moreCards(); i++) {

                Card c = deck.getCard();

                if (i == 1 && c.getRank() == 1) {
                    fail++;
                }

                if (i == 2 && c.getRank() == 2) {
                    fail++;
                }

                if (i == 3 && c.getRank() == 3) {
                    fail++;
                }
            }
        }
        if (fail >= 1) {
            bad++;      
        }
        else{
            good++;
        }
    }
    System.out.println("Good: " + good + " Bad: " + bad);
    result = good / bad;
    System.out.println("Result= " + result);
}

}

它的作用是计算我的套牌成功完成的概率：

它一边数1-2-3、1-2-3，一边抽牌。现在，如果卡片在计数为“1”时恰好是 ACE，则当前牌组将失败。当程序计数“2”等时，等级 2 的卡片也是如此。它完成而不会失败一次的概率是 0.8% 。

这是我正在创建的 CardDeck 类：

import se.lth.cs.ptdc.cardGames.Card;

import java.util.Random;

public class CardDeck {
    private Card[] cards;
    private int current;
    private static Random rand = new Random();

    public CardDeck() {
        cards = new Card[52];
        for(int suit = Card.SPADES; suit <= Card.CLUBS; suit++) {
            for (int i = 0; i < 13; i++) {
                cards[i * suit] = new Card(suit, i);
            }
        }
        current = 0;
    }

    public void shuffle() {
        Card k;
        for(int i = 1000; i > 0; i--) {
            int nbr = rand.nextInt(52);
            int nbr2 = rand.nextInt(52);
            k = cards[nbr2];
            cards[nbr2] = cards[nbr];
            cards[nbr] = k;
        }
    }

    /**
     *Checks for more cards
     */
    public boolean moreCards() {
        if(current > 51) {
            return false;
        } else {
            return true;
        }
    }

    /**
     *Draws the card lying on top.
     */
    public Card getCard() {
        return cards[current++];

    }
}

这是import se.lth.cs.ptdc.cardGames.Card;如果需要，它是创建卡片的类。

package se.lth.cs.ptdc.cardGames;

public class Card {
    public static final int SPADES = 1;
    public static final int HEARTS = SPADES + 1;
    public static final int DIAMONDS = SPADES + 2;
    public static final int CLUBS = SPADES + 3;

    private int suit;
    private int rank;

    public Card(int suit, int rank) {
        this.suit = suit;
        this.rank = rank;
    }

    public int getSuit() {
        return suit;
    }

    public int getRank() {
        return rank;
    }
}

（请注意，我不应该更改上述课程）

4

3 回答 3

4

你的问题在这里：

cards[i * suit] = new Card(suit, i);

如果您将其更改为：

cards[i + ((suit - 1) * 13)] = new Card(suit, i);

它会做你所期望的。

有两件事要考虑：首先，数组是从零开始的，所以你的第一张牌需要在索引 0 处。其次，乘以花色，你会得到该数字的倍数，例如：

  • 黑桃：1、2、3、4、5、6、7、8、9、10、11、12、13
  • 心：2、4、6、8、10 ...
  • 钻石：3、6、9、12 ...
  • 俱乐部：4、8、12、16 ...

所以有些元素会被多次填充（12 被填充四次），而一些（特定的素数> 13）元素（例如 23）将为空。一般来说，用另一个变量表示索引可能就足够了，如下所示：

int cardIndex = 0;
for (int suit = Card.SPADES; suit <= Card.CLUBS; suit++) {
    for (int i = 0; i < 13; i++) {
        cards[cardIndex++] = new Card(suit, i);
    }
}
于 2012-11-15T09:52:19.730 回答
1

该数组CardDeck.cards包含null元素，因为i * suit当您使用基于 1 的索引时（您Card.SPADES在

于 2012-11-15T09:49:55.937 回答
0

似乎您getCard()的CardDeck类方法返回null而不是返回 Card 对象。在返回之前检查getCard()方法并打印 Card obj。

于 2012-11-15T09:49:29.347 回答