NYCPHP Meetup

NYPHP.org

[nycphp-talk] Minor rant, pass by reference during method invocation depreciated

Jake McGraw jmcgraw1 at gmail.com
Fri Jul 2 11:13:52 EDT 2010


On Fri, Jul 2, 2010 at 10:58 AM, John Campbell <jcampbell1 at gmail.com> wrote:
>> $grantUser object updated with the new levels or not. If I know I am dealing
>> with the current user, than I want the admin authority to immediately be
>> applied so it will flow to all future function calls:
>> $return= Application::setUserRole(&$grantUser, USER_ADMIN);
>> Wheras if I am dealing with the previous user, I may very well want to keep
>> the original user to see if any changes where made:
>> $return = Application::setUserRole($grantUser, USER_ADMIN);
>
> Objects are always passed by reference.  You don't have a choice.
> What is the point of this rant?  Passing strings / ints by reference
> is a pointless feature of php.

The whole: MyClass::myFunc(&$var); is, IMO, bad form because you are
then leaving a vital component of functionality OUTSIDE of the method
definition. There should be one place for developers to check how a
function will operate, and that should be the function definition.

- jake

>
> -John Campbell
> _______________________________________________
> New York PHP Users Group Community Talk Mailing List
> http://lists.nyphp.org/mailman/listinfo/talk
>
> http://www.nyphp.org/Show-Participation
>



More information about the talk mailing list