La segunda opción está mas clara igual, “self::$instancia = new self();”

Creo que hay un poco de confusión, el & se usaba en PHP4 para pasar variables u objetos por referencia, pero en PHP5 ya todo es por referencia, por lo que el & deja de ser útil, y me parece que usar el self::instancia seria mezclarlo, más bien algo así:

$instancia =& new $class;

En la forma en que me muestras el método getInstance no funcionaría correctamente ya que no hay validaciones de que si no existe previamente el objeto cree uno nuevo y el método debería llevar un & en el nombre, quedaría algo así en PHP4 ese método:


function &getInstance ($class) {
static $instancia;
if (!isset( $instancia )) {
$instancia =& new $class;
}
return $instancia;
}


Y el método en PHP5 se podría simplificar de la siguiente manera:


public static function getInstance () {
if (!isset(self::$instancia)) {
self::$instancia = new self();
}
return self::$instancia;
}


public static function getInstance () {
if (!isset(self::$instancia)) {
$obj = __CLASS__;
self::$instancia = new $obj;
}
return self::$instancia;
}

podría simplificarse en:

public static function getInstance () {
self::$instancia =& $this;
}

Vos que sos mas conocedor, fijate si tiene alguna desventaja que no estoy viendo.