Convert ExpressionCommand and History collector to use Rational instead of PRAT

src/CalcManager/CEngine/History.cpp

@@ -12,6 +12,7 @@
 constexpr int ASCII_0 = 48;
 
 using namespace std;
+using namespace CalcEngine;
 
 void CHistoryCollector::ReinitHistory()
 {
@@ -51,7 +52,7 @@ CHistoryCollector::~CHistoryCollector()
     }
 }
 
-void CHistoryCollector::AddOpndToHistory(wstring_view numStr, PRAT hNoNum, bool fRepetition)
+void CHistoryCollector::AddOpndToHistory(wstring_view numStr, Rational rat, bool fRepetition)
 {
     std::shared_ptr<CalculatorVector<int>> commands = std::make_shared<CalculatorVector<int>>();
     // Check for negate
@@ -91,8 +92,7 @@ void CHistoryCollector::AddOpndToHistory(wstring_view numStr, PRAT hNoNum, bool
         }
     }
 
-    auto operandCommand = std::make_shared<COpndCommand>(commands, fNegative, fDecimal, fSciFmt);
+    auto operandCommand = std::make_shared<COpndCommand>(commands, rat, fNegative, fDecimal, fSciFmt);
-    operandCommand->Initialize(hNoNum);
     int iCommandEnd = AddCommand(operandCommand);
     m_lastOpStartIndex = IchAddSzToEquationSz(numStr, iCommandEnd);
 

src/CalcManager/CEngine/scicomm.cpp

@@ -211,9 +211,7 @@ void CCalcEngine::ProcessCommandWorker(WPARAM wParam)
         if (!m_HistoryCollector.FOpndAddedToHistory())
         {
             // if the prev command was ) or unop then it is already in history as a opnd form (...)
-            PRAT curRat = m_currentVal.ToPRAT();
+            m_HistoryCollector.AddOpndToHistory(m_numberString, m_currentVal);
-            m_HistoryCollector.AddOpndToHistory(m_numberString, curRat);
-            destroyrat(curRat);
         }
 
         /* m_bChangeOp is true if there was an operation done and the   */
@@ -310,10 +308,7 @@ DoPrecedenceCheckAgain:
         {
             if (!m_HistoryCollector.FOpndAddedToHistory())
             {
-
+                m_HistoryCollector.AddOpndToHistory(m_numberString, m_currentVal);
-                PRAT curRat = m_currentVal.ToPRAT();
-                m_HistoryCollector.AddOpndToHistory(m_numberString, curRat);
-                destroyrat(curRat);
             }
 
             m_HistoryCollector.AddUnaryOpToHistory((INT)wParam, m_bInv, m_angletype);
@@ -339,9 +334,7 @@ DoPrecedenceCheckAgain:
         if(wParam == IDC_PERCENT)
         {
             CheckAndAddLastBinOpToHistory();
-            PRAT curRat = m_currentVal.ToPRAT();
+            m_HistoryCollector.AddOpndToHistory(m_numberString, m_currentVal, true);
-            m_HistoryCollector.AddOpndToHistory(m_numberString, curRat, true);
-            destroyrat(curRat);
         }
 
         /* reset the m_bInv flag and indicators if it is set
@@ -464,9 +457,7 @@ DoPrecedenceCheckAgain:
 
         if (!m_HistoryCollector.FOpndAddedToHistory())
         {
-            PRAT curRat = m_currentVal.ToPRAT();
+            m_HistoryCollector.AddOpndToHistory(m_numberString, m_currentVal);
-            m_HistoryCollector.AddOpndToHistory(m_numberString, curRat);
-            destroyrat(curRat);
         }
 
         do {
@@ -485,9 +476,7 @@ DoPrecedenceCheckAgain:
                     m_currentVal = m_holdVal;
                     DisplayNum(); // to update the m_numberString
                     m_HistoryCollector.AddBinOpToHistory(m_nOpCode, false);
-                    PRAT curRat = m_currentVal.ToPRAT();
+                    m_HistoryCollector.AddOpndToHistory(m_numberString, m_currentVal); // Adding the repeated last op to history
-                    m_HistoryCollector.AddOpndToHistory(m_numberString, curRat); // Adding the repeated last op to history
-                    destroyrat(curRat);
                 }
 
                 // Do the current or last operation.
@@ -595,9 +584,7 @@ DoPrecedenceCheckAgain:
 
             if (!m_HistoryCollector.FOpndAddedToHistory())
             {
-                PRAT curRat = m_currentVal.ToPRAT();
+                m_HistoryCollector.AddOpndToHistory(m_numberString, m_currentVal);
-                m_HistoryCollector.AddOpndToHistory(m_numberString, curRat);
-                destroyrat(curRat);
             }
 
             // Get the operation and number and return result.
@@ -701,9 +688,7 @@ DoPrecedenceCheckAgain:
 
             if (!m_HistoryCollector.FOpndAddedToHistory())
             {
-                PRAT curRat = m_currentVal.ToPRAT();
+                m_HistoryCollector.AddOpndToHistory(m_numberString, m_currentVal);
-                m_HistoryCollector.AddOpndToHistory(m_numberString, curRat);
-                destroyrat(curRat);
             }
 
             PRAT curRat = m_currentVal.ToPRAT();

src/CalcManager/ExpressionCommand.cpp

@@ -7,6 +7,7 @@
 #include "ExpressionCommand.h"
 
 using namespace std;
+using namespace CalcEngine;
 
 constexpr wchar_t chNegate = L'-';
 constexpr wchar_t chExp = L'e';
@@ -95,25 +96,12 @@ void CBinaryCommand::Accept(_In_ ISerializeCommandVisitor &commandVisitor)
 }
 
 COpndCommand::COpndCommand(_In_ shared_ptr<CalculatorVector<int>> const &commands,
-        _In_ bool fNegative,
+        Rational const& rat,
-        _In_ bool fDecimal,
+        bool fNegative,
-        _In_ bool fSciFmt) :
+        bool fDecimal,
-    m_commands(commands), m_fNegative(fNegative), m_fDecimal(fDecimal), m_fSciFmt(fSciFmt)
+        bool fSciFmt) :
-{
+    m_commands(commands), m_value{ rat }, m_fNegative(fNegative), m_fDecimal(fDecimal), m_fSciFmt(fSciFmt)
-    m_hnoNum = nullptr;
+{}
-}
-
-
-void COpndCommand::Initialize(_In_ PRAT hNoNum)
-{
-    assert(&m_hnoNum != nullptr);
-    if (m_hnoNum != nullptr)
-    {
-        destroyrat(m_hnoNum);
-        m_hnoNum = nullptr;
-    }
-    DUPRAT(m_hnoNum, hNoNum);
-}
 
 const shared_ptr<CalculatorVector<int>> & COpndCommand::GetCommands() const
 { 
@@ -294,19 +282,15 @@ const wstring & COpndCommand::GetToken(wchar_t decimalSymbol)
 
 wstring COpndCommand::GetString(uint32_t radix, int32_t precision, wchar_t decimalSymbol)
 {
-    wstring numString{};
+    PRAT valRat = m_value.ToPRAT();
-    if (m_hnoNum != nullptr)
+    auto result = NumObjToString(valRat, radix, eNUMOBJ_FMT::FMT_FLOAT, precision);
-    {
+    destroyrat(valRat);
-        numString = NumObjToString(m_hnoNum, radix, eNUMOBJ_FMT::FMT_FLOAT, precision);
-    }
 
-    return numString;
+    return result;
 }
 
 COpndCommand::~COpndCommand()
-{
+{}
-    destroyrat(m_hnoNum);
-}
 
 void COpndCommand::Accept(_In_ ISerializeCommandVisitor &commandVisitor)
 {

src/CalcManager/ExpressionCommand.h

@@ -3,7 +3,8 @@
 
 #pragma once
 #include "ExpressionCommandInterface.h"
-#include "Header Files\CalcEngine.h"
+#include "Header Files/CalcEngine.h"
+#include "Header Files/Rational.h"
 
 class CParentheses : public IParenthesisCommand
 {
@@ -49,11 +50,11 @@ class COpndCommand : public IOpndCommand
 {
 public:
     COpndCommand(_In_ std::shared_ptr<CalculatorVector<int>> const &commands,
-        _In_ bool fNegative,
+        CalcEngine::Rational const& rat,
-        _In_ bool fDecimal,
+        bool fNegative,
-        _In_ bool fSciFmt);
+        bool fDecimal,
+        bool fSciFmt);
     ~COpndCommand();
-    void Initialize(_In_ PRAT hNoNum);
 
     const std::shared_ptr<CalculatorVector<int>> & GetCommands() const;
     void SetCommands(std::shared_ptr<CalculatorVector<int>> const& commands);
@@ -74,7 +75,7 @@ private:
     bool m_fSciFmt;
     bool m_fDecimal;
     std::wstring m_token;
-    PRAT m_hnoNum;
+    CalcEngine::Rational m_value;
     void ClearAllAndAppendCommand(CalculationManager::Command command);
 };
 

src/CalcManager/Header Files/History.h

@@ -5,6 +5,7 @@
 
 #include "ICalcDisplay.h"
 #include "IHistoryDisplay.h"
+#include "Rational.h"
 
 // maximum depth you can get by precedence. It is just an array's size limit.
 static constexpr size_t MAXPRECDEPTH = 25;
@@ -16,7 +17,7 @@ class CHistoryCollector {
 public:
     CHistoryCollector(ICalcDisplay *pCalcDisplay, std::shared_ptr<IHistoryDisplay> pHistoryDisplay, wchar_t decimalSymbol); // Can throw errors
     ~CHistoryCollector();
-    void AddOpndToHistory(std::wstring_view numStr, PRAT hNoNum, bool fRepetition = false);
+    void AddOpndToHistory(std::wstring_view numStr, CalcEngine::Rational rat, bool fRepetition = false);
     void RemoveLastOpndFromHistory();
     void AddBinOpToHistory(int nOpCode, bool fNoRepetition = true);
     void ChangeLastBinOp(int nOpCode, bool fPrecInvToHigher);