3

嗨,我已经用谷歌搜索了,但无法弄清楚为什么我的 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 以防出现问题,但事实并非如此。

有任何想法吗?

4

2 回答 2

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

  2. 将此Object作为局部变量加载,并且只加载一次

  3. paintComponent()叫做

    • 隐含地,当JComponent需要重绘时,或

    • 明确地,例如在每个鼠标事件上,如果paintComponent()要从MouseMotionListener.

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

于 2012-10-02T14:33:10.937 回答
5
  • 您永远不应该调用paintComponent()repaint()方法来合并所有更改组件的请求(在屏幕刷新之间可能有多个重绘请求)。它将更新请求添加到 GUI 事件队列,以便更新与其他 GUI 操作正确协调(Swing 和 AWT 不是线程安全的)。这个更新请求,在处理时,调用update(),调用paint(),调用你的paintComponent()

  • 为什么会有这个:

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

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

  • 这可能不相关,但在您的代码中我看到了这一点:

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

不要用于与值equals()进行比较,null因为这可能会抛出 a NullPointerException,因为您试图尊重空指针,例如此代码会抛出 a NPE

    String s=null;
    if(!s.equals(null)) {//throws NPE
        System.out.println("Here");//is never printed
    }
  • 同样正如 mKorbel 所说(对他 +1)不要在全局paintComponent()声明中执行长时间运行的任务Image,在构造函数中分配它,然后在paintComponent(). Fi

宁可这样做:

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);
                }
    }
}
于 2012-10-02T14:35:34.190 回答