New bar event in EA

http://docs.mql4.com/series/itime

Why not use iTime("EURUSD",PERIOD_M1,0)

p.ej.

si(New_Time_M1 != iTime("EURUSD",PERIOD_M1,0)) // Compare time --> new bar is born
{
New_Time_M1=iTime("EURUSD",PERIOD_M1,0); // New time set
New_Bar_M1=true; // A new bar detected

}

Thanks for this I am trying with custom timerames, if I set it up for a 3 hour candle for example eg 180 minutos. When does this time start to count? eg form midnight every 3 hours or does it depend when the indicatoe is placed on the chart?

thanks again

Hola.

thx for this ideas, ur right.

1. Accessing the arrays is more faster than use switch - case element. (in this example 4 veces), some test:

  start = GetTickCount();

  para(test=0; test<10000000; test++)
   para(i=0; i<MAX; i++)
    int value1 = utils.periodIndexToPeriod[i];

  registro("Elapsed : " + (GetTickCount()-comenzar)); // 2100 ms

  start = GetTickCount();

  para(test=0; test<10000000; test++)
   para(i=0; i<MAX; i++)
    int value2 = utils2.periodIndexToPeriod(i);

  registro("Elapsed : " + (GetTickCount()-comenzar)); // 8000 EM

but how accsess this 10.000.000*MAX in a row? 😀 and i think much more readble switch - case options beacuse it is more similar the other rutins. Easily readable by someone who not a coder.

2. use local variable in loop save some times but not so much i think so test it: in half year backtest, where do some loop in onBar event, ur code done in 6006ms, my 5990ms on ~ 72Kbar event so its same

3. Tiempo[0] / iTime(NULO,0,0) does not affect the result.

So i change the loop code for ur versions (more readable), but not change the array access rutin. btw thx a lot, i swear i will not publish a code, después 2 soy, when i am too tired 😛

1. veces[curIndex]-(MathMod(veces[curIndex]/60,utils.periodIndexToPeriod(i)))*60

   for efficiency and readability, simplify

   veces[curIndex]- MathMod(veces[curIndex],utils.periodIndexToPeriod(i)*60);

   ---

2. for efficiency and readability and to avoid unnecessary function calls, I'd replace the functions with an array and save results

   int utils.periodIndexToPeriod   = {
       PERIOD_M1,  PERIOD_M5,  PERIOD_M15, PERIOD_M30, PERIOD_H1,  PERIODO_H4,
       PERIOD_D1,  PERIOD_W1,  PERIOD_MN1, 20,         50  };
   
   para(int i=curIndex+1; i<MAX; i++)
       int period  = utils.periodIndexToPeriod[i];
           seconds = period*60,
           time0   = times[curIndex] - MathMod(veces[curIndex],segundos);
       si (veces[i] != time0) {
           veces[i] = time0;
           onBar(período);
   }   }

   ---

3. And avoid unnecessary function calls with the simpler

   veces[curIndex] = Time[0]; // = iTime(NULO,0,0);