Details
-
Improvement
-
Status: Closed
-
Minor
-
Resolution: Implemented
-
Trunk
-
None
Description
- Diamond Operators (removing unnecessary parameters)
- unnecessary else (removing unnecessary else-phrase, e.g. when the if-phrase always ends in a return)
- multi catches (using multi catches instead of multiple single ones, if possible)
- enhance for-loops:
for (int zahl : zahlen) { System.out.print(zahl + " "); } - remove exceptions which aren't thrown (some methods claim to throw exceptions, which aren't actually thrown, they will be removed)
- remove old commented unnecessary code
- put {} around every if and else (this is to make the code more even, because some coders use it and some don't)
there will be separate patches for each class of the package for easier reviewing and patching purposes
Attachments
1.
|
[Refactoring] Package org.apache.ofbiz.product.category |
|
Patch Available | Michael Brohl |