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;} } } }
Не помогло,
взял классический MACD Sample, не хочет закрывать открытые другими ордера
Не помогло,
взял классический MACD Sample, не хочет закрывать открытые другими ордера
Что не помогло? - Вы ничего не изменили в логике, а она некорректна.
Не помогло,
взял классический 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.
А вот как я бы его переписал. Далее по коду внесите те же измения и возможно эксперт не будет пропускать момент закрытия.
А я бы, вообще его убрал, он ни на что не влияет.)))
Ну, то, что не открывает, ясно, у вас переменная а всегда равна нулю, а для открытия требуется единица.
Длинные ордера должен закрывать, а закрытие коротких было с ошибкой, вам уже указали,
Убрать действительно можно. Но скорректировать именно этот кусок кода было самым легким и наглядным способом указать на одну явную ошибку и на более правильный (на мой взгляд) способ проверки наличия открытых бай и селл. А далее по коду это уже сам пользователь должен исправлять. Он же научиться хочет и понять. Или нет?
В данном случае Ваш взгляд ошибся, нет никакой разницы каким образом перебирать ордера, удаления ведь нет.
- Бесплатные приложения для трейдинга
- 8 000+ сигналов для копирования
- Экономические новости для анализа финансовых рынков
Вы принимаете политику сайта и условия использования
Советник перестал закрывать ордера
Есть у меня советник сделанный из MACD Sample, открывает ордер по условию, и закрывает - по условию.
Открытие ордеров в советнике я отключил - в условие для открытия ордера внес ключ 1 или 0 и использовал советник только для закрытия открытых ордеров - советник работал, меня все устраивало...
В данный момент советник ордера открытые вручную или другими советниками не закрывает - ошибок в журнале Эксперты также нет.
С месяц назад советник перестал закрывать открытые вручную или другими советниками ордера, я проверил на тестере стратегий - если включить ключ - то советник исправно ордера открывает и закрывает.
Прошу подсказать как мне сделать что бы закрывались ордера