Overriding Methods on JRadioButton and JButton

Question

public class CustomRadioButton extends JRadioButton {
    private final Color pressedForeground = Color.YELLOW;
    private final Color foreground = Color.WHITE;
    private final ImageIcon icon = new ImageIcon(getClass().getResource("/com/project/Graphics/Button_UP.gif"));
    private final ImageIcon pressedIcon = new ImageIcon(getClass().getResource("/com/project/Graphics/Button_DOWN.gif"));
    public CustomRadioButton() {
        setBorderPainted(false);
        setFocusPainted(false);
        setIcon(icon);
        setForeground(foreground);
        setSelectedIcon(pressedIcon);
        setContentAreaFilled(false);
        setHorizontalTextPosition(SwingConstants.CENTER);
        setFont(new Font("Arial", Font.BOLD, 12));
        setMargin(new Insets(0, 0, 0, 0));
        setMinimumSize(new Dimension(101,25));
        setMaximumSize(new Dimension(101,25));
        setPreferredSize(new Dimension(101,25));
        setBorder(null);
    @Override
    public Icon getIcon() {
        setForeground(foreground);
        return super.getIcon();
    @Override
    public Icon getSelectedIcon() {
        setForeground(pressedForeground);
        return super.getSelectedIcon();
}Why does this code cause an infinite loop when it's in a selected state? I ran a profiler on my project and found that it infinitely calls painComponent(), getIcon(), and getSelectedIcon() when it's pressed. Yet when I create my object as a regular JRadioButton with all of these setting and an ActionListener to change the foreground color there is no infinite loop. Am I missing something here?

Answer 1

A paintComponent override doen't seem to trigger the repaint loop.import java.awt.*;
import javax.swing.*;
public class SelectedForegroundRadioButtonTest {
   public static void main(String[] args) {
      SwingUtilities.invokeLater(new Runnable() {
         @Override
         public void run() {
            new SelectedForegroundRadioButtonTest().makeUI();
   public void makeUI() {
      ButtonGroup group = new ButtonGroup();
      JPanel panel = new JPanel(new GridLayout(0, 1));
      for (int i = 0; i < 5; i++) {
         JRadioButton button = new SelectedForegroundRadioButton("Button " + i);
         panel.add(button);
         group.add(button);
      JFrame frame = new JFrame();
      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      frame.add(panel);
      frame.pack();
      frame.setLocationRelativeTo(null);
      frame.setVisible(true);
class SelectedForegroundRadioButton extends JRadioButton {
   private final Color pressedForeground = Color.ORANGE;
   private final Color defaultForeground = Color.BLUE;
   private final int iconSize = 16;
   public SelectedForegroundRadioButton(String text) {
      super(text);
      setIcon(new DefaultIcon());
      setPressedIcon(new PressedIcon());
      setSelectedIcon(new SelectedIcon());
   @Override
   public void paintComponent(Graphics g) {
      setForeground(isSelected() ? pressedForeground : defaultForeground);
      super.paintComponent(g);
   class DefaultIcon implements Icon {
      public void paintIcon(Component c, Graphics g, int x, int y) {
         g.setColor(isSelected() ? pressedForeground : defaultForeground);
         g.drawOval(0, 0, iconSize, iconSize);
         g.drawOval(1, 1, iconSize - 2, iconSize - 2);
      public int getIconWidth() {
         return iconSize;
      public int getIconHeight() {
         return iconSize;
   class PressedIcon extends DefaultIcon {
      @Override
      public void paintIcon(Component c, Graphics g, int x, int y) {
         super.paintIcon(c, g, x, y);
         g.setColor(Color.GREEN);
         g.fillOval(3, 3, iconSize - 6, iconSize - 6);
   class SelectedIcon extends PressedIcon {
      @Override
      public void paintIcon(Component c, Graphics g, int x, int y) {
         super.paintIcon(c, g, x, y);
         g.setColor(Color.RED);
         g.fillOval(5, 5, iconSize - 10, iconSize - 10);
}db