Question

¿Qué es necesario cambiar en la implementación del patrón singleton siguiente para realmente lograr que sólo una instancia de la clase pueda ser instanciada?

public class BD {
    static BD singleton ;
    static {
        singleton = new BD() ;
    }
    public BD() {
        //....
    }
    static BD instancia() {
        return singleton
    }
}

Comments

Como bien te dice Humberto, la definición del constructor debe ser privada para permitir sólo su invocación desde la propia clase.

Answer 1

El constructor de clase debe ser privado, de esta forma aseguras que ninguna otra clase pueda crear instancias de la clase Singleton.

Answer 2

En caso de trabajar con multi-hilo, podrías tener problemas de sincronización al tener el metodo instancia() sin sincronizar. Declarar el método instancia como

synchronyzed static BD instancia(){
/../
}

podría ser una solución, además de lo dicho anteriormente por el compañero

Answer 3

Según he visto en varios sitios, una forma muy efectiva y que suelo usar es esta:

public class BD 
{
    private static class Singleton 
    {
        private static BD instance = new BD();
    }

    public static BD getInstance() 
    {
        return Singleton.instance;
    }

    private BD()
    {
        // Código del constructor
    }
}   

No he probado a ver si hay alguna forma de "cargarse" esto en un entorno multihilo, pero yo nunca he tenido problemas y suelo trabajar mucho con Singleton en entornos multihilo.

Answer 4

Según Joshua Bloch, que de esto sabe un rato, la mejor forma es mediante un Enum.

public enum Foo {
   INSTANCE;

}

Joshua Bloch explicó esta alternativa en la charla de I/O Google 2008 his Effective Java Reloaded talk at Google I/O 2008:

The Right Way to Implement a Serializable Singleton

 public enum Elvis {
       INSTANCE;
       private final String[] favoriteSongs =
           { "Hound Dog", "Heartbreak Hotel" };
       public void printFavorites() {
           System.out.println(Arrays.toString(favoriteSongs));
       }
   }

Edit: En su libro Effective Java explica lo siguiente:

"This approach is functionally equivalent to the public field approach, except that it is more concise, provides the serialization machinery for free, and provides an ironclad guarantee against multiple instantiation, even in the face of sophisticated serialization or reflection attacks. While this approach has yet to be widely adopted, a single-element enum type is the best way to implement a singleton."