问题与Java join（）方法

Question

首先，这里有一些代码段：

public void startThread() { 
    this.animationThread = new Thread(this); 
    this.animationThread.start(); 
    try { 
     this.animationThread.join(); 
    } catch (InterruptedException e) { 
     e.printStackTrace(); 
    } 
} 

@Override 
public void run() {  
    pirateMainAnimation.animate(); 
} 

public void animate() { 
    for (int j = 0; j < 9; j++) { 
     try { 
      Thread.sleep(250); 
     } catch (InterruptedException e) { 
      break; 
     } 
     PirateAnimationPanel.getInstance().setCurrent(j); 
     PirateAnimationPanel.getInstance().repaint(); 
    } 
} 

我试图动画一些图片。问题是我想要主线程等待动画线程完成并继续。我搜索了一下，读了一下，然后决定使用join（）方法。它完美地等待线程完成，但我没有正确动画。 repaint（）方法被调用2次而不是9次。我想也许问题是因为我使用了单身人士。这是单例实现。

import java.awt.Graphics; 
import java.awt.MediaTracker; 

import javax.swing.JPanel; 

import uk.ac.aber.dcs.piratehangman.animation.PirateMainAnimation; 
import uk.ac.aber.dcs.piratehangman.utilityclasses.AnimationThread; 

@SuppressWarnings("serial") 
public class PirateAnimationPanel extends JPanel { 
    private int current; 
    private MediaTracker mTracker; 
    private PirateMainAnimation pirateMainAnimation; 
    private AnimationThread animationThread; 

    private PirateAnimationPanel() { 
     this.current = 0; 
     this.pirateMainAnimation = new PirateMainAnimation(); 
     mTracker = new MediaTracker(this); 
     this.animationThread = new AnimationThread(); 
     setMediaTracker(); 
     repaint(); 
    } 

    private void setMediaTracker() { 
     for (int i = 0; i < 9; i++) { 
      mTracker.addImage(
        this.pirateMainAnimation.getImagesForAnimation()[i], 
        this.pirateMainAnimation.getImagesForAnimationID()[i]); 
      try { 
       mTracker.waitForID(this.pirateMainAnimation 
         .getImagesForAnimationID()[i]); 
      } catch (InterruptedException e) { 
       System.out.println("Error loading image: " + i); 
      } 
     } 
    } 

    public void playAnimation() { 
     this.animationThread.startThread(); 
    } 

    public void paintComponent(Graphics g) { 
     super.paintComponents(g); 
     System.out.println("called"); 
     g.drawImage(this.pirateMainAnimation.getImagesForAnimation()[current], 
       0, 0, this); 
    } 

    private static class PirateAnimationPanelHolder { 
     private static final PirateAnimationPanel pirateAnimationPanel = 
      new PirateAnimationPanel(); 
    }; 

    public static PirateAnimationPanel getInstance() { 
     return PirateAnimationPanelHolder.pirateAnimationPanel; 
    } 

    public void setCurrent(int current) { 
     this.current = current; 
    } 

    public int getCurrent() { 
     return current; 
    } 
} 

Answer 1

我认为你的意思是paintComponent（）方法只被调用两次。此外，我认为如果您将组件填充到背景颜色，您应该能够移除对super.paintComponents（）的调用。

repaint（）方法只会将该组件标记为脏，并请求在下一个paint上重新渲染。

我会期望Swing线程能够在250ms内重绘，但我不确定还有哪些其他工作正在完成/呈现。您可能想在动画之前调用MediaTracker.waitForAll（）。

虽然静态单例没有增加太多，但我不认为这是造成问题（在这种情况下）。

更新： 所以问题是，join（）是在Swing事件线程，阻止组件重绘。我提出了一个调用，比如下面显示的“最后的动画完成后，新的游戏对话框：

SwingUtilities.invokeLater(new Runnable() { 
    public void run() { showDialog(); } 
}) 

Answer 2

‘请注意，事件被发布到EventQueue可以被合并，’这也许可以解释的差异另外，可以肯定。打造你的GUI上event dispatch thread进行更详细的讨论，请参阅A More Complex Image Icon Example

附录：。Improving Perceived Performance When Loading Image Icons有一个很好的SwingWorker的例子，可以简化卸载