In your code, why not just have the enum do it’s own processing?

switch (parserState) {
06. case DOMAIN:
07. processDomain();

Instead, just throw away the switch state and create a process method and do this:

parserState.process()

An Enum in Java can have an abstract method definition requiring all declared Enums to implement it. If you have an “abstract void process()” method declared, each of your Enums would have to implement a method for handling that call.

So if they all have process method, then each Enum executes its own processing code. It is similar to a Command class executing whatever it’s logic is; the caller need not be aware of the Command doing the executing. It is only aware of the type.

The switch statement isn’t needed and loses a bit of the power of it.

vtd-xml