Misuse of Java visibility declaration
The public, protected, and private keywords (as well as "default") visibility in Java are there for a reason. Their purpose is to make the compiler catch mistakes, implement encapsulation and data hiding, and protect sensitive code from unauthorized callers.
I am seeing more and more Java code in which methods are blindly, lazily declared as public. This is a disservice to anyone who has to read the code. If the architecture is so broken that proper encapsulation and data hiding are not possible, then the correct response is to refactor the broken code, not exacerbate it by added sloppiness.
Comments:
