Singleton Database tilgang

Hmmm... er ikke meget for at have "Access" og "database" i samme sætning :-)

Fra en anden klasse skal I blot benytte: DBadgang.getDBadgang()

Det er det vi gør, men vi kan ikke finde ud af at kalde "databasen".
Dvs sql-statements.

Det er eksempelvis fra en klasse som denne vi vil lave kaldende fra.

public class VareKatalog
{
    private DBadgang database;
    public VareKatalog()
    {
        database.getDBadgang();
    }
   
    public int getVareNr(int Nr)
    {
        // Her skal vi lave sql-sætningen...
    }
   
    public String getBeskrivelse(int Nr)
    {
       
    }
}

I mangler nogle metoder i DBadgang til at håndtere jeres SQL kald (I har jo en connection at gå ud fra).

Jeg ville nok gemme Connection objektet fremfor Statement objektet.

Prøv denne her:

import java.sql.*;

public class DatabaseConnection {
   
    // variabler
    private String username = "";  // brugerid
    private String password = "";  // kodeord for brugeren
    private Connection con;
   
   
    // *************************************************************
    //  konstruktør - skabe adgang til databasen
    // *************************************************************
    public DatabaseConnection(String input) {
        try {
            String url = "jdbc:odbc:" + input;                          // forbindel gennem denne.
            Class.forName( "sun.jdbc.odbc.JdbcOdbcDriver" );
            con = DriverManager.getConnection(url, username, password); // Log ind i database
            con.setAutoCommit(true);                                    // autogem - funktion
        }
        catch (Exception e) {
            System.out.println("Kunne ikke initialisere JDBC connection. " + e);
        }
   
    }// end DBconnection
   
    // *************************************************************
    //  funktion til at lukke databasen!
    // *************************************************************
    public void closeDB() {
        try {
            con.close();
        }
        catch (SQLException e) {
            System.out.println("Kunne ikke lukke connection. " + e);
        }
               
    }// end closeDB   
   
    // *************************************************************
    //  funktion at lave insert, update og delete i database.
    //    Den bruges fordi der ikke kommer et output ud fra DB igen.
    //
    // *************************************************************
    public int exeUpdate(String sqlstring) {
        int tal = 0;
                try {
            Statement stmt = con.createStatement(); // byg et nyt statement
            tal = stmt.executeUpdate(sqlstring);    // eksekvere update i databasen
            con.commit();                          // sikre commit
        }
        catch (SQLException e) {
            System.out.println("Kunne ikke opdatere i DB. " + e);
        }
                return tal;
               
    }// end exeUpdate
   
    // *************************************************************
    //  Funktion at lave select statements i database.
    //    Der kommer et resultset tilbage fra funktionen.
    //
    // *************************************************************
    public ResultSet exeQuery(String sqlstring) {
        ResultSet rs = null;
        try {
            Statement stmt = con.createStatement(); // byg et nyt statement
            rs = stmt.executeQuery(sqlstring);      // flytter data fra tabel over i resultset   
            con.commit();                          // sikre commit
        }
        catch (SQLException e) {
            System.out.println("Kunne ikke udtage data fra DB. " + e);
        }
        return rs;
               
    } // end exeQuery
       
}// class DatabaseConnection

I kunne måske nøjes med at lave en "getConnection" metode.



Det er eksempelvis fra en klasse som denne vi vil lave kaldende fra.

public class VareKatalog
{
    private DBadgang database;
    public VareKatalog()
    {
        database = DBadgang.getDBadgang();
    }
   
    public int getVareNr(int Nr)
    {
        Connection conn = database.getConnection();
        // Her skal vi lave sql-sætningen...     
    }
   
    public String getBeskrivelse(int Nr)
    {
       
    }
}

Vi har fået det til at virke med inspiration fra jer begge - tak for det.
Vi bibeholder vores Singleton og laver så nogle get-metoder dertil.

Overvej *meget* grundigt hvordan jeres klasse vil virke
i et multi-threaded environment.

En connection pool er en oplagt singleton.

Men hverken connection eller statement er særligt
egnede som singletons.

Du skriver at en connection pool er en oplagt singleton.
Hvordan fungerer den?
Har du evt. et eksempel?

En singleton i et multi-threaded environment vil have multiple
threads som samtidigt bruger det samme objekt.

Med Connection og Statement så kan det gå grueligt galt.

Med en connection pool som har N connections som threads kan
acquire og release igen, så er singleton lige det man skal bruge.

Eller er det en connection pool implementation I leder efter ?

Jeg må indrømme, at jeg ikke er så meget inde i brugen at patterns og de klasser der hører til. Så du bliver nok nødt til at uddybe dig lidt nærmere.
Det skal måske lige siges, at det er et simpelt system med kun én bruger.
Jeg ved ikke om det gør nogen forskel mht. connection pool???

Hvis der kun er en bruger og database kun skal bruges i en tråd, så
har I næppe brug for en connection pool, og I får ikke problemer
med jeres klasse.

Fint. Det andet med flere tråde er vist et niveau for højt endnu.
Men tak fordi du gjorde os opmærksom på det...