Задержка в четверть секунды может пропустить первый тик - Статьи и техническая библиотека по автоматическому трейдингу

Alexey Viktorov 2022.06.29 10:04 #21

Владимир, вот код

/********************************************************************\
|   Expert tick function                                             |
\********************************************************************/
void OnTick()
 {
//---
  Sleep(250);
  if(iOpen(_Symbol, PERIOD_CURRENT, 0) && iVolume(_Symbol, PERIOD_CURRENT, 0) <= 1)
    Print("Ура! Появился новый бар! ",TimeTradeServer());
  datetime tm = iTime(_Symbol, PERIOD_CURRENT, 0);
  static datetime time = tm;
  if(time == tm)
    return;
  long volume = iVolume(_Symbol, PERIOD_CURRENT, 0);
  if(volume > 1)
    Print("***** Вот и ошибка? volume = ", volume);
  time = tm;
 }/******************************************************************/

и вот результат работы

2022.06.29 12:51:55.339 Test_NewBar (GBPUSD,M1) ***** Вот и ошибка? volume = 2
2022.06.29 12:52:57.520 Test_NewBar (GBPUSD,M1) ***** Вот и ошибка? volume = 3
2022.06.29 12:53:55.377 Test_NewBar (GBPUSD,M1) ***** Вот и ошибка? volume = 2
2022.06.29 12:54:55.449 Test_NewBar (GBPUSD,M1) ***** Вот и ошибка? volume = 3
2022.06.29 12:55:55.417 Test_NewBar (GBPUSD,M1) Ура! Появился новый бар! 2022.06.29 12:55:55
2022.06.29 12:56:55.444 Test_NewBar (GBPUSD,M1) ***** Вот и ошибка? volume = 3
2022.06.29 12:57:55.458 Test_NewBar (GBPUSD,M1) Ура! Появился новый бар! 2022.06.29 12:57:55
2022.06.29 12:58:55.450 Test_NewBar (GBPUSD,M1) Ура! Появился новый бар! 2022.06.29 12:58:55
2022.06.29 12:59:55.656 Test_NewBar (GBPUSD,M1) ***** Вот и ошибка? volume = 2

Конечно задержка в четверть секунды многовато, но ведь мне надо было показать, что задержка может пропустить первый тик, а задержка может произойти в самый неожиданный момент…

Nikolai Semko 2022.06.29 12:56 #22

Yuriy Bykov #:

С этим можно легко столкнуться в мультивалютном советнике, который работает на графике одной валютной пары. Советник в OnTick() запускает цикл по нескольким парам. Допустим, по первой паре наступил новый бар и советник решил открыть по ней позицию или несколько отложенных ордеров. На это с учетом возможных задержек может уйти несколько секунд. За эти секунды по второй паре уже пришло два тика, и когда советник доберется в цикле до обработки второй пары, то ваше условие нового бара уже не будет выполнено. Новый бар для второй пары будет пропущен.

Поэтому я для себя реализовывал более сложный код для определения нового бара, в основе которого примерно та же идея, как и в этом ответе, только вместо iTime используется iVolume.

Да не нужно здесь сложного кода.

Все элементарно и очевидно.

В баре есть только один уникальный атрибут - это время открытия. Это его PK(Primary Key). Поэтому через него и надо идентифицировать бар и реализовывать функцию IsNewBar(). Эта функция состоит из 4 строк кода:

Статическая переменная time_open_last_bar = 0
получение time_open_cur_bar через iTime
сравнивание time_open_cur_bar с time_open_last_bar
time_open_last_bar = time_open_cur_b

Volume не уникален для бара и поэтому не годится для его идентификации.

Все просто.

[АРХИВ]Любой вопрос новичка, чтоб Разговор с искусственным интеллектом Мт4 Конец поддержке.

Nikolai Semko 2022.06.29 16:06 #23

Реализация.
Для текущего ТФ и символа сгодится такая функция:

bool IsNewBar() {
   static datetime last_bar_time_open = 0;
   datetime cur_bar_time_open = iTime(NULL, 0, 0);
   bool result = cur_bar_time_open != last_bar_time_open;
   last_bar_time_open = cur_bar_time_open;
   return result;
}

для мультивалютных сов и с разными ТФ лучше реализовать через класс, когда для каждого символа и (или) ТФ нужен свой экземпляр класса.
Например так:

class CIsNewBar {
private:
   datetime          last_bar_time_open;
   ENUM_TIMEFRAMES   tf;
   string            symbol;
public:
                     CIsNewBar(ENUM_TIMEFRAMES _tf = 0, string _symbol = NULL) {
      symbol = _symbol;
      tf = _tf;
      last_bar_time_open = iTime(symbol, tf, 0);
   };
                    ~CIsNewBar();
   bool              IsNewBar() {
      datetime cur_bar_time_open = iTime(symbol, tf, 0);
      bool result = cur_bar_time_open != last_bar_time_open;
      last_bar_time_open = cur_bar_time_open;
      return result;
   }
};

код не проверял. Вдруг где-то накосячил. Хотя вряд ли. ))

Yuriy Bykov 2022.06.29 16:37 #24

Nikolai Semko #:

Реализация.

код не проверял. Вдруг где-то накосячил. Хотя вряд ли. ))

А если в одном вызове OnTick() надо проверить в нескольких местах событие наступления нового бара? Ваш код, похоже, позволяет сделать это только один раз. После этого IsNewBar() будет возвращать false внутри того же вызова OnTick().

Если же такого не требуется, то ваш код вполне достаточен.
А если добавить еще второй класс, который будет занимается централизованным хранением объектов CIsNewBar для разных символов и таймфреймов, чтобы можно было одной функцией определять новый бар для разных комбинаций, то объем кода еще увеличится.

P.S. Посмотрел, что я потом переделал немного реализацию, сделав ее одинаковой для MT5/MT4, избавившись от CHashMap и упростив.

Работа советника внутри эксперта Любые вопросы новичков по MetaTrader 5 Strategy Tester!

Nikolai Semko 2022.06.29 16:50 #25

Yuriy Bykov #:

А если в одном вызове OnTick() надо проверить в нескольких местах событие наступления нового бара? Ваш код, похоже, позволяет сделать это только один раз. После этого IsNewBar() будет возвращать false внутри того же вызова OnTick().

Как это так - в нескольких местах?
Идентификация нового бара должна происходить в OnTick и обработка этого события должна происходить в одном месте. Иначе нужно менять алгоритм.
Ну если никак не обойти этого (не представляю ситуации - когда это может быть нужно), то создавайте несколько экземпляров класса с одними и теми же параметрами TF и Symbol - для каждого места свой.

ЗЫ: заглянул в ваш код краем глаза и увидел там циклы и поиск символа. Явный перебор с алгоритмом. Какой же там перфоманс?! Жесть!

Заметил глюк: открывается 2 Любые вопросы новичков по Вопрос о функции OnTradeTransaction

MrBrooklin 2022.06.29 17:19 #26

Alexey Viktorov #:

Владимир, вот код

и вот результат работы

Конечно задержка в четверть секунды многовато, но ведь мне надо было показать, что задержка может пропустить первый тик, а задержка может произойти в самый неожиданный момент…

Добрый вечер, Алексей! Задержку и мне приходилось ставить, чтобы сымитировать вне штатную ситуацию. У меня правда была 900 мск. Видел такие пропуски по несколько минут на таймфрейме М1. В общем подвожу результат всего сказанного в обсуждении:

Решение по определению появления нового бара не верное - согласен. Но меня устраивает.
В однострочном коде реализовать условие для идеального определения появления нового бара - не возможно. Да и бог с ним!
Спасибо всем за активное участие в дискуссии!!!

С уважением ко всем, Владимир.

Любые вопросы новичков по Замки и ключи к Самообучение языку MQL5 с

Alexey Viktorov 2022.06.29 17:40 #27

Nikolai Semko #:

Как это так - в нескольких местах?
Идентификация нового бара должна происходить в OnTick и обработка этого события должна происходить в одном месте. Иначе нужно менять алгоритм.
Ну если никак не обойти этого (не представляю ситуации - когда это может быть нужно), то создавайте несколько экземпляров класса с одними и теми же параметрами TF и Symbol - для каждого места свой.

ЗЫ: заглянул в ваш код краем глаза и увидел там циклы и поиск символа. Явный перебор с алгоритмом. Какой же там перфоманс?! Жесть!

Зачем??? Разве не достаточно объявить bool переменную и ей присвоить значение функции IsNewBar? И потом во всех местах где нужно только проверить значение этой переменной.

НО!!! Артём Тришкин писал мне класс для мультивалютников и где-то я уже давал этот класс для применения и обсуждения.

Пожелания к синтаксису языка Вопросы по ООП в Ошибки, баги, вопросы

Nikolai Semko 2022.06.29 18:05 #28

Alexey Viktorov #:

Зачем??? Разве не достаточно объявить bool переменную и ей присвоить значение функции IsNewBar? И потом во всех местах где нужно только проверить значение этой переменной.

НО!!! Артём Тришкин писал мне класс для мультивалютников и где-то я уже давал этот класс для применения и обсуждения.

Да, можно и так.

Причем, лучше чтобы она(переменная) находилась внутри класса CIsNewBar, была бы паблик и выставлялась бы внутри метода IsNewBar() значением result, а метод IsNewBar() в таком случае может вызываться только из OnTick и поэтому логичнее его переименовать в NewTick или OnTick, а саму переменную в IsNewBar.

class CIsNewBar {
private:
   datetime          last_bar_time_open;
   ENUM_TIMEFRAMES   tf;
   string            symbol;
public:
                     CIsNewBar(ENUM_TIMEFRAMES _tf = 0, string _symbol = NULL) {
      symbol = _symbol;
      tf = _tf;
      last_bar_time_open = iTime(symbol, tf, 0);
      IsNewBar = false;
   }
                    ~CIsNewBar();
   bool              OnTick() {
      datetime cur_bar_time_open = iTime(symbol, tf, 0);
      IsNewBar = cur_bar_time_open != last_bar_time_open;
      last_bar_time_open = cur_bar_time_open;
      return IsNewBar;
   }
   bool IsNewBar;
};

OnTick() не работает в Новый бар в мултивалютном Ошибки, баги, вопросы

Petr Zharuk 2022.06.29 21:49 #29

Доброго времени, коллеги!

Я пишу код только месяц. Но хочу поделится своим способом определения нового бара.

datetime LastTime;
datetime NowTime;

void OnTick()
{

        NowTime = iTime(NULL, PERIOD_CURRENT, 0);//Получаем текущее время

        if(LastTime != NowTime) //Если время предыдущего действия не равно текущему времени, значит это новая свеча.
         {
        ///
                Тело функции выполняемое один раз за свечу...
        ///
         }
        LastTime = NowTime;

Как по мне это самый коротки вариант.

Подводных камней пока не встретил.

От гуру послушал бы рекомендаций.

Предложения для эксперта (от Синхронизация баров H4 и вопрос по MQL4

Nikolai Semko 2022.06.29 22:02 #30

Petr Zharuk #:

Доброго времени, коллеги!

Я пишу код только месяц. Но хочу поделится своим способом определения нового бара.

Как по мне это самый коротки вариант.

Подводных камней пока не встретил.

От гуру послушал бы рекомендаций.

ровно тоже самое, что я давал здесь (вариант функции - когда используется только текущий ТФ и текущий символ), только с глобальной переменной, а не статической, и не в виде функции, а в теле OnTick.
Статическая переменная по сути тоже самое, что и глобальная, только удобнее, т.к. не "засоряет" шапку и зона видимости только в пределах функции, что безопаснее.
Использование функции в данной ситуации только лишь улучшает читаемость кода, но можно конечно же обойтись и без функции. На скорость не влияет.

void OnTick()
{
        static datetime LastTime = 0;
        datetime NowTime = iTime(NULL, PERIOD_CURRENT, 0);//Получаем текущее время

        if(LastTime != NowTime) //Если время предыдущего действия не равно текущему времени, значит это новая свеча.
         {
        ///
                Тело функции выполняемое один раз за свечу...
        ///
         }
        LastTime = NowTime;

Любые вопросы новичков по Обсуждение статьи "Как использовать Как кодировать?

Советники: New_Bar (условие, определяющее появление нового бара, реализованно в виде микросоветника) - страница 3