When I see a question like this I often ask myself if it would be appropiate to tell the person that his code does not match the Java Styling Conventions. I'm not talking about brace or indent style, as it compiles down to the exact same.
However, something like
- Wrong method name case
- Wrong variable / field name case
- No data encapsulation /
public
fields without getters
really bothers me as it does matter in some cases and could be bad.
Example: The method AddNewListings
should instead be addNewListings
to match Java's "lower camel case".
Is it rude, inappropiate, or bad in any way to tell the person?