Animation headache

Question

i've been working on this program for almost two weeks now, and still couldn't get it. the problem now is it's not animated. when click on "start" button. the program "sleep" for a while -- which means the "timer" is working, but it loads the last pic in the array, instead of loading each pic one by one. here is the code. i think the problem is might be within the "FeetPanel1.actionPerformed()". would you mind to look into it?
import java.awt.*;
import javax.swing.*;
import java.awt.event.*;
import javax.swing.border.*;
public class FeetPanelTest
public static void main(String args[])
FeetFrame frame=new FeetFrame();
frame.setSize(400, 300);
frame.setLocation(350, 250);
frame.setVisible(true);
frame.setTitle("Test FeetPanel");
class FeetFrame extends JFrame implements ActionListener
FeetPanel1 feetPanel=new FeetPanel1();
JButton jbStart=new JButton("Start");
public FeetFrame()
feetPanel.setBorder(new LineBorder(Color.black, 1));
getContentPane().add(feetPanel, BorderLayout.CENTER);
getContentPane().add(jbStart, BorderLayout.SOUTH);
jbStart.addActionListener(this);
public void actionPerformed(ActionEvent e)
if(e.getSource()==jbStart)
feetPanel.start();
//feet.start();
class FeetPanel1 extends JPanel implements ActionListener
protected int numOfPlanes=26;
protected int currentPlaneIndex=0;
protected Image planeArray[];
protected Image planeToDisplay;
private Timer timer;
public FeetPanel1()
planeArray=new Image[numOfPlanes];
for(int i=0; i<planeArray.length; i++)
planeArray=Toolkit.getDefaultToolkit().getImage
("C:/forte4j/bin/JavaExcercise/sampledir/Project4099/Planes/Plane"
+(i+1)+".gif");
setLayout(new BorderLayout());
timer=new Timer(10, this);
public void paintComponent(Graphics g)
super.paintComponent(g);
planeToDisplay=planeArray[currentPlaneIndex];
g.drawImage(planeToDisplay, 0, 0, getSize().width, getSize().height, this);
public void actionPerformed(ActionEvent e)
for(currentPlaneIndex=0;
currentPlaneIndex<planeArray.length-1; currentPlaneIndex++)
//I think the problem is possibly here
currentPlaneIndex=currentPlaneIndex;
repaint();
public void start()
timer.start();
thanx a lot!!

Answer 1

Oh, where to begin with?
I think you should rethink your little animation viewer and change quite everything... but maybe you could get something displayed also with a little modification, in your loop use paintImmediately() instead of repaint(), this should avoid that all the very close calls to repaint() be coalesced.