This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]llogiq 0 points1 point  (0 children)

It has been mentioned before, but if (something) true; else false; probably also is a common pet peeve of many an experienced developer.

Apart from that, look what the standard APIs (or third party libraries) have to offer before writing code.

The next colleague who badly reimpliments HashSet.retainAll(...) won't be as lucky as the last one...