JTable Zeile wird nicht in MySQL gelöscht

[didi577]

Moin,

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.

btnLoeschen.addActionListener(new LoeschenBtnAction() {
           @Override
           public void actionPerformed(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 = new DefaultTableModel(new MySQL().holeFragen(), new String [] {"fragen_id", "kurs", "ort", "dozent", "datum"});
           }
       });

Kann mir jemand helfen ???

 

[Thallius]

String sql = "DELETE FROM fragen WHERE '"+row+"'";

Übersetzt das mal wenn row z.B. 3 ist und sage mir dann ob das in Deinen Augen Sinn ergibt....

Lösche von fragen wo 3

ausserdem wirst du dich wundern wenn du die Tabelle mal anders sortierst. Dann wird nämlich die falsche Zeile gelöscht...

 

[W9ND3R]

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

DELETE FROM fragen WHERE ROW='" + row + "';

Siehe dazu auch http://www.1keydata.com/de/sql/sql-delete.php

 

[didi577]

"DELETE FROM fragen WHERE fragen_id = '"+row+"'";

mit der Spaltenangabe wird leider auch nocht gelöscht :-(
kann man da irgendeine Spalte aus der My SQL Tabelle benennen?

 

[W9ND3R]

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

"DELETE FROM fragen WHERE fragen_id=" + row;

 

[Joose]

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

 

[didi577]

das ist meine JTable, in dieser ist ein TableModel, also fragen_id wird in der JTable angezeigt

    tblLoeschen.setModel(tblm);
    tblm = new DefaultTableModel(new MySQL().holeFragen(), new String [] {"fragen_id", "kurs", "ort", "dozent", "datum"});
       tblm.addTableModelListener( new TableAction() {

ich hole mir also den Wert aus der selektierten 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

"DELETE FROM fragen WHERE fragen_id=" + row;

so klappt es auch nicht

 

[Joose]

das ist meine JTable, in dieser ist ein TableModel, also fragen_id wird in der JTable angezeigt

    tblLoeschen.setModel(tblm);
    tblm = new DefaultTableModel(new MySQL().holeFragen(), new String [] {"fragen_id", "kurs", "ort", "dozent", "datum"});
       tblm.addTableModelListener( new TableAction() {

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.

ich hole mir also den Wert aus der selektierten Row

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(...)";

 

[didi577]

die Reihenfolge ist so:

tblm = new DefaultTableModel(new MySQL().holeFragen(), new String [] {"fragen_id", "kurs", "ort", "dozent", "datum"});
       tblm.addTableModelListener( new TableAction() {
           @Override
           public void tableChanged(TableModelEvent arg0) {
                               
           }
       });
       tblLoeschen = new JTable();
       tblLoeschen.setModel(tblm);

das heißt ich füge im ActionListener des Löschen Button

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?

 

[Joose]

das heißt ich füge im ActionListener des Löschen Button

tblm.getValueAt();

ein?

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.

Die Werte sind ja unbestimmt da ich ein getSelectedRow löschen möchte..
Wie kann ich das umsetzen?

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.

 

[didi577]

ich denke ich habe es 🙂

btnLoeschen.addActionListener(new LoeschenBtnAction() {
           @Override
           public void actionPerformed(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 = new DefaultTableModel(new MySQL().holeFragen(), new String [] {"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?

 

[Joose]

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)

 

[didi577]

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

 

[Joose]

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

Definitiv JA! Eine Verbindung sollte nur solange offen bleiben wie sie gebraucht wird, und das ist nun mal nur solange eine Aktion durchgeführt wird.

 

[Thallius]

Definitiv JA! Eine Verbindung sollte nur solange offen bleiben wie sie gebraucht wird, und das ist nun mal nur solange eine Aktion durchgeführt wird.

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.

Gruß

Claus

 

[didi577]

ok Danke

ich werde die goldene Mitte suchen und nur dort offen bleiben wo viele Aktionen laufen

 

[Joose]

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 😉