I could see moving the find user by id or name to the user class, not sure where the other should go. Any suggestions to improve readability? I could do something like
```ruby
begin
 @current_user = User.byNameOrId(x)
rescue => ex
 log ex
end

not_found... if user.nil?
@current_user
```