Тут конечно религиозные предпочтения.
public static function getInstance()
{
return self::$instance ? : self::$instance = new static;
}
С точки зрения программиста такой код привлекательней и быстрей. Однако для командной работы лучше старый вариант. Иначе к вам придет злобный программист, который искал после вас баг и вынесет мозг.