Violation | Line |
---|
Perhaps 'mAssetsPerSecondIncludingCheckin' could be replaced by a local variable. | 608 |
Avoid unnecessary constructors - the compiler will generate these for you | 618 - 620 |
It is a good practice to call super() in a constructor | 618 - 620 |
Assigning an Object to null is a code smell. Consider refactoring. | 647 |
Assigning an Object to null is a code smell. Consider refactoring. | 648 |
Assigning an Object to null is a code smell. Consider refactoring. | 649 |
Assigning an Object to null is a code smell. Consider refactoring. | 650 |
Assigning an Object to null is a code smell. Consider refactoring. | 655 |
A method should have only one exit point, and that should be the last statement in the method | 730 |
This statement may have some unnecessary parentheses | 730 |
A method should have only one exit point, and that should be the last statement in the method | 757 |
This statement may have some unnecessary parentheses | 757 |
A method should have only one exit point, and that should be the last statement in the method | 771 |
This statement may have some unnecessary parentheses | 771 |
A method should have only one exit point, and that should be the last statement in the method | 782 |
This statement may have some unnecessary parentheses | 782 |
A method should have only one exit point, and that should be the last statement in the method | 806 |
This statement may have some unnecessary parentheses | 806 |
A method should have only one exit point, and that should be the last statement in the method | 814 |
This statement may have some unnecessary parentheses | 814 |
This statement may have some unnecessary parentheses | 885 |
Avoid unnecessary return statements | 899 |
A method should have only one exit point, and that should be the last statement in the method | 964 |
A method should have only one exit point, and that should be the last statement in the method | 973 |
A method should have only one exit point, and that should be the last statement in the method | 1021 |
This statement may have some unnecessary parentheses | 1021 |
A method should have only one exit point, and that should be the last statement in the method | 1030 |
This statement may have some unnecessary parentheses | 1030 |
A method should have only one exit point, and that should be the last statement in the method | 1043 |
This statement may have some unnecessary parentheses | 1043 |
A method should have only one exit point, and that should be the last statement in the method | 1072 |
This statement may have some unnecessary parentheses | 1072 |
A method should have only one exit point, and that should be the last statement in the method | 1105 |
This statement may have some unnecessary parentheses | 1105 |
A method should have only one exit point, and that should be the last statement in the method | 1127 |
This statement may have some unnecessary parentheses | 1127 |
A method should have only one exit point, and that should be the last statement in the method | 1141 |
This statement may have some unnecessary parentheses | 1141 |
A method should have only one exit point, and that should be the last statement in the method | 1167 |
This statement may have some unnecessary parentheses | 1167 |
This statement may have some unnecessary parentheses | 1171 |
This statement may have some unnecessary parentheses | 1230 |
Avoid unnecessary return statements | 1256 |
A method should have only one exit point, and that should be the last statement in the method | 1298 |
This statement may have some unnecessary parentheses | 1298 |
This statement may have some unnecessary parentheses | 1329 |
Avoid unnecessary return statements | 1361 |
Avoid empty catch blocks | 1407 - 1410 |
Avoid empty catch blocks | 1411 - 1414 |
Avoid empty catch blocks | 1426 - 1428 |
A method should have only one exit point, and that should be the last statement in the method | 1472 |
This statement may have some unnecessary parentheses | 1472 |
A method should have only one exit point, and that should be the last statement in the method | 1482 |
This statement may have some unnecessary parentheses | 1482 |
Avoid assigments in operands | 1491 - 1641 |
This statement may have some unnecessary parentheses | 1711 |
A method should have only one exit point, and that should be the last statement in the method | 1729 |
This statement may have some unnecessary parentheses | 1729 |
This statement may have some unnecessary parentheses | 1732 |
Avoid empty catch blocks | 1797 - 1799 |
Avoid empty catch blocks | 1810 - 1812 |
A method should have only one exit point, and that should be the last statement in the method | 1830 |
This statement may have some unnecessary parentheses | 1830 |
A method should have only one exit point, and that should be the last statement in the method | 1861 |
This statement may have some unnecessary parentheses | 1861 |
A method should have only one exit point, and that should be the last statement in the method | 1883 |
This statement may have some unnecessary parentheses | 1883 |
This statement may have some unnecessary parentheses | 1890 |
A method should have only one exit point, and that should be the last statement in the method | 1904 |
This statement may have some unnecessary parentheses | 1904 |
A method should have only one exit point, and that should be the last statement in the method | 1926 |
This statement may have some unnecessary parentheses | 1926 |
This statement may have some unnecessary parentheses | 1933 |
A method should have only one exit point, and that should be the last statement in the method | 1945 |
This statement may have some unnecessary parentheses | 1945 |
A method should have only one exit point, and that should be the last statement in the method | 1956 |
This statement may have some unnecessary parentheses | 1956 |
This statement may have some unnecessary parentheses | 1959 |
A method should have only one exit point, and that should be the last statement in the method | 1971 |
This statement may have some unnecessary parentheses | 1971 |
This statement may have some unnecessary parentheses | 1974 |