[php] optymalny zapis Opcje

[S3L41N]

Witam, poszukuję optymalnego sposobu na zapis poniższej funkcji (metoda w klasie, której nie wklejam):

[PHP] pobierz, plaintext 
public function __get($key)
{
	if (isset($this->$key))
		$value = $this->$key;
	elseif (isset($this->data[$key]))
		$value = $this->data[$key];
	else
		$value = NULL;
	return $value;
}
[PHP] pobierz, plaintext 

Gdyby w isset() można było dokonać przypisania, to całość sprowadzałaby się do:

[PHP] pobierz, plaintext 
public function __get($key)
{
	return (isset($value = $this->$key) OR isset($value = $this->data[$key])) ? $value : NULL;
}
[PHP] pobierz, plaintext 

Niestety się nie da. Ma ktoś może jakiś pomysł jak to zoptymalizować?

[S3L41N]

No dobrze, ale są jakieś konkretne wytyczne kiedy używać jednego zapisu, a kiedy drugiego?
Których z zapisów jest bardziej poprawny, czytelny i zalecany w następujących sytuacjach?
Sytuacja 1:

[PHP] pobierz, plaintext 
//1.1
$syshelpers = file_exists($syspath = trim(FRAMEWORKSDIR.'/'.HELPERSDIR, '/')) ? array_slice(scandir($syspath), 2) : array();
//1.2
if (file_exists($syspath = trim(FRAMEWORKSDIR.'/'.HELPERSDIR, '/')))
     $syshelpers = array_slice(scandir($syspath), 2);
else
     $syshelpers = array();
//1.3
$syshelpers = array();
if (file_exists($syspath = trim(FRAMEWORKSDIR.'/'.HELPERSDIR, '/')))
     $syshelpers = array_slice(scandir($syspath), 2);
[PHP] pobierz, plaintext 

Sytuacja 2:

[PHP] pobierz, plaintext 
//2.1
$type = (substr($type, -1) == 'y') ? substr($type, 0, -1).'ies' : $type.'s';
//2.2
if (substr($type, -1) == 'y')
     $type = substr($type, 0, -1).'ies';
else
     $type = $type.'s';
[PHP] pobierz, plaintext 

Sytuacja 3:

[PHP] pobierz, plaintext 
//3.1
return (isset($this->$key) OR isset($this->data[$key])) ? TRUE : FALSE;
//3.2
if (isset($this->$key) OR isset($this->data[$key]))
     return TRUE;
return FALSE;
//3.3
if (isset($this->$key) OR isset($this->data[$key]))
     return TRUE;
else
     return FALSE;
[PHP] pobierz, plaintext 

Sytuacja 4:

[PHP] pobierz, plaintext 
//4.1
return (isset($this->$key) ? $this->$key : (isset($this->data[$key]) ? $this->data[$key] : NULL));
//4.2
return (isset($this->$key) ? $this->$key :
           (isset($this->data[$key]) ? $this->data[$key] :
           NULL));
//4.3
if (isset($this->$key))
     return $this->$key;
elseif (isset($this->data[$key])
     return $this->data[$key];
else
     return NULL;
//4.4
if (isset($this->$key))
     return $this->$key;
else if (isset($this->data[$key])
     return $this->data[$key];
else
     return NULL;
//4.5
if (isset($this->$key))
     return $this->$key;
if (isset($this->data[$key])
     return $this->data[$key];
return NULL;
[PHP] pobierz, plaintext 

Sytuacja 5:

[PHP] pobierz, plaintext 
//5.1
return isset(self::$instance) ? self::$instance : self::$instance = new self;
//5.2
if (isset(self::$instance))
     return self::$instance;
else
     return self::$instance = new self;
//5.3
if (isset(self::$instance))
     return self::$instance;
return self::$instance = new self;
//5.4
if (!isset(self::$instance))
     self::$instance = new self;
return self::$instance;
[PHP] pobierz, plaintext 

Które z nich, a może jakieś inne są najbardziej poprawne?