Stała+string jako argument domyślny funkcji, ... czy to możliwe? Opcje

[kiler129]

Witajcie!
Natrafiłem na dziwne ograniczenie którego nie potrafię przeskoczyć:

[PHP] pobierz, plaintext 
<?php
define("TEST", "abc");
 
function foo($a = TEST) { print($a); } //Wynik: abc
function bar($a = "def") { print($a); } //Wynik: def
function foobar($a = TEST."def") { print($a); } //Wynik: Parse error: syntax error, unexpected '.', expecting ')'
?>
[PHP] pobierz, plaintext 

Oczywiście mogę zrobić to tak:

[PHP] pobierz, plaintext 
<?php
function foobar($a = TEST) { $a .= "def"; print($a); } //Wynik: abcdef
?>
[PHP] pobierz, plaintext 

Ale w żaden sposób nie kojarzy się to z dobrym stylem pisania i przejrzystością kodu. Czy w PHP niemożliwe jest zdefiniowanie argumentu domyślnego funkcji składającego się z 2 części?

[everth]

Manual i cytat

Domyślna wartość musi być stałym wyrażeniem, a nie na przykład zmienną, członkiem klasy czy wywołaniem funkcji

Czy to chcesz zrobić ma sens? Może ma, nie spotkałem się z czymś takim ani nie bardzo wiem czemu miałoby to służyć - wartość domyślna jest stała, nie powinna zależeć od innych warunków - jeśli zależy to należy ją przekazać. Do operacji na zmiennych mamy ciało funkcji a nie parametry.

[kiler129]

Manual i cytat

Czy to chcesz zrobić ma sens? Może ma, nie spotkałem się z czymś takim ani nie bardzo wiem czemu miałoby to służyć - wartość domyślna jest stała, nie powinna zależeć od innych warunków - jeśli zależy to należy ją przekazać. Do operacji na zmiennych mamy ciało funkcji a nie parametry.

Niestety natrafiłem na sytuację gdzie jest to przydatne. Pokazuję to niżej:

[PHP] pobierz, plaintext 
define("APATH", "/Users/Grzegorz/Sites/uCMS");
 
foo($safe_dir=APATH."/download") { ... }
[PHP] pobierz, plaintext 

Co prawda problem da się obejść tak:

[PHP] pobierz, plaintext 
define("APATH", "/Users/Grzegorz/Sites/uCMS");
define("SAFE_BROWSE_PATH", APATH."/download"); //path where user can safety view without violating security
 
foo($safe_dir=SAFE_BROWSE_PATH) { ... }
[PHP] pobierz, plaintext 

To że nie powinna od niczego zależeć i może być z całą definicją funkcji uznana jako stała to wiem ;] Tyle że połączenie nawet 2 stringów ("foo"."bar") skutkuje błędem.

Ten post edytował kiler129 29.08.2010, 03:50:09

[michaJlS]

[PHP] pobierz, plaintext 
<?php
function foobar($a = TEST) { $a .= "def"; print($a); } //Wynik: abcdef
?>
[PHP] pobierz, plaintext 

Ale w żaden sposób nie kojarzy się to z dobrym stylem pisania i przejrzystością kodu. Czy w PHP niemożliwe jest zdefiniowanie argumentu domyślnego funkcji składającego się z 2 części?

[PHP] pobierz, plaintext 
function foobar($a=null){
if(is_null($a)) $a = TEST.'def';
// ...
}
[PHP] pobierz, plaintext 

to co chcesz osiągnąć, to jeszcze "gorszy" styl pisania, bo w ogóle niepoprawny;) Ty to źle nazywasz, stąd ci się wydaje, że tak powinno się dać. To nie jest definiowanie domyślnego argumentu składającego się z dwóch części. To jest po prostu definiowanie domyślnego argumentu jako wynik pewnej operacji, a to jak sam wiesz jest zabronione.

Ten post edytował michaJlS 29.08.2010, 12:40:59

[tehaha]

Niestety natrafiłem na sytuację gdzie jest to przydatne. Pokazuję to niżej:

[PHP] pobierz, plaintext 
define("APATH", "/Users/Grzegorz/Sites/uCMS");
 
foo($safe_dir=APATH."/download") { ... }
[PHP] pobierz, plaintext 

Co prawda problem da się obejść tak:

[PHP] pobierz, plaintext 
define("APATH", "/Users/Grzegorz/Sites/uCMS");
define("SAFE_BROWSE_PATH", APATH."/download"); //path where user can safety view without violating security
 
foo($safe_dir=SAFE_BROWSE_PATH) { ... }
[PHP] pobierz, plaintext 

To że nie powinna od niczego zależeć i może być z całą definicją funkcji uznana jako stała to wiem ;] Tyle że połączenie nawet 2 stringów ("foo"."bar") skutkuje błędem.

Oba przypadki są jak dla mnie pozbawione sensu i są złą praktyką, przecież jeżeli przy przekazywaniu do funkcji będziesz składał sobie zmienną to ta funkcja jest kompletnie nieelastyczna, powinieneś to tak rozwiązać:

[PHP] pobierz, plaintext 
define("APATH", "/Users/Grzegorz/Sites/uCMS");
 
$path = APATH."/download";
foo($path);
[PHP] pobierz, plaintext 

funkcja przyjmuje jakieś parametry i wykonuje na nich operacje, jeżeli jakieś parametry mają wartość domyślną to ją ustawiasz, ale jeżeli te wartości trzeba poskładać to robisz to przed funkcją

Ten post edytował tehaha 29.08.2010, 12:59:23

[everth]

To raczej jest ograniczenie Zend Engine - PHP pozwala przecież na osadzanie stałych jako parametrów domyślnych - one nie zmieniają wartości. Logicznie rzecz biorąc złączenie dwóch stałych stringów również daje stały ciąg tekstowy. Chyba że tu chodzi np. o operacje na liczbach zmiennoprzecinkowych (gdzie wynik już nie do końca jest stały).