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

[zelu]

Nie wiem dlaczego tak jest (IMG:style_emoticons/default/winksmiley.jpg) Tak wynikało z moich testów (IMG:style_emoticons/default/winksmiley.jpg)

[skarabe.pl]

Nie wiem dlaczego tak jest ;) Tak wynikało z moich testów ;)

Trochę bez sensu, no ale :)

[Fifi209]

Trochę bez sensu, no ale (IMG:style_emoticons/default/smile.gif)

Dlaczego bez sensu? Czy coś co jest mniej czytelne, musi być wydajniejsze? (IMG:style_emoticons/default/smile.gif)

[skarabe.pl]

Dlaczego bez sensu? Czy coś co jest mniej czytelne, musi być wydajniejsze? :)

Nie nie nie nie - czytelność jest kwestią gustu, bez sensu jest tylko to, że zapis:

[PHP] pobierz, plaintext 
$var = condition ? true : false;
[PHP] pobierz, plaintext 

który znaczy dokładnie to samo co:

[PHP] pobierz, plaintext 
if (condition) {
  $var = true;
} else {
  $var = false;
}
[PHP] pobierz, plaintext 

może być wolniejszy od tego drugiego. Tak przy okazji - nie wyobrażam sobie zapisania skryptu typu:

[PHP] pobierz, plaintext 
$zmienna_1 = isset($_POST['zmienna_1']) ? $_POST['zmienna_1'] : '';
$zmienna_2 = isset($_POST['zmienna_2']) ? $_POST['zmienna_2'] : '';
$zmienna_3 = isset($_POST['zmienna_3']) ? $_POST['zmienna_3'] : '';
/* ... */
$zmienna_100 = isset($_POST['zmienna_100']) ? $_POST['zmienna_100'] : '';
[PHP] pobierz, plaintext 

W taki sposób:

[PHP] pobierz, plaintext 
if (isset($_POST['zmienna_1'])) {
  $zmienna_1 = $_POST['zmienna_1'];
} else {
  $zmienna_1 = '';
}
if (isset($_POST['zmienna_2'])) {
  $zmienna_2 = $_POST['zmienna_2'];
} else {
  $zmienna_2 = '';
}
if (isset($_POST['zmienna_3'])) {
  $zmienna_3 = $_POST['zmienna_3'];
} else {
  $zmienna_3 = '';
}
/* ... */
if (isset($_POST['zmienna_100'])) {
  $zmienna_100 = $_POST['zmienna_100'];
} else {
  $zmienna_100 = '';
}
[PHP] pobierz, plaintext 

nawet w tej, w sposób oczywisty mniej wydajnej wersji:

[PHP] pobierz, plaintext 
$zmienna_1 = '';
if (isset($_POST['zmienna_1'])) {
  $zmienna_1 = $_POST['zmienna_1'];
}
[PHP] pobierz, plaintext 

Dla mnie to pierwsze jest znacznie bardziej czytelne. Ale jak już napisałem - kwestia gustu.

[Fifi209]

Nie nie nie nie - czytelność jest kwestią gustu, bez sensu jest tylko to, że zapis:
Dla mnie to pierwsze jest znacznie bardziej czytelne. Ale jak już napisałem - kwestia gustu.

Jeżeli robisz to w sposób, który przedstawiłeś na forum to nie skomentuję...
Co do wydajności - wrzuć sobie w pętlę z 100 tysięcy raz, wyciągnij czasy, podaj czas średni i porównaj - może się mylimy... (IMG:style_emoticons/default/smile.gif) Ja nie idę w zapartego, że jest tak jak My piszemy i nie podważam za wszelką cenę Twoich teorii.

Ale to trochę tak jak z ++$i zamiast $i++ - nikt nikomu nie każe przeglądać wszystkich swoich skryptów i zamieniać jedno na drugie.

No i tutaj byś się zdziwił, jest różnica czy używasz ++$i czy $i++
Miłego czytania

[kapuch]

No i tutaj byś się zdziwił, jest różnica czy używasz ++$i czy $i++
Miłego czytania

wystarczy ze wklepie echo (IMG:style_emoticons/default/winksmiley.jpg)

[PHP] pobierz, plaintext 
$i = 0;
echo $i++; // 0
echo $i; // 1
echo ++$i; // 2
[PHP] pobierz, plaintext 

Ten post edytował kapuch 3.08.2010, 00:01:40