[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ć?

[Crozin]

Od kiedy to optymalny równa się maksymalnie krótki i nieczytelny?

[PHP] pobierz, plaintext 
if (isset($this->$key)) {
    return $this->$key;
} else if (isset($this->data[$key])) {
    return $this->data[$key];
}
 
return null;
[PHP] pobierz, plaintext 

Nie kombinuj tam gdzie nie potrzeba.

EDIT: Ogólnie to masz paskudną strukturę danych.

Ten post edytował Crozin 2.08.2010, 17:55:36

[zelu]

Od kiedy to optymalny równa się maksymalnie krótki i nieczytelny?

A to inna sprawa. Zapis typu ?: jest wolniejszy niz zwykly if - else, wiec Twoje rozwiązanie jest najlepsze (IMG:style_emoticons/default/smile.gif) Źle się zasugerowałem postem kolegi (IMG:style_emoticons/default/smile.gif)

Pozdro

[skarabe.pl]

A to inna sprawa. Zapis typu ?: jest wolniejszy niz zwykly if - else (...)

Czemu?