为什么我得到一个空指针异常,我应该使用反射?

时间:2014-11-13 23:54:01

标签: java

我正在编写一个应该玩HiLo游戏的程序。 HiLo是一款游戏,会生成随机数,玩家必须猜测它。如果他们猜错了,他们被告知如果答案高于或低于他们的猜测,那么他们就可以再次猜测。如果需要,我的程序应该能够限制猜测的数量,设置可以从中选择答案的值的范围(因此也可以选择猜测的范围)。我还需要选择一个人是否会玩游戏,或者我是否会让电脑玩,如果是的话,还需要选择什么技能等级(坏,中,好)。

我写了3个课程:GameManager,HiLo和Player。

GameManager类允许玩多个游戏并跟踪获胜次数。

public class GameManager {

private HiLo game;
private int wins;
private int totalRounds;
private int player;

/**
 * Constructor for objects of class GameManager
 */
public GameManager()
{
    player = 2;
}

public void playGames(int rounds)
{
    int i = 0;
    while(i < rounds) {
        game = new HiLo(player);
        game.play();
        wins += game.getHasWon();
        i++;
    }
    totalRounds += rounds;
}

public void changePlayer(int playerType)
{
    player = playerType;
}
} //This should be indented better, but it won't format properly, apologies

HiLo课程基本上设置并播放HiLo的个人游戏。

public class HiLo {

private int guessLimit;
private int lowerLimit;
private int upperLimit;
private int guesses;
private int answer;
private boolean hasWon;
private Player player;
private String result;
/**
 * Constructor for objects of class HiLo
 */
public HiLo(int playerType) //Set default values 
{
    guessLimit = 10;
    lowerLimit = 0;
    upperLimit = 50;
    hasWon = false;
    player = new Player(playerType);
}
public HiLo(int guessLimit, int lowerLimit, int upperLimit, int playerType) //Overload
{
    this.guessLimit = guessLimit;
    this.lowerLimit = lowerLimit;
    this.upperLimit = upperLimit;
    hasWon = false;
    player = new Player(playerType);
}
//Returns lower limit
public int getLowerLimit()
{
    return lowerLimit;
}
//Returns upper limit
public int getUpperLimit()
{
    return upperLimit;
}
//Returns 1 if hasWon is true and 0 if hasWon is false
public int getHasWon()
{
    if(hasWon) {
        return 1;
    } else {
        return 0;
    }
}
//Returns the number of guesses so far
public int getGuesses()
{
    return guesses;
}
//Play a game of HiLo
public void play()
{
    answer = randInt(lowerLimit, upperLimit);
    guesses = 0;
    while(guesses < guessLimit) {
        int guess = player.guess(); //This gets a guess from the player
        result = isCorrect(guess); //Was the guess correct?
        if(result.equals("Win")) {
            break; //If the player has won, break from the while loop
        }
        guesses++;
    }

}
//Checks if the guess was correct
public String isCorrect(int guess)
{
    if(guess == answer) {
        hasWon = true;
        return "Win";
    } else if(guess < answer) {
        lowerLimit = guess;
        return "Higher";
    } else {
        upperLimit = guess;
        return "Lower";
    }

}

public int randInt(int min, int max) 
{
    Random rand = new Random();
    int randomNum = rand.nextInt((max - min) + 1) + min;
    return randomNum;
}
}

玩家等级确定正在玩哪个玩家(0 =人类,1 = badComp,2 = medComp,3 = goodComp)以及他们将用于猜测的策略。

import java.util.Random;
import java.util.Scanner;

public class Player {

private int playerType;
private HiLo game;
private int range;
private int lowestLimit; //This is the starting value of lowerLimit

/**
 * Constructor for objects of class Player
 */
public Player(int playerType)
{
   if(playerType < 0 || playerType > 3) {
       System.out.println("ERROR. Please pick a valid playerType");
   }
   this.playerType = playerType;
   range = 0;
   range = setRange();
   lowestLimit = game.getLowerLimit();
}

public int guess()
{   
    if(playerType == 0) {
        return humanPlayer();
    } else if(playerType == 1) {
        return badComputerPlayer();
    } else if(playerType == 2) {
        return mediumComputerPlayer();
    } else {
        return goodComputerPlayer();
    }
}
//Guess from a good computer player
public int goodComputerPlayer()
{    
   return (game.getUpperLimit() - game.getLowerLimit())/2;
}
//Guess from a medium computer player
public int mediumComputerPlayer()
{
    return randInt(game.getLowerLimit(), game.getUpperLimit());
}
//Guess from a bad computer player
public int badComputerPlayer()
{
    return randInt(lowestLimit, range);
}

public int humanPlayer()
{
    Scanner keyboard = new Scanner(System.in);
    System.out.println("enter an integer");
    int myint = keyboard.nextInt();
    return myint;
}
//Generates a random number in the range maz-min
public int randInt(int min, int max) 
{
    Random rand = new Random();
    int randomNum = rand.nextInt((max - min) + 1) + min;
    return randomNum;
}
//Set the range of values to guess from for bad player
public int setRange()
{
    int r = game.getUpperLimit() - game.getLowerLimit();
    return r;
}
}

当我从HiLo类创建一个对象并输入所有参数时,我得到一个NullPointException:

java.lang.NullPointerException
at Player.setRange(Player.java:75)
at Player.<init>(Player.java:26)
at HiLo.<init>(HiLo.java:35)

我是编程和Java的新手,这是我的第一个NullPointException错误。我已经阅读了其他一些帖子,但我无法弄清楚。

我也不认为玩家类正在做我想要的。 Player类的每个新实例在概念上都与创建该实例的HiLo游戏的实例相关联,因为HiLo游戏一次只能有一个玩家。我希望Player类的新实例引用创建它的HiLo类的实例:

lowestLimit = game.getLowerLimit();

我试图将lowestLimit =设置为创建此Player实例的HiLo游戏实例的下限,但我认为我只是引用了从此创建的对象游戏HiLo类,而不是创建此Player实例的实例。对不起,如果这令人困惑,我会尽力解释:)

我在Player类中的其他几个方法中也这样做。我已经阅读了一些关于使用反射来解决这个问题的事情,但我真的不确定我是否理解它。

非常感谢任何帮助和/或想法,

感谢。

编辑:只是想我告诉你我正在使用BlueJ环境。

2 个答案:

答案 0 :(得分:2)

setRange()方法取决于game不是null。因为在setRange()的构造函数中调用Player,所以它总是抛出NullPointerException,因为game实例变量尚未初始化。

解决此问题的一种方法是让Player的构造函数接受HiLo参数,并在调用game之前将setRange()设置为该值:

public Player(int playerType, HiLo game) {
    this.game = game;
    // ...
}

现在您需要更新player = new Player(playerType);语句以使用正确的参数。由于该语句位于HiLo的实例方法中,因此我们只需将this作为第二个参数传递:

player = new Player(playerType, this);

答案 1 :(得分:0)

以下是setRange代码:

public int setRange() {
    int r = game.getUpperLimit() - game.getLowerLimit();
    return r;
}

投掷NPE的方法只有一种可能的解释。那是 - gamenull

使用它...弄清楚为什么gamenull ...并修复它。

提示:代码中有两个不同的game字段。你正在初始化其中一个......但不是另一个。


  

......我应该使用反射吗?

反思不是NPE的解决方案。