sonar presentation
DESCRIPTION
dummy sonar pptTRANSCRIPT
PowerPoint Presentation
Sonar Throwable.printStackTrace(...) should never be calledUse play.Logger;
System.out and System.err should not be used as loggersAgain, use Loggers
Constant names should comply with a naming conventionUse only uppercase words separated by underscore _ and numbers (not at the beginning)
if/else/for/while/do statements should always use curly braces
if/else/for/while/do statements should always use curly braces
For this case, use ternary operator
Strings literals should be placed on the left side when checking for equalityUse someText.equals(aVar) , because in this way u dont need to validate null for aVar
Class variable fields should not have public accessibility
Use private modifier and setter and getter methodsClass variable fields should not have public accessibility
Local variable and method parameter names should comply with a naming convention
Dont use underscore on variable namesField names should comply with a naming convention
Dont use underscore _ on local variable namesUnused local variables should be removed
Just delete these!
Method parameters, caught exceptions and foreach variables should not be reassignedDont reassigned method parameters, use:Log[] otherVariable = contactsIds; and use this, because the parameter could be used after this method by his original source
Right curly brace and next "else", "catch" and "finally" keywords should be located on the same line
AVOIDUSEClass names should comply with a naming convention
http://howtodoinjava.com/2013/04/04/java-exception-handling-best-practices/