refactor: Uses MovingAverage_Engine for smoothing

Include/MyIncludes/StochasticSlow_Calculator.mqh

@@ -1,38 +1,40 @@
 //+------------------------------------------------------------------+
 //|                                     StochasticSlow_Calculator.mqh|
-//|      VERSION 1.20: Optimized for incremental calculation.        |
+//|      VERSION 2.00: Uses MovingAverage_Engine for smoothing.      |
 //|                                        Copyright 2025, xxxxxxxx  |
 //+------------------------------------------------------------------+
 #property copyright "Copyright 2025, xxxxxxxx"
 
-#include <MyIncludes\HeikinAshi_Tools.mqh>
+#include <MyIncludes\MovingAverage_Engine.mqh>
 
 //+==================================================================+
-//|           CLASS 1: CStochasticSlowCalculator (Base Class)        |
+//|           CLASS: CStochasticSlowCalculator                       |
 //+==================================================================+
 class CStochasticSlowCalculator
   {
 protected:
-   int               m_k_period, m_d_period, m_slowing_period;
+   int               m_k_period;
-   ENUM_MA_METHOD    m_slowing_ma_type, m_d_ma_type;
 
-   //--- Persistent Buffers for Incremental Calculation
+   //--- Composition: Two MA Engines
+   CMovingAverageCalculator m_slowing_engine; // For Slow %K
+   CMovingAverageCalculator m_signal_engine;  // For %D
+
+   //--- Persistent Buffers
    double            m_src_high[], m_src_low[], m_src_close[];
-   double            m_raw_k[]; // Stores Fast %K
+   double            m_raw_k[]; // Stores Fast %K (intermediate)
 
    double            Highest(int period, int current_pos);
    double            Lowest(int period, int current_pos);
 
-   //--- Updated: Accepts start_index
    virtual bool      PrepareSourceData(int rates_total, int start_index, const double &open[], const double &high[], const double &low[], const double &close[]);
 
 public:
                      CStochasticSlowCalculator(void) {};
    virtual          ~CStochasticSlowCalculator(void) {};
 
-   bool              Init(int k_p, int slow_p, ENUM_MA_METHOD slow_ma, int d_p, ENUM_MA_METHOD d_ma);
+   //--- Init now takes ENUM_MA_TYPE for both smoothings
+   bool              Init(int k_p, int slow_p, ENUM_MA_TYPE slow_ma, int d_p, ENUM_MA_TYPE d_ma);
 
-   //--- Updated: Accepts prev_calculated
    void              Calculate(int rates_total, int prev_calculated, const double &open[], const double &high[], const double &low[], const double &close[],
                                double &k_buffer[], double &d_buffer[]);
   };
@@ -40,33 +42,32 @@ public:
 //+------------------------------------------------------------------+
 //| Init                                                             |
 //+------------------------------------------------------------------+
-bool CStochasticSlowCalculator::Init(int k_p, int slow_p, ENUM_MA_METHOD slow_ma, int d_p, ENUM_MA_METHOD d_ma)
+bool CStochasticSlowCalculator::Init(int k_p, int slow_p, ENUM_MA_TYPE slow_ma, int d_p, ENUM_MA_TYPE d_ma)
   {
-   m_k_period        = (k_p < 1) ? 1 : k_p;
+   m_k_period = (k_p < 1) ? 1 : k_p;
-   m_slowing_period  = (slow_p < 1) ? 1 : slow_p;
+
-   m_slowing_ma_type = slow_ma;
+// Initialize both engines
-   m_d_period        = (d_p < 1) ? 1 : d_p;
+   if(!m_slowing_engine.Init(slow_p, slow_ma))
-   m_d_ma_type       = d_ma;
+      return false;
+   if(!m_signal_engine.Init(d_p, d_ma))
+      return false;
+
    return true;
   }
 
 //+------------------------------------------------------------------+
-//| Main Calculation (Optimized)                                     |
+//| Main Calculation                                                 |
 //+------------------------------------------------------------------+
 void CStochasticSlowCalculator::Calculate(int rates_total, int prev_calculated, const double &open[], const double &high[], const double &low[], const double &close[],
       double &k_buffer[], double &d_buffer[])
   {
-   if(rates_total <= m_k_period + m_slowing_period + m_d_period)
+// Check minimum bars required
+   int min_bars = m_k_period + m_slowing_engine.GetPeriod() + m_signal_engine.GetPeriod();
+   if(rates_total <= min_bars)
       return;
 
-//--- 1. Determine Start Index
+   int start_index = (prev_calculated == 0) ? 0 : prev_calculated - 1;
-   int start_index;
-   if(prev_calculated == 0)
-      start_index = 0;
-   else
-      start_index = prev_calculated - 1;
 
-//--- 2. Resize Buffers
    if(ArraySize(m_src_high) != rates_total)
      {
       ArrayResize(m_src_high, rates_total);
@@ -75,11 +76,10 @@ void CStochasticSlowCalculator::Calculate(int rates_total, int prev_calculated,
       ArrayResize(m_raw_k, rates_total);
      }
 
-//--- 3. Prepare Source Data (Optimized)
    if(!PrepareSourceData(rates_total, start_index, open, high, low, close))
       return;
 
-//--- 4. Calculate Raw %K (Fast %K)
+//--- 1. Calculate Raw %K (Fast %K)
    int loop_start_k = MathMax(m_k_period - 1, start_index);
 
    for(int i = loop_start_k; i < rates_total; i++)
@@ -94,87 +94,26 @@ void CStochasticSlowCalculator::Calculate(int rates_total, int prev_calculated,
          m_raw_k[i] = (i > 0) ? m_raw_k[i-1] : 50.0;
      }
 
-//--- 5. Calculate Slow %K (Main Line)
+//--- 2. Calculate Slow %K (Main Line) using Slowing Engine
-   int k_slow_start = m_k_period + m_slowing_period - 2;
+// Offset for Raw %K is (K - 1)
-   int loop_start_slow = MathMax(k_slow_start, start_index);
+   int raw_k_offset = m_k_period - 1;
 
-   for(int i = loop_start_slow; i < rates_total; i++)
+// Output goes to k_buffer (this is the main Slow Stochastic line)
-     {
+   m_slowing_engine.CalculateOnArray(rates_total, prev_calculated, m_raw_k, k_buffer, raw_k_offset);
-      switch(m_slowing_ma_type)
-        {
-         case MODE_EMA:
-         case MODE_SMMA:
-            if(i == k_slow_start)
-              {
-               double sum=0;
-               for(int j=0; j<m_slowing_period; j++)
-                  sum+=m_raw_k[i-j];
-               k_buffer[i]=sum/m_slowing_period;
-              }
-            else
-              {
-               if(m_slowing_ma_type==MODE_EMA)
-                 {
-                  double pr=2.0/(m_slowing_period+1.0);
-                  k_buffer[i]=m_raw_k[i]*pr+k_buffer[i-1]*(1.0-pr);
-                 }
-               else
-                  k_buffer[i]=(k_buffer[i-1]*(m_slowing_period-1)+m_raw_k[i])/m_slowing_period;
-              }
-            break;
-         case MODE_LWMA:
-           {double sum=0,w_sum=0; for(int j=0; j<m_slowing_period; j++) {int w=m_slowing_period-j; sum+=m_raw_k[i-j]*w; w_sum+=w;} if(w_sum>0) k_buffer[i]=sum/w_sum;}
-         break;
-         default:
-           {double sum=0; for(int j=0; j<m_slowing_period; j++) sum+=m_raw_k[i-j]; k_buffer[i]=sum/m_slowing_period;}
-         break;
-        }
-     }
 
-//--- 6. Calculate %D (Signal Line)
+//--- 3. Calculate %D (Signal Line) using Signal Engine
-   int d_start = m_k_period + m_slowing_period + m_d_period - 3;
+// Offset for Slow %K is (Raw_Offset + Slowing_Period - 1)
-   int loop_start_d = MathMax(d_start, start_index);
+   int slow_k_offset = raw_k_offset + m_slowing_engine.GetPeriod() - 1;
 
-   for(int i = loop_start_d; i < rates_total; i++)
+// Input is k_buffer (Slow %K), Output is d_buffer
-     {
+   m_signal_engine.CalculateOnArray(rates_total, prev_calculated, k_buffer, d_buffer, slow_k_offset);
-      switch(m_d_ma_type)
-        {
-         case MODE_EMA:
-         case MODE_SMMA:
-            if(i == d_start)
-              {
-               double sum=0;
-               for(int j=0; j<m_d_period; j++)
-                  sum+=k_buffer[i-j];
-               d_buffer[i]=sum/m_d_period;
-              }
-            else
-              {
-               if(m_d_ma_type==MODE_EMA)
-                 {
-                  double pr=2.0/(m_d_period+1.0);
-                  d_buffer[i]=k_buffer[i]*pr+d_buffer[i-1]*(1.0-pr);
-                 }
-               else
-                  d_buffer[i]=(d_buffer[i-1]*(m_d_period-1)+k_buffer[i])/m_d_period;
-              }
-            break;
-         case MODE_LWMA:
-           {double sum=0,w_sum=0; for(int j=0; j<m_d_period; j++) {int w=m_d_period-j; sum+=k_buffer[i-j]*w; w_sum+=w;} if(w_sum>0) d_buffer[i]=sum/w_sum;}
-         break;
-         default:
-           {double sum=0; for(int j=0; j<m_d_period; j++) sum+=k_buffer[i-j]; d_buffer[i]=sum/m_d_period;}
-         break;
-        }
-     }
   }
 
 //+------------------------------------------------------------------+
-//| Prepare Source Data (Standard - Optimized)                       |
+//| Prepare Source Data (Standard)                                   |
 //+------------------------------------------------------------------+
 bool CStochasticSlowCalculator::PrepareSourceData(int rates_total, int start_index, const double &open[], const double &high[], const double &low[], const double &close[])
   {
-// Optimized copy loop
    for(int i = start_index; i < rates_total; i++)
      {
       m_src_high[i]  = high[i];
@@ -185,7 +124,7 @@ bool CStochasticSlowCalculator::PrepareSourceData(int rates_total, int start_ind
   }
 
 //+------------------------------------------------------------------+
-//| Highest                                                          |
+//| Helpers                                                          |
 //+------------------------------------------------------------------+
 double CStochasticSlowCalculator::Highest(int period, int current_pos)
   {
@@ -202,7 +141,7 @@ double CStochasticSlowCalculator::Highest(int period, int current_pos)
   }
 
 //+------------------------------------------------------------------+
-//| Lowest                                                           |
+//|                                                                  |
 //+------------------------------------------------------------------+
 double CStochasticSlowCalculator::Lowest(int period, int current_pos)
   {
@@ -225,19 +164,16 @@ class CStochasticSlowCalculator_HA : public CStochasticSlowCalculator
   {
 private:
    CHeikinAshi_Calculator m_ha_calculator;
-   // Internal HA buffers
    double            m_ha_open[], m_ha_high_temp[], m_ha_low_temp[], m_ha_close_temp[];
-
 protected:
    virtual bool      PrepareSourceData(int rates_total, int start_index, const double &open[], const double &high[], const double &low[], const double &close[]) override;
   };
 
 //+------------------------------------------------------------------+
-//| Prepare Source Data (Heikin Ashi - Optimized)                    |
+//|                                                                  |
 //+------------------------------------------------------------------+
 bool CStochasticSlowCalculator_HA::PrepareSourceData(int rates_total, int start_index, const double &open[], const double &high[], const double &low[], const double &close[])
   {
-// Resize internal HA buffers
    if(ArraySize(m_ha_open) != rates_total)
      {
       ArrayResize(m_ha_open, rates_total);
@@ -245,12 +181,7 @@ bool CStochasticSlowCalculator_HA::PrepareSourceData(int rates_total, int start_
       ArrayResize(m_ha_low_temp, rates_total);
       ArrayResize(m_ha_close_temp, rates_total);
      }
-
+   m_ha_calculator.Calculate(rates_total, start_index, open, high, low, close, m_ha_open, m_ha_high_temp, m_ha_low_temp, m_ha_close_temp);
-//--- STRICT CALL: Use the optimized 10-param HA calculation
-   m_ha_calculator.Calculate(rates_total, start_index, open, high, low, close,
-                             m_ha_open, m_ha_high_temp, m_ha_low_temp, m_ha_close_temp);
-
-//--- Copy to source buffers (Optimized loop)
    for(int i = start_index; i < rates_total; i++)
      {
       m_src_high[i]  = m_ha_high_temp[i];