SWT addPaintListener选中图片无法直接显示到Canvas问题

两个逻辑错误导致该现象：

1. 没有主动触发画布重绘：SWT的Canvas不会在你修改关联资源（这里指dogImage）的时候自动刷新，只有接收到系统级重绘事件（窗口最小化后还原、控件被遮挡后重新露出、控件尺寸变更）时才会回调PaintListener的绘图方法。你加载完图片后没有发起重绘请求，自然不会立刻显示图片。
2. 监听器注册位置错误：你把PaintListener的注册逻辑写在了按钮点击回调里，每次选文件都会给Canvas重复添加新的绘图监听器，不仅会造成重复绘图的性能损耗，长期运行还会因为旧Image对象没有及时销毁引发SWT资源泄漏。另外你注释掉了dogImage != null的判空逻辑，首次未选图片触发绘图时会直接抛出空指针异常。

按以下步骤调整代码即可：

• 把PaintListener的注册逻辑从按钮点击事件中移出，仅在Canvas初始化完成后注册一次，不要重复添加
• 每次加载完新图片后，主动调用Canvas的重绘方法，强制立刻刷新画布显示新内容
• 加载新图片前先销毁已存在的旧Image对象，避免SWT图形资源泄漏
• 恢复绘图逻辑里的空判断，避免空指针异常

修正后的核心代码如下：

// 1. 初始化Canvas时仅注册一次绘图监听器
dogPhoto.addPaintListener(new PaintListener() {
    public void paintControl(PaintEvent event) {
        // 恢复空判断，无图片时不执行绘图
        if (dogImage != null && !dogImage.isDisposed()) {
            event.gc.drawImage(dogImage, 0, 0);
        }
    }
});

browse.addSelectionListener(new SelectionAdapter() {
    public void widgetSelected(SelectionEvent event) {
        fileName = new FileDialog(shell).open();
        System.out.println("fileName  "+fileName);
        if (fileName != null) {
            // 加载新图前先销毁旧图资源，防止内存泄漏
            if (dogImage != null && !dogImage.isDisposed()) {
                dogImage.dispose();
            }
            dogImage = new Image(display, fileName);
            // 2. 主动标记画布需要重绘，并立刻执行重绘逻辑
            dogPhoto.redraw();
            dogPhoto.update();
        }
        newText = newText + "File Selected - "+fileName + System.lineSeparator();
        textArea.setText(newText);      
    }
});

说明：redraw()方法会给Canvas标记一个重绘区域，update()方法会让UI线程立刻处理所有待执行的重绘请求，二者配合就能实现图片加载后立刻显示，不需要等待系统触发重绘事件。

内容的提问来源于stack exchange，提问作者Fanish