[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]

[PHP] pobierz, plaintext 
return isset( $this->key ) ? $this->key : ( isset( $this->data[$key] ) ? $this->data[$key] : NULL );
[PHP] pobierz, plaintext 

[Wicepsik]

[PHP] pobierz, plaintext 
return $this->$key ? $this->$key : ($this->data[$key] ? $this->data[$key] : null);
[PHP] pobierz, plaintext 

[zelu]

[PHP] pobierz, plaintext 
return $this->$key ? $this->$key : ($this->data[$key] ? $this->data[$key] : null);
[PHP] pobierz, plaintext 

Twoja wersja zwróci Notice'y jeżeli nie będzie któreś z wartości

Pozdro

[S3L41N]

No zgadza się, isset()-y celowo dałem. Dzięki za pomoc Panowie.

[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 Źle się zasugerowałem postem kolegi

Pozdro

[skarabe.pl]

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

Czemu?

[zelu]

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

[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

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

[croc]

A to inna sprawa. Zapis typu ?: jest wolniejszy niz zwykly if - else, wiec Twoje rozwiązanie jest najlepsze

Wątpię - a nawet jeśli, to co z tego? Tworzysz grę, która ma śmigać w 60 FPS? Panowie, nie popadajcie w paranoję. Zapis ? : jest tu jak najbardziej w porządku.

[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.

[Crozin]

Zacznijmy od tego, że jak masz taką konstrukcję to jest coś nie tak z Twoimi umiejętnościami. Ale rozumiem, że to tylko na potrzeby przykładu.

Jeszcze tak raz podkreślę: nie szukaj wydajności w takich miejscach - to, że zyskasz 0.00000213 sekundy nic Ci nie da.

[zelu]

W Twojej wersji jak najbardziej zapis skrócony jest sensowniejszy. Kwesta kompromisu pomiędzy czasem wykonania a przejrzystością kodu. Poza tym rozchodzi się o czas rzędu 3E-6 Więc strasznie dużej straty na tym nie ma. Niemniej jednak takowa występuje.

Jednak przy pierwotnym problemie w tym temacie zapis Crozina wydaje mi się najsensowniejszy. Szybszy i jednak bardziej przejrzysty niż zagnieżdżanie ?:

Pozdro

[skarabe.pl]

Jeszcze tak raz podkreślę: nie szukaj wydajności w takich miejscach - to, że zyskasz 0.00000213 sekundy nic Ci nie da.

Spoko spoko - nie ja podniosłem kwestię wydajności. 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. Ale nie zaszkodzi, jeśli to pierwsze wejdzie w krew (i szablony kodu :)).

[S3L41N]

No dobrze, ważna jest czytelność kodu, a czy poniższy zapis jest nieczytelny?

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

Jeżeli ktoś posiada konkretną wiedzę czy i jeśli tak to kiedy wskazane jest używanie skróconego zapisu za pomocą ternary operator-a będę niezmiernie wdzięczny za wytyczne.

[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... 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

[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

[Fifi209]

wystarczy ze wklepie echo

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

No i moim zdaniem jest to BARDZO duża różnica i jeżeli ktoś nie widzi tej różnicy lub o tym nie wie, znając życie przyleci na forum i będzie pytał: Dlaczego mi skrypt źle działa?