From 0a9202c3ef0d3255ae3d57d0df68f7232396b580 Mon Sep 17 00:00:00 2001
From: Robin Moussu <robin.moussu+git@gmail.com>
Date: Sat, 26 May 2018 20:08:08 +0200
Subject: add "R" (Reroll until) operator

---
 node/rerolldicenode.cpp | 36 ++++++++++--------------------------
 node/rerolldicenode.h   | 13 +++++--------
 2 files changed, 15 insertions(+), 34 deletions(-)

(limited to 'node')

diff --git a/node/rerolldicenode.cpp b/node/rerolldicenode.cpp
index a24ffa9..f7df0fa 100644
--- a/node/rerolldicenode.cpp
+++ b/node/rerolldicenode.cpp
@@ -1,8 +1,11 @@
 #include "rerolldicenode.h"
 #include "parsingtoolbox.h"
 
-RerollDiceNode::RerollDiceNode()
-    : m_diceResult(new DiceResult()),m_adding(false),m_validator(nullptr)
+RerollDiceNode::RerollDiceNode(bool reroll, bool addingMode)
+    : m_diceResult(new DiceResult())
+    , m_validator(nullptr)
+    , m_reroll(reroll)
+    , m_adding(addingMode)
 {
     m_result=m_diceResult;
 }
@@ -38,26 +41,12 @@ void RerollDiceNode::run(ExecutionNode* previous)
             for(int i = 0; i < list.size() ; ++i)
             {
                 auto die = list.at(i);
-                if(m_validator->hasValid(die,false))
+                while(m_validator->hasValid(die,false))
                 {
-                    if(m_instruction != nullptr)
+                    die->roll(m_adding);
+                    if(m_reroll)
                     {
-                        m_instruction->run(this);
-                        auto lastNode = ParsingToolBox::getLatestNode(m_instruction);
-                        if(lastNode != nullptr)
-                        {
-                            auto lastResult = dynamic_cast<DiceResult*>(lastNode->getResult());
-                            if(lastResult != nullptr)
-                            {
-                                toRemove.append(die);
-                                list.append(lastResult->getResultList());
-                            }
-                            lastResult->clear();
-                        }
-                    }
-                    else
-                    {
-                        die->roll(m_adding);
+                        break;
                     }
                 }
             }
@@ -96,10 +85,6 @@ QString RerollDiceNode::toString(bool wl) const
 	}
 	//return QString("RerollDiceNode [label=\"RerollDiceNode validatior:%1\"");
 }
-void RerollDiceNode::setAddingMode(bool b)
-{
-    m_adding = b;
-}
 qint64 RerollDiceNode::getPriority() const
 {
     qint64 priority=0;
@@ -113,9 +98,8 @@ qint64 RerollDiceNode::getPriority() const
 }
 ExecutionNode* RerollDiceNode::getCopy() const
 {
-    RerollDiceNode* node = new RerollDiceNode();
+    RerollDiceNode* node = new RerollDiceNode(m_reroll, m_adding);
     node->setValidator(m_validator);
-    node->setAddingMode(m_adding);
     if(nullptr!=m_nextNode)
     {
         node->setNextNode(m_nextNode->getCopy());
diff --git a/node/rerolldicenode.h b/node/rerolldicenode.h
index c999dda..8bef6dd 100644
--- a/node/rerolldicenode.h
+++ b/node/rerolldicenode.h
@@ -18,8 +18,9 @@ public:
     enum ReRollMode {EQUAL,LESSER,GREATER};
 	/**
 	 * @brief RerollDiceNode
+	 * @param reroll If true reroll the dice only once, otherwise until the condition is false
 	 */
-    RerollDiceNode();
+    RerollDiceNode(bool reroll, bool addingMode);
 
 	/**
 	 * @brief ~RerollDiceNode
@@ -40,12 +41,6 @@ public:
 	 * @return
 	 */
     virtual QString toString(bool )const;
-
-
-	/**
-	 * @brief setAddingMode
-	 */
-    virtual void setAddingMode(bool);
 	/**
 	 * @brief getPriority
 	 * @return
@@ -63,9 +58,11 @@ public:
 
 private:
     DiceResult* m_diceResult = nullptr;
-    bool m_adding;
     Validator* m_validator  = nullptr;
     ExecutionNode* m_instruction = nullptr;
+
+    const bool m_reroll;
+    const bool m_adding;
 };
 
 #endif // REROLLDICENODE_H
-- 
cgit v1.2.3-70-g09d2