Loggers should be "private static final" and should share a naming convention
Collection.isEmpty() should be used to test for emptiness
Added a TODO to SimpleObserver