Datenbank verbindung

[max514]

Hi Leute, ich hab folgendes problem: ich möchte meinen jTable in meiner Datenbank speichern. Hier ist der Code:

try{

Class.forName("com.mysql.jdbc.Driver").newInstance();
con = DriverManager.getConnection("jdbc:mysql://localhost/meinedb","root","root123");
con.setAutoCommit(false);
  int rows=jTable.getRowCount();

  for(int row = 0; row<rows; row++)
  {  
    String auftragsdatum = (String)jTable.getValueAt(row, 0);
    String beschreibung = (String) jTable.getValueAt(row, 1);
    String auftraggeber = (String)jTable.getValueAt(row, 3);
    String query = "Insert into meinedb(auftragsdatum,beschreibung,auftraggeber) values ('"+auftragsdatum+"','"+beschreibung+"','"+auftraggeber+"')";

    pst = con.prepareStatement(query);
    pst.execute();    
  }
  JOptionPane.showMessageDialog(null, "Successfully Save");
}
catch(Exception e){
  JOptionPane.showMessageDialog(this,e.getMessage());
}

wenn ich jetzt keine daten zum jtable hinzufüge funktioniert das aber sobald ein Eintrag drinne ist sagt mir der messagedialog das der table meinedb.meinedb nicht existiert

 

[httpdigest]

Der Grund für diese Fehlermeldung ist, dass die Tabelle "meinedb" im Schema "meinedb" nicht existiert. Hast du die Tabelle denn bereits mit einem CREATE TABLE in deinem Programm oder über einen anderen SQL-Editor bzw. eine andere Datenbank-Pflegeoberfläche angelegt?

Desweiteren gibt es noch einige andere Fehler und Unschönheiten:
1. Du stellst die Connection auf autocommit=false, committest dann aber die Änderungen niemals.
2. Du verwendest zwar ein PreparedStatement, nutzt es aber nicht als solches, insbesondere was das Setzen von Parameterwerten angeht.

 

[max514]

so sieht des im moment aus bei mir ich hab halt noch nie mit datenbanken gearbeitet

 

[max514]

nein sry so

 

[httpdigest]

Na dann musst du auch die Tabelle "Aufträge" im INSERT Statement verwenden.
Hint: Umlaute zu verwenden ist immer eher schlecht.

 

[max514]

und der table den ich angelegt habe heißt aufträge

 

[max514]

ok und wie geht das kannst du mir ein kleines bsp. geben ??

 

[httpdigest]

"Insert into 'aufträge'(..."

 

[max514]

oh man =) vielen lieben dank ich glaub ohne des Forum wäre ich wahnsinnig geworden =)

 

[httpdigest]

Jetzt mal ehrlich: Wenn deine Tabelle schon "aufträge" heißt, bist du nicht selber mal auf die Idee gekommen, dass dann dieser Name wohl auch iiiiirgendwo in deinen SQL-Statements (z.B. im INSERT Statement) auftauchen muss?
Oder einfach mal nach der generellen Syntax eines SQL INSERT Statements zu googlen und dann zu sehen, dass nach INSERT INTO ja der Tabellenname kommt?

 

[M.L.]

Stichwort SQL Injection i.V. mit Java: https://de.wikipedia.org/wiki/SQL-Injection#Java_(JDBC) (s.a. Punkt 2. der 1.Antwort)

 

[Thallius]

Ich denke Prepared Statements bei einer JAVA Client App sind ziemlich überflüssig. Da ist es eh überhaupt kein Problem sich die DB Verbindungsdaten aus dem Code zu holen und mit der DB zu machen was man will. Da muss ich nicht mit irgendwelchen Injections rumbasteln...

 

[M.L.]

mit irgendwelchen Injections rumbasteln...

Bei einer privaten Sache spielt das sicher keine Rolle. Aber der Verweis auf den Artikel diente als Hinweis, zudem könnte man irgendwann für einen grösseren Personenkreis programmieren...

 

[Thallius]

Bei einer privaten Sache spielt das sicher keine Rolle. Aber der Verweis auf den Artikel diente als Hinweis, zudem könnte man irgendwann für einen grösseren Personenkreis programmieren...

Hat ja nichts mit privat oder großer Kreis zu tun. In dem Moment wo Du die Verbindungsdaten in den Code schreibst hast Du verloren...

 

[mihe7]

Ich denke Prepared Statements bei einer JAVA Client App sind ziemlich überflüssig.

Das denke ich nicht. Es geht ja nicht nur um böswillige Absichten. Mal abgesehen davon, dass PreparedStatements gecached werden können, übernimmt der JDBC-Treiber das Escaping.

 

[max514]

Ne ich hab mir ein youtube video angeschaut bei dem man nicht gesehen hat das der table genau so wie die DB heißt von dem her war ich verwirrt warum des bei dem Funktioniert und bei mir nicht. Außerdem ist des meine erste Datenbank die ich je gemacht.

 

[mihe7]

ich hab mir ein youtube video angeschaut

Das erklärt einiges 😛

 

[max514]

Nochmal ne frage 🙁 des funktionier jetzt zwar aber jedes mal wenn ich auf speichern drück überträgt es mir den ganzen jTable . gibt es eine möglichkeit den alten table in der Database zu entfernen bevor ich abspeicher und wenn ja wie ?? Ich hab zwar den Sql befehl Delete From Where ... gefunden weiß aber nicht so richtig wie ich auf den table zugreif ???

(Hier vor code ganzen alten table entfernen)

for(int row = 0; row<rows; row++)
  { 
    Integer auftragsnummer = (Integer)jTable.getValueAt(row, 2);
    String beschreibung = (String) jTable.getValueAt(row, 1);
    String auftraggeber = (String)jTable.getValueAt(row, 3);
    String auftragsdatum = (String)jTable.getValueAt(row, 0);
    String sql = "INSERT INTO db.mytable (auftragsdatum, beschreibung, auftraggeber, auftragsnummer) VALUES ('"+auftragsdatum+"','"+beschreibung+"','"+auftraggeber+"','"+auftragsnummer+"')";
    pst = con.prepareStatement(sql);
    pst.execute();   
  }
  JOptionPane.showMessageDialog(null, "Successfully Save");
}
catch(Exception e){
  JOptionPane.showMessageDialog(this,e.getMessage());
}
[/java]

 

[httpdigest]

https://www.w3schools.com/sql/sql_delete.asp

"DELETE FROM db.mytable"

 

[max514]

ok vielen dank =) vorallem das ihr so geduldig mit mir seit =)