Question

我首先想说这是一项任务。我不想把答案给我吃，但我想知道是什么原因导致了我的问题。

我目前正在实施康威的生命游戏。单击单元格应更改颜色，以表示正在切换到活动状态的单元格。如果再次单击，它应该返回默认颜色。

当我点击窗口中的任何地方时，程序会在第56行抛出一个空指针异常。在最后一天左右一直停留在此，所以任何帮助都表示赞赏。谢谢！

下面是代码：

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

public class VisibleGrid extends JPanel implements MouseListener, KeyListener{ 

  CellGrid cellGrid;
  Graphics rect;

  public VisibleGrid(){
    addMouseListener(this);
    cellGrid = new CellGrid();
  }

  //Draw the grid of cells, 7px wide, 75 times to create 75x75 grid
  public void paint(Graphics g){
    for(int i=0; i<525;i=i+7){
      for(int j = 0; j<525; j=j+7){
        g.drawRect(i ,j,7,7);       
      }
    }   
  }

  //auxillary method called to fill in rectangles
  public void paint(Graphics g, int x, int y){
    g.fillRect(x, y, 7, 7);
    repaint();

  }

  //main method, adds this JPanel to a JFrame and sets up the GUI  
  public static void main(String[] args){
    JFrame j = new JFrame("Conway's Game of Life");
    j.setLayout(new BorderLayout());
    j.add(new VisibleGrid(), BorderLayout.CENTER);
    JTextArea info = new JTextArea("Press S to Start, E to End");
    info.setEditable(false);
    j.add(info, BorderLayout.SOUTH);
    j.setSize(530,565);
    j.setVisible(true);
  }

  //these methods are to satisfy the compiler/interface
  //Begin Mouse Events
  public void mouseExited(MouseEvent e){}
  public void mouseEntered(MouseEvent e){}
  public void mouseReleased(MouseEvent e){}
  public void mousePressed(MouseEvent e){}
  public void mouseClicked(MouseEvent e){
    //fill the selected rectangle
    rect.fillRect(e.getX(), e.getY(), 7,7); 
    repaint();

    //set the corresponding cell in the grid to alive
    int row = e.getY() /7;
    int column = e.getX() /7;
     cellGrid.getCell(row, column).setAlive(true);
  } 
  //End Mouse Events

//These methods are to satisfy the compiler/interface
//Begin KeyEvents
  public void keyReleased(KeyEvent e){}
  public void keyPressed(KeyEvent e){}
  public void keyTyped(KeyEvent e){}



}

Answer 1

此处的问题是，您的rect字段永远不会设置为任何内容，因此它保持为null。调用rect.drawRect将导致您看到的NullPointerException。

如果我没记错的话，Swing Graphics物品当你不期望你做任何绘画时，真的不喜欢你画画。因此，我建议您不要在Graphics等字段中调用paint()来调用rect对象。如果你想要重新绘制部分窗口，最好告诉Swing窗口的哪个部分需要重新绘制，然后让它调用你的paint()方法。

在mouseClicked()方法中，我删除了对rect.fillRect()的调用，并将调用移至repaint()到方法的末尾。我还修改了paint()方法以绘制填充矩形（如果单元格处于活动状态），否则填充未填充矩形。执行此操作后，您的代码似乎有效，因为我可以单击某些单元格，它们会变黑。

我有一些改进代码的建议。我将把最后两个作为练习留给你：

我建议将行j.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);添加到main()。关闭窗口时，此行使应用程序正常退出。
目前，每次单个单元格更改时，您的代码都会重新绘制整个75×75网格。应该可以更改代码，使其仅重新绘制已更改的单元格。你可以pass a Rectangle to the repaint() method告诉Swing'我的组件的这部分只需要重新绘制'。在paint方法中，您可以使用the getClipBounds() method of the Graphics class获取此矩形，并使用它来确定要重绘的单元格。
drawRect仅绘制矩形的轮廓。如果单元格死亡，则paint方法将不会清除网格中现有的黑色矩形。您可以通过将死细胞绘制为顶部带有黑色轮廓矩形的白色填充矩形来解决此问题。

Answer 2

您确定CellGrid个对象已被细胞填充吗？我不是Java专家，但我没有在你的代码中看到这个初始化......

Java：填充网格上的矩形

2 个答案: