Из цикла убери вызов этой функции - MQL4 и MetaTrader 4

Artyom Trishkin 2021.04.14 23:02 #14441

Vitaly Muzichenko:

Спасибо!

Попробовал и получил ошибку, что-то делаю не так:

А buffer[] распределён? Вижу объявление, но не вижу установку размера

Vitaly Muzichenko 2021.04.15 01:08 #14442

Artyom Trishkin:

А buffer[] распределён? Вижу объявление, но не вижу установку размера

Установил размер, ошибка ещё где-то, не работает.

Artyom Trishkin 2021.04.15 05:29 #14443

Vitaly Muzichenko:

Установил размер, ошибка ещё где-то, не работает.

В begin что передаёшь?

Vitaly Muzichenko 2021.04.15 08:40 #14444

Artyom Trishkin:

В begin что передаёшь?

   double Array_1[];
   double Array_2[];
   ArraySetAsSeries(close, true);
   int count = rates_total-1;
   if(count < 0)
      return (-1);
   int i = 1;
   int period = (int)MathFloor(MathSqrt(a));
   int val_1 = (int)MathFloor(a / 1.9);
   int counter = Bars(_Symbol,_Period) - count + a + 1;
   if(counter > Bars(_Symbol,_Period))
      counter = Bars(_Symbol,_Period);
  // ArraySetAsSeries(Array_1, true);
   ArrayResize(Array_1, counter);
  // ArraySetAsSeries(Array_2, true);
   ArrayResize(Array_2, counter);
   double d_close = close[1];
   double buff[];
   ArrayResize(buff, counter);
   for(i = 0; i < counter; i++)
      Array_1[i] = 2.0 * ma_1(i, val_1) - ma_1(i, a);
   for(i = 0; i < counter - a; i++) {
      // buf_3[i] = iMAOnArray(Array_1, 0, period, 0, MODE_SMMA, i);
      // buf_3[i] = iMA(NULL, 0, period, 0, MODE_SMMA, PRICE_LOW);
      //  CopyBuffer(10,0,i,1,buff);
      SmoothedMAOnBuffer(counter, prev_calculated, i, period, Array_1, buff);
      buf_3[i] = buff[0];
      }
     .....

//+------------------------------------------------------------------+
int SmoothedMAOnBuffer(const int rates_total,const int prev_calculated,const int begin,const int period,const double& price[],double& buffer[])
{
//--- check period
   if(period<=1 || period>(rates_total-begin))
      return(0);
//--- save as_series flags
   bool as_series_price=ArrayGetAsSeries(price);
   bool as_series_buffer=ArrayGetAsSeries(buffer);
   ArraySetAsSeries(price,false);
   ArraySetAsSeries(buffer,false);
//--- calculate start position
   int start_position;
   if(prev_calculated==0) { // first calculation or number of bars was changed
      //--- set empty value for first bars
      start_position=period+begin;
      for(int i=0; i<start_position-1; i++)
         buffer[i]=0.0;
      //--- calculate first visible value
      double first_value=0;
      for(int i=begin; i<start_position; i++)
         first_value+=price[i];
      buffer[start_position-1]=first_value/period;
   } else
      start_position=prev_calculated-1;
//--- main loop
   for(int i=start_position; i<rates_total; i++)
      buffer[i]=(buffer[i-1]*(period-1)+price[i])/period;
//--- restore as_series flags
   ArraySetAsSeries(price,as_series_price);
   ArraySetAsSeries(buffer,as_series_buffer);
//---
   return(rates_total);
}

Alexey Viktorov 2021.04.15 08:52 #14445

Vitaly Muzichenko:

Виталий, а зачем копировать функцию из инклюдника? Не проще подключить библиотеку?

#include <MovingAverages.mqh>

Artyom Trishkin 2021.04.15 09:09 #14446

Vitaly Muzichenko:

Вообще из цикла убери вызов этой функции. В ней самой уже есть полный цикл. В begin нужно передавать начало значимых данных в том массиве, на основании которого делаешь сглаживание. Для некоторых типов сглаживания необходим предварительный просчет. Для SMMA - это примерно два периода расчета.

Что нужно знать об Элитные показатели :) Помощь: Почему советник не

Alexey Viktorov 2021.04.15 09:23 #14447

Artyom Trishkin:

Вообще из цикла убери вызов этой функции. В ней самой уже есть полный цикл. В begin нужно передавать начало значимых данных в том массиве, на основании которого делаешь сглаживание. Для некоторых типов сглаживания необходим предварительный просчет. Для SMMA - это примерно два периода расчета.

А в таком случае, при добавлении нового бара не будет пересчитываться весь массив? Да и при изменении текущего значения в массиве буфера не мешало-бы пересчитывать. И как тогда без цикла?

Один раз при запуске индикатора по всему буферу и потом только последний индекс на каждом тике? Это экспромт. Я так не делал пока… надо проверить.

Функция iLowest Тонкости работы функции IndicatorCounted() CopyBuffer копирует массив не

Artyom Trishkin 2021.04.15 09:38 #14448

Alexey Viktorov:

А в таком случае, при добавлении нового бара не будет пересчитываться весь массив? Да и при изменении текущего значения в массиве буфера не мешало-бы пересчитывать. И как тогда без цикла?

Один раз при запуске индикатора по всему буферу и потом только последний индекс на каждом тике? Это экспромт. Я так не делал пока… надо проверить.

Ну код-то погляди - его Виталий прямо выложил же

Vitaly Muzichenko 2021.04.15 10:12 #14449

Artyom Trishkin:

Ну код-то погляди - его Виталий прямо выложил же

Сейчас пришёл к тому, чтобы отладить конструкцию "SmoothedMAOnBuffer()" в мт4.

Что не так, вообще не понимаю

int OnCalculate(const int rates_total,
                const int prev_calculated,
                const datetime &time[],
                const double &open[],
                const double &high[],
                const double &low[],
                const double &close[],
                const long &tick_volume[],
                const long &volume[],
                const int &spread[])
{
   double Array_1[];
   double Array_2[];
   int count = IndicatorCounted();
   if(count < 0)
      return (-1);
   int i = 1;
   int period = (int)MathFloor(MathSqrt(a));
   int val_1 = (int)MathFloor(a / 1.9);
   int counter = Bars - count + a + 1;
   if(counter > Bars)
      counter = Bars;
   ArraySetAsSeries(Array_1, TRUE);
   ArrayResize(Array_1, counter);
   ArraySetAsSeries(Array_2, TRUE);
   ArrayResize(Array_2, counter);
   double d_close = close[1];
   double buff[];
   ArrayResize(buff, counter);
   for(i = 0; i < counter; i++)
      Array_1[i] = 2.0 * ma_1(i, val_1) - ma_1(i, a);
   for(i = 0; i < counter - a; i++) {
     // buf_3[i] = iMAOnArray(Array_1, 0, period, 0, MODE_SMMA, i); // Так индикатор работает
      SmoothedMAOnBuffer(counter, prev_calculated, i, period, Array_1, buff); Совсем не работает, но и ошибок в журнале нет
      buf_3[i] = buff[0]; ???
   }
   for(i = counter - a; i > 0; i--) {
      Array_2[i] = Array_2[i + 1];
      if(buf_3[i] > buf_3[i + 1])
         Array_2[i] = 1;
      if(buf_3[i] < buf_3[i + 1])
         Array_2[i] = -1;
      if(Array_2[i] > 0.0) {
         buf_1[i] = buf_3[i];
         if(Array_2[i + 1] < 0.0)
            buf_1[i + 1] = buf_3[i + 1];
         if(Array_2[i + 1] < 0.0) {
         }
         buf_2[i] = EMPTY_VALUE;
      } else {
         if(Array_2[i] < 0.0) {
            buf_2[i] = buf_3[i];
            if(Array_2[i + 1] > 0.0)
               buf_2[i + 1] = buf_3[i + 1];
            if(Array_2[i + 1] > 0.0) {
            }
            buf_1[i] = EMPTY_VALUE;
         }
      }
   }
   for(i = 0; i < counter-1; i++) {
      if(buf_2[i + 1] == EMPTY_VALUE && buf_2[i] != EMPTY_VALUE)
         buf_4[i] = buf_2[i];
      if(buf_1[i + 1] == EMPTY_VALUE && buf_1[i] != EMPTY_VALUE)
         buf_5[i] = buf_1[i];
   }
   return(rates_total);
}
//+------------------------------------------------------------------+
//+------------------------------------------------------------------+
int SmoothedMAOnBuffer(const int rates_total,const int prev_calculated,const int begin,const int period,const double& price[],double& buffer[])
{
//--- check period
   if(period<=1 || period>(rates_total-begin))
      return(0);
//--- save as_series flags
   bool as_series_price=ArrayGetAsSeries(price);
   bool as_series_buffer=ArrayGetAsSeries(buffer);
   ArraySetAsSeries(price,false);
   ArraySetAsSeries(buffer,false);
//--- calculate start position
   int start_position;
   int i=0;
   if(prev_calculated==0) { // first calculation or number of bars was changed
      //--- set empty value for first bars
      start_position=period+begin;
      for(i=0; i<start_position-1; i++)
         buffer[i]=0.0;
      //--- calculate first visible value
      double first_value=0;
      for(i=begin; i<start_position; i++)
         first_value+=price[i];
      buffer[start_position-1]=first_value/period;
   } else
      start_position=prev_calculated-1;
//--- main loop
   for(i=start_position; i<rates_total; i++)
      buffer[i]=(buffer[i-1]*(period-1)+price[i])/period;
//--- restore as_series flags
   ArraySetAsSeries(price,as_series_price);
   ArraySetAsSeries(buffer,as_series_buffer);
//---
   return(rates_total);
}

Artyom Trishkin 2021.04.15 10:14 #14450

Vitaly Muzichenko:

Сейчас пришёл к тому, чтобы отладить конструкцию "SmoothedMAOnBuffer()" в мт4.

Что не так, вообще не понимаю

Из цикла убери

Любые вопросы новичков по MQL4 и MQL5, помощь и обсуждение по алгоритмам и кодам - страница 1445