ФорумРазделы

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

Новый комментарий
PolarSeaman:

Всё равно вылетает.

у вас [i+1], нужно проверять на +2 больше

вообще нужно делать примерно так

limit=(rates_total<cb || cb<=0) ? rates_total-1 : cb;
if (limit<=0) return(0);

 

Ihor Herasko:

Потому что проверка в корне неверная. Допустим, Bars вернула 1000, а cb тоже 1000. Затем в цикле на первой итерации i получает значение 1000. В первом же условии тела цикла:

сразу два выхода за пределы массива: обращение к бару с индексом 1000 и с индексом 1001. Ведь если на графике 1000 баров, то первый бар имеет индекс 0, а последний - 999.

Дальше по телу цикла идет обращение к еще более далеким барам а истории:

Все это нужно учесть при изначальной проверке.

Как делать правильно проверку? посмотрите в примере по функции IndicatorCounted(), в справке по MQL4. Только теперь IndicatorCounted() лучше заменить на совместное использование переменных rates_total (это Bars) и prev_calculated (это IndicatorCounted()).

Спасибо, Нашел, 

 int counted_bars=IndicatorCounted(); 
     if(counted_bars>0) counted_bars--;

 с чем его сравнивать? в примере начинает с limit

 limit=Bars-counted_bars;

с чем мне эту "cb" сравнивать?

PolarSeaman:

Спасибо, Нашел, 

 с чем его сравнивать? в примере начинает с limit

с чем мне эту "cb" сравнивать?

Если нужно ограничивать количество баров, на которых отображаются данные индикатора, то лучше сделать так:

int GetRecalcIndex(int& total, const int ratesTotal, const int prevCalculated)
{
   total = ratesTotal - 2 - barsig;                                                                         
                                                   
   if (cb > 0 && cb < total)
      total = MathMin(cb, total);                      
                                                   
   if (prevCalculated < ratesTotal - 1)                     
   {       
      InitializeBuffers();    // Это функция, которая должна заново инициализировать все индикаторные буфера, т. к. имеем дело с первой загрузкой индикатора или подкачкой истории
      return (total);
   }
   
   return (MathMin(ratesTotal - prevCalculated, total));                            
}

Пользоваться так:

int total;   
int limit = GetRecalcIndex(total, rates_total, prev_calculated);                                

for (int i = limit; i >= 0; --i)
{
  ...
}
Значение total - это индекс самого глубокого бара в истории, к которому можно обращаться, исходя из значений настроечных параметров индикатора.
Taras Slobodyanik:

у вас [i+1], нужно проверять на +2 больше

вообще нужно делать примерно так

 

Сделал, точно так, пишет ...array out of range in 'HiLo.mq4' (122,15)

что проверять на +2 больше?


Ihor Herasko:

Если нужно ограничивать количество баров, на которых отображаются данные индикатора, то лучше сделать так:

Пользоваться так:

Значение total - это индекс самого глубокого бара в истории, к которому можно обращаться, исходя из значений настроечных параметров индикатора.

Компилятор ругается

'InitializeBuffers' - function not defined HiLo.mq4 161 7

и в справке об этой функции ничего нет
PolarSeaman:

Компилятор ругается

'InitializeBuffers' - function not defined HiLo.mq4 161 7

и в справке об этой функции ничего нет

Я же написал в комментарии, что это функция, которая должна выполнить инициализацию всех буферов индикатора. Это пользовательская функция. У меня она такая:

void InitializeBuffers()
{
   ArrayInitialize(g_indValues, EMPTY_VALUE);
   ArrayInitialize(g_tempBuffer, EMPTY_VALUE);
}

У вас будет другая, т . к. другие буфера. В случае же, если индикатор работает с графическими объектами, то здесь нужно все их удалить, т. к. предстоит начальная отрисовка показаний.

Ihor Herasko:

Я же написал в комментарии, что это функция, которая должна выполнить инициализацию всех буферов индикатора. Это пользовательская функция. У меня она такая:

У вас будет другая, т . к. другие буфера. В случае же, если индикатор работает с графическими объектами, то здесь нужно все их удалить, т. к. предстоит начальная отрисовка показаний.

Спасибо, но ничего не изменилось ...array out of range in 'HiLo.mq4' (130,15)

. Что не так?

#property copyright "Copyright ©  november 2015"
#property strict 

#property indicator_chart_window
#property indicator_buffers 6

#property indicator_color1 RoyalBlue       //DodgerBlue
#property indicator_color2 Crimson         //OrangeRed
#property indicator_color3 Black  //White
#property indicator_color4 Black  //White
#property indicator_color5 Black            //White
#property indicator_color6 Black         //Red

#property indicator_width1 2
#property indicator_width2 2

#property indicator_style3 STYLE_DOT
#property indicator_style4 STYLE_DOT

input int    p        = 10;    // Период
input int    s        = 5;     // Угол наклона
input double distance = 2.0;   // Ширина канала
input bool   showBb   = false;  // Границы канала
input bool   showCl   = true;  // Центральная линия
input int    barsig   = 1;     // Сигнальная свеча (номер)
input int    arrots   = 0;    // Стрелка (отступ)
input int    arrsz    = 0;     // Стрелка (размер)
input int    ATR      = 1000;  // Период ATR
input int    cb       = 1000;  // Сколько свечей в истории

double fx1[],fx2[],hp[];
double z1,z2,ki;
int fs;

double upper[],lower[];
double upar[],dnar[];

//+------------------------------------------------------------------+
//| Custom indicator initialization function                         |
//+------------------------------------------------------------------+
int OnInit(void)
  {
//--- indicator buffers mapping
IndicatorBuffers(7);
SetIndexBuffer(0,fx1);
SetIndexBuffer(1,fx2);
SetIndexBuffer(2,lower);
SetIndexBuffer(3,upper);
SetIndexBuffer(4,upar);
SetIndexBuffer(5,dnar);
SetIndexBuffer(6,hp);


SetIndexStyle (4,DRAW_ARROW,0,arrsz);
SetIndexArrow (4,233);
SetIndexStyle (5,DRAW_ARROW,0,arrsz);
SetIndexArrow (5,234);

   if(showBb)
   {SetIndexStyle(2,DRAW_LINE);
    SetIndexStyle(3,DRAW_LINE);
   }
   else
   {SetIndexStyle(2,DRAW_NONE);
    SetIndexStyle(3,DRAW_NONE);
   }
   
    if(showCl)
   {SetIndexStyle(0,DRAW_LINE);
    SetIndexStyle(1,DRAW_LINE);
   }
   else
   {SetIndexStyle(0,DRAW_NONE);
    SetIndexStyle(1,DRAW_NONE);
   }

SetIndexEmptyValue(0,0.0);
SetIndexEmptyValue(1,0.0);

//---
   return(INIT_SUCCEEDED);
  }
  
//+------------------------------------------------------------------+
//| Custom indicator iteration function                              |
//+------------------------------------------------------------------+
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[])
  {
//---
   int i,limit;
//   limit=(rates_total<cb || cb<=0) ? rates_total-2 : cb;
//if (limit<=0) return(0);
 int total;   
 limit = GetRecalcIndex(total, rates_total, prev_calculated);                                

for (int i = limit; i >= 0; --i)
{ 
  
  
   SetIndexDrawBegin(0,Bars-cb);
   SetIndexDrawBegin(1,Bars-cb);

double avg;

ki=2.0/(p+1);

for (i=cb; i>=0; i--) {fx1[i]=Close[i];}

for (int m=0; m<=s; m++)
{
z1=fx1[0];
for (i=0; i<=cb; i++) {z1=z1+(fx1[i]-z1)*ki; hp[i]=z1;}

z2=fx1[cb];
for (i=cb; i>=0; i--) {z2=z2+(fx1[i]-z2)*ki; fx1[i]=(hp[i]+z2)/2;}
}

fs=0;
for (i=cb; i>=0; i--)
{
if (fx1[i]>fx1[i+1]) fs=1;
if (fx1[i]<fx1[i+1]) {if (fs==1) fx2[i+1]=fx1[i+1]; fs=2;}
if (fs==2) fx2[i]=fx1[i]; else fx2[i]=0.0;

avg = iATR(NULL,0,ATR, i+10);
upper[i] = hp[i] + distance*avg;
lower[i] = hp[i] - distance*avg;

if(Close[i+1+barsig]<upper[i+1+barsig] && Close[i+barsig]>upper[i+barsig])
 dnar[i] = High[i]+arrots*Point; else dnar[i] = EMPTY_VALUE;
 
if(Close[i+1+barsig]>lower[i+1+barsig] && Close[i+barsig]<lower[i+barsig])
 upar[i] = Low[i]-arrots*Point; else upar[i] = EMPTY_VALUE; 
}
}
//--- return value of prev_calculated for next call
   return(rates_total);
  }
  

int GetRecalcIndex(int& total, const int ratesTotal, const int prevCalculated)
{
   total = ratesTotal - 2 - barsig;                                                                         
                                                   
   if (cb > 0 && cb < total)
      total = MathMin(cb, total);                      
                                                   
   if (prevCalculated < ratesTotal - 1)                     
   {       
      InitializeBuffers();    // Это функция, которая должна заново инициализировать все индикаторные буфера, т. к. имеем дело с первой загрузкой индикатора или подкачкой истории
      return (total);
   }
   
   return (MathMin(ratesTotal - prevCalculated, total));                            
}

void InitializeBuffers()
{
   ArrayInitialize(fx1, EMPTY_VALUE);
   ArrayInitialize(fx2, EMPTY_VALUE);
   ArrayInitialize(lower, EMPTY_VALUE);
   ArrayInitialize(upper, EMPTY_VALUE);
   ArrayInitialize(upar, EMPTY_VALUE);
   ArrayInitialize(dnar, EMPTY_VALUE);
   ArrayInitialize(hp, EMPTY_VALUE);
}
PolarSeaman:

Спасибо, но ничего не изменилось. Что не так?

Конечно, результат не изменится. Ведь главную причину (цикл по cb) Вы не убрали. Вот этот цикл организован неправильно:

for (i=cb; i>=0; i--)

Его нужно убирать и вместо него ставить цикл по limit. В обоих местах.

Ihor Herasko:

Конечно, результат не изменится. Ведь главную причину (цикл по cb) Вы не убрали. Вот этот цикл организован неправильно:

Его нужно убирать и вместо него ставить цикл по limit. В обоих местах.

Таких циклов 3. Заменил их, терминал завис.

Сделал, не зависает, не вылетает, но в первом буфере (fx2) значений бывает 3: цена, 0.0 и 164874239.218492.

sell_1_B=NormalizeDouble(iCustom(Symbol(),0,"HiLo",1,1),Digits);

если значение sell_1_B!=EMPTY_VALUE и sell_1_B!=0 это не значит что там цена.

Как получить сигнал?

1...560561562563564565566567568569570571572573574...2748
Новый комментарий