repaint（）方法不重绘JPanel

Question

好的，所以首先我知道关于此问题还有很多其他线程，但是他们只是说尝试使用invalidate（）和validate（）对我没有影响。

我的问题是-标题很明显-JPanel的repaint（）方法不会重新绘制。 我在paintComponent（）方法中添加了一些调试消息，这是控制台的输出：

Debug
-10 421 010 441
repainted
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse
Moved mouse

注意，在程序开始时，您只看到debug ， (co-ordinates)和repainted一次，但是我调用了repaint（）;。 每次移动鼠标。

我将五个类合而为一，因为我相信它是调试的首选：

package com.trtld.spacewar;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseMotionListener;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.Timer;

public class GroupClass {
     private static Content content = new Content();

    public static void main(String[] args) {
        JFrame window = new JFrame("Spacewar!");
        content.setVisible(true);
        window.setContentPane(content);
        MouseInput listener = new MouseInput();
        content.addMouseMotionListener(listener);
        content.addMouseListener(listener);
        window.setSize(600, 480);
        window.setLocation(100, 100);
        window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        window.setVisible(true);
        //window.setResizable(false);
        System.out.println("Debug");
    }


}

 class Target {

    private int x1, y1;
    private int x2, y2;

    public Target(int x1, int y1, int x2, int y2){
        this.x1 = x1; 
        this.y1 = y1;
        this.x2 = x2;
        this.y2 = y2;

    }

    public int getX1(){
        return x1;
    }
    public int getX2(){
        return x2;
    }
    public int getY1(){
        return y1;
    }
    public int getY2(){
        return y2;
    }
}
class Content extends JPanel{

        private static Content instance;
        private Target[] targets = new Target[9];   
        private Bullet[] bullets = new Bullet[999];//assuming no more than 1000 bullets are on the screen at
                                                  //once (they'd have to have a really fancy-nice auto-clicker ha ha)
        int spaceShipX = (getWidth()/2); 





        public Content (){
            int yLevel = (((int)Math.random())*(getHeight() / 4));
            int xPos = ((int)Math.random())*getWidth();
            for(int i = 0;!(i == 9); i++){                                         //Create the randomly located targets
                targets[i] = new Target(xPos, yLevel, xPos-15, yLevel-10);
            }
        }

        public void paintComponent(Graphics g){
             super.paintComponent(g);  
             g.setColor(Color.RED);
             System.out.println(spaceShipX-10 + " " + (getHeight()-20) + " " + spaceShipX+10 + " " + getHeight());
             System.out.println("repainted");
             g.fillRect(spaceShipX+50, getHeight()-20 ,spaceShipX+100 , getHeight());

             /**  //commented out for simplicity + needs to be fixed:
                for(Target t : this.targets){        //paint targets
                    g.drawRect(t.getX1(), t.getY1(), t.getX2(), t.getY2());
                }
                for(Bullet b : this.bullets){         //paint bullets and remove invalid ones plus check for hits
                    if(b != null){


                    g.drawRect(b.getX1(), b.getY1(), b.getX2(), b.getY2());

                    if(!(b.checkTargetPresence() == null)){
                        for(Target target : targets){
                            if(b.checkTargetPresence() == target){
                                target = null;
                            }
                        }
                    }

                    if(b.isValid() == false){
                        b = null;
                    }
                }
                }*/
        }


        public static Content getInstance(){
            if(instance == null) {
                instance = new Content();     //TODO: Singletons are out-dated and not preferred; fix this
                }
                return instance;
        }


    //Getters and setters:

        public Bullet[] getBullets(){
            return bullets;
        }

        public Target[] getTargets(){
            return targets;
        }

        public void setTarget(Target t, int loc){
            this.targets[loc] = t;
        }

        public void setBullet(Bullet b){
            int location = -1;
            if(bullets[0] == null){
            this.bullets[0] = b;
            return;
            }
            for(Bullet bullet: bullets){
                location++;
            if(bullets[location] == null){ //TODO: Messy, clean this up
                bullets[location] = b;
                return;
            }   
            }

        }
        public int getSpaceShipX(){
        return spaceShipX;
        }
        public void setSpaceShipX(int x){
            spaceShipX = x-10;
            repaint();

        }

    }
class Bullet implements ActionListener{

    int x1, x2;
    int y1 = 10, y2 = 5;

    Timer timer = new Timer(250, this);

    public Bullet(int x1, int x2){
        this.x1 = x1;
        this.x2 = x2;
        timer.start();
    }

    public void actionPerformed(ActionEvent evt) {
        y1 = y1 + 2;
        y2 += 2;
        Content.getInstance().repaint();
    }

    public Target checkTargetPresence(){
        for(Target t : Content.getInstance().getTargets()){
            if(x1 <= t.getX1() && x1 >= t.getX2()){
                if(y1 <= t.getY1() && y1 >= t.getY2()){
                         //HIT!
                    return t;
                }
            }
        }
        return null; //TODO

    }

    public boolean isValid(){//Checks if the bullet is still on the screen

        if(y1 >= Content.getInstance().getHeight() || y2 >= Content.getInstance().getHeight()){
            return false;
        }else{
            return true;
        }

    }
    public int getX1(){
        return x1;
    }
    public int getX2(){
        return x2;
    }
    public int getY1(){
        return y1;
    }
    public int getY2(){
        return y2;
    }
}
class MouseInput implements MouseListener, MouseMotionListener {

    public void mouseDragged(MouseEvent arg0) {     
    }

    public void mouseMoved(MouseEvent evt) {
        Content.getInstance().setSpaceShipX(evt.getX());
        Content.getInstance().repaint(); // I try to repaint here and in the Content class, but nothing happens.
        System.out.println("Moved mouse");
    }

    public void mouseClicked(MouseEvent arg0) {

    }

    public void mouseEntered(MouseEvent arg0) {     
    }

    public void mouseExited(MouseEvent arg0) {      
    }

    public void mousePressed(MouseEvent evt) {  
    Content.getInstance().setBullet(new Bullet(evt.getX()-2, evt.getX()+2));
    System.out.println("clicked");
    }

    public void mouseReleased(MouseEvent arg0) {    
    }

}

我将不胜感激，如果需要格式化其他格式，请告诉我。 请注意，这是我的第一个GUI程序。

Answer 1

我认为问题是您创建了Content类，然后将其添加到框架中。

但是，在MouseListener代码中，您将调用Content类的getInstance()方法，并且“ instance”变量为null，因此将创建一个新的Content实例（但不会添加到框架中，因此它永远不会被绘制）。

因此，这种基本逻辑是错误的，您不想创建新实例。

而是在Content类的构造函数中执行以下操作：

instance = this;

然后，getInstance（）方法将仅返回变量，因为它将始终具有值。

同样，MouseListeners应该是Content类中的内部类。 那就是您应该创建侦听器并将其添加到您的类。 当您这样做时，您甚至不需要静态的getInstance（）方法，因为您可以直接访问实例变量。

但是他们只说尝试使用invalidate（）和validate（）对我没有影响。

您不应该使用这些方法。 这些用于AWT。 在Swing中，您可以使用：

revalidate();
repaint();

当您在GUI可见后或在自定义类中更改属性后，将组件添加到GUI中时。 revalidate（）将调用布局管理器。 repaint（）将绘制组件。 在您的类中，您不会更改任何内容的大小，因此只需要在setter方法中使用repaint（）即可。

编辑：

介意分享您添加的代码？

我添加了一行代码：

    public Content (){
        int yLevel = (((int)Math.random())*(getHeight() / 4));
        int xPos = ((int)Math.random())*getWidth();
        for(int i = 0;!(i == 9); i++)   { 
             //Create the randomly located targets
            targets[i] = new Target(xPos, yLevel, xPos-15, yLevel-10);
        }

        instance = this; // added
    }

一次进行一次更改。 一旦您验证了我的假设，只需添加一行代码即可。 然后，您可以整理课程的其余部分。 如果您一次进行多个更改，则不知道是哪个引起了问题。