Array aus JCheckBox? 1000 if's UPDATE: Spaltennamen auslesen

[norman]

mahlzeit,

mit meiner applikation möchte ich spalten einer tabelle auswählen und anschließend die tabelle ausgeben. das klappt bisher auch, nur ist mir der code etwas zu umständlich:

ich erstelle zunächst für jede mögliche spalte (zunächst 10) eine JCheckBox mit entpsrechendem namen und text.
in einer methode auswahl() wird dann einem string TXT (anfangswert="") der spaltenname und ein ',' angehangen.
das ist bisher mit sovielen if's realisert, wie es auch spalten gibt. also derzeit 10.
Auszug:

if (cb_vorname.isSelected())           //wenn JCheckBox cb_vorname aktiv hänge
        txt=txt.concat("vorname,");   //"vorname," an txt an
if (cb_name.isSelected())
	txt=txt.concat("name,");

meine methode abfrage() sieht etwa so aus:
sende_an_datenbank_die_abfrage("SELECT " + TXT + "FROM" + tabelle);

wenn ich die anwendung erweitern möchte, muss ich folglich immer mehr JCheckBox und immer mehr if's schreiben. hat hier jemand eine elegantere möglichkeit? die aktuelle version ist bei 3 spalten schon ziemlich mies.

grüße
norman

 

[m@nu]

entweder eine schleife über dein array machen:

for(int i = 0, l = checkBoxArray.length; i < l; i++) {
  // if...
}

oder eine JList mit entsprechendem Renderer verwenden (such mal nach tutorials/posts hier im forum... gibts einige )

 

[Mag1c]

Hi,

Zusatz zum Array-Vorschlag:

den entsprechenden Spaltennamen könntest du im "name" der JCheckBox speichern, so daß du ihn in der for-Schleife per getName() in deine SQL-Anfrage einbauen kannst. Wenn dir das mit "name" nicht gefällt, kannst du auch eine Ableitung von JCheckBox() machen, welche dann per getColumnName() den Spaltennamen liefern kann.

Gruß
Mag1c

 

[m@nu]

gute idee, aaaaaber:
wieso einfach wenns au kompliziert geht?

speicher den spaltennamen per

checkbox.setActionCommand("spaltenname");

anschliessend kannst du per

checkbox.getActionCommand();

das ganze wieder auslesen.

 

[Sky]

gute idee, aaaaaber:
wieso einfach wenns au kompliziert geht?

speicher den spaltennamen per

checkbox.setActionCommand("spaltenname");

anschliessend kannst du per

checkbox.getActionCommand();

das ganze wieder auslesen.

Die Lösung finde ich ziemlich unsauber. ActionCommand ist kein Speicher für beliebige Strings, die irgendwie zur Box dazugehören könnten!

Die Idee mit "name" bzw. "getColumnName()" von Mag1c finde ich besser!

 

[norman]

danke..läuft soweit )

habe es jetzt so:

for (int i=0;i<checkboxarray.length; i++) {
			if (checkboxarray[i].isSelected()) {
				txt=txt.concat(checkboxarray[i].getText());		
				txt=txt.concat(",");							
				}		
		}

	private JCheckBox vorname		= new JCheckBox("Vorname");
	private JCheckBox name 			= new JCheckBox("Name");
	private	JCheckBox nummer		= new JCheckBox("Nummer");
	private	JCheckBox strasse		= new JCheckBox("Straße");
	private	JCheckBox plz			= new JCheckBox("PLZ");
	private	JCheckBox ort			= new JCheckBox("Ort");
	private	JCheckBox titel			= new JCheckBox("Titel");
	private	JCheckBox isbn			= new JCheckBox("ISBN");
	private	JCheckBox genre			= new JCheckBox("Genre");
	private	JCheckBox verfuegbar	= new JCheckBox("Verfuegbar?");

private JCheckBox[] checkboxarray	={vorname, name, nummer, strasse, plz, ort, titel, isbn, genre, verfuegbar};

=> kann man die jcheckboxen auch im array deklarieren? ist ja immernoch ziemlich viel code so.

 

[m@nu]

@sky: hab ich mir auch überlegt... grundsätzlich deiner meinung. denke aber, da keine ActionListener auf die CheckBox's zugewiesen werden, könnte man da doch ein auge zukneifen

@norman: lös' die instanzierung des arrays doch auch mit einer schleife...

String[] cols = new String[]{"Vorname","Name","Nummer","Strasse","PLZ","Ort","Titel","ISBN","Genre","Verfügbar?");
JCheckBox[] checkboxArray = new JCheckBox[cols.length];

for(int i = 0, l = cols.length; i < l; i++) {
  checkboxArray[i] = new JCheckBox(cols[i]);

  // Falls deine Spalten in der DB nicht gleich heisen, wie die beschriftung lauten soll,
  // erstell einfach ein zweites String-Array mit den entsprechenden Bezeichner aus
  // der DB
  checkboxArray[i].setActionCommand(cols[i]);
  // oder auch checkboxArray[i].setColumnName(cols[i]);  ;-)
}

als erweiterung könntest du die spaltennamen von der datenbank einlesen und entsprechend deine spaltenauswahl generieren lassen :arrow: flexibler

 

[Mag1c]

Hi,

String[] columns = new String[] {"Vorname", "Name", "Nummer", "Strasse", "Plz", "Ort", "Titel", "ISBN", "Genre", "Verfuegbar"};
JCheckBox[] boxes;

boxes = new JCheckBox[columns.length];
for (int i=0; i < columns.length; i++) {
    boxes[i] = new JCheckBox(columns[i]);
}

EDIT: da war ich wohl ein My zu langsam

Gruß
Mag1c

 

[norman]

gibt es synchron-programmier-weltmeisterschaften?

naja..danke jedenfalls.

PS: das mit dem einlesen der spalten habe ich auch schon in betracht gezogen..das könnte dann auch den fehler vermeiden, wenn ich spalten auswähle, die nicht zur ausgewählten tabelle passen..

schönen tag dann noch..

 

[norman]

@norman: lös' die instanzierung des arrays doch auch mit einer schleife...

String[] cols = new String[]{"Vorname","Name","Nummer","Strasse","PLZ","Ort","Titel","ISBN","Genre","Verfügbar?");
JCheckBox[] checkboxArray = new JCheckBox[cols.length];

for(int i = 0, l = cols.length; i < l; i++) {
  checkboxArray[i] = new JCheckBox(cols[i]);

  // Falls deine Spalten in der DB nicht gleich heisen, wie die beschriftung lauten soll,
  // erstell einfach ein zweites String-Array mit den entsprechenden Bezeichner aus
  // der DB
  checkboxArray[i].setActionCommand(cols[i]);
  // oder auch checkboxArray[i].setColumnName(cols[i]);  ;-)
}

als erweiterung könntest du die spaltennamen von der datenbank einlesen und entsprechend deine spaltenauswahl generieren lassen :arrow: flexibler

also ich habe jetzt geschafft die tabellenauswahl (das ist eine Choice bei mir) automatisch zu generieren. das sieht jetzt so aus:

	void set_user_tabellen(){
		try {
			Connection con = DriverManager.getConnection("jdbc:oracle:thin:@172.20.20.225:1592:ora92","nrudnik","tasse");    
			Statement stmt = con.createStatement();
		    ResultSet rset = stmt.executeQuery("SELECT table_name FROM user_tables");
		    
		    while (rset.next()){
		    	tabellen_auswahl.add(rset.getString(1));
		    }
		}
		catch (SQLException e) {
			e.printStackTrace();
		}
	}

so ähnlich wollte ich das auch mit den checkboxen (entsprechen den spalten der ausgewählten tabelle) machen:

		    while (rset.next()){
		    	JCheckBox checkbox = new JCheckBox("/*SpaltenName*/");
		    	checkboxen.add(checkbox);
		    }

allerdings habe ich keine idee, wie ich an den spaltennamen rankomme...

 

[Mag1c]

Moin,

con.getMetaData().getColumns(...)

Gruß
Mag1c

 

[norman]

okay, deine variante ist wohl kürzer mag1c. danke
(hatte es jetzt so gelöst

		    ResultSet rset = stmt.executeQuery("SELECT * FROM " + tabelle);	    
		    ResultSetMetaData cols = rset.getMetaData();
		    
		    for (int i=1; i <= cols.getColumnCount(); i++) {
		    	JCheckBox checkbox = new JCheckBox(cols.getColumnName(i));
		    	checkboxen.add(checkbox);
		    }

 

[m@nu]

diesesmal war er schneller gg