为什么Java repaint()方法不起作用?

时间:2018-03-28 19:43:09

标签: java swing paint repaint

下面的代码是一个非常简单的测试,涉及一个图像。 每当我发送" a"它应该重新绘制图像。到System.in并且它应该在我发送" q"。

时退出程序

问题是只有退出工作: 方法paint()永远不会被调用,我不知道为什么。

我检查了对" super.paint()"的调用,尝试用paintCompoenent(Graphics g)替换paint(Graphics g),但似乎没有任何工作:只是没有调用

涉及扫描仪的问题是否在main()中?

程序中的路径与我使用的路径不一样,第一个涂料是正确的,所以问题不应该存在。

注意,如果它有用,我使用Eclipse Oxygen和Java9 SE

感谢所有人!

代码粘贴:

public class TestImagePanel extends JPanel {

    private BufferedImage image;
    private int xpos = 0;
    private int ypos = 0;
    private String _imagePath = "//myFolder//image.png";

    public TestImagePanel() {
        try {
            image = ImageIO.read(new File(_imagePath));
        } catch (IOException ex) {}
    }

    public void paint(Graphics g) {
        super.paint(g);
        System.out.println("painting LOG");
        g.drawImage(image, this.xpos++, this.ypos++, this);
    }

    public void update(String a) {
        System.out.print("Receiving:" + a + "---" + xpos + ":" + ypos);
        if (a.equals("a"))
            repaint();
        else if (a.equals("q")) {
            System.out.println("LOGOUT");
            System.exit(0);
        }
    }

    public static void main(String[] args) {
        JFrame frame = new JFrame("test");
        TestImagePanel testimg = new TestImagePanel();
        frame.add(new TestImagePanel());
        frame.setSize(new Dimension(600, 600));
        frame.setVisible(true);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        Scanner in = new Scanner(System.in);
        while (true)
            testimg.update( in .next());
    }
}

2 个答案:

答案 0 :(得分:4)

所以,有一些错误......

让我们从这里开始......

JFrame frame = new JFrame("test");
TestImagePanel testimg = new TestImagePanel();
frame.add(new TestImagePanel());

//...

Scanner in = new Scanner(System.in);
while (true)
    testimg.update( in .next());

您正在创建TestImagePanel的两个实例,而您只是更新不在屏幕上的实例

像...一样的东西。

JFrame frame = new JFrame("test");
TestImagePanel testimg = new TestImagePanel();
frame.add(testimg);

//...

Scanner in = new Scanner(System.in);
while (true)
    testimg.update( in .next());

会有所帮助。

下一步...

public void paint(Graphics g) {
    super.paint(g);
    System.out.println("painting LOG");
    g.drawImage(image, this.xpos++, this.ypos++, this);
}

好的,你应该避免覆盖paint,作为一般偏好,建议改为覆盖paintComponent

因为绘画可以在任何时候出于各种原因发生,所以你永远不应该从绘画方法中更新或修改UI的状态,绘画是用于绘制当前状态

所以,它应该更像......

protected void paintComponent(Graphics g) {
    super.paint(g);
    g.drawImage(image, this.xpos, this.ypos, this);
}

好的,那么,我们如何更新xposypos值?在您的情况下,update方法可能是显而易见的选择....

public void update(String a) {
    xpos++;
    ypos++;
    System.out.print("Receiving:" + a + "---" + xpos + ":" + ypos);
    if (a.equals("a"))
        repaint();
    else if (a.equals("q")) {
        System.out.println("LOGOUT");
        System.exit(0);
    }
}

现在,这引发了一个问题。 xpos方法需要ypospaintComponent,这意味着不应在事件调度线程的上下文之外更新这些值

一个简单的解决方法可能是做一些像......

public void update(String a) {
    if (!EventQueue.isDispatchThread()) {
        EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                update(a);
            }
        });
    }
    xpos++;
    ypos++;
    System.out.print("Receiving:" + a + "---" + xpos + ":" + ypos);
    if (a.equals("a")) {
        repaint();
    } else if (a.equals("q")) {
        System.out.println("LOGOUT");
        System.exit(0);
    }
}

这确保了update方法的内容在EDT的上下文中执行。

这是恕我直言,有点乱。更好的解决方案是使用SwingWorker

SwingWorker<Void, String> worker = new SwingWorker<Void, String>() {
    @Override
    protected Void doInBackground() throws Exception {
        Scanner in = new Scanner(System.in);
        while (true) {
            publish(in.next());
        }
    }

    @Override
    protected void process(List<String> chunks) {
        for (String text : chunks) {
            testimg.update(text);
        }
    }

};

这需要为我们将更新添加到EDT中。

这会生成一个看起来像这样的解决方案......

import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
import java.util.List;
import java.util.Scanner;
import javax.imageio.ImageIO;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingWorker;

public class TestImagePanel extends JPanel {

    private BufferedImage image;
    private int xpos = 0;
    private int ypos = 0;
    private String _imagePath = "//myFolder//image.png";

    public TestImagePanel() {
        try {
            image = ImageIO.read(new File(_imagePath));
        } catch (IOException ex) {
        }
    }

    @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        System.out.println("painting LOG");
        g.drawImage(image, this.xpos, this.ypos, this);
    }

    public void update(String a) {
        System.out.print("Receiving:" + a + "---" + xpos + ":" + ypos);
        if (a.equals("a")) {
            xpos++;
            ypos++;
            repaint();
        } else if (a.equals("q")) {
            System.out.println("LOGOUT");
            System.exit(0);
        }
    }

    public static void main(String[] args) {
        JFrame frame = new JFrame("test");
        TestImagePanel testimg = new TestImagePanel();
        frame.add(new TestImagePanel());
        frame.setSize(new Dimension(600, 600));
        frame.setVisible(true);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        SwingWorker<Void, String> worker = new SwingWorker<Void, String>() {
            @Override
            protected Void doInBackground() throws Exception {
                Scanner in = new Scanner(System.in);
                while (true) {
                    publish(in.next());
                }
            }

            @Override
            protected void process(List<String> chunks) {
                for (String text : chunks) {
                    testimg.update(text);
                }
            }

        };

    }
}

现在,问题是,为什么要在GUI程序中从控制台获取输入?你应该通过GUI输入数据?以上内容可能是从文件或其他自动源读取内容的一个很好的解决方案,但应该避免用户输入...这不是GUI的工作方式。

答案 1 :(得分:0)

首先,你不应该覆盖paint()方法;你应该;而是覆盖paintComponent()。

其次,您没有在EventDispatchThread上执行此操作。即使你是这样,将更新放在一个循环中也会阻塞事件派发线程,直到循环结束,结果只有一次最终重绘。