JTable Zeile wird nicht in MySQL gelöscht

didi577

Bekanntes Mitglied
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.

Java:
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

Top Contributor
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...
 

didi577

Bekanntes Mitglied
Java:
"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

Aktives Mitglied
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
Code:
"DELETE FROM fragen WHERE fragen_id=" + row;
 

Joose

Top Contributor
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

Bekanntes Mitglied
das ist meine JTable, in dieser ist ein TableModel, also fragen_id wird in der JTable angezeigt
upload_2016-12-19_12-6-21.png

Java:
    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
Code:
"DELETE FROM fragen WHERE fragen_id=" + row;
so klappt es auch nicht
 

Joose

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

Java:
    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

Bekanntes Mitglied
die Reihenfolge ist so:
Java:
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
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?
 

Joose

Top Contributor
das heißt ich füge im ActionListener des Löschen Button
Java:
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

Bekanntes Mitglied
ich denke ich habe es :)
Java:
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

Top Contributor
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

Bekanntes Mitglied
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

Top Contributor
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

Top Contributor
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
 

Joose

Top Contributor
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 ;)
 
Ähnliche Java Themen
  Titel Forum Antworten Datum
D Falsche Zeile wird in JTable gelöscht Java Basics - Anfänger-Themen 6
G jtable listener wenn zeile markiert wird Java Basics - Anfänger-Themen 6
G JTable, Zeile auswählen und Ergebnis an Schaltfläche übergeben Java Basics - Anfänger-Themen 4
Crazynet jTable erste Zeile mit deffinierten Werten Java Basics - Anfänger-Themen 0
S Zeile entfernen aus JTable Java Basics - Anfänger-Themen 15
S In JTable Zeile selektieren mit Mausklick Java Basics - Anfänger-Themen 16
F Zeile bei JTable hinzufügen Java Basics - Anfänger-Themen 6
S Farbe eine Zeile in JTable ändern, wenn JButton geklickt wurd Java Basics - Anfänger-Themen 4
G JTable: Inhalt einer selektierten Zeile speichern Java Basics - Anfänger-Themen 2
L Neue Zeile JTable Java Basics - Anfänger-Themen 13
E Nächste freie Zeile in jTable ansprechen Java Basics - Anfänger-Themen 5
G JTable aus Hibernate Zeile löschen Java Basics - Anfänger-Themen 4
T Text einer JTable-Zeile farblich ändern Java Basics - Anfänger-Themen 16
G jtable, zeile automatisch hinzufügen Java Basics - Anfänger-Themen 3
S JTable: Wie kann ich die Werte aus der selektierten Zeile au Java Basics - Anfänger-Themen 11
richis-fragen JTable den angezeigten WERT nicht den Wert aus dem Model ausgeben. Java Basics - Anfänger-Themen 3
richis-fragen JTable Header ausgeblendete (width = 0) nicht per mouseDragged aufziehen. Java Basics - Anfänger-Themen 9
richis-fragen JTable sowohl Spaltennamen wie auch Spaltenbeschriftungen Java Basics - Anfänger-Themen 7
richis-fragen JTable effektiv angezeigter Text ausgeben nicht Inhalt vom Model Java Basics - Anfänger-Themen 9
G JTable Listselectionlistener friert das Programm ein Java Basics - Anfänger-Themen 8
Mady Daten von JList & Combobox in JTable adden Java Basics - Anfänger-Themen 2
W Liste mit Listen in JTable darstellen Java Basics - Anfänger-Themen 1
J Zelleninhalt einer Jtable löschen Java Basics - Anfänger-Themen 2
E JTable einzelne Zelle färben Java Basics - Anfänger-Themen 2
thobren jtable arraylist Java Basics - Anfänger-Themen 12
thobren JTable Icon Java Basics - Anfänger-Themen 1
R Compiler-Fehler JTable mit XML befüllen | The constructor JTable(Object[], String[]) is undefined Java Basics - Anfänger-Themen 10
H Kapselung JProgressBar in JTable, aber getValueAt() greift nicht Java Basics - Anfänger-Themen 7
J Jtable Eingabe nach Klick ausserhalb der Tabelle übernehmen Java Basics - Anfänger-Themen 6
J JTable Spalteninhalt löschen Java Basics - Anfänger-Themen 1
J JTable Titel wird nicht angezeigt Java Basics - Anfänger-Themen 6
B jTable Spalte summieren Java Basics - Anfänger-Themen 7
N JTable auslesen Java Basics - Anfänger-Themen 6
O JTable in Excel mit Farben Java Basics - Anfänger-Themen 8
O Kommentar auf JTable Zelle Java Basics - Anfänger-Themen 2
M Jtable Reenderer Java Basics - Anfänger-Themen 0
O Popoup Menü im JTable richtig anzeigen Java Basics - Anfänger-Themen 6
M Jtable änderung updaten Java Basics - Anfänger-Themen 2
O JTable Suchfunktion Java Basics - Anfänger-Themen 2
M jTable bekommt null Java Basics - Anfänger-Themen 1
M JTable an andere Klasse übergeben Java Basics - Anfänger-Themen 2
M Datenbank in jTable Laden Java Basics - Anfänger-Themen 49
M Klasse in JTable einfügen Java Basics - Anfänger-Themen 7
S JTable Java Basics - Anfänger-Themen 16
S JTable mit Daten füllen Java Basics - Anfänger-Themen 7
L JTable Tagebuch Spaltenhöhe verändern Java Basics - Anfänger-Themen 3
S JTable - Filter an anderen Colums Java Basics - Anfänger-Themen 2
R JTable Suchfunktion mit SQL Daten Java Basics - Anfänger-Themen 2
C JTable update mit MySQL Datenbank Java Basics - Anfänger-Themen 1
C Best Practice JTable in MVC Pattern Java Basics - Anfänger-Themen 7
J Daten einer Textdatei in ein JTable importieren. Java Basics - Anfänger-Themen 3
A zykl. Aktualisierne JTable Java Basics - Anfänger-Themen 9
C Endlosschleife bei füllen von Daten im JTable Java Basics - Anfänger-Themen 5
C Werte aus JTable auslesen Java Basics - Anfänger-Themen 4
A Probleme beim zykl. aktulisieren von Daten in JTable Java Basics - Anfänger-Themen 3
V JTable welcher Listener ? Java Basics - Anfänger-Themen 7
D MySQL Abfrage in JTable speichern Java Basics - Anfänger-Themen 43
D JTable Zeilen löschen Java Basics - Anfänger-Themen 5
C Klassen JTable wird ohne Header aufgebaut Java Basics - Anfänger-Themen 6
K (JTable) Text einer Zelle auf der linken Seite kürzel Java Basics - Anfänger-Themen 2
B Kniffel JTable Java Basics - Anfänger-Themen 5
N JTable flackert Java Basics - Anfänger-Themen 8
T JTable Daten aus txt datei Java Basics - Anfänger-Themen 3
J ArrayList wird in JTable falsch angezeigt Java Basics - Anfänger-Themen 0
J Eintragen von Personen in JTable Java Basics - Anfänger-Themen 4
X JTable mit grünen und roten Punkten Java Basics - Anfänger-Themen 2
LexeB4F DEL --> JTable Zelleninhalt Java Basics - Anfänger-Themen 3
R JTable Auslesen Java Basics - Anfänger-Themen 1
K Collections Sortieren nach zweiter Spalte in JTable Java Basics - Anfänger-Themen 18
J JTable Wert gleich überschreiben Java Basics - Anfänger-Themen 6
S JTable clonen Java Basics - Anfänger-Themen 5
H Best Practice PDF JTable Java Basics - Anfänger-Themen 4
D JTable Probleme beim Sortieren von Zahlen. Java Basics - Anfänger-Themen 6
M JTable mit XML datei befüllen Java Basics - Anfänger-Themen 1
K JTable Bild einfügen Java Basics - Anfänger-Themen 1
M [JTable] getValue throws ArrayOutOfBoundException Java Basics - Anfänger-Themen 1
B JTable - Highlighter ??? Java Basics - Anfänger-Themen 3
S JTable LinkedList <Objekt> befüllen Java Basics - Anfänger-Themen 1
S JTable dynamisch mit Datenbankinhalten füllen Java Basics - Anfänger-Themen 6
W JTable mit einem JButton-Array füllen Java Basics - Anfänger-Themen 4
O JScrollPane zu gross für JTable Java Basics - Anfänger-Themen 2
L JTable Row selected -> fireTableDataChange do nothing. Java Basics - Anfänger-Themen 3
E JTable + TableModel updaten? Java Basics - Anfänger-Themen 1
O java.lang.IndexOutOfBoundsException JTable autoSort Java Basics - Anfänger-Themen 5
F JTable adding Row Java Basics - Anfänger-Themen 5
P jTable getColumnClass, mit unterschiedlichen Klassen in einer Column? Java Basics - Anfänger-Themen 5
M Eingabe in JTable bei Eingabe korrigieren Java Basics - Anfänger-Themen 2
Z jtable problem (das tausendste??) Java Basics - Anfänger-Themen 12
J JTable Java Basics - Anfänger-Themen 7
T JTable Java Basics - Anfänger-Themen 2
T JTable einzelne Zeilen löschen Java Basics - Anfänger-Themen 3
Uzi21 jTable / Inhalt speichern Java Basics - Anfänger-Themen 2
M Problem mit JTable und Model Java Basics - Anfänger-Themen 3
F Methoden JTable + 2 For-Schleifen Java Basics - Anfänger-Themen 4
C jtextfield und jtable Java Basics - Anfänger-Themen 34
X JTable mit Inhalten aus JTextField o.ä. füllen Java Basics - Anfänger-Themen 4
G JTable: SelectionListener Problem Java Basics - Anfänger-Themen 2
G JTable: Werte in Tabelle direkt ansprechen Java Basics - Anfänger-Themen 3
S Icons in JTable per ResultSet Java Basics - Anfänger-Themen 5
G Spalte in JTable unsichtbar machen, Zugriff auf Daten ermöglichen Java Basics - Anfänger-Themen 2

Ähnliche Java Themen

Neue Themen


Oben