Новое барное событие в EA

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

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

например.

если(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 минуты. 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

Привет.

thx for this ideas, ur right.

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

  start = GetTickCount();

  для(test=0; тест<10000000; test++)
   для(я=0; я<MAX; я++)
    int value1 = utils.periodIndexToPeriod[я];

  log("Elapsed : " + (GetTickCount()-начинать)); // 2100 ms

  start = GetTickCount();

  для(test=0; тест<10000000; test++)
   для(я=0; я<MAX; я++)
    int value2 = utils2.periodIndexToPeriod(я);

  log("Elapsed : " + (GetTickCount()-начинать)); // 8000 РС

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. Время[0] / iTime(НУЛЕВОЙ,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, после 2 являюсь, when i am too tired 😛

1. раз[curIndex]-(MathMod(раз[curIndex]/60,utils.periodIndexToPeriod(я)))*60

   for efficiency and readability, simplify

   раз[curIndex]- MathMod(раз[curIndex],utils.periodIndexToPeriod(я)*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,  PERIOD_H4,
       PERIOD_D1,  PERIOD_W1,  PERIOD_MN1, 20,         50  };
   
   для(int i=curIndex+1; я<MAX; я++)
       int period  = utils.periodIndexToPeriod[я];
           seconds = period*60,
           time0   = times[curIndex] - MathMod(раз[curIndex],секунды);
       если (раз[я] != time0) {
           раз[я] = time0;
           onBar(период);
   }   }

   ---

3. And avoid unnecessary function calls with the simpler

   раз[curIndex] = Время[0]; // = iTime(НУЛЕВОЙ,0,0);