@ratboy667 said:
Actually, I like this code. The conditional reads positively, and is immediately visually correct. The action case is the else, but who cares? !(cond) may be preferred by some, but -- if (list == null || list.size() == 0) { do stuff } is much more prone to error when it reaches maintainance.
No "wtf" here...
I hope I never have to maintain your code if you think that is a good idea.
If you really think (list == null || list.size() == 0) is hard to maintain, then put it in a method and call that. E.g. "if (listIsEmpty(list))...". I could handle that and it reads well. Abusing control structures is not how you write maintainable code.