Nothings wrong with it (except the obvious things that are, but that's another discussion). You can support 2 and 3 in the same codebase.