In class OfflineImageReconstructor, it uses many SectionProcessors to process xml files and load the subtree of the XML into a Node structure. But there are lots of places that node removes key by directively writing value in methods rather than define them first. Like this:
We could improve this to define them in Node and them invoked like this way:
And it will be good to manager node key's name in the future.