Question

I have a applet that simulates a coffee vending machine. I am trying to get the text Ready to be display for 5 secs. But it doesn’t seem to be working. can anyone tell me what’s wrong with my timer ?

Class qn1 is the class for JApplet.
init() will be called to instantiate the variables.

I add actionlistener to the 2 buttons so they will react when i click them.

    import java.awt.*;
    import java.awt.event.*;

    import javax.swing.*;
    import javax.swing.event.*;

    public class qn1 extends JApplet implements ActionListener {

        private static int FULL = 4;
        private int coffees = FULL;
        private JPanel p;
        private JButton jbtw, jbtb;
        private Dispenser dis;
        private JLabel jlbl;
        private Timer timer;

        public void init() {
            setLayout(new BorderLayout(5, 5));
            setSize(400, 500);

            p = new JPanel(new GridLayout(1, 2));
            jbtw = new JButton("White");
            jbtb = new JButton("Black");
            jbtw.addActionListener(this);
            jbtb.addActionListener(this);
            p.add(jbtw);
            p.add(jbtb);

            add(p, BorderLayout.NORTH);

            dis = new Dispenser();
            add(dis, BorderLayout.CENTER);

            jlbl = new JLabel("Select", SwingConstants.CENTER);
            add(jlbl, BorderLayout.SOUTH);

            timer = new Timer(50, this);
        }

        public void actionPerformed(ActionEvent e) {
            if (e.getSource().equals(jbtb) | e.getSource().equals(jbtw)) {
                if (coffees - 1 >= 0) {
                    coffees--;
                    jlbl.setText("Ready " + coffees);               
                }

                timer.setInitialDelay(5000);
                timer.start();  

                if (coffees == 0) {
                    jlbl.setText("Empty");
                } else {
                    jlbl.setText("Select" + coffees);
                }
            }
        }
    }

    class Dispenser extends JPanel {
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            g.drawLine(0, 0, 25, 50);
            g.drawLine(25, 50, 55, 50);
            g.drawLine(55, 50, 80, 0);
        }
    }

Answer 1

Your timer is called, but if the source is not one of your button, you don’t perform anything:

    if (e.getSource().equals(jbtb) | e.getSource().equals(jbtw)) {

To answer your comments, you might want something like that:

    public void actionPerformed(ActionEvent e) {
        if (e.getSource().equals(jbtb) | e.getSource().equals(jbtw)) {
            timer.setInitialDelay(5000);
            timer.start();  

            if (coffees == 0) {
                jlbl.setText("Empty");
            } else {
                jlbl.setText("Select " + coffees);
            }
        }
        else {
            if (coffees - 1 >= 0) {
                coffees--;
                jlbl.setText("Ready " + coffees);               
            }
        }
    }