I would change this to `current_user.present?`, you want to know that a current user is found, not just that there is some value in the session. For instance the id in the session could be invalid and not actually refer to a User.

Also, I would create a `logged_in?` helper method, which I would use in the view code below.