Might be good to have a destructor too

[popboot.git] / planner.cpp

diff --git a/planner.cpp b/planner.cpp
index 5e1719e11b48619a3a805801893e4bb0fa46408f..6d70ae40df2114d32e6cf4f3ab611155e96b6aa6 100644 (file)
--- a/planner.cpp
+++ b/planner.cpp
@@ -6,12 +6,13 @@ using namespace __gnu_cxx;
 
 Planner::Planner(std::vector<Action> actions, literals init, literals goal){
 
-       _start = new Node(Action("",literals(), init));
-       Node* finish = new Node(Action("",goal,literals()));
+       _start = new Node(Action("start",literals(), init));
+       addNode(_start);
+       Node* finish = new Node(Action("finish",goal,literals()));
 
        for(vector<Action>::iterator action = actions.begin(); action != actions.end(); ++action){
-               literals effects = action->effects();
-               for (literals::iterator effect = effects.begin(); effect != effects.end(); ++effect){
+               const literals& effects = action->effects();
+               for (literals::const_iterator effect = effects.begin(); effect != effects.end(); ++effect){
                        cerr << "Adding effect: " << *effect << endl;
                        _actions[*effect] = *action;
                }
@@ -19,15 +20,24 @@ Planner::Planner(std::vector<Action> actions, literals init, literals goal){
        makePlan(finish);
 }
 
+Planner::~Planner(){
+       for (hash_map<string,Node*>::iterator node = _addedNodes.begin(); node != _addedNodes.end(); ++node){
+               cerr << "Deleting node " << node->second->action().executable() << endl;
+               delete node->second;
+       }
+}
+
 
 void Planner::makePlan(Node* node){
-       literals preconds = node->action().preconditions();
+       cerr << "Fetching preconditions for action: " << node->action().executable() << ".. ";
+       const literals& preconds = node->action().preconditions();
+       cerr << "done" << endl;
 
        if (preconds.size() == 0){
                cerr << "Found no preconds" << endl;
                _start->addChild(node);
        }else{
-               for (literals::iterator precond = preconds.begin(); precond != preconds.end(); ++precond){
+               for (literals::const_iterator precond = preconds.begin(); precond != preconds.end(); ++precond){
                        cerr << "Looking for: " << *precond << endl;
                        hash_map<string,Node*>::iterator addedNode = _addedNodes.find(*precond);
                        if(addedNode != _addedNodes.end()){
@@ -50,10 +60,15 @@ void Planner::makePlan(Node* node){
 }
 
 void Planner::addNode(Node* node){
-       literals effects = node->action().effects();
+       const literals& effects = node->action().effects();
 
-       for (literals::iterator effect = effects.begin(); effect != effects.end(); ++effect){
+       for (literals::const_iterator effect = effects.begin(); effect != effects.end(); ++effect){
                cout << "Adding node for effect: " << *effect << endl;
                _addedNodes[*effect] = node;
        }
 }
+
+
+void Planner::execute(){
+       _start->execute(literals());
+}