Re: 1 last error to fix before the application is done!

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jochem Maas wrote:
Colin Guthrie schreef:
  if(!empty($_SESSION['userInfo']['loggedin'])) {

if (Sess::userLoggedIn()) { /* ... :-) */ }


Well, yes, that's how I do it in my apps too, but in internally in that function you may want to use the empty() call :)

I agree that when checking for values other than boolean and such you need to do more checks but quite often in practical coding, I find that using empty cuts down quite a lot on function call overhead and double statements.

Everyone's different tho' so each to their own way :)

Col

--

Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
  Mandriva Linux Contributor [http://www.mandriva.com/]
  PulseAudio Hacker [http://www.pulseaudio.org/]
  Trac Hacker [http://trac.edgewall.org/]


--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php


[Index of Archives]     [PHP Home]     [Apache Users]     [PHP on Windows]     [Kernel Newbies]     [PHP Install]     [PHP Classes]     [Pear]     [Postgresql]     [Postgresql PHP]     [PHP on Windows]     [PHP Database Programming]     [PHP SOAP]

  Powered by Linux