|
|
|
|
|
by pohl
5608 days ago
|
|
The static methods need to go. That's just bad form, because of unit tests, or the ability to use the decorator pattern to enhance an EventManager (or instrument one), or to make an EventManager injectable via something like Guice. I'm not sure why you're bothered by inner classes, though. Anonymous inner classes are a very common way to write event-handling code in java, especially if the handler needs to capture some enclosing scope for later use. Edit: this project is more interesting in the context of the owner's other two projects: http://code.google.com/u/skjervold/ |
|