Panel mit Radiobuttons als eigene Klasse

[buster]

Hallo,

ich möchte für mein Programm ein Panel als Klasse erstellen, wo schon Radiobuttons drauf sind (es soll so eine Art interaktiver Fragebogen werden, wo z.B. immer wieder als Antwort "Ja" oder "Nein" vorkommt).

Da ich ein ziemlicher Newbie in Java bin, hab ich jetzt ein Problem (was auch sonst... ) : das Panel wird nicht dargestellt

Das Hauptprogramm:

import java.awt.Dimension;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;

import javax.swing.BorderFactory;
import javax.swing.JFrame;
import javax.swing.JPanel;

import panels.PanelJaNein;

public class PTestMain extends JFrame {
	public static void main(String[] args) {
		PTestMain f = new PTestMain();
		f.setTitle("PTest");
		JPanel sp2 = new JPanel();
		sp2.setPreferredSize(new Dimension(500,250));
		sp2.setBorder(BorderFactory.createEtchedBorder());
		sp2.setLayout(new GridBagLayout());
		GridBagConstraints gbc1 = new GridBagConstraints();
		PanelJaNein pn1 = new PanelJaNein("Ja");
		pn1.setPreferredSize(new Dimension(250,50));
	    gbc1.gridheight = 1;
	    gbc1.weightx = 1.0;
	    gbc1.weighty = 1.0;
	    gbc1.gridwidth = GridBagConstraints.REMAINDER;
	    gbc1.anchor = GridBagConstraints.NORTHWEST;
		sp2.add(pn1, gbc1);
	    f.add(sp2);
		f.pack();
		f.setVisible(true);
	}

}

und hier die Klasse mit dem Panel:

package panels;

import java.awt.FlowLayout;

import javax.swing.ButtonGroup;
import javax.swing.JPanel;
import javax.swing.JRadioButton;

public class PanelJaNein extends JPanel {

	public String antwort;
	public JPanel pn1;
	
	public PanelJaNein(String antwort) {
		super();
		this.antwort = antwort;
		pn1 = new JPanel();
		pn1.setLayout(new FlowLayout());
		ButtonGroup bg1 = new ButtonGroup();
		JRadioButton rbJa = new JRadioButton("Ja");
		JRadioButton rbNein = new JRadioButton("Nein");
		bg1.add (rbJa);
		bg1.add (rbNein);
		if (antwort == "Ja") {
			rbJa.setSelected(true);
		}
		else if (antwort == "Nein") {
			rbNein.setSelected(true);
		}
		pn1.add(rbNein, FlowLayout.LEFT);
		pn1.add(rbJa, FlowLayout.LEFT);
	}

}

Rufe ich das nur falsch auf, oder habe ich es schon falsch deklariert???

Bin für jede Hilfe dankbar...

Beste Grüße,
der Buster

 

[Beni]

Wieso erstellst du in deiner PanelJaNein-Klasse noch ein weiteres Panel? Wirf es raus (das "pn1"), du kannst die Buttons direkt deinem PanelJaNein hinzufügen.

 

[The_S]

desweiteren, Strings vergleicht man mit equals und nicht ==

 

[Eldaron]

Der Fehler liegt ... da drin

public PanelJaNein(String antwort) { 
      super(); 
      this.antwort = antwort; 
      pn1 = new JPanel(); 
      pn1.setLayout(new FlowLayout()); 
      ButtonGroup bg1 = new ButtonGroup(); 
      JRadioButton rbJa = new JRadioButton("Ja"); 
      JRadioButton rbNein = new JRadioButton("Nein"); 
      bg1.add (rbJa); 
      bg1.add (rbNein); 
      if (antwort == "Ja") { 
         rbJa.setSelected(true); 
      } 
      else if (antwort == "Nein") { 
         rbNein.setSelected(true); 
      } 
      pn1.add(rbNein, FlowLayout.LEFT); 
      pn1.add(rbJa, FlowLayout.LEFT); 
   }

werf mal sämtliche Verwendungen von pn1 raus ... so dass das da rauskommt:

public PanelJaNein(String antwort) { 
      super(); 
      this.antwort = antwort; 
      setLayout(new FlowLayout()); 
      ButtonGroup bg1 = new ButtonGroup(); 
      JRadioButton rbJa = new JRadioButton("Ja"); 
      JRadioButton rbNein = new JRadioButton("Nein"); 
      bg1.add (rbJa); 
      bg1.add (rbNein); 
      if (antwort.equals("Ja")) { 
         rbJa.setSelected(true); 
      } 
      else if (antwort.equals("Nein")) { 
         rbNein.setSelected(true); 
      } 
      add(rbNein, FlowLayout.LEFT); 
      add(rbJa, FlowLayout.LEFT); 
   }

 

[buster]

Danke für die schnellen Antworten, jetzt funktionierts! *freu*
Ich hab wohl ganz übersehen, dass PanelJaNein ja schon von JPanel geerbt hat...

Wenn ich jetzt herausfinden will, welche Antworten geklickt wurden (z.B. beim Schließen der Seite), brauch ich dafür sicher eine Methode in der Art "pn1.getAntwort()", und das über alle erzeugten Instanzen von PanelJaNein. Und da ich ja vorher nicht weiss, wieviele Fragen der Fragebogen enthält, sollte pn1 vielleicht ein array sein (pn)? Wo ich dann alle einzeln abfragen kann? Oder gibts da auch andere (bessere) Möglichkeiten?

Nochmals vielen Dank,
der Buster