Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [riena-dev] Time for ITreeRidget.bindToModel() refactoring?

Hi Elias,

great Idea!! I like it...

Greetings, Flo


2010/3/29 Elias Volanakis <elias@xxxxxxxxxxxxxxxxx>
"If you methods have more than 7 parameters, you probably forgot a few
already" (anonymous)

I'm not sure if we should change this, but I'm a bit scared by the
growing bindToModel(...) methods in ITreeRidget. We now have 5
variants. The latest addition has 10 parameters.

Maybe it's time to introduce bindToModel(TreeRidgetBinder
helperObject) and potentially remove some of the more recent
bindToModel(...) variants.

Having a helperObject also means we can add additional parameters
without having to add bindToModel(...) methods (=API breakage).

Food for discussion,
Elias.

--
Elias Volanakis | Technical Lead | http://eclipsesource.com
elias@xxxxxxxxxxxxxxxxx | +1 503 929 5537 | @evolanakis
_______________________________________________
riena-dev mailing list
riena-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/riena-dev


Back to the top