Closed frizbog closed 8 years ago
GedcomParser is a magic class, with too much in it. A better way to deal with this would probably be a whole package of classes centered around parsing tags, and have GedcomParser orchestrate that activity, rather than doing it all itself.
Visitor pattern may apply nicely here.
I've started work on this.
This is complete in the v3-development branch.
Released in v3.0.0
GedcomParser is a magic class, with too much in it. A better way to deal with this would probably be a whole package of classes centered around parsing tags, and have GedcomParser orchestrate that activity, rather than doing it all itself.
Visitor pattern may apply nicely here.