Changeset 8d9d38 for molecuilder/src/World.cpp

Timestamp:

Feb 25, 2010, 4:43:02 PM (16 years ago)

Author:

Tillmann Crueger <crueger@…>

Children:

3db67e

Parents:

2e6496

Message:

Made the world solely responsible for creating and erasing molecules.

File:

• molecuilder/src/World.cpp (modified) (4 diffs)

molecuilder/src/World.cpp

@@ -47 +47 @@
   OBSERVE;
   molecule *mol = NULL;
-  mol = new molecule(periode);
-  molecules_deprecated->insert(mol);
+  mol = NewMolecule();
   assert(!molecules.count(currMoleculeId));
+  mol->setId(currMoleculeId++);
   // store the molecule by ID
-  molecules[currMoleculeId++] = mol;
+  molecules[mol->getId()] = mol;
   mol->signOn(this);
   return mol;
+}
+
+void World::destroyMolecule(molecule* mol){
+  OBSERVE;
+  destroyMolecule(mol->getId());
+}
+
+void World::destroyMolecule(moleculeId_t id){
+  OBSERVE;
+  molecule *mol = molecules[id];
+  assert(mol);
+  DeleteMolecule(mol);
+  molecules.erase(id);
 }
 
@@ -83 +96 @@
 }
 
-void World::destroyAtom(int id) {
+void World::destroyAtom(atomId_t id) {
   OBSERVE;
   atom *atom = atoms[id];
@@ -134 +147 @@
     currMoleculeId(0),
     periode(new periodentafel),
-    molecules_deprecated(new MoleculeListClass),
+    molecules_deprecated(new MoleculeListClass(this)),
     atoms(),
     molecules()
@@ -145 +158 @@
   delete molecules_deprecated;
   delete periode;
-  AtomSet::iterator iter;
-  for(iter=atoms.begin();iter!=atoms.end();++iter){
-    DeleteAtom((*iter).second);
+  MoleculeSet::iterator molIter;
+  for(molIter=molecules.begin();molIter!=molecules.end();++molIter){
+    DeleteMolecule((*molIter).second);
+  }
+  molecules.clear();
+  AtomSet::iterator atIter;
+  for(atIter=atoms.begin();atIter!=atoms.end();++atIter){
+    DeleteAtom((*atIter).second);
   }
   atoms.clear();