Return 4806 when create a iMACD handle in OnTick and useiGetArray to get buffer data, what`s the problem?

[Hardy5012]

The following is my test code

void OnTick()
  {
//---
   tick_cnt++;
   if(tick_cnt % 100 == 0)
     {
      signal_period++;
      int handle =iMACD(Symbol(),PERIOD_CURRENT,120,260,signal_period, PRICE_CLOSE);
      if(handle==INVALID_HANDLE)
        {
         PrintFormat("Failed to create handle of iMACD indicator for the symbol %s/%s, signal_period:%d, error code %d",
                     Symbol(),
                     EnumToString(Period()),
                     signal_period,
                     GetLastError());
         return;
        }
      double macd_buffer[];
      ArraySetAsSeries(macd_buffer,true);
      double signal_buffer[];
      ArraySetAsSeries(signal_buffer,true);
      if(!getMacdBuffer(handle, 0, 6, macd_buffer, signal_buffer))
        {
         PrintFormat("getMacdBuffer failed!%d", handle);
        }
        else{
          PrintFormat("getMacdBuffer OK!%d", handle);
        }
      IndicatorRelease(handle);
     }

  }

and the log

2020.09.16 09:08:22.250 2020.07.02 13:21:11   ERROR! EA: macd_demo.mq5, FUNCTION: iGetArray, amount to copy: 6, copied: -1, error code 4806
2020.09.16 09:08:22.250 2020.07.02 13:21:11   INFO! EA: macd_demo.mq5, FUNCTION: getMacdBuffer, iGetArray 0 failed! curr_time:2020.07.02 13:21
2020.09.16 09:08:22.250 2020.07.02 13:21:11   getMacdBuffer failed!636
2020.09.16 09:08:22.287 2020.07.02 13:24:24   ERROR! EA: macd_demo.mq5, FUNCTION: iGetArray, amount to copy: 6, copied: -1, error code 4806
2020.09.16 09:08:22.287 2020.07.02 13:24:24   INFO! EA: macd_demo.mq5, FUNCTION: getMacdBuffer, iGetArray 0 failed! curr_time:2020.07.02 13:24
2020.09.16 09:08:22.287 2020.07.02 13:24:24   getMacdBuffer failed!637
2020.09.16 09:08:22.313 2020.07.02 13:27:41   ERROR! EA: macd_demo.mq5, FUNCTION: iGetArray, amount to copy: 6, copied: -1, error code 4806
2020.09.16 09:08:22.313 2020.07.02 13:27:41   INFO! EA: macd_demo.mq5, FUNCTION: getMacdBuffer, iGetArray 0 failed! curr_time:2020.07.02 13:27
2020.09.16 09:08:22.313 2020.07.02 13:27:41   getMacdBuffer failed!638
2020.09.16 09:08:22.328 2020.07.02 13:30:00   ERROR! EA: macd_demo.mq5, FUNCTION: iGetArray, amount to copy: 6, copied: -1, error code 4806
2020.09.16 09:08:22.328 2020.07.02 13:30:00   INFO! EA: macd_demo.mq5, FUNCTION: getMacdBuffer, iGetArray 0 failed! curr_time:2020.07.02 13:30

[Vladimir Karputov]

Hardy5012 :

The following is my test code

and the log

You are making a gross mistake! In MQL5, the indicator handle SHOULD be received ONLY ONCE! This is done in OnInit ().

• Example from help: iMACD
• An example of how to create an indicator handle and get indicator values from an expert: iMACD value on chart

[Hardy5012]

Vladimir Karputov:

You are making a gross mistake! In MQL5, the indicator handle SHOULD be received ONLY ONCE! This is done in OnInit ().

• Example from help: iMACD
• An example of how to create an indicator handle and get indicator values from an expert: iMACD value on chart

Thanks again for your answers.There is a requirement that create a new macd handle as a quotation changed.

[Vladimir Karputov]

Hardy5012 :

Thanks again for your answers. There is a requirement that create a new macd handle as a quotation changed.

I did not understand the highlighted phrase . Can you ask your question differently?

[Hardy5012]

Vladimir Karputov:

I did not understand the highlighted phrase . Can you ask your question differe

Dynamically created new macd handle(change signal_period paramter) as the market quotaion changed. Like the following code , I use iRSI change signal_period paramter with  iMacd .

void OnTick()
  {
//---
   tick_cnt++;
   if(tick_cnt % 100 == 0)
     {
      double rsi_buffer[];
      ArraySetAsSeries(rsi_buffer,true);
      if(!iGetArray(handle_ris,0,0,1,rsi_buffer))
        {
         PrintFormat("INFO! EA: %s, FUNCTION: %s,  macd_type:%s,  iGetArray 0 failed! curr_time:%s",
                     __FILE__,__FUNCTION__, EnumToString(macd_type), TimeToString(time_current));
         return(false);
        }
      double rsi_value = rsi_buffer[0];
      int signal_period = int((rsi_value - 20) * rsi_macd_coeff + 2);
      int handle =iMACD(Symbol(),PERIOD_CURRENT,120,260,signal_period, PRICE_CLOSE);

[Vladimir Karputov]

Hardy5012 :

Dynamically created new macd handle(change signal_period paramter) as the market quotaion changed. Like the following code , I use iRSI change signal_period paramter with  iMacd .

Bad, very bad example.

All indicators must be created in advance in OnInit ().

[Carl Schreiber]

Hardy5012:

Dynamically created new macd handle(change signal_period paramter) as the market quotaion changed. Like the following code , I use iRSI change signal_period paramter with  iMacd .

One option might be the usage of IndicatorCreate() together with IndicatorParameters and IndicatorRelease.

But the better way would be to calculate the MACD yourself and just change the coefficients of the used ema for the fast and slow MA