Changes in src/Actions/WorldAction/RemoveSphereOfAtomsAction.cpp [952f38:d74077]

File:

• src/Actions/WorldAction/RemoveSphereOfAtomsAction.cpp (modified) (5 diffs)

src/Actions/WorldAction/RemoveSphereOfAtomsAction.cpp

@@ -12 +12 @@
 #include "Descriptors/AtomDescriptor.hpp"
 #include "atom.hpp"
-#include "Helpers/Log.hpp"
+#include "element.hpp"
+#include "log.hpp"
 #include "molecule.hpp"
-#include "LinearAlgebra/Vector.hpp"
-#include "Helpers/Verbose.hpp"
+#include "vector.hpp"
+#include "verbose.hpp"
 #include "World.hpp"
 
@@ -25 +26 @@
 #include "UIElements/UIFactory.hpp"
 #include "UIElements/Dialog.hpp"
-#include "Actions/ValueStorage.hpp"
+#include "UIElements/ValueStorage.hpp"
 
 const char WorldRemoveSphereOfAtomsAction::NAME[] = "remove-sphere";
@@ -42 +43 @@
 };
 
-Dialog* WorldRemoveSphereOfAtomsAction::fillDialog(Dialog *dialog) {
-  ASSERT(dialog,"No Dialog given when filling action dialog");
+Dialog* WorldRemoveSphereOfAtomsAction::createDialog() {
+  Dialog *dialog = UIFactory::getInstance().makeDialog();
 
   dialog->queryDouble(NAME, ValueStorage::getInstance().getDescription(NAME));
@@ -62 +63 @@
   vector<molecule *> molecules = World::getInstance().getAllMolecules();
   for (vector<atom*>::iterator AtomRunner = AllAtoms.begin(); AtomRunner != AllAtoms.end(); ++AtomRunner) {
-    if (point.DistanceSquared((*AtomRunner)->x) > radius*radius) { // distance to first above radius ...
+    if ((*AtomRunner)->DistanceSquared(point) > radius*radius) { // distance to first above radius ...
+//      cout << "Removing " << (*AtomRunner)->getType()->symbol << (*AtomRunner)->getId() << " at " << (*AtomRunner)->getPosition() << " as distance is " << sqrt((*AtomRunner)->DistanceSquared(point)) << endl;
       // TODO: This is not necessary anymore when atoms are completely handled by World (create/destroy and load/save)
       for (vector<molecule *>::iterator iter = molecules.begin();iter != molecules.end();++iter)
@@ -68 +70 @@
       World::getInstance().destroyAtom(*AtomRunner);
     }
+//    else {
+//      cout << "Keeping" << (*AtomRunner)->getType()->symbol << (*AtomRunner)->getId() << " at " << (*AtomRunner)->getPosition() << " as distance is " << sqrt((*AtomRunner)->DistanceSquared(point)) << endl;
+//    }
   }
   return Action::success;