JDBC modular

[allievo]

Hi,

um die Gemüter gleich zu beruhigen - ich habe kein Problem mit JDBC Zugriffen :wink: den Zugriff bekomme ich hin.
Da ich allerdings kaum Programmiererfahrung habe, weiß ich nicht, wie ich in einem Progrämmchen den Zugriff auf Datenbanken so modular wie möglich gestalte. Sprich, dass es meinem Programm als solches egal ist ob ich auf eine Access, MySQL oder PostgressQL Datenbank zugreife.
Mein bisheriges Werk habe ich zur Erläuterung mal angehängt ...

public class Hauptklasse {
  public static void main(String[] args) {
    DBConnect db = new DBConnect("ODBCName","user", "password");
    db.setSQL("SELECT * FROM tabelle1;");
    db.getDaten();
    db.closeAll();
  }
}

import java.sql.*;

public class DBConnect {
    
    private final String url = "jdbc:odbc:";
    private String dbname;
    private final String driver = "sun.jdbc.odbc.JdbcOdbcDriver";
    
    private String user;
    private String pwd;    
    private String sql;
    
    private Connection con;
    private Statement stmt;
    private ResultSet result;

    public DBConnect() {
    }
    
    public DBConnect(String dbname, String user, String pwd) {
        
        try {
            Class.forName(driver);
            con = DriverManager.getConnection(url+dbname,user,pwd);
            System.out.println("Verbinde mit " + dbname + " ...");
        }
        catch(SQLException e) {
            System.out.println("Fehler beim Verbindungsaufbau!");
            System.exit(0);
        }
        catch (ClassNotFoundException e) {
            System.out.println("JDBC over ODBC Treiber nicht gefunden!");
            System.exit(0);
        }
        
        System.out.println("Verbindungsaufbau erfolgreich\n");
    }
    
    public void setSQL(String sql) {
        this.sql = sql;
    }
    
    public String getDaten() {
        
        try {
            stmt = con.createStatement();
            result = stmt.executeQuery(sql);
            while (result.next()) {
                ...
                ...
                ...
            }
        }
        catch(SQLException e) {
            System.out.println("Fehler bei der Ausgabe!" + e);
            System.exit(0);
        }
        return "";
    }
        
    public void closeAll() {
        try {
            result.close();
            stmt.close();
            con.close();
            System.out.println("\n\nResultSet und Statement wurden erfolgreich geschlossen!");
        }
        catch(SQLException e) {
            System.out.println("Fehler im close()-Argument!" + e);
            System.exit(0);
        }
    }
}

Würde jetzt gerne mal wissen wie Ihr das angestellt hättet. Mir bleibt es ja grundsätzlich selbst überlassen ob ich bspw. eine getDaten() in der gleichen Klasse beherberge, doch im Hinblick auf die Modularität wäre es doch sinnvoller die ganzen getterMethoden (falls es jetzt mehrere davon gäbe) in eine andere Klasse zu verschieben? Vielleicht sogar die DBConnect als abstrakt definieren und die extra Klasse als Unterklasse der abstrakten DBConnect definieren?
Oder gar einen ganzen anderen Ansatz, dass man die getterMethoden direkt in die Klassen einfügt, aus denen sie benötigt werden???
Also Ihr seht schon - kann mich unter tausenden von Variatonen nicht entscheiden. Vielleicht könnt Ihr mir auf die Sprünge helfen und mir verraten was in der Praxis häufig Anwendung findet.

Grüße
Allievo

 

[foobar]

Du mußt den Namen des Treibers auch noch mit an die Klasse DbConnect übergeben. Ausserdem sollte die Klasse ein Interface implementieren, daß eine Methode

public Connection getConnection();

beinhaltet. Dann hast du den Datenbankzugriff besser gekapselt, denn alles was die anderen Klassen Interessiert ist die Connection. Wo die Connection herkommt und auf welche Db diese ZUgreift spielt keine Rolle.

Ansonsten guck dir mal die Pattern DAO und Factory an.

 

[Bleiglanz]

mein senf dazu

import java.sql.*;

public class DBConnect {
    
    private final String url = "jdbc:odbc:";
    private String dbname;
    private final String driver = "sun.jdbc.odbc.JdbcOdbcDriver";  
    
    private String user; // sollte final sein
    private String pwd; // sollte final sein   
    private String sql;
    
    private Connection con;  // sollte final sein  
    private Statement stmt;  // 
    private ResultSet result;  // 

    public DBConnect() {
    }
    
    public DBConnect(String dbname, String user, String pwd) {
        
        try {
            Class.forName(driver);
            con = DriverManager.getConnection(url+dbname,user,pwd);
            System.out.println("Verbinde mit " + dbname + " ...");
        }
        catch(SQLException e) {
            System.out.println("Fehler beim Verbindungsaufbau!"+e);
            System.exit(0); // TOTALER MIST, VERZICHTE AUF SYSTEM.EXIT!
        }
        catch (ClassNotFoundException e) {  // FALSCHE REIHENFOLGE, KOMMT DIE NICHT VOR DER SQLException?
            System.out.println("JDBC over ODBC Treiber nicht gefunden!"+e);
            System.exit(0); // TOTALER MIST, VERZICHTE AUF SYSTEM.EXIT!
        }
        // SEHR SCHLECHT GEMACHT! WAS SOLL DIE AUSGABE AUF DIE KONSOLE?
        // WENN DIE NICHT "SICHTBAR" IST, WAS DANN - DER KONSTRUKTOR
        // KEHRT NORMAL ZURÜCK OBWOHL con=null usw.
        System.out.println("Verbindungsaufbau erfolgreich\n");
    }
    
    public void setSQL(String newSql) {
        this.sql = newSql;
    }
    
    public String getDaten() {
        
        try {
            stmt = con.createStatement();
            result = stmt.executeQuery(sql);
            while (result.next()) {
                ...
                ...
                ...
            }
        }
        catch(SQLException e) {
            System.out.println("Fehler bei der Ausgabe!" + e);
            System.exit(0); // TOTALER MIST, VERZICHTE AUF SYSTEM.EXIT!
        }
        return ""; // hä? Was soll da zurückkommen??
        // wie soll das überhaupt realisiert werden? Man übergibt einen
        // sql String ung bekommt einen Zurück? Welchen????
    }
        
    public void closeAll() {
        try {
            result.close(); // ACHTUNG: Wenn hier ein Fehler, wird con.close() nicht mehr aufgerufen!!!!
            stmt.close();
            con.close();
            System.out.println("\n\nResultSet und Statement wurden erfolgreich geschlossen!");
        }
        catch(SQLException e) {
            System.out.println("Fehler im close()-Argument!" + e); // was ist ein "close()-Argument"
            System.exit(0); // TOTALER MIST, VERZICHTE AUF SYSTEM.EXIT!
        }
    }
}

[/quote]
toll gemacht, dein Programm "verschluckt" wirklich alle Ausnahmen!

warum willst du dein Programm gleich beenden, wenn eine ex auftaucht? Kopfschüttel

wie soll getDaten denn funktionieren (Ergebnis eines SQL-Befehls als String)? Kopfschüttel

generell zwingst du den Client, closeAll aufzurufen (was, wenn er das vergisst?) - finde ich
nicht so gut (was passiert wenn er danach nochmal getDaten aufruft?)

BTW: warum gibts this.sql als private member - könnten doch genausogut
als Paramter für getDaten herhalten?