SQLite Ich muss für mein Projekt meine Datenbank Kapseln

[java2323]

Ich würde gerne die Datenbank aufrufe über eine Methode machen aber es funktioniert nicht so wie ich es gerne hätte. Der Aufruf an sich funktioniert nur es geschieht nichts in der Datenbank.
Mein Code:

public void actionPerformed(ActionEvent e) {                new Datenbank().mitarbeiterSpeichern();
//                try {
//                    String query="Update MitarbeiterInfo set "
//                            + "Vorname='"+textField.getText()+"' ,"
//                            + "Nachname='"+textField_1.getText()+"' ,"
//                            + "Geburtsdatum='"+textField_2.getText()+"' ,"
//                            + "Wohnadresse='"+textField_3.getText()+"' ,"
//                            + "Postleitzahl='"+textField_4.getText()+"',"
//                            + "Eintrittsdatum='"+textField_5.getText()+"',"
//                            + "Handynummer='"+textField_6.getText()+"',"
//                            + "Email='"+textField_7.getText()+"',"
//                            + "ID='"+fieldID.getText()+"' "
//                            + "where ID='"+fieldID.getText()+"' ";
//
//                    PreparedStatement pst=connection.prepareStatement(query);
//
//
//                    pst.execute();
//
//                    JOptionPane.showMessageDialog(null, "Mitarbeiter aktualisiert");
//
//                    pst.close();
//                } catch (Exception b) {
//                    b.printStackTrace();
//                }
                refreshTable();
            }
        });

Datenbank Klasse:

import java.sql.*;

import javax.swing.*;
public class Datenbank {
    Connection conn=null;
    public static Connection dbConnector(){
        try{
            Class.forName("org.sqlite.JDBC");
            Connection conn=DriverManager.getConnection("jdbc:sqlite:C:\\Users\\Audi\\Documents\\workspace\\MitarbeiterDatenbank.sqlite");
            
            return conn;
        }catch(Exception e){
            JOptionPane.showMessageDialog(null, e);
        }
        return null;
    }
    Connection connection=null;
    public void mitarbeiterSpeichern(){
        connection=Datenbank.dbConnector();
        GUI gui = new GUI();
        try {
            String query="Update MitarbeiterInfo set "
                    + "Vorname='"+gui.getTextField().getText()+"' ,"
                    + "Nachname='"+gui.getTextField_1().getText()+"' ,"
                    + "Geburtsdatum='"+gui.getTextField_2().getText()+"' ,"
                    + "Wohnadresse='"+gui.getTextField_3().getText()+"' ,"
                    + "Postleitzahl='"+gui.getTextField_4().getText()+"',"
                    + "Eintrittsdatum='"+gui.getTextField_5().getText()+"',"
                    + "Handynummer='"+gui.getTextField_6().getText()+"',"
                    + "Email='"+gui.getTextField_7().getText()+"',"
                    + "ID='"+gui.getFieldID().getText()+"' "
                    + "where ID='"+gui.getFieldID().getText()+"' ";

            PreparedStatement pst=connection.prepareStatement(query);
            pst.execute();
            JOptionPane.showMessageDialog(null, "Mitarbeiter aktualisiert");
            pst.close();
        } catch (Exception b) {
            b.printStackTrace();
        }
    }
}

 

[Psypsy]

Moin,

hast du es mal mit einer einfacheren Tabelle versucht wo nur eine Spalte drin ist, also nur ein Wert gesetzt werden muss?
Bist du dir sicher das du eine Verbindung zur Datenbank hast?

By
PsyPsy

 

[Joose]

Das Problem liegt darin das du in der Methode "mitarbeiterSpeichern" der Klasse "Datenbank" ein neues Objekt der Klasse "GUI" erzeugt. Von diesem Objekt fragst du dann die Werte der Textfelder ab ... diese sind aber nicht beschrieben da es das falsche Objekt ist!

Einfache Möglichkeit: Schreibe eine Klasse Mitarbeiter (oder MitarbeiterInfo), welche alle benötigten Informationen gesetzt bekommen kann. In der "actionPerformed" Methode erstellst du ein Objekt dieser Klasse und setzt alle Attribute mit den Werten der Textfelder.
Diese Mitarbeiter Objekt übergibst du dann als Parameter an die "mitarbeiterSpeichern" Methode. Und für das Query verwendest du dann die Attribute des Objektes.

Kleiner Tipp: Du verwendest zwar PreparedStatements aber baust dir dein Query jedesmal neu zusammen! Schau mal auf Google wie man ein PreparedStatement wirklich schreiben sollte