My code on flashing alarm

Question

Hi,
please advise on why is that my code only shows red "omg" than to gray and it stopped.I would like to make it blink red, gray, red, gray, red, gray continuously.
Please give any comments and guide along.Thanks
public class Alarm extends JLabel
   private String text;
    private JLabel label;
    Timer time;
    public Alarm( JLabel label, String text)
       this.label = label;
      this.text=text;
triggerAlarm();
time = new Timer( 1000, new ActionListener() {
                    public void actionPerformed( ActionEvent e ) {
    triggerAlarm();
    trigger2();
time.start();
    public void triggerAlarm(){
       setForeground(Color.RED);
      setText(text);
      repaint();
    public void trigger2(){
         setForeground(Color.gray);
       setText(text);
        repaint();
  public class testAlarm {
public static void main(String[] args){
     String text= "omg";
JLabel test = new JLabel();
    JLabel label = new Alarm(test, text);
      JFrame frame = new JFrame();
       frame.add(label);
      frame.setSize(500,500);
      frame.setVisible(true);
      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
}

Answer 1

seah_ly wrote:
public class Alarm extends JLabel
time = new Timer( 1000, new ActionListener() {
                    public void actionPerformed( ActionEvent e ) {
triggerAlarm();
trigger2();
The code above will call trigger2() immediately after triggerAlarm. You're better off flipping a boolean variable between true and false in your timer and then setting the color based on the value of your boolean. Either that or if you want to use more than 2 colors, create a Color[] array and increment a class-level index variable, mod it by the array length and use this as the index to the color array to choose a color from the array, setting your label's text this way. For e.g.,
import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.*;
public class BlinkingLabel {
  private static final Color[] COLORS = {
    Color.gray, Color.red
  private static final int TIMER_DELAY = 300;
  private static int colorIndex = 0;
  private static void createAndShowUI() {
    final JLabel label = new JLabel("My Label", SwingConstants.CENTER);
    Timer time = new Timer(TIMER_DELAY, new ActionListener() {
      public void actionPerformed(ActionEvent e) {
        colorIndex++;
        colorIndex %= COLORS.length;
        label.setForeground(COLORS[colorIndex]);
    time.start();
    JFrame frame = new JFrame("BlinkingLabel");
    frame.getContentPane().add(label);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.pack();
    frame.setLocationRelativeTo(null);
    frame.setVisible(true);
  public static void main(String[] args) {
    java.awt.EventQueue.invokeLater(new Runnable() {
      public void run() {
        createAndShowUI();
}