使用 KeyListener 使用 setText 时出现 NullPointerException

Question

这是一个猜词游戏。每次都显示第一个字母，而其余字母则不显示。每当用户键入我希望它更改板上的字母时，这里是一个屏幕截图：http://puu.sh/7moQS.jpg

错误：

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at BoardUI.keyTyped(BoardUI.java:48)

BoardUI.java:48 行在哪里

b.getLetters()[b.position(b.getCurrentRow(), b.getCurrentColumn())].setText(typed + "");

Game.java

public class Game {
}

testGUI.java

public class testGUI {
public static void main(String[] args) {
    new GUI("Lingo");
}
}

BoardUI.java

import javax.swing.*;
import java.awt.*;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

public class BoardUI extends JPanel implements KeyListener {

private final Game game;
private Board b = new Board();
private Color hoverColor = new  Color(100, 100, 255);

public BoardUI(LayoutManager layout, Game game, String word) {
    super(layout);
    this.b = new Board();
    this.game = game;
    initComponents(word);
}

public BoardUI(Game game) {
    this.game = game;
}

private void initComponents(String word) {
    this.setLayout(new GridLayout(5, 5, 2, 2));
    for (int i = 0; i < 5 * 5; i++) {
        b.getLetters()[i] = new Label();
        b.getLetters()[i].setBackground(Color.WHITE);
        b.getLetters()[i].setForeground(Color.BLACK);
        b.getLetters()[i].setAlignment(Label.CENTER);
        b.getLetters()[i].setFont(new Font("Big", Font.BOLD, 48));
        this.add(b.getLetters()[i]);
    }

        for (int i = 0; i < 5; i++) {
            b.getLetters()[i].setText(word.charAt(i) + "");
            b.getLetters()[i].setBackground(Color.BLUE);
        }
    b.setCurrentColumn(0);
    b.setCurrentRow(0);
    }

@Override
public void keyTyped(KeyEvent e) {
    char typed = e.getKeyChar();

    if (Character.isLetter(typed) && b.getCurrentColumn() < 5) {
        typed = Character.toUpperCase(typed);
        b.getLetters()[b.position(b.getCurrentRow(), b.getCurrentColumn())].setText(typed + "");
        b.getLetters()[b.position(b.getCurrentRow(), b.getCurrentColumn())].setBackground(Color.BLUE);

        if (b.getCurrentColumn() == 0) {
            for (int i = 1; i < 5; i++) {
                b.getLetters()[b.position(b.getCurrentRow(), i)].setText(".");
                b.getLetters()[b.position(b.getCurrentRow(), i)].setBackground(Color.BLUE);
            }

            b.nextColumn(true);

            if (b.getCurrentColumn() < 5) {
                b.getLetters()[b.position(b.getCurrentRow(), b.getCurrentColumn())].setBackground(hoverColor);
            }

            if (typed == 8 && b.getCurrentColumn() >= 0) {
                this.oneBack();
            }
        }
    }
}

private void oneBack() {
    if (b.getCurrentColumn() < 5) {
        b.getLetters()[b.position(b.getCurrentRow(), b.getCurrentColumn())].setBackground(Color.BLUE);
        b.nextColumn(false);
        b.getLetters()[b.position(b.getCurrentRow(), b.getCurrentColumn())].setText(".");
        b.getLetters()[b.position(b.getCurrentRow(), b.getCurrentColumn())].setBackground(Color.BLUE);
    }
}

@Override
public void keyPressed(KeyEvent e) {

}

@Override
public void keyReleased(KeyEvent e) {

}
}

Board.java：

import java.awt.*;

public class Board {
private Reader reader = new Reader();
private Label[] letters = new Label[5 * 5];
private String[] words = new String[100];
private int currentRow, currentColumn;

public Label[] getLetters() {
    return letters;
}

public String firstWord() {
    return getWord().charAt(0) + "....";
}

public String getWord() {
    int count = reader.LeesWoordenVanFile(words);
    int x = (int) (Math.random() * 100);
    int gokX = (x % count);
    return words[gokX].toUpperCase();
}

public int position(int row, int column) {
    return row * 5 + column;
}

public void nextColumn(boolean next) {
    if (next) {
        currentColumn++;
    } else {
        currentColumn--;
    }
}

public int getCurrentRow() {
    return currentRow;
}

public int getCurrentColumn() {
    return currentColumn;
}

public void setCurrentRow(int currentRow) {
    this.currentRow = currentRow;
}

public void setCurrentColumn(int currentColumn) {
    this.currentColumn = currentColumn;
}
}

GUI.java：

import javax.swing.*;
import java.awt.*;

public class GUI extends JFrame {

private JPanel gamePanel;
private Game game;
private Board board = new Board();
private BoardUI boardUI = new BoardUI(game);

public GUI(String title) throws HeadlessException  {
    super(title);
    game = new Game();
    setupVieuw();
    showFrame();
    this.addKeyListener(boardUI);
}

public void setupVieuw() {
    getContentPane().removeAll();
    initComponents(board.firstWord());
    layoutComponents();
    this.validate();
}

public void layoutComponents() {
    getContentPane().add(gamePanel, BorderLayout.CENTER);
}

private void initComponents(String word) {
    gamePanel = new JPanel(new BorderLayout());
    gamePanel.add(new BoardUI(new BorderLayout(),game, word));
}

private void showFrame() {
    this.setBounds(50, 50, 800, 800);
    this.setPreferredSize(new Dimension(600, 600));
    this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    this.pack();
    this.setVisible(true);
    this.doLayout();
}
}

Answer 1

这真的很令人困惑，因为您有一些冗余代码并且在某些情况下会执行两次，但这是我发现的。

在 GUI.java 中，您在此处实例化 BoardUI：

private Game game;
private Board board = new Board();
private BoardUI boardUI = new BoardUI(game);

请注意，

1. 您正在使用空的 Game 对象初始化 BoardUI

2. 你正在调用这个 BoardUI 构造函数：

   public BoardUI(Game game) {
       this.game = game;
   }
   

   and not 这个构造函数 - 又名。实际上用new Label() 对象填充您的letters 数组：

   public BoardUI(LayoutManager layout, Game game, String word) {
       super(layout);
       this.b = new Board();
       this.game = game;
       initComponents(word);
   }
   

仍在GUI.java 中，您最终确实调用了不同的initComponents()，但您使用新的BoardUI 实例调用它

private void initComponents(String word) {
    gamePanel = new JPanel(new BorderLayout());
    gamePanel.add(new BoardUI(new BorderLayout(),game, word));
}

在返回之后，您又回到了GUI 类构造函数的最后一行，并将键侦听器设置为尚未运行的BoardUI initComponents()：

this.addKeyListener(boardUI);

TL;DR：您正在向尚未运行 initComponents() 的 BoardUI 对象添加键侦听器