Changing the icon of a JButton based on a JRadioButton selection

Question

I have a JRadioButton with an ActionListener but cannot figure out how to trigger an icon change for a JButton in a different panel when it's clicked. The code for both is listed below. The image needs to switch from the left button to the right when the correct radio button is selected.

package gui;

public class ExampleGUI extends JFrame {

private static final long serialVersionUID = 1L;
private JPanel contentPane;
ImageIcon icon = new ImageIcon(ExampleGUI.class
        .getResource("/gui/schlange1.gif"));

/**
 * Launch the application.
 */
public static void main(String[] args) {
    EventQueue.invokeLater(new Runnable() {
        public void run() {
            try {
                ExampleGUI frame = new ExampleGUI();
                frame.setVisible(true);
            } catch (Exception e) {
                e.printStackTrace();
            }
        }
    });
}

/**
 * Create the frame.
 */
public ExampleGUI() {
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setBounds(100, 100, 450, 300);
    contentPane = new JPanel();
    contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
    contentPane.setLayout(new BorderLayout(0, 5));
    setContentPane(contentPane);

    JLabel lblExampleGui = new JLabel("Example GUI");
    lblExampleGui.setFont(new Font("Tahoma", Font.PLAIN, 18));
    lblExampleGui.setHorizontalAlignment(SwingConstants.CENTER);
    contentPane.add(lblExampleGui, BorderLayout.NORTH);

    JPanel radioButtonPanel = new JPanel();
    contentPane.add(radioButtonPanel, BorderLayout.SOUTH);

    JPanel imagePanelBoxes = mainImagePanel();
    contentPane.add(imagePanelBoxes, BorderLayout.CENTER);

    JButton leftImage = leftClickImage();
    imagePanelBoxes.add(leftImage);

    JButton rightImage = rightClickImage();
    imagePanelBoxes.add(rightImage);

    JRadioButton leftRadioButton = leftRadioButton();
    radioButtonPanel.add(leftRadioButton);

    JRadioButton rightRadioButton = rightRadioButton();
    radioButtonPanel.add(rightRadioButton);

    ButtonGroup group = new ButtonGroup();
    group.add(leftRadioButton);
    group.add(rightRadioButton);
}

private JPanel mainImagePanel() {
    JPanel imagesPanel = new JPanel();
    imagesPanel.setBorder(new EmptyBorder(0, 5, 0, 5));
    imagesPanel.setLayout(new GridLayout(0, 2, 10, 0));
    return imagesPanel;
}

private JRadioButton leftRadioButton() {
    final JRadioButton leftRadioButton = new JRadioButton("LEFT");
    leftRadioButton.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent e) {
            changeIcon(leftClickImage(), icon);
        }
    });
    leftRadioButton.setSelected(true);
    return leftRadioButton;
}

private JRadioButton rightRadioButton() {
    final JRadioButton rightRadioButton = new JRadioButton("RIGHT");
    rightRadioButton.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent e) {
            changeIcon(rightClickImage(), icon);
        }
    });
    rightRadioButton.isSelected();
    return rightRadioButton;
}

private JButton leftClickImage() {
    JButton leftImage = new JButton("");
    leftImage.setIcon(new ImageIcon(ExampleGUI.class
            .getResource("/gui/schlange1.gif")));
    leftImage.setBackground(Color.BLACK);
    return leftImage;
}

private JButton rightClickImage() {
    final JButton rightImage = new JButton("");
    rightImage.setBackground(Color.BLACK);
    return rightImage;
}

public void changeIcon(JButton jb, ImageIcon icon) {
    jb.setIcon(icon);
}

}

You neither have a button nor a radiobutton. you have two methods who are returning those. Do you need those methods, or do you want to achieve, what you described above? If last, see below, if not, clarify your needs — meister_reineke, Mar 18 '15 at 15:03
I added the complete code. The image is defaulted to show on the left button and needs to switch to the right button when the right radio button is clicked. — KMKZ509, Mar 18 '15 at 15:41
its deffinitly not the complete code, your class-deffnition is missing XD — meister_reineke, Mar 18 '15 at 16:08

e-oj · Answer 1 · 2015-03-18T15:04:48.037

public class SwitchButton {
public static void main(String [] args){
    SwitchButton sb = new SwitchButton();
}

JFrame jfButtons =  new JFrame();
JPanel jpButtons =  new JPanel();
JRadioButton jrb = new JRadioButton("if you click me");
JButton jb = new JButton("I'll change");

public SwitchButton(){
    jrb.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
            changeColor(jb, Color.blue);
        }
    });
    jpButtons.add(jrb);
    jpButtons.add(jb);
    jfButtons.add(jpButtons);
    jfButtons.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
    jfButtons.setVisible(true);
    jfButtons.pack();
}

public void changeColor(JButton jbtn, Color color){
    jbtn.setBackground(color);
}

}

This does basically what you're trying to do. You'll just have to change changeColor() to changeIcon()

Still new with Java so I am not sure how to implement your code into mine. I have added my complete code if you want to look it over. — KMKZ509, Mar 18 '15 at 15:48

score 0 · Answer 2 · answered Mar 18 '15 at 14:56

0

Just simply

yourJButton.setIcon(yourIconToSet);

and this should be ofc called from action listener on radio button

leftRadioButton.addActionListener(new ActionListener() {
    public void actionPerformed(ActionEvent arg0) {
        // ? <- here invoke code to change your button's label
    }
});

answered Mar 18 '15 at 14:56

Antoniossss

31,590
6
57
99

I tried that but it did not work. I added the complete code if you want to check it out. – KMKZ509 Mar 18 '15 at 15:45

score 0 · Answer 3 · answered Mar 18 '15 at 16:03

You could provide the Button, who is to be changed as an argument of your JRadioButton-creator, but that looks quite unreadable and ill-designed

private JRadioButton leftRadioButton(JButton affectedButton) {
    JRadioButton leftRadioButton = new JRadioButton("LEFT");
    leftRadioButton.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent arg0) {
            if (leftRadioButton.isSelected())
                affectedButton=rightRadioButton();
            else 
                affectedButton=leftRadioButton();
        }
    });
    leftRadioButton.setSelected(true);
    return leftRadioButton;
}

I would rather use not an in-line deffinition of your actionListener but, implement it in your frame (or whatelse class you use) to get access to the Buttons and Labels etc used in that class. It gets quite more readable, if you dont have too many items to listen to.

public class buttonchanger extends JFrame implements ActionListener{
    JPanel radioButtonPanel;
    JPanel imagePanelBoxes;
    JButton leftImage;
    JButton rightImage;
    JRadioButton leftRadioButton;
    JRadioButton rightRadioButton;

    public initGUI() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(100, 100, 450, 300);
        contentPane = new JPanel();
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
        contentPane.setLayout(new BorderLayout(0, 5));
        setContentPane(contentPane);

        JLabel lblExampleGui = new JLabel("Example GUI");
        lblExampleGui.setFont(new Font("Tahoma", Font.PLAIN, 18));
        lblExampleGui.setHorizontalAlignment(SwingConstants.CENTER);
        contentPane.add(lblExampleGui, BorderLayout.NORTH);

        JPanel radioButtonPanel = new JPanel();
        contentPane.add(radioButtonPanel, BorderLayout.SOUTH);

        JPanel imagePanelBoxes = mainImagePanel();
        contentPane.add(imagePanelBoxes, BorderLayout.CENTER);

        JButton leftImage = leftClickImage();
        imagePanelBoxes.add(leftImage);

        JButton rightImage = rightClickImage();
        imagePanelBoxes.add(rightImage);

        JRadioButton leftRadioButton = leftRadioButton();
        leftRadioButton.addActionListener(this);
        radioButtonPanel.add(leftRadioButton);

        JRadioButton rightRadioButton = rightRadioButton();
        rightRadioButton.addActionListener(this);
        radioButtonPanel.add(rightRadioButton);

        ButtonGroup group = new ButtonGroup();
        group.add(leftRadioButton);
        group.add(rightRadioButton);
    }

    public void ActionListener(ActionEvent actE){
        Object obj=actE.getSource();
        if (obj==leftRadioButton){
            leftImage.setIcon(yourIcon);
            //or do whatever you intend to do
        }
    }

}

hope this is more of the solution you were looking for. I still dont know what button should change to what state after the radioButton-Event

Thank you for answering, unfortunately I am still having issues. I am going to check with a tutor to see if they can help. — KMKZ509, Mar 18 '15 at 17:56

score 0 · Answer 4 · answered Mar 18 '15 at 18:07

0

ImageIcon icon = new ImageIcon("img src"); 

rightRadiobutton.addActionListener(new ActionListener(ActionEvent ae){
   changeIcon(rightButton, icon); 
});

public void changeIcon(JButon jb, ImageIcon icon){
    jb.setIcon(icon);
}

You do the same for leftRadioButton. Also you dont need separate methods to create components; you can just assign and use them like:

JRadioButton rightJrb = new JRadioButton("I am a radio button");
rightJrb.addActionListener();

answered Mar 18 '15 at 18:07

e-oj

209
2
5

I edited the question code with the most recent code minus imports. I am only using one class. After trying your suggestions it still will not show an image in the right button while the right radio button is selected. I also tried using an if statement to check if the radioButton isSelected but still nothing. – KMKZ509 Mar 18 '15 at 18:50

Changing the icon of a JButton based on a JRadioButton selection

4 Answers4