plotrb icon indicating copy to clipboard operation
plotrb copied to clipboard

Turn Transform methods not handling state into a module

Open pjotrp opened this issue 10 years ago • 1 comments

The transform methods do not need to keep track of state. I would make this a module, so as to force a more functional style. All you are using is the type, which can be passed in with every method, or fetched from the surrounding class(name). If you need state, add a separate state class and include the module.

Split out methods that can be 'pure' from methods that really need 'state'.

pjotrp avatar Jul 06 '13 12:07 pjotrp

OK, you are also tracking properties. Let me rephrase what I wrote: put all methods that are not handling state into a separate module.

pjotrp avatar Jul 06 '13 13:07 pjotrp