[
https://issues.apache.org/jira/browse/OFBIZ-9836?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dennis Balkir updated OFBIZ-9836:
---------------------------------
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
was:
- 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)
> General refactoring and code improvements
> -----------------------------------------
>
> Key: OFBIZ-9836
> URL: https://issues.apache.org/jira/browse/OFBIZ-9836
> Project: OFBiz
> Issue Type: Improvement
> Components: ALL COMPONENTS
> Affects Versions: Trunk
> Reporter: Dennis Balkir
> Priority: Minor
>
> - 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
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
