|
Here is a non exhaustive list of what we need to amend before release any stable version:
- The complexity of many methods are a way to highÂ
- We should not throw RuntimeException or Exception directly, we should create our own exception
- We miss code comment and java doc comment on most methods (whit the right number of @param attributes)
- We need to remove all the FIXME and TODO
- We need to remove the e.printStackTrace()
- We should use Objects.equals() whenever we want to check whether 2 objects are equal
- We should remove all the string concatenation and replace them with StringBuilder of String.format()
- We need to simplify our if statements
- We need to externalise the duplicate code
- We need to cover the code with unit test
|