Changes between Version 2 and Version 3 of Ticket #35658, comment 63
- Timestamp:
- 06/28/2016 02:28:16 AM (8 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #35658, comment 63
v2 v3 3 3 That worries me because it portends a proliferation of extra optional parameters having to be passed around making function signatures more complicated and harder to remember. 4 4 5 Our team went down that path with our Sunrise library several years ago and it was probably the biggest architectural mistake we made. What we found was that we ended up having to pass two values around as we were building plugins and it was an explosion of complexity that you can start to see in the patch. I explained it [https://bitbucket.org/newclarity/sunrise-1/overview '''here''']; scroll down to the section entitled ''"Use an "Object Type" Class "''5 Our team went down that path with our Sunrise library several years ago and it was probably the biggest architectural mistake we made. What we found was that we ended up having to pass two values around as we were building plugins and it was an explosion of complexity that you can start to see in the patch. I explained it [https://bitbucket.org/newclarity/sunrise-1/overview '''here''']; scroll down to the section entitled ''"Use an "Object Type" Class."'' 6 6 7 7 Of course it took several years painful experience to realize we had made that mistake; it really did seem like a great idea at the time.