重绘没有被调用

Question

您好，我用 google 搜索过，不知道为什么我的 paintComp 方法没有被调用

我有以下代码 包 com.vf.zepto.view;

import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.util.Properties;

import javax.imageio.ImageIO;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;

import com.vf.zepto.view.interfaces.ProcessorPanel;

public class CountryDetailsPanel extends JPanel implements ProcessorPanel, Runnable {
    private GridBagConstraints c = new GridBagConstraints();
    private String countryName;
    private Properties prop = new Properties();
    private BufferedImage image;

    public CountryDetailsPanel() {
        try {
            prop.load(new FileInputStream("country.props"));
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        } catch (IOException e) {
            e.printStackTrace();
        }

        //this.setLayout(new GridBagLayout());

        c.gridx = 0;
        c.gridy = 0;
        c.fill = GridBagConstraints.BOTH;
        c.insets = new Insets(5, 5, 5, 5);

        this.setPreferredSize(new Dimension(200, 200));
    }

    @Override
    public void paintComponent(Graphics g) {
        super.paintComponent(g);

        try {
            if(countryName != null) {
                String asset = prop.getProperty(countryName+".flag");

                if(!asset.equals(null)) {
                    image = ImageIO.read(new File(asset));
                    g.drawImage(image, 0, 0, null);
                }
            }
        } 
        catch (IOException e) {
            e.printStackTrace();
        }
    }

    @Override
    public void updateDetails(Object o) {
        countryName = (String)o;
        SwingUtilities.invokeLater(this);
    }

    @Override
    public void run() {
        this.repaint();
    }
}

当调用 this.repaint() 时，期望调用 paintComponent 方法，但不是为了爱和金钱。

已尝试强制它使用 EDT，以防出现问题但不是。

有什么想法吗？

Answer 1

• 您永远不应该调用paintComponent() repaint() 方法合并所有更改组件的请求（在屏幕刷新之间可能有多个重绘请求）。它将更新请求添加到 GUI 事件队列，以便更新与其他 GUI 操作正确协调（Swing 和 AWT 不是线程安全的）。此更新请求在处理后会调用update()，它会调用paint()，它会调用您的paintComponent()

• 为什么会有这个：

  @Override
  public void run() {
      this.repaint();
     }
  

它似乎没有任何用处（创建一个新线程来重绘一次？更不用说线程不在 EDT 上，而是在 JPanel 实例上调用 repaint() （如果在外部进行了修改，你甚至不应该担心). 但是要启动修改 UI 组件使用的线程 SwingTimer/SwingWorker 或 SwingUtilities#invokeXXX()

• 这可能不相关，但在您的代码中我看到了：

          if(!asset.equals(null)) {
              image = ImageIO.read(new File(asset));
              g.drawImage(image, 0, 0, null);
          }
  

不要使用equals() 与null 值进行比较，因为这可能会抛出NullPointerException，因为您正在尝试 尊重一个空指针，例如这段代码抛出一个NPE：

    String s=null;
    if(!s.equals(null)) {//throws NPE
        System.out.println("Here");//is never printed
    }

• 也正如 mKorbel 所说（对他 +1）不要在 paintComponent() 中执行长时间运行的任务，全局声明您的 Image，在构造函数中分配它，然后在 paintComponent() 中使用它。 F.i

宁可这样做：

public class TestPanel extends JPanel {

private Image image;

public TestPanel() {
image = ImageIO.read(new File(asset));
}

    @Override
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
                if(asset!=null) {
                    g.drawImage(image, 0, 0, null);
                }
    }
}

Answer 2

1. 不要不在paintComponent中加载图像或其他硬或长时间运行的代码

2. 将此Object 加载为局部变量且仅加载一次

3. paintComponent() 被调用

   • 隐式地，当JComponent 需要重绘时，或

   • 明确地，例如在每个鼠标事件上，如果要从 MouseMotionListener 调用 paintComponent()。

4. 如果有动画，则使用Swing Timer并调用repaint()。