Variablen Parameter bekommen

[Lucaaa]

Hallo!
Ich versuche mein Problem mal zu beschreiben:
* Ich habe eine GUI mit einer Combobox, 3 Textfeldern, und einem Colorpicker.
* In einer Datenbank habe ich Daten zu Schulfächern gespeichert. (ID, Name, Leher, Raum, Hintergrundfarbe)
* In der GUI sollen diese Daten bearbeitet werden (entsprechende Funktion habe ich schon, und ist nicht mein problem.)
* Die Combobox soll die Namen der Schulfächer anzeigen (zur Auswahl)
* Die Textfelder zeigen: Name, Lehrer und Raum, der Colorpicker die Farbe.
* Die Daten hole ich aus durch ein SQL statement

SELECT * FROM Subjects WHERE ID=9846518949898;  // Nur Beispiel

So bekomme ich dann eine Liste aus ID, Name, Lehrer […]

* Die Combobox hat einen ActionListener.
* Durch Klicken eines Items sollen nun die Felder Aktualisiert werden (Auf die daten des Faches.

* Meine Frage: Wie bekomme ich die daten? Die Combobox speichert nur den Namen und für die Funktion zum laden wird die ID benötigt. Ich muss diese also irgendwie speichern.

* Mein Gedanke
- Combobox aus HBoxen
- Einfach unsichtbares Label das die ID speichert.
- mit:

ArrayList<String> Infos = getSubjectInfos(event.getSource().getChildreen.get(1).getText() );
// nur als Bsp,

die Daten laden, und setzen.

Kann / sollte ich das so machen oder
Gibt es da eine bessere Lösung?

Vielen Dank schon mal!

 

[Flown]

Sprechen wir hier von JavaFX oder Swing?

 

[Lucaaa]

Sprechen wir hier von JavaFX oder Swing?

JavaFX

 

[Flown]

Warum verwaltest du nicht deine Daten in einer ObservableList und befüllst sie dann richtig in einer onMouseClick?

 

[Lucaaa]

Warum verwaltest du nicht deine Daten in einer ObservableList und befüllst sie dann richtig in einer onMouseClick?

Du meinst, ich soll ALLE Daten zum Anfang einmal laden?, und dann einfach entsprechend einsetzen?

 

[Lucaaa]

[/Code]

Warum verwaltest du nicht deine Daten in einer ObservableList und befüllst sie dann richtig in einer onMouseClick?

Und wie speichere ich mehrere Elemente in einem ListItem ? mit String[] ?

 

[Flown]

Du meinst, ich soll ALLE Daten zum Anfang einmal laden?, und dann einfach entsprechend einsetzen?

Ich kenn die Details nicht. Ich würde es beim aufbau der jeweiligen View laden. Oder eben, wenn auf diese Combobox geklickt wird.

Und wie speichere ich mehrere Elemente in einem ListItem ? mit String[] ?

Richtige Objekte!

 

[Lucaaa]

Ich kenn die Details nicht. Ich würde es beim aufbau der jeweiligen View laden. Oder eben, wenn auf diese Combobox geklickt wird.

Es ist ein Dialog.
Ich lade es jetzt, beim Aufbau des Dialoges.
So:

public ArrayList<String[]> loadSubjects() {
  ArrayList<String[]> subjects = new ArrayList<>();
 
  try {
   Statement stat = connection.createStatement();
   stat.executeUpdate("CREATE TABLE IF NOT EXISTS Subjects ('ID', 'Name', 'Teacher', 'Room', 'Color');");
   ResultSet rs = stat.executeQuery("SELECT * FROM Subjects;");
   
   while(rs.next()) {
    String[] sub = new String[] {
     rs.getString(1), rs.getString(2),
     rs.getString(3), rs.getString(4),
     rs.getString(5)
    };
    subjects.add(sub);
   } 
   
   
   
   stat.close();
   rs.close();
 
  } catch (SQLException e) {
   // TODO Auto-generated catch block
   e.printStackTrace();
  }
 
  return subjects;
 }

und dann:

@Override
 public void handle(ActionEvent event) {
  if(event.getSource()==combSubjects) {
   String item = combSubjects.getValue();
   int itemIndex = combSubjects.getItems().indexOf(item);
   tfName.setText(subjects.get(itemIndex)[1]);
   tfTeacher.setText(subjects.get(itemIndex)[2]);
   tfRoom.setText(subjects.get(itemIndex)[3]);
   btnColor.setValue(Color.web((subjects.get(itemIndex)[1]).replace("#", "")));
  } else if(event.getSource()==btnColor) {
   
  } else if(event.getSource()==btnOk) {
   save();
  }
 }

Ob das jetzt die beste Lösung ist weiß ich nicht, aber es funktioniert wie es soll, und ich muss auch nur einmal auf die Datenbank zugreifen anstatt 20x.

 

[Flown]

Ein String-Array ist kein ordentliches Objekt!

 

[Lucaaa]

Ein String-Array ist kein ordentliches Objekt!

Was soll ich sonst verwenden?
Und wieso ist das kein ordentliches Objekt?

 

[Robat]

Was soll ich sonst verwenden?

Beispielhaft :

class Subject {
    private int id;
    private String name;
    private String teacher; // eventuell ist Teacher auch ein eigenes Objekt
    private String room; // eventuell ist Room auch ein eigenes Objekt
    private Color color;

    // Constructor

    // Getter & Setter
}

 

[Lucaaa]

Beispielhaft :

Danke!
Aber nochmal die Frage:
Warum ist String[] kein richtiges Objekt?
String[] ist doch ein Array der Klasse String, die wiederunm als komplexer Datentyp ein Objekt ist oder nicht?

 

[Robat]

Technisch gesehen ist ein String[] ein Objekt. Hier ist es aber so gemeint, dass dein String[] eine Ansammlung von Eigenschaften einer Klassendefinition sind. Anstatt immer das Array mitzuschleppen und händisch über die Indizes die einzelnen Eigenschaften abzufragen, kannst du das ganze auch "sauber" in eine eigene Klasse wrappen, welche die Attribute ID, Name, .. besitzt. So können die Attribute auch die Datentypen bekommen, die sie verdienen und musst nicht alles als Strings auffassen. Eine ID ist i.d.R. nun mal kein String.

 

[Lucaaa]

sauber" in eine eigene Klasse wrappen

Ok.
Fazit ► ► ►
Wenn ich mehrere Objekte Habe (gleicher typ, verschiedene Eigenschaften), speichere ich diese in einer Klasse (Objekt). ► Objektorientierung