Thread: Trouble with repainting a traffic light using radio buttons: Any Suggestions?

import javax.swing.*;
import java.awt.*;
import java.awt.Graphics;
import java.awt.event.*;
 
public class TrafficLight extends JFrame {
 
  private NewLabel lights = new NewLabel();
 
  private JRadioButton red, yellow, green;
 
  public TrafficLight() {
 
    JPanel p1 = new JPanel();
 
    p1.setLayout(new GridLayout(1,3));
 
    add(lights, BorderLayout.CENTER);
    add(p1, BorderLayout.SOUTH);
 
  	p1.add(red = new JRadioButton("Red"));
	p1.add(yellow = new JRadioButton("Yellow"));
	p1.add(green = new JRadioButton("Green"));
 
	//Button Group
	ButtonGroup group = new ButtonGroup();
	group.add(red);
	group.add(yellow);
	group.add(green);
 
  }
 
  class LightListener implements ActionListener {
  	public void actionPerformed (ActionEvent e) {
 
  	red.addActionListener(new LightListener());
	green.addActionListener(new LightListener());
	yellow.addActionListener(new LightListener());
 
	if (e.getSource() == red)
		System.out.println("red");
 
	else if (e.getSource() == yellow)
		System.out.println("yellow");
 
	else if (e.getSource() == green)
		System.out.println("green");
 
  	}
  }
 
 
  public static void main(String[] args) {
    TrafficLight frame = new TrafficLight();
    frame.setTitle("TrafficLight");
    frame.setLocationRelativeTo(null); // Center the frame
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setSize(200, 250);
    frame.setVisible(true);
  }
 
class NewLabel extends JPanel {
 
 
  protected void paintComponent(Graphics g) {
    super.paintComponent(g);
 
 
    g.drawRect(10,10,50,150);
    g.drawOval(10,10,50,50);
    g.drawOval(10,60,50,50);
    g.drawOval(10,110,50,50);
 
 
  }
}
}

I can't make it so that when you click the radio button it repaints the circle the appropriate color. In this case, the colors of a traffic light.

If the source is red: It needs to paint the top circle red

If the source is yellow: It needs to paint the center circle yellow

If the source is red: It needs " " " red...

I'm having trouble repainting it after the radio buttons are selected. I also don't even know if the radio buttons are doing anything... help is appreciated.

-Nate

I don't see any code that changes any color.

I don't see when you add a LightListener to anything, except in the LightListener itself, which doesn't make sense.

Originally Posted by Ha1luciNate

Thank you for re-stating my question

No problem, especially considering that your original post does not include an actual question (usually they end in question marks). You were obviously not far from a solution, but you didn't say what confused you or what you were expecting to happen. You simply posted an incomplete solution without an actual question. What kind of a response were you expecting?

Anyway, I did my best to point out the things you were missing in the hopes that you would attempt them and come back if you got stuck, maybe with a more specific question. The attitude is unnecessary, and it's only going to decrease your chances of getting help in the future.

Glad you got it figured out though.