terryzhu/addressbook — Issue #5: everything related with production code
Author: honnix
Created: 2012-03-07 07:53:29+00:00
State: open
Labels: enhancement

* AddressItem, why fileds are all `protected`?

* you need to invoke `super()` in constructor

* LOGGER should be `private`

* you don't need to do `String ret = ""` because in any path, it will be assigned with a string, so just `String ret = null`

* do not `e.printStackTrace` directly, log it