Singleton-Implementation

[prolahmierer]

Hallo,

bisher haben meine Singleton Klassen immer eine private Variable

 private static X instance = new X();

und eine getInstance () Funktion:

    public static DBAdmin getInstance() {
	return instance;
    }

Wenn ich jetzt ein Interface implementiere, dass die Methode getInstance() verspricht darf die Methode nicht mehr static sein. Hat das irgendwelche Einflüsse auf irgendwas? (Multithreading, etc.) ?

 

[maki]

Wenn ich jetzt ein Interface implementiere, dass die Methode getInstance() verspricht darf die Methode nicht mehr static sein.

Ja, es ist kein S. mehr

Um Objektmethoden aufzurufen, braucht du ein Objekt, aber nach diesem Muster kannst du keines erstellen, nur die Klasse selbst.

S. und Interface beissen sich in Java, ist aber nicht weiter schlimm, S. sollten sowieso vermieden werden.

 

[prolahmierer]

Aber es hat keinen Einfluss auf die static-Variable oder? alles klaro, danke

 

[Landei]

Was du willst, geht nicht. static Methoden werden nicht wirklich vererbt, und Interfaces können deshalb keine statischen Methoden enthalten. Das ist ein wichtiger Punkt, der gegen Singletons spricht, und zu Ideen wie Dependency Injection geführt hat.

Schreibe eine "normale" Klasse gegen ein Interface, und gib sie allen anderen Klassen, die sie brauchen, mit.

interface FooProvider {
  public Foo getFoo();
}

class FooProviderImpl extends FooProvider {
  public Foo getFoo() { return new Foo(); }
}

class NeedsFoo {
  private final FooProvider fooProvider;
  public NeedsFoo(FooProvider fooProvider) {
     this.fooProvider = fooProvider
  }
}

//an einer einzigen, zentralen Stelle im Programm:
FooProvider fooProviderInstance = new FooProviderImpl(); 

NeedsFoo needsFoo = new NeedsFoo(fooProviderInstance);

Wird das zu aufwendig, schau dir das Factory-Pattern oder eben Dependency Injection (z.B. Google Guice) an.

 

[Spacerat]

Genau so muss es sein. Singletons haben eine statische private Konstante einer Instanz der eigenen Klasse und eine öffentliche statische Methode, mit welcher diese Instanz besorgt werden kann. Singletons sind von Natur aus finale Klassen und als solche zu deklarieren. Singletons haben einen privaten parameterlosen Konstruktor. Singletons dürfen Interfaces implementieren. Über die Methoden dieses Interfaces sollten aber keine anderen Instanzen dieses Interfaces erreichbar sein. Ein Musterbeispiel liefert z.B. die Implementation diverser Server-Dienste. Das Interface heisst also "Dienst". Dienste können gestartet und gestoppt werden.

public interface Dienst
{
  boolean start();
  boolean stop();
}

Konkrete Dienste könnten z.B. HTTPDaemon und FTPDaemon heissen

public final class HTTPDaemon
implements Dienst
{
  private final static HTTPDaemon instance;

  static
  {
    // So wird die Instanz beim Initialisieren der Klasse
    // statt beim ersten Aufruf von "getInstance()"
    // instanziert.
    instance = new HTTPDaemon();
  }

  private HTTPDaemon()
  {
    // ...
  }

  public static HTTPDaemon getInstance()
  {
    return instance;
  }

  public boolean start()
  {
    // ...
    return started;
  }


  public boolean stop()
  {
    // ...
    return stopped;
  }

  // ...
}

public final class FTPDaemon
implements Dienst
{
  private final static FTPDaemon instance;

  static
  {
    // So wird die Instanz beim Initialisieren der Klasse
    // statt beim ersten Aufruf von "getInstance()"
    // instanziert.
    instance = new FTPDaemon();
  }

  private FTPDaemon()
  {
    // ...
  }

  public static FTPDaemon getInstance()
  {
    return instance;
  }

  public boolean start()
  {
    // ...
    return started;
  }


  public boolean stop()
  {
    // ...
    return stopped;
  }

  // ...
}

@maki: ...gottogottogott... was hab' isch denn da angerichtet...

 

[maki]

@maki: ...gottogottogott... was hab' isch denn da angerichtet...

Hehehe... dachte schon es würde niemanden auffallen

 

[Landei]

@SpaceRat: Die "halboffizielle" Empfehlung von Sun ist inzwischen, enums für Singletons zu benutzen. Vorteile: Der Code ist kürzer, man kann nix vergessen und selbst über Serialisierung bekommt man keine "doppelten" Singletons.

enum Singleton {
   INSTANCE;
   //alle gewünschten Methoden
}

 

[Spacerat]

@Landei: Das nenn' ich mal 'ne Empfehlung... :applaus:

 

[prolahmierer]

Huhu, danke für die wertvollen Hinweise

Grüße