Just to add to the other responses, because the else
is usually used, I would consider it good practice to at least have a comment explaining that it is deliberately omitted and why. I've commonly seen the else
included but with no statements, just to show to future maintainers that it is not an error but by design. It may also be worthwhile to add a logging call in the else
so it won't appear to your user, but it might provide useful information to you on what real-world data looks like. You may think now that you know all possible input cases, but the world has a way of changing things without telling you. At least my world does!
I'd like to be able to assign to an luser