使用 .getWidth() 时找不到空指针的原因
Cannot locate cause of nullpointer when using .getWidth()
我很难找到为什么我在这段代码中返回了一个空指针:
import java.awt.Graphics2D;
import java.awt.Rectangle;
public class Move {
private static final int DIAMETER = 30;
int x = 0;
int y = 0;
int x_1 = 1;
int y_1 = 1;
private GameStart game;
public void Ball(GameStart game) {
this.game = game;
}
void moveBall() {
if (x + x_1 < 0) {
x_1 = 1;
}
if (x + x_1 > game.getWidth() - DIAMETER) {
x_1 = -1;
}
if (y + y_1 < 0) {
y_1 = 1;
}
if (y + y_1 > game.getHeight() - DIAMETER) {
game.gameOver();
}
if (collision()) {
y_1 = -1;
y = game.racquet.getTopY() - DIAMETER;
}
x = x + x_1;
y = y + y_1;
}
private boolean collision() {
return game.racquet.getBounds().intersects(getBounds());
}
public void paint(Graphics2D g) {
g.fillOval(x, y, 30, 30);
}
public Rectangle getBounds() {
return new Rectangle(x, y, DIAMETER, DIAMETER);
}
}
GameStart class 在这里:
import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.event.KeyEvent;
@SuppressWarnings("serial")
public class GameStart extends JPanel {
Move move_ball = new Move();
Racquet racquet = new Racquet(this);
public GameStart() {
addKeyListener(new MyKeyListener() {
public void keyTyped(KeyEvent e) {
}
public void keyReleased(KeyEvent e) {
racquet.keyReleased(e);
}
public void keyPressed(KeyEvent e) {
racquet.keyPressed(e);
}
});
setFocusable(true);
}
private void move() {
move_ball.moveBall();
racquet.move();
}
public void paint(Graphics g) {
super.paint(g);
Graphics2D graphics = (Graphics2D) g;
graphics.setRenderingHint(RenderingHints.KEY_ANTIALIASING,
RenderingHints.VALUE_ANTIALIAS_ON);
move_ball.paint(graphics);
racquet.paint(graphics);
}
public void gameOver() {
JOptionPane.showMessageDialog(this, "Spil slut", "Du tabte", JOptionPane.YES_NO_OPTION);
System.exit(ABORT);
}
public static void main (String[] args) throws InterruptedException {
JFrame mainWindow = new JFrame("Matematikken");
GameStart game = new GameStart();
mainWindow.add(game);
mainWindow.setSize(300, 400);
mainWindow.setVisible(true);
mainWindow.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
while (true) {
game.move();
game.repaint();
Thread.sleep(10);
}
}
}
我在行 if (x + x_1 > game.getWidth() - DIAMETER) {
上得到一个空指针。我在另一个 class 中以相同的方式使用 .getWidth()
并且它工作正常 - 但由于某种原因它不会在这里。我试过打印结果,但这只是另一个空指针。我知道我遗漏了一些东西,但我找不到它。
你没有打电话
public void Ball(GameStart game) {
this.game = game;
}
这就是为什么在您的 Move
class 中,GameStart
仍然为空。
您尚未将类型 Move
的实例 move_ball
的成员变量 game
设置为 null
的值,因此抛出 NullPointerException .
您需要实例化此成员变量,例如通过调用 move_ball.Ball(<some Game instance>)
.
此外,请不要让非构造函数的方法以大写字母开头,请始终使用驼峰命名法,而不是在名称格式之间切换。它使您的代码难以阅读。
看来你想要那个方法
public void Ball(GameStart game) { this.game = game; }
真正成为构造函数:
public Move(GameStart game) { this.game = game; }
然后,在实例化 Move
:
时必须使用此构造函数(因为不再有无参数构造函数)
Move move_ball = new Move(this);
您没有通过 Ball
方法将 GameStart
传递给 Move
。这是一个非常具有误导性的名字。它应该是 setGame
或其他东西。
您应该为 Move class 创建一个构造函数,它将游戏作为参数,并将该参数存储在 class.
中的一个变量中
示例:
Public Move(GameStart mygame){
game = mygame;
}
这应该允许您使用游戏中的方法来创建动作。它基本上将取代您拥有的 "Ball" 方法。
我很难找到为什么我在这段代码中返回了一个空指针:
import java.awt.Graphics2D;
import java.awt.Rectangle;
public class Move {
private static final int DIAMETER = 30;
int x = 0;
int y = 0;
int x_1 = 1;
int y_1 = 1;
private GameStart game;
public void Ball(GameStart game) {
this.game = game;
}
void moveBall() {
if (x + x_1 < 0) {
x_1 = 1;
}
if (x + x_1 > game.getWidth() - DIAMETER) {
x_1 = -1;
}
if (y + y_1 < 0) {
y_1 = 1;
}
if (y + y_1 > game.getHeight() - DIAMETER) {
game.gameOver();
}
if (collision()) {
y_1 = -1;
y = game.racquet.getTopY() - DIAMETER;
}
x = x + x_1;
y = y + y_1;
}
private boolean collision() {
return game.racquet.getBounds().intersects(getBounds());
}
public void paint(Graphics2D g) {
g.fillOval(x, y, 30, 30);
}
public Rectangle getBounds() {
return new Rectangle(x, y, DIAMETER, DIAMETER);
}
}
GameStart class 在这里:
import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.event.KeyEvent;
@SuppressWarnings("serial")
public class GameStart extends JPanel {
Move move_ball = new Move();
Racquet racquet = new Racquet(this);
public GameStart() {
addKeyListener(new MyKeyListener() {
public void keyTyped(KeyEvent e) {
}
public void keyReleased(KeyEvent e) {
racquet.keyReleased(e);
}
public void keyPressed(KeyEvent e) {
racquet.keyPressed(e);
}
});
setFocusable(true);
}
private void move() {
move_ball.moveBall();
racquet.move();
}
public void paint(Graphics g) {
super.paint(g);
Graphics2D graphics = (Graphics2D) g;
graphics.setRenderingHint(RenderingHints.KEY_ANTIALIASING,
RenderingHints.VALUE_ANTIALIAS_ON);
move_ball.paint(graphics);
racquet.paint(graphics);
}
public void gameOver() {
JOptionPane.showMessageDialog(this, "Spil slut", "Du tabte", JOptionPane.YES_NO_OPTION);
System.exit(ABORT);
}
public static void main (String[] args) throws InterruptedException {
JFrame mainWindow = new JFrame("Matematikken");
GameStart game = new GameStart();
mainWindow.add(game);
mainWindow.setSize(300, 400);
mainWindow.setVisible(true);
mainWindow.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
while (true) {
game.move();
game.repaint();
Thread.sleep(10);
}
}
}
我在行 if (x + x_1 > game.getWidth() - DIAMETER) {
上得到一个空指针。我在另一个 class 中以相同的方式使用 .getWidth()
并且它工作正常 - 但由于某种原因它不会在这里。我试过打印结果,但这只是另一个空指针。我知道我遗漏了一些东西,但我找不到它。
你没有打电话
public void Ball(GameStart game) {
this.game = game;
}
这就是为什么在您的 Move
class 中,GameStart
仍然为空。
您尚未将类型 Move
的实例 move_ball
的成员变量 game
设置为 null
的值,因此抛出 NullPointerException .
您需要实例化此成员变量,例如通过调用 move_ball.Ball(<some Game instance>)
.
此外,请不要让非构造函数的方法以大写字母开头,请始终使用驼峰命名法,而不是在名称格式之间切换。它使您的代码难以阅读。
看来你想要那个方法
public void Ball(GameStart game) { this.game = game; }
真正成为构造函数:
public Move(GameStart game) { this.game = game; }
然后,在实例化 Move
:
Move move_ball = new Move(this);
您没有通过 Ball
方法将 GameStart
传递给 Move
。这是一个非常具有误导性的名字。它应该是 setGame
或其他东西。
您应该为 Move class 创建一个构造函数,它将游戏作为参数,并将该参数存储在 class.
中的一个变量中示例:
Public Move(GameStart mygame){
game = mygame;
}
这应该允许您使用游戏中的方法来创建动作。它基本上将取代您拥有的 "Ball" 方法。