basic radiobutton question

[Adam Confino]

Dear Swing Gurus,

I am trying to work through some beginner tutorials and noticed that my radiobutton would allow you to check all of the options. Do the buttons need to be placed in some type of special container? My code is below:

    
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JRadioButton;

public class Swing4 extends JFrame {

	public Swing4() {
		super();
		this.setSize(400, 200);
		this.getContentPane().setLayout(null);
		this.add(getJLabel(), null);
		this.add(addRadioButton1());
		this.add(addRadioButton2());
		this.add(addRadioButton3());
		this.setTitle("Be-Da-Be-Da-Be");
	}

	private JRadioButton addRadioButton1() {
		JRadioButton radio1 = new JRadioButton("OOooo, pick me!");
		radio1.setBounds(85, 50, 125, 30);
		return radio1;
	}
	
	private JRadioButton addRadioButton2() {
		JRadioButton radio2 = new JRadioButton("No, no, pick me!");
		radio2.setBounds(85, 80, 125, 30);
		return radio2;
	}
	
	private JRadioButton addRadioButton3() {
		JRadioButton radio3 = new JRadioButton("You can only pick one!");
		radio3.setBounds(85, 110, 125, 30);
		return radio3;
	}

	private JLabel getJLabel() {
		JLabel jLabel = new javax.swing.JLabel();
		jLabel.setBounds(14, 49, 80, 18);
		jLabel.setText("JRadioBox:");

		return jLabel;
	}

	public static void main(String[] args) {
		Swing4 swing4 = new Swing4();
		swing4.setVisible(true);
	}

}

Thanks. I appreciate your time and help very much,
Adam

[Brian Cole]

Adam Confino wrote:I am trying to work through some beginner tutorials and noticed that my radiobutton would allow you to check all of the options. Do the buttons need to be placed in some type of special container?

You need to use a ButtonGroup. It's not a container in the GUI sense, but it's a non-graphical object that deselects the other buttons in the group when a button is selected.

If you're looking at beginner tutorials, they should mention ButtonGroup along with JRadioButton. Sun's tutorial does.


Then again, a beginner tutorial should not be having you do setLayout(null) and setBounds(...) either. It's brittle and in the long run no easier than doing it the right way.

[Adam Confino]

Thanks a bunch! I appreciate your help!