search

iut-ibk / DynaMind-ToolBox

DynaMind-ToolBox
www.dance4water.org
GNU General Public License v2.0
9 stars 6 forks source link

Systems as container #224

Closed zacharias2k closed 10 years ago

zacharias2k commented 11 years ago

While building the new view system ( #223 ) i tried to cleanup old methods, which may not be necessary anymore. One of them is System::System(const System& ref) - the copy constructor.

This method clones all components and also generates a db-entry for the system. Thus i strongly advise against the useage of this method. I will try to remove this feature and come up with a better solution.

@christianurich @abroxos

christianurich commented 11 years ago

relates to https://github.com/iut-ibk/DynaMind-ToolBox/issues/98

zacharias2k commented 11 years ago

Yes, indeed. But i still think it is possible and advisable to use them without copying them. The system itself as container is still useful and should stay the way it is.

Just a though: when working entirely with the new DataViewer, we could exchange the stream-root-system with this class, moving all objects and the successor handling to it and thus enabling us to downsize the system. Ultimately getting rid of complicated db operations when working with subsystems.