Несложный советник,прошу помощи

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

Советник на основе известного встроеного в метатрейдер советника МА.
При работе и тестировании выяснилось,что на часовых свечах открывает 2
ордера,а нужен только один. Первый открывается и сразу закрывается с убытком в спред.
Затем открывает второй-рабочий.Первый нужно убрать из-за него убытки.
Код прилагаю.Кто может посмотрите опытным глазом.Спасибо.

//+------------------------------------------------------------------+
//|                                               Moving Average.mq4 |
//|                      Copyright © 2005, MetaQuotes Software Corp. |
//|                                       https://www.metaquotes.net// |
//+------------------------------------------------------------------+
#define MAGICMA  20050610
extern double Lots               = 0.1;
extern double MovingPeriod       = 55;
extern double MovingShift        = 0;
extern double peremen;
//+------------------------------------------------------------------+
//| Calculate open positions                                         |
//+------------------------------------------------------------------+
int CalculateCurrentOrders(string symbol)
  {
   int buys=0,sells=0;
//----
   for(int i=0;i<OrdersTotal();i++)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==false) break;
      if(OrderSymbol()==Symbol() && OrderMagicNumber()==MAGICMA)
        {
         if(OrderType()==OP_BUY)  buys++;
         if(OrderType()==OP_SELL) sells++;
        }
     }
//---- return orders volume
   if(buys>0) return(buys);
   else       return(-sells);
  }
//+------------------------------------------------------------------+
//| Check for open order conditions                                  |
//+------------------------------------------------------------------+
void CheckForOpen()
  {
   double ma;
   int    res;
//---- go trading only for first tiks of new bar
//  if(Volume[0]>1) return;
//---- get Moving Average 
   ma=iMA(NULL,0,MovingPeriod,MovingShift,MODE_SMA,PRICE_CLOSE,0);
//---- sell conditions
   if(Bid>ma && (Bid-Open[0])*100000>260)  
     {
      res=OrderSend(Symbol(),OP_SELL,Lots,Bid,3,0,0,"",MAGICMA,0,Red);
      return;
     }
//---- buy conditions
   if(Ask<ma && (Open[0]-Ask)*100000>260)  
     {
      res=OrderSend(Symbol(),OP_BUY,Lots,Ask,3,0,0,"",MAGICMA,0,Blue);
      return;
     }
//----
  }
//+------------------------------------------------------------------+
//| Check for close order conditions                                 |
//+------------------------------------------------------------------+
void CheckForClose()
  {
   for(int i=0;i<OrdersTotal();i++)
     {
      if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==false)        break;
      if(OrderMagicNumber()!=MAGICMA || OrderSymbol()!=Symbol()) continue;
      //---- check order type 
      if(OrderType()==OP_BUY)
        {
 if (peremen!=Bars)
     {
     OrderClose(OrderTicket(),OrderLots(),Bid,3,White);
     peremen=Bars;
    }
        }
        
      if(OrderType()==OP_SELL)
         {
    if (peremen!=Bars)    
         {
         OrderClose(OrderTicket(),OrderLots(),Ask,3,White);
     peremen=Bars;
         }
      }
    }
  }
//+------------------------------------------------------------------+
//| Start function                                                   |
//+------------------------------------------------------------------+
void start()
  {
//---- check for history and trading
   if(Bars<100 || IsTradeAllowed()==false) return;
//---- calculate open orders by current symbol
   if(CalculateCurrentOrders(Symbol())==0) CheckForOpen();
   else                                    CheckForClose();
//----
  }
//+------------------------------------------------------------------+
 
pistolet >>:

Советник на основе известного встроеного в метатрейдер советника МА.
При работе и тестировании выяснилось,что на часовых свечах открывает 2
ордера,а нужен только один. Первый открывается и сразу закрывается с убытком в спред.
Затем открывает второй-рабочий.Первый нужно убрать из-за него убытки.
Код прилагаю.Кто может посмотрите опытным глазом.Спасибо.

Изменил, оставив способ входа-выхода и прочие несуразности в неприкосновенности.

Файлы:
movingxaverage_.mq4  4 kb
 
Valmars писал(а) >>

Изменил, оставив способ входа-выхода и прочие несуразности в неприкосновенности.


Спасибо,несложно а я не додумался, не разбираюсь в циклах.
В целом идея я думаю стоящая,то есть торговать бай от низинок и селл от вершинок.
Нужно лишь находить "примерно" эти экстремумы, или закрывать по небольшому тр и стоп-лосс.
Новый комментарий