Closed jonnycomes closed 2 years ago
Done. The Chemistry constructor now takes a split function instead of a splitting strategy. The SplittingStrategy has been removed completely. There is a new SplittingFactory which will be responsible for all the good that was lost with SplittingStrategy.
Currently the SplittingStrategy class constructor is not usable in the most common (maybe all?) situations. It would be better to have a constructor which initiates the splitting function to do no splitting, and then have some sort of method that allows for adding a splitting condition based on Left and Right pairs.