Adding button events stops core functonality!

Question

This should be an easy one; here is a piece of code that i have adapted from the Java website!
The problem occured when i added the code for the buttons in Public void actionPerformed.
Instead of animating; the picture is static and will only advance by 1 frame if i press either the play or stop buttons! if u remove the code for the buttons the animation works fine!
what am i doing wrong?
(T1 to T10 can be any random gif pictures)
I was wondering if i need a new actionPerformed class or something????
hope you can help me
kind regards
RSH
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
public class ImageSequenceTimer extends JFrame
                                implements ActionListener {
    ImageSQPanel imageSQPanel;
    static int frameNumber = -1;
    int delay;
    Thread animatorThread;
    static boolean frozen = false;
    Timer timer;
     //Invoked only when this is run as an application.
    public static void main(String[] args) {
        Image[] waving = new Image[10];
        for (int i = 1; i <= 10; i++) {
            waving[i-1] =
                Toolkit.getDefaultToolkit().getImage("images/T"+i+".gif");
        JFrame f = new JFrame("ImageSequenceTimer");
        f.addWindowListener(new WindowAdapter() {
            public void windowClosing(WindowEvent e) {
                System.exit(0);
        ImageSequenceTimer controller = new ImageSequenceTimer();
        controller.buildUI(f.getContentPane(), waving);
        controller.startAnimation();
        f.setSize(new Dimension(350, 350));
        f.setVisible(true);
    //Note: Container must use BorderLayout, which is the
    //default layout manager for content panes.
    void buildUI(Container container, Image[] dukes) {
        int fps = 10;
        //How many milliseconds between frames?
        delay = (fps > 0) ? (1000 / fps) : 100;
        //Set up a timer that calls this object's action handler
        timer = new Timer(delay, this);
        timer.setInitialDelay(0);
        timer.setCoalesce(true);
        JPanel buttonPanel = new JPanel();
        JButton play = new JButton("PLAY");
        play.addActionListener(this);
        JButton stop = new JButton("STOP");
        stop.addActionListener(this);
        JButton restart = new JButton("RESTART");
        JButton back = new JButton("Back");
        imageSQPanel = new ImageSQPanel(dukes);
        container.add(imageSQPanel, BorderLayout.CENTER);
        imageSQPanel.addMouseListener(new MouseAdapter() {
            public void mousePressed(MouseEvent e) {
                if (frozen) {
                    frozen = false;
                    startAnimation();
                } else {
                    frozen = true;
                    stopAnimation();
       container.add(buttonPanel, BorderLayout.SOUTH);
       buttonPanel.add(play);
       buttonPanel.add(stop);
       buttonPanel.add(restart);
       buttonPanel.add(back);
    public void start() {
        startAnimation();
    public void stop() {
        stopAnimation();
    public synchronized void startAnimation() {
        if (frozen) {
            //Do nothing.  The user has requested that we
            //stop changing the image.
        } else {
            //Start animating!
            if (!timer.isRunning()) {
                timer.start();
    public synchronized void stopAnimation() {
        //Stop the animating thread.
        if (timer.isRunning()) {
            timer.stop();
    public void actionPerformed(ActionEvent e) {
        //Advance the animation frame.
        frameNumber++;
        //Display it.
        imageSQPanel.repaint();
        {JButton button = (JButton) e.getSource();
         String label = button.getText();
         if("PLAY".equals(label))
         { start();
         else if("STOP".equals(label))
         { stop();
    class ImageSQPanel extends JPanel{
        Image dukesWave[];
        public ImageSQPanel(Image[] dukesWave) {
            this.dukesWave = dukesWave;
        //Draw the current frame of animation.
        public void paintComponent(Graphics g) {
            super.paintComponent(g); //paint background
            //Paint the frame into the image.
            try {
                g.drawImage(dukesWave[ImageSequenceTimer.frameNumber%10],
                            0, 0, this);
            } catch (ArrayIndexOutOfBoundsException e) {
                //On rare occasions, this method can be called
                //when frameNumber is still -1.  Do nothing.
                return;
}

Answer 1

You get ClassCastException at JButton button = (JButton) e.getSource(); therefore
1. Always look if there are any exceptions thrown
2. In this case e.getSource() does not always return JButton object reference, actionPerformed is also called by Timer. This is a source of your exception: you cast correctly only once - when button is pressed; then Timer calls actionPerformed, exception is thrown and animation stops.
Here s corrected code
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
public class Test extends JFrame
                                implements ActionListener {
    ImageSQPanel imageSQPanel;
    static int frameNumber = -1;
    int delay;
    Thread animatorThread;
    static boolean frozen = false;
    Timer timer;
    JButton play, stop; // <------------------------------------
     //Invoked only when this is run as an application.
    public static void main(String[] args) {
        Image[] waving = new Image[10];
        for (int i = 1; i <= 10; i++) {
            waving[i-1] =
                Toolkit.getDefaultToolkit().getImage("images/T"+i+".gif");
        JFrame f = new JFrame("ImageSequenceTimer");
        f.addWindowListener(new WindowAdapter() {
            public void windowClosing(WindowEvent e) {
                System.exit(0);
        Test controller = new Test();
        controller.buildUI(f.getContentPane(), waving);
        controller.startAnimation();
        f.setSize(new Dimension(350, 350));
        f.setVisible(true);
    //Note: Container must use BorderLayout, which is the
    //default layout manager for content panes.
    void buildUI(Container container, Image[] dukes) {
        int fps = 10;
        //How many milliseconds between frames?
        delay = (fps > 0) ? (1000 / fps) : 100;
        //Set up a timer that calls this object's action handler
        timer = new Timer(delay, this);
        timer.setInitialDelay(0);
        timer.setCoalesce(true);
        JPanel buttonPanel = new JPanel();
        play = new JButton("PLAY"); // <------------------------------------
        play.addActionListener(this);
        stop = new JButton("STOP"); // <------------------------------------
        stop.addActionListener(this);
        JButton restart = new JButton("RESTART");
        JButton back = new JButton("Back");
        imageSQPanel = new ImageSQPanel(dukes);
        container.add(imageSQPanel, BorderLayout.CENTER);
        imageSQPanel.addMouseListener(new MouseAdapter() {
            public void mousePressed(MouseEvent e) {
                if (frozen) {
                    frozen = false;
                    startAnimation();
                } else {
                    frozen = true;
                    stopAnimation();
       container.add(buttonPanel, BorderLayout.SOUTH);
       buttonPanel.add(play);
       buttonPanel.add(stop);
       buttonPanel.add(restart);
       buttonPanel.add(back);
    public void start() {
        startAnimation();
    public void stop() {
        stopAnimation();
    public synchronized void startAnimation() {
        if (frozen) {
            //Do nothing.  The user has requested that we
            //stop changing the image.
        } else {
            //Start animating!
            if (!timer.isRunning()) {
                timer.start();
    public synchronized void stopAnimation() {
        //Stop the animating thread.
        if (timer.isRunning()) {
            timer.stop();
    public void actionPerformed(ActionEvent e) {
        //Advance the animation frame.
        frameNumber++;
        //Display it.
        imageSQPanel.repaint();
         if(e.getSource() == play) // <------------------------------------
         { start();
         else if(e.getSource() == stop) // <------------------------------------
         { stop();
    class ImageSQPanel extends JPanel{
        Image dukesWave[];
        public ImageSQPanel(Image[] dukesWave) {
            this.dukesWave = dukesWave;
        //Draw the current frame of animation.
        public void paintComponent(Graphics g) {
            super.paintComponent(g); //paint background
            //Paint the frame into the image.
            try {
                g.drawImage(dukesWave[Test.frameNumber%10],
                            0, 0, this);
            } catch (ArrayIndexOutOfBoundsException e) {
                //On rare occasions, this method can be called
                //when frameNumber is still -1.  Do nothing.
                return;
}