On Aug 10, 2012, at 11:45 AM, Tedd Sperling <t...@sperling.com> wrote:

> On Aug 9, 2012, at 5:16 PM, Jim Lucas <li...@cmsws.com> wrote:
>> You are relying on PHP's loose typing.  This is a poor check.
>> 
>> session_id() returns a string, not boolean.
>> 
>> You should do this instead.
>> 
>> if ( session_id() === '' )
>> ....
>> 
>> 
>> -- 
>> Jim Lucas
> 
> Thanks Jim -- you're right.
> 
> What about?
> 
> if (!defined(SID))
>       {
>       session_start();
>       }

Before you answer, the "(!defined(SID))" is over 50 times slower than "( 
session_id() === '' )"

Your way is better.

Cheers,

tedd

_____________________
t...@sperling.com
http://sperling.com
--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to