You might want to "memoize" this to make performance better (as you did in the second course).  As it is, every time you refer to the current_user method you are doing database I/O.  
The memoization takes the format "@user ||= User.find..."