Very basic question, please

[airbag]

If one were to change this working code...

function verifyAcc()
{
$user = ClassUser::currentUser();
if ( !$user->isLoggedIn() )
return false;
else
return true;
}

to

function verifyAcc()
{
// $user = ClassUser::currentUser();
if ( !ClassUser::currentUser()->isLoggedIn() )
return false;
else
return true;
}

i.e. create the class instance and use it in the same statement,
should it continue to work?

[laserlight]

Yes, and in fact I might write it as:

function verifyAcc()
{
    return ClassUser::currentUser()->isLoggedIn();
}

[airbag]

@... thank you for the quick reply.

I much prefer your version to the original. I'm reading all this code and was bothered by the amount of what looked like unnecessary code and extra variables being created every where.

I do realise that sometimes it makes more sense to create the instance variable for use later, but couldn't understand why it needed to be done every time, even for a single use.
Thanks very much!