zysimplelife/Addressbook — Issue #7: production code related stuffs
Author: honnix
Created: 2012-03-06 09:23:06+00:00
State: open
Labels: bug, enhancement

* add javadoc to public method, class

* remove all `TODO`s if they are not really a `TODO`s

* logger should be declare as static member instead of getting it each time when using

`private static final Logger LOGGER = LoggerFactory.getLogger(Command.class)`

* think about member method/field visibility, do we need default one?

* certain fields like `cmdPackage` is not used by anybody, so please remove dead code

* why `help()` and `phraseParam` are all empty?

* method name like `Validating` is not as java convention

* do not `println` anywhere

* log exception instead of just `e.printStackTrace()`