On Jul 31, 5:12 pm, 7stud -- <[email protected]> wrote: > > What do you think about this: > > > def current_user > > @current_user ||= user_from_remember_token > > end > > > I see two problems with that function. First off, that is a getter > > function, yet it can set the @current_user variable. Getter functions > > should not set instance variables. Secondly, the function directly sets > > the @current_user variable instead of calling the current_user= setter > > function. >
Memoization is an extremely common practice. I find the existence of current_user= a little odd. Fred > -- > Posted viahttp://www.ruby-forum.com/. -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en.
