Te contestó desde el móvil, así que disculpas si no queda del todo organizado.
Para empezar un singleton NO DEBE BORRARSE, ya que el puntero "instance" no se entera y se queda apuntando a una dirección no válida. Para evitar ese tipo de tentaciones lo mejor es devolver una referencia, ya que a nadie se le ocurre convertir una referencia en puntero para borrarla.
En segundo lugar, queda más limpio si evitas usar una variable estática a nivel de clase, es mejor declararla estática a nivel de función:
Código C++:
Ver originalclases Singleton
{
public:
static Singleton& Instance()
{
static Singleton instance;
return instance;
}
};
Además, lo suyo es declarar los constructores privados, para evitar que se hagan copias no autorizadas del objeto. No sólo el constructor por defecto, sino también el constructor copia y, por costumbre, el operador de asignación.
La excepción te salta porque getInstance es un método estático... Es decir, hay que llamarlo S=Singleton::getInstance(). Además que S es un puntero no inicializado... cualquier uso de ese puntero te dará error.
Y para terminar, Singleton se usa cuando se necesita una única instancia del objeto y que este sea global a toda la aplicación. No veo por qué querrías crear un objeto con argumentos... Y si después pides otro con argumentos diferentes?? En esos casos no necesitas un Singleton sino objetos comunes.
Un saludo