Buttons not re-displaying?

[Ryan Taubert]

What i'm doing is a simple image viewer & everything is fine other than my buttons.

I can't seem to display the buttons so that I click on 1 & it replaces the IconImage with another one. If you don't get what I'm trying to explain, maybe my bit of code for the buttons will be a better explanation.

private void radioButtons() {
		try {
		BufferedImage btn14 = ImageIO.read(Launcher.class.getResource("/sprites/btn14.png"));
		
		Radio1 = new JButton();
		Radio2 = new JButton();
		Radio3 = new JButton();
		Radio4 = new JButton();
		Radio5 = new JButton();
		Radio6 = new JButton();
		Radio7 = new JButton();
		
		if (SlideId == 1) {
			Radio1.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn14.png")));
			} else {
			Radio1.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn13.png")));
			}
		if (SlideId == 2) {
			Radio2.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn14.png")));
			} else {
			Radio2.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn13.png")));
			}
		if (SlideId == 3) {
			Radio3.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn14.png")));
			} else {
			Radio3.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn13.png")));
			}
		if (SlideId == 4) {
			Radio4.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn14.png")));
			} else {
			Radio4.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn13.png")));
			}
		if (SlideId == 5) {
			Radio5.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn14.png")));
			} else {
			Radio5.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn13.png")));
			}
		if (SlideId == 6) {
			Radio6.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn14.png")));
			} else {
			Radio6.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn13.png")));
			}
		if (SlideId == 7) {
			Radio7.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn14.png")));
			} else {
			Radio7.setIcon(new ImageIcon(Launcher.class.getResource("/sprites/btn13.png")));
			}
		
			Radio1.setRolloverIcon(new ImageIcon(btn14));
			Radio1.addActionListener(new ActionListener() {
				public void actionPerformed(ActionEvent arg0) {
					System.out.println("Radio Button 1 Pressed [Slider]");
					if (SlideId != 1) {
						SlideId = 1;
						radioButtons();
						System.out.println(SlideId);
					}
				}
			});
			Radio1.setBorder(new EmptyBorder(0, 0, 0, 0));
			Radio1.setOpaque(false);
			Radio1.setContentAreaFilled(false);
			Radio1.setBorderPainted(false);
			Radio1.setFocusPainted(false);
			Radio1.setToolTipText("Select");
			Radio1.setBounds(510, 296, 16, 16);
			contentPane.add(Radio1);

			Radio2.setRolloverIcon(new ImageIcon(btn14));
			Radio2.addActionListener(new ActionListener() {
				public void actionPerformed(ActionEvent arg0) {
					System.out.println("Radio Button 1 Pressed [Slider]");
				}
			});
			Radio2.setBorder(new EmptyBorder(0, 0, 0, 0));
			Radio2.setOpaque(false);
			Radio2.setContentAreaFilled(false);
			Radio2.setBorderPainted(false);
			Radio2.setFocusPainted(false);
			Radio2.setToolTipText("Select");
			Radio2.setBounds(548, 296, 16, 16);
			contentPane.add(Radio2);
		
			Radio3.setRolloverIcon(new ImageIcon(btn14));
			Radio3.addActionListener(new ActionListener() {
				public void actionPerformed(ActionEvent arg0) {
					System.out.println("Radio Button 1 Pressed [Slider]");
				}
			});
			Radio3.setBorder(new EmptyBorder(0, 0, 0, 0));
			Radio3.setOpaque(false);
			Radio3.setContentAreaFilled(false);
			Radio3.setBorderPainted(false);
			Radio3.setFocusPainted(false);
			Radio3.setToolTipText("Select");
			Radio3.setBounds(586, 296, 16, 16);
			contentPane.add(Radio3);
			
		//Radio Button End
		} catch (IOException e) {
			e.printStackTrace();
		}
	}

In the first button's actionevent you'll get a better understanding of what I'm trying to do. (Click on the button, change the image icon).

[Tony Docherty]

In the first button's actionevent you'll get a better understanding of what I'm trying to do. (Click on the button, change the image icon).

You aren't changing the buttons icon, you are calling the radioButtons() method again which will create a whole new set of buttons. You need to call the button's setIcon() method.

BTW Can I suggest you use the Java Naming Conventions as it makes the code easier for everyone to read eg class names start with an uppercase letter, variable and method names start with an lowercase letter etc.