Работает малость не так. Помогите уважаемые форумчане.

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

Добрый день.

Решил заняться программированием, вот первые труды.

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

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

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

extern double TakeProfit = 50;
extern double Lots = 0.1;
extern double TrailingStop = 30;
extern int    risk   = 10;
double  LOT;


//+------------------------------------------------------------------+
//|                                                                  |
//+------------------------------------------------------------------+
int start()
  {
   double UP,LOW,Z, Z1, S, S1;
   double SignalPrevious, MaCurrent, MaPrevious;
   int cnt, ticket, total;
// 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

UP=iCustom(Symbol(),0,"MA in Color_wAppliedPrice",12, 3, 6, 1,0);
LOW=iCustom(Symbol(),0,"MA in Color_wAppliedPrice",12, 3, 5, 2,0);

   total=OrdersTotal();
   if(total<1) 
     {
      // no opened orders identified
      if(AccountFreeMargin()<(10*Lots))
        {
         Print("We have no money. Free Margin = ", AccountFreeMargin());
         return(0);  
        }
      // check for long position (BUY) possibility
      if(LOW>UP)
        {LOT = LOT(risk,1);
         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(UP>LOW )
        {LOT = LOT(risk,1);
         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=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
           {
            // should it be closed?
            if(UP>LOW)
                {
                 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(LOW>UP)
              {
               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.

double LOT(int risk,int ord)
{
   double MINLOT = MarketInfo(Symbol(),MODE_MINLOT);
   LOT = AccountFreeMargin()*risk/100/MarketInfo(Symbol(),MODE_MARGINREQUIRED)/ord;
   if (LOT>MarketInfo(Symbol(),MODE_MAXLOT)) LOT = MarketInfo(Symbol(),MODE_MAXLOT);
   if (LOT<MINLOT) LOT = MINLOT;
   if (MINLOT<0.1) LOT = NormalizeDouble(LOT,2); else LOT = NormalizeDouble(LOT,1);
   return(LOT);
}
 

Нужно организовать цикл:

 Проходим по всем ордерам. Если среди закрытых ордеров есть ордер, закрытый сегодня, то ставим флаг запрета торгов. Иначе флаг снимаем.

 

А не могли бы на примере показать? А то я еще нуб, я этого советника из стандартного слепил не могли бы помочь ?

добавил индикатор в этот шаблон

https://www.mql5.com/ru/code/9835

Тоже самое ((( а охота проверить стратегию до закрытия рынка

 

bool  FlagTorga=true;

for (i=OrdersHistoryTotal()-1;i>=0;i--) {
  if (!OrderSelect(i, SELECT_BY_POS, MODE_HISTORY)) { // если ордер выбрать не удалось, то пишем сообщение об ошибке}
else {
  if(OrderSymbol()!= Symbol() || OrderMagicNumber()!= MAGIC){ continue;}// магик нужно проинициализировать заранее, например, в пользовательских переменных
  if(OrderCloseTime()>=iTime(Symbol(),PERIOD_D1,0)){ 
  FlagTorga=false;// снимаем флаг торга
}
}
}

 
Что то куда не вставлял не работает ( я не спец помогите пожалуйста!
Новый комментарий
Причина обращения: