System.out.println(someObject.toString());Ugh. 1) redundant. 2) buggy. serious codesmell (tm)
why is it buggy? if someObject is null, you have NullPointerException. if you didn't have the toString() call, println() would safely print null for you.
I'm gonna fidget with PMD's syntax and see if I can make a rule for it.
Just make *all* direct calls to toString() errors:
http://www.brunningonline.net/simon/blog/archives/000793.html
Simon Brunning [simon@brunningonline.net]