🚀 go-pugleaf

W dniu 2012-01-24 15:25, PawelS pisze:

> OK tylko, że z tym returnem to trochę pojechałeś,
> może niezbyt czytelnie wyszedł ten [...inline_ciach...],
> w założeniu funkcja/metoda ma zwrócić typ boolean,

to trzeba było napisać, że musi być boolean (a potrafisz uzasadnić 
dlaczego?) gdyż sam dokonujesz testowania bliżej nieokreślonej wartości 
(bez przyrównywania do żadnej innej), zatem (być może o tym nie wiedząc 
;)) posiłkujesz się ekwiwalentami logicznymi innych typów.

> w zależności od wartości jaka się znajdzie w tablicy,
> a tutaj w tablicy USERPERMISSIONS[string_key]
> mogą być zapisane różne wartości,
> także Twój przykład kodu równie dobrze można zapisać:
> return rand(); :P

coś bzdury gadasz, gdyż poprzednio sam wkleiłeś:
if ($this->USERPERMISSIONS[...inline_ciach...])
co w zależności od spełnienia warunku zwraca potem true lub false.
Zatem true zwrócisz gdy if potraktuje 
$this->USERPERMISSIONS[...inline_ciach...] jako prawdę, i analogicznie 
dla fałszu.
Jakim cudem return ma zwracać random, a if nie?

To ode mnie, też gratis, dostaniesz:

return!!$this->USERPERMISSIONS[...inline_ciach...];

ok, masz "po ludzku":
return(bool)$this->USERPERMISSIONS[...inline_ciach...];

To teraz możesz szukać takiego $bzdet dla którego:

return (bool)$bzdet;

to coś innego niż:

$bzdet?true:false;

albo:

if($bdet){
	return true;
}else{
	return false;
};

Ja wiem, że kod ma być przejrzysty, ale przejrzyste to jest pytanie "czy 
pada deszcz" które zwraca "TAK" lub "NIE".
Nie jest bardziej przejrzyste: "jeżeli pada deszcz odpowiedz TAK w 
przeciwnym przypadku odpowiedz NIE".

Nie martw się, bo widziałem już: "jeżeli odpowiedź na pytanie (czy pada 
deszcz) brzmi TAK to odpowiedz TAK w przeciwnym wypadku odpowiedz NIE".
Czyli coś w stylu:

if($deszcz==true){
	return true;
}else{
	return false;
}

zamiast normalnego:

return $deszcz;
(z opcjonalnym rzutowaniem typu, gdy $deszcz nie jest boolowskie a na 
wyjściu _musi_ być boolean).

Ty plasujesz się gdzieś pośrodku ;p

Więc zapewne nie napiszesz
return $x>0;
tylko sklecisz jakiegoś if'a z dwoma returnami, choć $x>0 samo w sobie 
jest już boolowskie.

v.
PS. po raz kolejny uprzejmie informuję, że to _nie_ jest 
pl.comp.lang.php, którą też czasami podczytuję ;)