通过事件从另一个窗口打开时，Java图形窗口不会绘制

Question

我有三节课，第一个是主窗口，像

public class Starter extends JFrame{
JButton b=new JButton("Game");
Starter(){

    setSize(200,200);
    add(b);
    b.addActionListener(new ActionListener(){
        public void actionPerformed(ActionEvent e){
            TryGraph gg=new TryGraph();
        }
    });
    setVisible(true);

}
public static void main(String [] args){
    Starter g= new Starter();

}

}

然后第二个类是一个具有面板的窗口，在该面板上将绘制图形

public class TryGraph {
static int w=640,h=480;
TryGraph(){
    JFrame mF=new JFrame();
    GPan pan=new GPan();

    mF.setLocationRelativeTo(null);
    mF.setResizable(false);
    mF.setSize(w,h);

    mF.add(pan);

    mF.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    mF.setVisible(true);

    pan.playGame();
}

public static void main(String []args){
    TryGraph t=new TryGraph();
}

}

然后最后是进行绘图的面板类

public class GPan extends JPanel{

private boolean running;
private BufferedImage image;
private Graphics2D g;
int x,y;

public GPan(){
    x=TryGraph.w;
    y=TryGraph.h;
    init();
}

public void init(){
    running=true;
    image=new BufferedImage(x,y,BufferedImage.TYPE_INT_RGB);
    g=(Graphics2D)image.getGraphics();
    g.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
}

public void playGame(){
    while(running){
         g.setColor(new Color(102,102,102));
        g.fillRect(0,0,x,y);
        g.setColor(new Color(255,0,0));
        g.fillOval(0, 0, 100,100);
        repaint();
    }
}

public void paintComponent(Graphics g){
    Graphics2D g2 = (Graphics2D)g;
    g2.drawImage(image,0,0,x,y,null);

    g2.dispose();
}

}

问题是如果我正在使用从Starter类窗口到TryGraph类窗口的事件，GPan面板在通过play game（）方法循环时将不会绘制图形。 但是当直接执行TryGraph类时，它可以正常工作

Answer 1

我注意到了几个问题。

1. 在TryGraph调用main时，将创建一个新实例。 但是当按下按钮时，您已经有一个TryGraph实例。

2. 不要扩展JFrame 。 这是不好的做法。 只需创建一个实例。

3. 仅使用one static main entry point 。

4. 最大的问题是您将repaint()置于一个紧密的循环中。 不要那样做 只需调用repaint（）。 请记住，重新绘制是在Event Dispatch Thread上运行的。 因此，如果该线程被捆绑，则其他任何操作均无效。

5. 不要处理您的图形上下文。 如果您创建一个新的，则可以执行此操作，否则不要放弃它。

6. 最后，将super.paintComponent（）作为您的JPanel paintComponent方法中的第一条语句。

我建议您在Java教程中阅读有关绘画和EDT的知识。

这是修改后的代码，全部在一个文件中。

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.image.BufferedImage;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class Starter {
   JButton b     = new JButton("Game");
   JFrame  frame = new JFrame();

   Starter() {

      frame.setSize(200, 200);
      frame.add(b);
      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      b.addActionListener(new ActionListener() {
         public void actionPerformed(ActionEvent e) {
            TryGraph gg = new TryGraph();
         }
      });
      frame.setVisible(true);

   }
   public static void main(String[] args) {
      Starter g = new Starter();

   }
}

class TryGraph {
   static int w = 640, h = 480;

   TryGraph() {
      JFrame mF = new JFrame();
      GPan pan = new GPan();

      mF.setLocationRelativeTo(null);
      mF.setResizable(false);
      mF.setSize(w, h);

      mF.add(pan);

      mF.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      mF.setVisible(true);

      pan.playGame();
   }

}

class GPan extends JPanel {

   private boolean       running;
   private BufferedImage image;
   private Graphics2D    g;
   int                   x, y;

   public GPan() {
      x = TryGraph.w;
      y = TryGraph.h;
      init();
   }

   public void init() {
      running = true;
      image = new BufferedImage(x, y, BufferedImage.TYPE_INT_RGB);
      g = (Graphics2D) image.getGraphics();
      g.setRenderingHint(RenderingHints.KEY_ANTIALIASING,
            RenderingHints.VALUE_ANTIALIAS_ON);
   }

   public void playGame() {
      repaint();

   }

   public void paintComponent(Graphics g) {
      super.paintComponent(g);
      Graphics2D g2 = (Graphics2D) g.create();

      g2.setColor(new Color(102, 102, 102));
      g2.fillRect(0, 0, x, y);
      g2.setColor(new Color(255, 0, 0));
      g2.fillOval(0, 0, 100, 100);
      // g2.drawImage(image,0,0,x,y,null);

      g2.dispose();
   }
}

另一项建议。 我相信在您的原始代码中，您通过在私有类中实现接口来实现了侦听器。 好主意，除了您应该扩展给定接口的适配器类。 例如，对于mouseListener ，扩展MouseAdapter 。 它提供了虚拟方法，因此您不必自己创建它们。 它有助于使您的代码更具可读性。 当然，单方法接口没有（或不需要）适配器。