So eine Art Singleton-Factory?

[tokitok]

Hallo,

in meiner Anwendung habe ich ein Singleton SQLServerReader (implements IReader).
Auf dieses Singleten greife ich an diversen (realtiv vielen) Stellen zu.

Jetzt erweitere ich die Anwendung und ich benötige einen OracleReader (implements IReader).
An all den Stellen, wo ich zur Zeit SQLServerReader.getInstance() stehen habe, müsste ich jetzt eine Fallunterscheidung einbauen, abhängig davon ob meine DBConnection eine SQLConnection oder eine OracleConnection ist, etwa so:

if (conn instanceof SQLConnection)
     iReader = SQLServerReader.getInstance();
else if (conn instanceof OracleConnection)
     iReader = OracleReader.getInstance();

Könnt Ihr mir Tips geben, wie ich das schöner lösen kann?

Vielen Dank!

tokitok

 

[Der Müde Joe]

vielleicht so

import java.sql.Connection;


public class ReaderFactory {

	private static final Read ORACLE = new OracleReader();
	private static final Read SQL = new SQLReader();

	public static Read get(Connection connection) {
		if (connection instanceof SQLConnection) {
			return SQL;
		} else if (connection instanceof OracleConnection) {
			return ORACLE;
		} else {
			throw new IllegalArgumentException("kein reader");
		}
	}

	private static interface Read {
	}

	static class OracleReader implements Read {
	}

	static class SQLReader implements Read {
	}
}

brauchst nicht mal mehr singletons

 

[AlexSpritze]

Vielleicht an die Connection hängen?

iReader = conn.getReader.getInstance();

Und in der Connection hast du dann diese obige if-Abfrage.

 

[DEvent]

Das ist genau wieso man keine Singletons verwenden sollte. Dependency Injection würde das Problem sehr elegant lösen. Anstelle das eine Klasse den Reader erstellt, solltest du den zu benutzenden Reader der Klasse übergeben.

Auf dieses Singleten greife ich an diversen (realtiv vielen) Stellen zu.

Für solche Fälle gibt es dann die DI Frameworks, wie guice, Spring, usw.

Also so z.B.:

public class Foo {
    public Foo(IReader reader) {
        this.reader = reader;
    }

    public void useReader() {
        reader.readData();
    }
}

Wenn du solche Späße wie instanceof verwenden musst, kann das auch ein Hinweis sein, dass dein Design kaputt ist. Normalerweise müsste es dem Reader egal sein, welche Connection es benutzt, da alles über JDBC angesprochen wird. Auch, wenn du Oracle benutzt, solltest du nicht eine Persistenz-Framework benutzen (wie z.B. Hibernate)? Scheint mir kein Hobby Projekt zu sein, wenn du solche Datenbanken verwendest.