Singleton Pattern

[Java_IRL]

Hallo.

Vermutlich nur eine kurze Frage.

Die Definition von Singleton lautet ja : "Ein Singleton ist eine Klasse, von welcher nur eine Instanz erzeugt werden kann!"

Betrachtet man nun folgenden Code

public class MySingleton{
    private static MySingleton mySingelton;
 
    private MySingleton(){
    }
 
    public static MySingleton getInstance(){
        if (mySingelton == null)
            mySingelton = new MySingleton();
        return mySingelton;
    }
 
    public void doSomething(){
        System.out.println("foo");
    }
}

Jetzt kann ich doch in der main-Funktion folgendes tun:

public static void main(String args[]){
	
	MySingleton sing1 = MySingleton.getInstance();
	MySingleton sing2 = MySingleton.getInstance();
	
	sing1.doSomething();
	sing2.doSomething();
	
}

Somit habe ich doch 2 Instanzen der Klasse erstellt, was ja eigentlich vermieden werden soll ?!?

Wo liegt mein Fehler ?

Vielen Dank

Gruß

 

[L-ectron-X]

Die Instanz der Klasse müsste final sein.

public class MySingleton {
   private static final MySingleton INSTANCE = new MySingleton();
   
   private MySingleton() {

   }
   
   public static MySingleton getInstance() {
       return INSTANCE;
   }
}

Übrigens: FAQ! http://www.java-forum.org/de/viewtopic.php?t=6090#28141

 

[SlaterB]

zur ursprünglichen Frage ungeachtet aller Stürme der Entrüstung, die dich beim Einsatz des gefährlichen Singleton-Patterns noch treffen können:

wieso glaubst du, dass du zwei Instanzen hast?
du hast nur zwei Variablen, die auf dasselbe eine Objekt zeigen,
sowas ist nie verboten,
z.B. geht auch

MySingleton sing1 = MySingleton.getInstance();
MySingleton sing2 = sing1;

 

[FArt]

Wie SlaterB!

Die Instanz muss nicht final sein, aber muss während der static initiaisation (wie bei L-ectron-X) instanziiert werden. Will man es lazy machen (also erst beim Aufruf von getInstance), dann muss getInstance synchronisiert sein.
Will man ohne Monitor auskommen, kann man die beiden Varianten kombinieren. Das heißt dann Singleton über Holder.

 

[Guest]

Wo liegt mein Fehler ?

Referenzvariablen speichern Referenzen auf Objekte. Nicht Objekte. In Java ist es nicht möglich, Objekte in Variablen zu speichern.

 

[Java_IRL]

Danke euch !

Hab's verstanden.