Du verwendest einen veralteten Browser. Es ist möglich, dass diese oder andere Websites nicht korrekt angezeigt werden. Du solltest ein Upgrade durchführen oder ein alternativer Browser verwenden.
Ich baue eine Datenabfrage aus einer Tabelle "fragen". Um die Antworten in der Anzeige zu filtern habe ich Comboboxen die auf Spalten in der Tabelle zugreifen:
Java:
public void ladenBoxenKurs() {
Connection con = null;
try {
con = DriverManager.getConnection("jdbc:mysql://localhost:3306/schulaus?user=root&password=#Patrick1");
System.out.println("[MySQL] Die Verbindung zur MySQL wurde hergestellt");
Statement stmt = (Statement) con.createStatement();
ResultSet rs1 = stmt.executeQuery("select DISTINCT kurs from fragen"); // DISTINCT verhindert Mehrfachnennung
while(rs1.next()) {
boxKursM.addElement(rs1.getString(1));
}
ResultSet rs2 = stmt.executeQuery("select DISTINCT dozent from fragen");
while(rs2.next()) {
boxDozentM.addElement(rs2.getString(1));
}
ResultSet rs3 = stmt.executeQuery("select DISTINCT ort from fragen");
while(rs3.next()) {
boxOrtM.addElement(rs3.getString(1));
}
ResultSet rs4 = stmt.executeQuery("select DISTINCT datum from fragen");
while(rs4.next()) {
boxDatumM.addElement(rs4.getString(1));
}
con.close();
} catch (SQLException e) {
System.out.println(e);
e.printStackTrace();
}
}
Ich möchte erreichen, dass mir die Comboboxen nur Werte aus der Tabelle anzeigen wenn es auch mindestens einen Werte mit Daten der anderen Comboboxen gibt. Die Boxen dienen akso also als Filter. Bis jetzt funktionieren Sie unabhängig voneinander. Die Daten befinden sich jeweils in einem DefaultComboboxModel.
Kann mir jemand helfen wie ich das Problem löse??
Erkläre das Zusammenspiel der ComboBoxen genauer.
Die gleiche Frage kann von mehreren Dozenten gestellt werden? Sprich wählst du eine Frage aus werden nur die Dozenten angezeigt welche diese Frage gestellt haben (und in weiterer Abhängigkeit noch Ort und Datum).
Wenn du wiederrum einen Dozenten auswählst, sollen nur die Fragen gezeigt werden die dieser Dozent gestellt hat?
Warum ladest du alle ComboBox gleich zu Beginn? Ich würde es der Reihe nach machen.
Also mal nur die Fragen laden, sobald eine Frage ausgewählt wurde ladest du alle Dozenten die diese Frage gestellt haben in die ComboBox.
Wenn ein Dozent gewählt wurde dann ladest du den Ort usw.
Also folgendes habe ich vor:
Wir haben in der Firma für Schulungsteilnehmer Fragebögen (Papier) die ausgefüllt werden. Ich muss diese auswerten. Dafür habe ich ein Programm geschrieben:
1. hiererfasse ich jeden Fragebogen (18 Fragen) und speichere ihn in einer MySQL Tabelle
2. Unter Bearbeiten rufe ich den Auswertungsdialog auf:
ich habe es jetzt wie empfohlen umgebaut. beim öffnen lädt nur Dozent. Alles andere soll in Abhängigkeit der Auswahl der Combobox und der vorhandenen erfassten Daten laden. Die Reihenfolge siehe Screenshot von oben nach unten. Die Daten sollen rechts im Fenster angezeigt werden. Diese sollen entsprechend der Comboboxen angezeigt werden. Wenn also ein Dozent ausgewählt ist sollen alle Werte für den Dozenten angezeigt werden. Wenn ich dann den Kurs auswähle sollen die Werte zum Dozenten und dem Kurs angezeigt werden etc. Wie ich die ausgewerteten Daten rechts anzeige ist noch offen. Ich dachte an eine Tabell mit zwei Zeilen und Spalten k.A. bis 1. in der ersten Zeile fix die Titel der JButton und in der zweiten Zeile die Summe der jeweils gespeicherten Werte.
Anbei mein bisheriger Code für die Abfrage der Boxen und die ActionLiustener:
Java:
public void ladenBoxDozent() {
Connection con = null;
try {
con = DriverManager.getConnection("jdbc:mysql://localhost:3306/schulaus?user=root&password=#Patrick1");
System.out.println("[MySQL] Die Verbindung zur MySQL wurde hergestellt");
Statement stmt = (Statement) con.createStatement();
ResultSet rs1 = stmt.executeQuery("select DISTINCT dozent from fragen");
while(rs1.next()) {
boxDozentM.addElement(rs1.getString(1));
}
con.close();
} catch (SQLException e) {
System.out.println(e);
e.printStackTrace();
}
}
public void ladenBoxKurs() {
Connection con = null;
try {
con = DriverManager.getConnection("jdbc:mysql://localhost:3306/schulaus?user=root&password=#Patrick1");
System.out.println("[MySQL] Die Verbindung zur MySQL wurde hergestellt");
Statement stmt = (Statement) con.createStatement();
ResultSet rs2 = stmt.executeQuery("select DISTINCT kurs from fragen where fragen="+boxDozentM.getSelectedItem()); // DISTINCT verhindert Mehrfachnennung
while(rs2.next()) {
boxKursM.addElement(rs2.getString(1));
}
con.close();
} catch (SQLException e) {
System.out.println(e);
e.printStackTrace();
}
}
public void ladenBoxOrt() {
Connection con = null;
try {
con = DriverManager.getConnection("jdbc:mysql://localhost:3306/schulaus?user=root&password=#Patrick1");
System.out.println("[MySQL] Die Verbindung zur MySQL wurde hergestellt");
Statement stmt = (Statement) con.createStatement();
ResultSet rs3 = stmt.executeQuery("select DISTINCT ort from fragen");
while(rs3.next()) {
boxOrtM.addElement(rs3.getString(1));
}
con.close();
} catch (SQLException e) {
System.out.println(e);
e.printStackTrace();
}
}
public void ladenBoxDatum() {
Connection con = null;
try {
con = DriverManager.getConnection("jdbc:mysql://localhost:3306/schulaus?user=root&password=#Patrick1");
System.out.println("[MySQL] Die Verbindung zur MySQL wurde hergestellt");
Statement stmt = (Statement) con.createStatement();
ResultSet rs4 = stmt.executeQuery("select DISTINCT datum from fragen");
while(rs4.next()) {
boxDatumM.addElement(rs4.getString(1));
}
con.close();
} catch (SQLException e) {
System.out.println(e);
e.printStackTrace();
}
}
private class BoxDozentAusAction implements ActionListener {
@Override
public void actionPerformed(ActionEvent e) {
if (boxDozent.getSelectedIndex()>1) {
boxKurs.setEnabled(true);
ladenBoxKurs();
}
}
}
private class BoxKursAusAction implements ActionListener {
@Override
public void actionPerformed(ActionEvent e) {
if (boxKurs.getSelectedIndex()>1) {
boxOrt.setEnabled(true);
ladenBoxOrt();
}
}
}
private class BoxOrtAusAction implements ActionListener {
@Override
public void actionPerformed(ActionEvent e) {
if (boxOrt.getSelectedIndex()>1) {
boxDatum.setEnabled(true);
ladenBoxDatum();
}
}
}
private class BoxDatumAusAction implements ActionListener {
@Override
public void actionPerformed(ActionEvent e) {
}
}
Leider kommt nach der Auswahl der ersten Box gleich ein Fehler:
com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'Berndt' at line 1
Berndt ist der Nachname eines Dozenten. Ich habe aber in der SPalte Dozent der Vor- und Nachnamen.
Den Fehler zeigt er hier an:
Java:
ResultSet rs2 = stmt.executeQuery("select DISTINCT kurs from fragen where fragen="+boxDozentM.getSelectedItem()); // DISTINCT verhindert Mehrfachnennung
ich denke insgesamt sehr ambitioniert für einen Anfänger aber ich beisse mich da durch und mnit Hilfe dieses Forums konnte ich bislang jede Nuss knacken
ResultSet rs2 = stmt.executeQuery("select DISTINCT kurs from fragen where fragen="+boxDozentM.getSelectedItem()); // DISTINCT verhindert Mehrfachnennung
ich denke insgesamt sehr ambitioniert für einen Anfänger aber ich beisse mich da durch und mnit Hilfe dieses Forums konnte ich bislang jede Nuss knacken
Ich nehme an beim Dozenten handelt es sich um einen String. Strings müssen in SQL Statements unter Anführungszeichen gesetzt werden! (Grundlagen von SQL!)
Ich habe dir schon in einem anderen Thread ans Herz gelegt, dir deine Statements nicht per String zusammenzubauen!
Verwende PreparedStatements, dadurch ersparst du dir auch das Escapen von Strings
Ein weiterer Kritikpunkt an deinem Code:
In jeder ladenBox*() Methode erstellst du deine Connection, öffnest sie, erstellst ein Statement führst es aus, wertest es auch und schließt am Ende dann die Connection -> DUPLICATE CODE, den man vermeiden sollte
Lagere das Erstellen und Schließen der Verbindung in eine eigene Datenbankklasse aus! Wenn sich das Passwort oder die URL zur Datenbank ändert musst du jede Methode anpassen.
Außerdem liefert dir con.createStatement(..) schon ein Statement zurück, warum castest du nochmal zu einem Statement?
Ich nehme an beim Dozenten handelt es sich um einen String. Strings müssen in SQL Statements unter Anführungszeichen gesetzt werden! (Grundlagen von SQL!)
Ich habe dir schon in einem anderen Thread ans Herz gelegt, dir deine Statements nicht per String zusammenzubauen!
Verwende PreparedStatements, dadurch ersparst du dir auch das Escapen von Strings
Ganz ehrlich? Doch bist Du. Du verstehst eigentlich nichts von dem was Du da machst. Du kopierst Dir Code aus dem Internet und hoffst das er so funktioniert wie Du es willst. Du hast null Konzept und keinerlei System.
Wie wäre es Du liest erstmal die Grundlagen von mySQL durch? Wenn Du nicht einmal weißt wie man einen Query richtig aufbaut, dann will ich gar nicht wissen wie deine DB aussieht. Normalisierung dürfte dann sicher genauso ein Fremdwort sein wie SQL-Injection und andere grundlegend wichtige Dinge.
Was Du da machst ist, als wenn ich mir heute ein Skalpell ausleihe und einfach mal Deinen Bauch aufschneide um zu sehen wie ich am besten den Blinddarm entferne. Wenn ich nicht weiter komme, schau ich halt kurz ins Internet wie es geht und probiere das dann aus.....
Nein! Bei dieser Frage zeigt sich was Thallius anspricht. Weißt du denn genau was diese Zeile macht? Wofür das (Statement) vor dem con.createStatement steht?
harte Worte Jungs...
Ich verstehe noch nicht alles. Richtig. Bisher habe ich immer zuerst gemacht und dann im Laufe der Zeit auch verstanden. Ich habe vor einiger Zeit auch mal einen Wochenendkurs zu mySQL besucht, ich habe den Java Viedeo Kurs gekauft, gesehen und geübt, die Insel Bücher durchforstet. Mein Problem ist hier ganz klar, das ich neues Terrain betrete und beides Neues gleich verbinden will (JAVA, MySQL). Als Feierabendprogrammierer ein dickes Brett. Als Führungskraft in einem Unternehmen schätze ich übrigens am meisten die Mitarbeiter, die sich die großen Aufgaben vornehmen und dabei auch scheitern und daraus lernen
Aber ok, ich danke euch trotzdem für die bislang guten Hinweise die mich meist auf den richtigen Weg bringen.
...und jetzt widme ich mich gleich wieder meiner Lieblingsbaustelle um meinen Query richtig zu bauen
