Вопросы от начинающих MQL4 MT4 MetaTrader 4 - страница 176

 

Ребята, помогите исправить советника. Вместо проверки ордеров, он закупается без остановки,при первом же сигнале, пока не потратит весь депозит. Заранее всех благодарю

//+------------------------------------------------------------------+
//|                                                      ProjectName |
//|                                      Copyright 2012, CompanyName |
//|                                       http://www.companyname.net |
//+------------------------------------------------------------------+
#property copyright "Copyright 2019, MetaQuotes Software Corp."
#property link      "https://www.mql5.com"
#property version   "1.00"
//+------------------------------------------------------------------+
extern double Lots            = 1;
extern int    TakeProfit      = 300;
extern int    StopLoss        = 50;
extern int    Magic           = 111;
extern int    Slippage        = 3;
//+------------------------------------------------------------------+
extern string TMA             = "Параметры индикатора TMA";
extern string TimeFrame       = "current time frame";
extern int    HalfLength      = 56;
extern int    Price           = PRICE_CLOSE;
extern double ATRMultiplier   = 2.0;
extern int    ATRPeriod       = 100;
extern bool   Interpolate     = true;
//+------------------------------------------------------------------+
double PriceHigh,PriceLow,SL,TP,OrderMagicNumber;
int ticket;
//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+
int OnInit()
  {
   if(Digits==3 || Digits==5)
     {
      TakeProfit *= 10;
      StopLoss   *= 10;
      Slippage   *= 10;
     }
   return(INIT_SUCCEEDED);
  }
//+------------------------------------------------------------------+
//| Expert deinitialization function                                 |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
  {
  }
//+------------------------------------------------------------------+
//| Expert tick function                                             |
//+------------------------------------------------------------------+
void OnTick()
  {
   PriceHigh= iCustom(NULL,0,"TMA_Fair",TimeFrame,HalfLength,Price,ATRMultiplier,ATRPeriod,Interpolate,1,0);
   PriceLow = iCustom(NULL,0,"TMA_Fair",TimeFrame,HalfLength,Price,ATRMultiplier,ATRPeriod,Interpolate,2,0);

   if(CountSell()==0 && Bid>=PriceHigh)
     {
      ticket=OrderSend(Symbol(),OP_SELL,Lots,Bid,Slippage,0,0,"ТМА",Magic,0,Red);
      if(ticket>0)
        {
         SL = NormalizeDouble(Bid + StopLoss*Point, Digits);
         TP = NormalizeDouble(Bid - TakeProfit*Point, Digits);
         if(OrderSelect(ticket,SELECT_BY_TICKET))
            if(!OrderModify(ticket,OrderOpenPrice(),SL,TP,0))
               Print("Ошибка модификации ордера на продажу");
        }
      else Print("Ошибка открытия ордера на продажу");
     }
   if(CountBuy()==0 && Ask<=PriceLow)
     {
      ticket=OrderSend(Symbol(),OP_BUY,Lots,Ask,Slippage,0,0,"ТМА",Magic,0,Blue);
      if(ticket>0)
        {
         TP = NormalizeDouble(Ask + TakeProfit*Point, Digits);
         SL = NormalizeDouble(Ask - StopLoss*Point, Digits);
         if(OrderSelect(ticket,SELECT_BY_TICKET))
            if(!OrderModify(ticket,OrderOpenPrice(),SL,TP,0))
               Print("Ошибка модификации ордера на покупку");
        }
      else Print("Ошибка открытия ордера на покупку");
     }
   if(Ask<=PriceLow && CountSell()>0)
     {
      for(int i=OrdersTotal() -1; i>=0; i--)
        {
         if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
           {
            if(OrderMagicNumber()==Magic && OrderType()==OP_SELL)
               if(!OrderClose(OrderTicket(),OrderLots(),Ask,Slippage,Black))
                  Print("Ошибка закрытия ордера на продажу");
           }
        }
     }
   if(Bid>=PriceHigh && CountBuy()>0)
     {
      for(i=OrdersTotal() -1; i>=0; i--)
        {
         if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
           {
            if(OrderMagicNumber()==Magic && OrderType()==OP_BUY)
               if(!OrderClose(OrderTicket(),OrderLots(),Bid,Slippage,Black))
                  Print("Ошибка закрытия ордера на покупку");
           }
        }
     }
  }
//+------------------------------------------------------------------+
int CountSell()
  {
   int count=0;
   for(int trade=OrdersTotal()-1; trade>=0; trade--)
     {
      if(OrderSelect(trade,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber && Magic && OrderType()==OP_SELL)
            count++;
        }
     }
   return(count);
  }
//+------------------------------------------------------------------+
int CountBuy()
  {
   int count=0;
   for(int trade=OrdersTotal()-1; trade>=0; trade--)
     {
      if(OrderSelect(trade,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber && Magic && OrderType()==OP_BUY)
            count++;
        }
     }
//+------------------------------------------------------------------+
 
Zelimhannahal00:

Ребята, помогите исправить советника. Вместо проверки ордеров, он закупается без остановки,при первом же сигнале, пока не потратит весь депозит. Заранее всех благодарю

Код можно нормально отформатировать прямо в редакторе кода, используя стилизатор кода: Ctrl + <

Я его отформатировал за вас - теперь он нормально читаемый.

 
Zelimhannahal00:

Ребята, помогите исправить советника. Вместо проверки ордеров, он закупается без остановки,при первом же сигнале, пока не потратит весь депозит. Заранее всех благодарю

Вы правильно определили, ошибка в функциях проверки позиций

//+------------------------------------------------------------------+
int CountSell()
  {
   int count=0;
   for(int trade=OrdersTotal()-1; trade>=0; trade--)
     {
      if(OrderSelect(trade,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber() == Magic && OrderType()==OP_SELL)
            count++;
        }
     }

   return(count);
  }
//+------------------------------------------------------------------+
int CountBuy()
  {
   int count=0;
   for(int trade=OrdersTotal()-1; trade>=0; trade--)
     {
      if(OrderSelect(trade,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber() == Magic && OrderType()==OP_BUY)
            count++;
        }
     }
//+----------------------

красным выделил, исправлено

Как то так.

 
Alekseu Fedotov:

Вы правильно определили, ошибка в функциях проверки позиций

красным выделил, исправлено

Как то так.

Исправил, всё так же. Проблема всё ещё актуальна
 
Zelimhannahal00:
Исправил, всё так же. Проблема всё ещё актуальна
//+------------------------------------------------------------------+
int CountSell()
  {
   int count=0;
   for(int trade=OrdersTotal()-1; trade>=0; trade--)
     {
      if(OrderSelect(trade,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber() == Magic && OrderType()==OP_SELL)
            count++;
        }
     }
   return(count);
  }
//+------------------------------------------------------------------+
int CountBuy()
  {
   int count=0;
   for(int trade=OrdersTotal()-1; trade>=0; trade--)
     {
      if(OrderSelect(trade,SELECT_BY_POS,MODE_TRADES))
        {
         if(OrderSymbol()==Symbol() && OrderMagicNumber() == Magic && OrderType()==OP_BUY)
            count++;
        }
     }
return(count);


там весь код скопировали? 

 
Vladislav Andruschenko:


там весь код скопировали?

Нечаянно не весь код скопировал. Там все нормально return(count);

}

Так заканчивается код, как положенно. При компиляции вообще нету ошибок и предупреждений. Походу в функциях где то я ошибся, не могу понять где именно. Как подсказали сверху,исправил. Так же нету ошибок, но сам советник работает так же(при первом сигнале, закупает на весь депозит, задаными лотами, например 1 лот выставляю и он закупает по 1 лоту сделок 200 и больше, на сколько хватает депозита. А так сигнал выдаёт правильно, покупает тоже правильно, только много)

 
Zelimhannahal00:

Нечаянно не весь код скопировал. Там все нормально return(count);

}

Так заканчивается код, как положенно. При компиляции вообще нету ошибок и предупреждений. Походу в функциях где то я ошибся, не могу понять где именно. Как подсказали сверху,исправил. Так же нету ошибок, но сам советник работает так же(при первом сигнале, закупает на весь депозит, задаными лотами, например 1 лот выставляю и он закупает по 1 лоту сделок 200 и больше, на сколько хватает депозита. А так сигнал выдаёт правильно, покупает тоже правильно, только много)

Проверил,исправил,работает.

Файлы:
ProjectName.mq4  11 kb
 
Как найти количество свечей между датами? 
 
Alexander Fedosov:
Как найти количество свечей между датами? 

если ТФ синхронизирован то любая CopyXXXX() функция :

Обращение по начальной и конечной датам требуемого интервала времени

int  CopyTime(
   string           symbol_name,     // имя символа
   ENUM_TIMEFRAMES  timeframe,       // период
   datetime         start_time,      // с какой даты
   datetime         stop_time,       // по какую дату
   datetime         time_array[]     // массив для копирования времени открытия
   );

Возвращаемое значение

Количество скопированных элементов массива либо -1 в случае ошибки.


можно с помощью  iBarShift() найти 2 бара по времени 


 
Igor Makanu:

если ТФ синхронизирован то любая CopyXXXX() функция :

Обращение по начальной и конечной датам требуемого интервала времени

int  CopyTime(
   string           symbol_name,     // имя символа
   ENUM_TIMEFRAMES  timeframe,       // период
   datetime         start_time,      // с какой даты
   datetime         stop_time,       // по какую дату
   datetime         time_array[]     // массив для копирования времени открытия
   );

Возвращаемое значение

Количество скопированных элементов массива либо -1 в случае ошибки.


Странно.. Не помню чтобы раньше были эти функции в мт4. Их что и пятёрки добавили? 
Причина обращения: