ich lösche per Klick auf einen JButton in einer JTable eine Zeile. Diese wird aber nicht in der dazugehörigen MySQL gelöscht. Der DELETE Befehl ist unvollständig. Habe auch schon ewig nach dem richtigen Befehl gesucht. In der GUI sieht alles gut aus.
Java:
btnLoeschen.addActionListener(newLoeschenBtnAction(){@OverridepublicvoidactionPerformed(ActionEvent e){DefaultTableModel tblm =(DefaultTableModel)tblLoeschen.getModel();int row = tblLoeschen.getSelectedRow();int modelRow = tblLoeschen.convertRowIndexToModel(row);
tblm.removeRow(modelRow);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 st =(Statement) con.createStatement();String sql =[SIZE=5]"DELETE FROM fragen WHERE '"+row+"'";[/SIZE]
st.executeUpdate(sql);}catch(SQLException ex){System.out.println(e);}finally{if( con !=null)try{ con.close();}catch(SQLException ex ){ ex.printStackTrace();}}
lblLoeschen.setVisible(true);
tblm =newDefaultTableModel(newMySQL().holeFragen(),newString[]{"fragen_id","kurs","ort","dozent","datum"});}});
Hi,
wie du selbst schon richtig sagst, ist der DELETE-Befehl unvollständig.
In der Bedingung fehlt dir noch die Spalten-Angabe. In deinem Fall vermutlich so etwas wie
Du solltest die Spalte deiner Tabelle nehmen, die eindeutige IDs hat. Ansonsten löschst du evtl. ungewollt Daten.
Wenn du auf Zahlen vergleichst, solltest du die Anführungszeichen weg lassen. Also lediglich
Sich einen SQL so zusammen zubauen ist schlecht -> SQL Injection. Verwende lieber Prepared Statements!
row entspricht bei dir der Zeile in der JTable, aber sicher nicht der gesuchten fragen_id. (weil zum Beispiel nicht alle Fragen angezeigt werden oder diese nicht nach der ID sondern etwas anderem sortiert sind bzw. die ID's nicht mehr fortlaufend sind, da schon einige Fragen gelöscht wurden)
Entweder steht in der JTable die fragen_id drinnen, dann solltest du dir den entsprechenden Wert aus der selektierten Row holen. Oder du musst eben überlegen wie du deine Fragen besser im Model verwaltest
Tipp: schreibe dir eine Klasse für deine Fragen (falls noch nicht vorhanden), ein eigenes TableModel und verwalte damit eine Liste der Frage Objekte. Siehe dazu diesen Link: http://stackoverflow.com/questions/12559287/how-to-set-a-custom-object-in-a-jtable-row
Du solltest die Spalte deiner Tabelle nehmen, die eindeutige IDs hat. Ansonsten löschst du evtl. ungewollt Daten.
Wenn du auf Zahlen vergleichst, solltest du die Anführungszeichen weg lassen. Also lediglich
Ich hoffe du hast den Code nicht in dieser Reihenfolge stehen.
Erst setzt du das Model und dann erstellst du ein neues Model? Warum?
So kann das meiner Meinung nach nicht wirklich funktionieren.
Nein du holst dir nur den Wert der selektierten Row. Sprich du weißt welches Zeile selektiert wurde, aber nicht die "fragen_id" aus dieser Zeile.
Du müsstest irgendwo einen Aufruf auf das Model haben mit der Methode "getValueAt(...)";
das heißt ich füge im ActionListener des Löschen Button
Java:
tblm.getValueAt();
ein?
In den Klammern muss ich ja getValueAt(int rowIndex, int columnIndex) eintragen. Die Werte sind ja unbestimmt da ich ein getSelectedRow löschen möchte..
Wie kann ich das umsetzen?
Ja! Um einen bestimmten Datensatz aus deiner Tabelle "fragen" zu löschen musst brauchst du die entsprechende "fragen_id".
Diese steht in deiner JTable ja drinnen. Sprich mit getValueAt fragst du dein TableModel nach dem Wert der Spalte "fragen_id" in der selektierten Zeile
Mit diesem Wert kannst du dann dein SQL Statement ausführen und den gewünschten Datensatz löschen.
Wieso sind diese Werte "unbestimmt"? Der rowIndex entspricht ja deiner selektierten Zeile, columnIndex wäre die Spalte in deinem Fall (laut Bild) eben 1 (oder 0, weiß nicht ob das TableModel bei 0 oder 1 anfängt zu zählen). Die Spalte wird ja immer gleich bleiben, die selektierte Zeile bleibt variabel.
btnLoeschen.addActionListener(newLoeschenBtnAction(){@OverridepublicvoidactionPerformed(ActionEvent e){DefaultTableModel tblm =(DefaultTableModel)tblLoeschen.getModel();int row = tblLoeschen.getSelectedRow();[B]Object selected = tblm.getValueAt(row,0);[/B]int modelRow = tblLoeschen.convertRowIndexToModel(row);
tblm.removeRow(modelRow);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 st =(Statement) con.createStatement();String sql ="DELETE FROM fragen WHERE fragen_id = [B]'"+selected+"'"[/B];
st.executeUpdate(sql);}catch(SQLException ex){System.out.println(e);}finally{if( con !=null)try{ con.close();}catch(SQLException ex ){ ex.printStackTrace();}}
lblLoeschen.setVisible(true);
tblm =newDefaultTableModel(newMySQL().holeFragen(),newString[]{"fragen_id","kurs","ort","dozent","datum"});}});
jetzt muss ich noch knobeln wie ich es schaffe mehrere Zeilen gleichzeitig zu löschen
genügt es anstatt getSelectedRow() alterntiv getSelectedRows() zu schreiben?
jetzt muss ich noch knobeln wie ich es schaffe mehrere Zeilen gleichzeitig zu löschen
genügt es anstatt getSelectedRow() alterntiv getSelectedRows() zu schreiben?
Nein genügt nicht, die eine Methode gibt dir einfach nur ein int zurück die andere ein int[].
Mach doch einfach eine Schleife um dieses int[] durchzugehen
(Aber bitte mach vor der Schleife deine Verbindung auf und nach der Schleife zu. Nicht bei jedem Durchlauf auf- bzw. zumachen)
das mehrfach löschen lass ich sein, da das löschen hier eher selten vorkommt
ist es überhaupt sinnvoll die Verbindung nach einer Action zu schliessen? Habe ja noch eine Speichern Action und irgendwann auch noch Abfragen die auf die DB zugreifen sollen
ist es überhaupt sinnvoll die Verbindung nach einer Action zu schliessen? Habe ja noch eine Speichern Action und irgendwann auch noch Abfragen die auf die DB zugreifen sollen
Ich finde das ist schon ein wenig Processabhängig. Wenn ich es ne App habe, die nur von ein paar wenigen benutzt wird dafür aber alle paar Sekunden auf die DB zugreift, dann macht es sicher mehr Sinn die Connection Ofen zu halten als wenn die DB von 100000 Apps angesprochen wird, diese aber nur einen Request machen wenn der User etwas. Aktiviert.
Ich finde das ist schon ein wenig Processabhängig. Wenn ich es ne App habe, die nur von ein paar wenigen benutzt wird dafür aber alle paar Sekunden auf die DB zugreift, dann macht es sicher mehr Sinn die Connection Ofen zu halten als wenn die DB von 100000 Apps angesprochen wird, diese aber nur einen Request machen wenn der User etwas. Aktiviert.
Da hast du natürlich recht!
Ich wollte es gestern noch ausführlicher beschreiben und eben auf den Unterschiede zwischen einer lokalen SingleUser Anwendung und einer Client-Server-Architektur mit mehreren 100/1000 Usern hinweisen, bin aber leider nicht mehr dazu gekommen
