Советник перестал закрывать ордера

Vasyl Kulyk 2014.01.26 18:03

Есть у меня советник сделанный из MACD Sample, открывает ордер по условию, и закрывает - по условию.

Открытие ордеров в советнике я отключил - в условие для открытия ордера внес ключ 1 или 0 и использовал советник только для закрытия открытых ордеров - советник работал, меня все устраивало...

В данный момент советник ордера открытые вручную или другими советниками не закрывает - ошибок в журнале Эксперты также нет.

С месяц назад советник перестал закрывать открытые вручную или другими советниками ордера, я проверил на тестере стратегий - если включить ключ - то советник исправно ордера открывает и закрывает.

Прошу подсказать как мне сделать что бы закрывались ордера

Файлы:

bfpwezlroqngjvt.mq4 7 kb

Ahmad Mehdiyev 2014.01.26 18:25 #1

Вот часть вашего кода :

   for(cnt=0;cnt<total;cnt++)
     {
      OrderSelect(cnt, SELECT_BY_POS, MODE_TRADES);
      if(OrderType()<=OP_SELL &&   // check for opened position 
         OrderSymbol()==Symbol())  // check for symbol
        {
         if(OrderType()==OP_BUY)   // long position is opened
           {buy=1;}
        

         else // go to short position
           {sell=1;}
      
        }
     }

А вот как я бы его переписал. Далее по коду внесите те же измения и возможно эксперт не будет пропускать момент закрытия.

   for(cnt=total-1;cnt>=0;cnt--)
      {
      if (OrderSelect(cnt, SELECT_BY_POS, MODE_TRADES)) 
         {
         if(OrderSymbol()==Symbol())  // check for symbol
            {
            if(OrderType()==OP_BUY)   // long position is opened
              {buy=1;}
            if(OrderType()==OP_SELL) // go to short position
              {sell=1;}
            }
         }
      }

Vasyl Kulyk 2014.01.26 19:11 #2

Не помогло,

взял классический MACD Sample, не хочет закрывать открытые другими ордера

Файлы:

macdnsample.mq4 6 kb

Алексей Тарабанов 2014.01.26 19:42 #3

fartoviy_kent:

Не помогло,

взял классический MACD Sample, не хочет закрывать открытые другими ордера

Что не помогло? - Вы ничего не изменили в логике, а она некорректна.

Alekseu Fedotov 2014.01.26 19:51 #4

fartoviy_kent:

Не помогло,

взял классический MACD Sample, не хочет закрывать открытые другими ордера

Пробуйте

//+------------------------------------------------------------------+
//|                                                  MACD Sample.mq4 |
//|                      Copyright © 2005, MetaQuotes Software Corp. |
//|                                       http://www.metaquotes.net/ |
//+------------------------------------------------------------------+

extern double TakeProfit = 120000;
extern double Lots = 0.1;
extern double TrailingStop = 999999;
extern double MACDOpenLevel=3;
extern double MACDCloseLevel=2;
extern double MATrendPeriod=26;

//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
int start()
  {
   double MacdCurrent, MacdPrevious, SignalCurrent;
   double SignalPrevious, MaCurrent, MaPrevious;
   int cnt, ticket, total,a;
// initial data checks
// it is important to make sure that the expert works with a normal
// chart and the user did not make any mistakes setting external 
// variables (Lots, StopLoss, TakeProfit, 
// TrailingStop) in our case, we check TakeProfit
// on a chart of less than 100 bars
   if(Bars<100)
     {
      Print("bars less than 100");
      return(0);  
     }
   if(TakeProfit<10)
     {
      Print("TakeProfit less than 10");
      return(0);  // check TakeProfit
     }
// to simplify the coding and speed up access
// data are put into internal variables
   MacdCurrent=iMACD(NULL,0,12,26,9,PRICE_CLOSE,MODE_MAIN,0);
   MacdPrevious=iMACD(NULL,0,12,26,9,PRICE_CLOSE,MODE_MAIN,1);
   SignalCurrent=iMACD(NULL,0,12,26,9,PRICE_CLOSE,MODE_SIGNAL,0);
   SignalPrevious=iMACD(NULL,0,12,26,9,PRICE_CLOSE,MODE_SIGNAL,1);
   MaCurrent=iMA(NULL,0,MATrendPeriod,0,MODE_EMA,PRICE_CLOSE,0);
   MaPrevious=iMA(NULL,0,MATrendPeriod,0,MODE_EMA,PRICE_CLOSE,1);
   a=0;

   total=OrdersTotal();
   if(total<2) 
     {
      // no opened orders identified
      if(AccountFreeMargin()<(0.1*Lots))
        {
         Print("We have no money. Free Margin = ", AccountFreeMargin());
         return(0);  
        }
      // check for long position (BUY) possibility
      if(a==1 && MacdPrevious<0 && MacdCurrent>0)
        {
         ticket=OrderSend(Symbol(),OP_BUY,Lots,Ask,3,0,Ask+TakeProfit*Point,"macd sample",16384,0,Green);
         if(ticket>0)
           {
            if(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES)) Print("BUY order opened : ",OrderOpenPrice());
           }
         else Print("Error opening BUY order : ",GetLastError()); 
         return(0); 
        }
      // check for short position (SELL) possibility
      if(a==1 && MacdPrevious>0 && MacdCurrent<0)
        {
         ticket=OrderSend(Symbol(),OP_SELL,Lots,Bid,3,0,Bid-TakeProfit*Point,"macd sample",16384,0,Red);
         if(ticket>0)
           {
            if(OrderSelect(ticket,SELECT_BY_TICKET,MODE_TRADES)) Print("SELL order opened : ",OrderOpenPrice());
           }
         else Print("Error opening SELL order : ",GetLastError()); 
         return(0); 
        }
      return(0);
     }
   // it is important to enter the market correctly, 
   // but it is more important to exit it correctly... 
  
    for(cnt=total-1;cnt>=0;cnt--)
     {
      OrderSelect(cnt, SELECT_BY_POS, MODE_TRADES);
      if(OrderType()<=OP_SELL &&   // check for opened position 
         OrderSymbol()==Symbol())  // check for symbol
        {
         if(OrderType()==OP_BUY)   // long position is opened
           {
            // should it be closed?
            if(SignalPrevious<0 && SignalCurrent>0)
                {
                 OrderClose(OrderTicket(),OrderLots(),Bid,3,Violet); // close position
                 return(0); // exit
                }
            // check for trailing stop
            if(TrailingStop>0)  
              {                 
               if(Bid-OrderOpenPrice()>Point*TrailingStop)
                 {
                  if(OrderStopLoss()<Bid-Point*TrailingStop)
                    {
                     OrderModify(OrderTicket(),OrderOpenPrice(),Bid-Point*TrailingStop,OrderTakeProfit(),0,Green);
                     return(0);
                    }
                 }
              }
           }
         else // go to short position
           {
            // should it be closed?
            if(SignalPrevious>0 && SignalCurrent<0)
              {
               OrderClose(OrderTicket(),OrderLots(),Ask,3,Violet); // close position
               return(0); // exit
              }
            // check for trailing stop
            if(TrailingStop>0)  
              {                 
               if((OrderOpenPrice()-Ask)>(Point*TrailingStop))
                 {
                  if((OrderStopLoss()>(Ask+Point*TrailingStop)) || (OrderStopLoss()==0))
                    {
                     OrderModify(OrderTicket(),OrderOpenPrice(),Ask+Point*TrailingStop,OrderTakeProfit(),0,Red);
                     return(0);
                    }
                 }
              }
           }
        }
     }
   return(0);
  }
// the end.

Alexander 2014.01.27 05:02 #5

pro_:

Вот часть вашего кода :

А я бы, вообще его убрал, он ни на что не влияет.)))

Ну, то, что не открывает, ясно, у вас переменная а всегда равна нулю, а для открытия требуется единица.

Длинные ордера должен закрывать, а закрытие коротких было с ошибкой, вам уже указали,

Типы ордеров - Принципы Типы ордеров - Принципы Расчет маржи: Биржевая модель

Ahmad Mehdiyev 2014.01.27 09:22 #6

Убрать действительно можно. Но скорректировать именно этот кусок кода было самым легким и наглядным способом указать на одну явную ошибку и на более правильный (на мой взгляд) способ проверки наличия открытых бай и селл. А далее по коду это уже сам пользователь должен исправлять. Он же научиться хочет и понять. Или нет?

Дополнительная защита программ (MQL5 Открытие и закрытие позиций Миграция - Виртуальный хостинг

Vasyl Kulyk 2014.01.27 10:06 #7

вот так работает (еще необходимо, наверно, хорошо проверить), условия те же, циклы те же...

Файлы:

afksryldoizrxue.mq4 3 kb

Alexander 2014.01.27 14:03 #8

pro_:
Убрать действительно можно. Но скорректировать именно этот кусок кода было самым легким и наглядным способом указать на одну явную ошибку и на более правильный (на мой взгляд) способ проверки наличия открытых бай и селл. А далее по коду это уже сам пользователь должен исправлять. Он же научиться хочет и понять. Или нет?

В данном случае Ваш взгляд ошибся, нет никакой разницы каким образом перебирать ордера, удаления ведь нет.

Ahmad Mehdiyev 2014.01.27 15:11 #9

Спорить все равно бессмысленно. У каждого свой взгляд и свой опыт. Удачи.

Новый комментарий