Since you aren't error checking your input, I'd suggest defining this method to just take an integer. If you still want it to take a user object, you should at least verify that it is one.

(Alternately, rename the methods with a leading underscore so they're internal, and then they can be more loose in what they check. My worry is that public-named methods might get used elsewhere by people who don't understand the calling conventions.)