allgemeine Architekturfrage

[mischer]

Hi,

ich bin gerade dabei eine Software zu entwerfen, bin aber mit meiner Architektur überhaupt noch nicht zufrieden.
Ein Problem ist folgendes:
Ich habe für die Datenbankanbindung eine eigene Klasse namens DBConn. Von dieser Klasse erzeuge ich in meiner Hauptklasse eine

public static DBConn db = new DBConn();

und greife dann später aus allen Klassen in der Art

 <Instanz der Hauptklasse>.db.machwas();

zu.
Das funktioniert zwar logischerweise, aber es ist absolut idiotisch, zumal es die Unterklassen nicht wiederverwendbar macht, da diese ja den Instanznamen der Main hart codiert haben.

Wie stellt man das richtig an?

Vielen Dank für eure Mühe!
MfG
mischer

 

[Guest]

Singleton?

DBCon db = DBCon.getInstance(); // Wo auch immer im Programm

 

[Guest]

public class DBCon ... {

  private static DBCon instance = null;
  ...

  public static DBCon getInstance() {
    synchronized(instance) {
      if(instance == null) {
        instance = new DBCon();
        Runtime.getRuntime().addShutdownHook(
          new Thread() {
            public void run() {
              instance.release(); // Verbindung zur DB abbauen etc.
            }
          }
        );
      }
    }
    return instance;
  }
...

 

[mischer]

Danke für die schnelle Antwort!

So einfach kann das manchmal sein - Singleton.

Aha addShutdownHook klingt ja interessant. Ich hätte es einfach bei Programmende mit

DBConn.getInstance().close();

gemacht, aber das werd ich mal probieren.

Danke nochmal!

MfG
mischer

 

[Guest]

Beachte aber, dass es bei Multithreading zu Problemen kommen kann,
wenn du nur eine Connection verwendest. (z.B. nur 1 ResultSets gleichzeitig offen)
Ausserdem immer alles bombensicher machen, damit ResultSets, PreparedStatements etc.
freigegeben werden. (finally-Block)
z.B.
gefährlich

try {
  PreparedStatement p = ...
  ...
  p.close(); // Wenn vor dieser Zeile eine Exception kommt, wird das Statement nicht freigegeben!!!
}
catch(SQLException e) {
  ...
}

OK

PreparedStatement p = null;
try {
  p = ...
  ...
}
catch(SQLException e) {
  ...
}
finally { // Wird auf jeden Fall ausgeführt, egal wo es knallt
  if(p != null) { 
    try { 
      p.close(); 
    } 
    catch(SQLException e) { 
      logger.error(e); // Für die Fehlerursachenforschungsabteilungeierköpfe ;)
    }
  }
}