error copying 4806

[samjesse]

Hi

I do not know if I am doing something wrong here for that error to appear.
the error only appears on the s_Ma buffer "last line"

            int x = Bars(pair.names[i], pair.periods[j]);  //-- x --- amount of bars to process ------
            int y = x-slow_iMa;                            //-- y --- amount of usable bars ---------- 
            // ... some other code here
            int f_iMa_Handle = iMA(pair.names[i], pair.periods[j],fast_iMa,0,MODE_SMA,PRICE_CLOSE);      
            int s_iMa_Handle = iMA(pair.names[i], pair.periods[j],slow_iMa,0,MODE_SMA,PRICE_CLOSE);
            if(CopyBuffer(f_iMa_Handle, 0, 0, y, f_Ma)<=0) Print("Error copying fast Ma ",GetLastError());
            if(CopyBuffer(s_iMa_Handle, 1, 0, y, s_Ma)<=0) Print("Error copying slow Ma ",GetLastError());

[samjesse]

If I switch the order of the last 2 lines, the last code line will produce the error copying 4806.

[Enivid]

It takes some time for MetaTrader to produce the indicator data, so it's possible that the second indicator isn't ready when you call CopyBuffer(). Try to catch that error and re-do copying until it disappears.

[samjesse]

It takes some time for MetaTrader to produce

so the code that needs to be repeated would be the code that has to do with producing the indicator? i.e.

int s_iMa_Handle = iMA(pair.names[i], pair.periods[j],slow_iMa,0,MODE_SMA,PRICE_CLOSE);

re-do copying until it disappears.

you mean this code?

if(CopyBuffer(s_iMa_Handle, 1, 0, y, s_Ma)<=0) Print("Error copying slow Ma ",GetLastError());

why redo the copy if the problem in the "produce" code that takes time?

[Enivid]

When you call iMA(), it takes some time for MetaTrader to calculate all the data. Until it's done, the handle doesn't point at the correct data.

There is no point in calling iMA() more than one time, as it will probably trigger MetaTrader to restart indicator calculation.

The point is to call CopyBuffer() in a cycle until it stops returning an error.

[samjesse]

is the following OK. if loops for a long time and I stop it manually.

           int f_iMa_Handle = iMA(nm, tf,fast_iMa,0,MODE_SMA,PRICE_CLOSE);      
            while(CopyBuffer(f_iMa_Handle, 0, 0, y, f_Ma)<=0) Print("Error copying fast Ma ",GetLastError());
            int s_iMa_Handle = iMA(nm, tf,slow_iMa,0,MODE_SMA,PRICE_CLOSE);
            while(CopyBuffer(s_iMa_Handle, 1, 0, y, s_Ma)<=0) Print("Error copying slow Ma ",GetLastError());

[Enivid]

Looks OK. Do you also check the handles for INVALID_HANDLE value? Maybe the problem is there.

[samjesse]

Originally Posted by Enivid

Looks OK. Do you also check the handles for INVALID_HANDLE value? Maybe the problem is there.

not sure how to do that. Also searched the docs "runtme errors" for INVALID_HANDLE for no avail.

[Enivid]

Something like this:

int f_iMa_Handle = iMA(nm, tf,fast_iMa,0,MODE_SMA,PRICE_CLOSE);
if (f_iMa_Handle == INVALID_HANDLE) Print("Error creating indicator handle);