tomgny
2021-02-24 22:31:53
Make sure that you didn't override TicketA or TicketB variable somewhere.
You can use OrderLots() function instead of using LotSize, especially when this value is changing during EA process. Additionally, by checking the OrderType(), you will avoid a mistake at the closing price.
Example:
if(yourCloseCondition){
if(OrderSelect(ticket, SELECT_BY_TICKET)){
if(OrderType() == OP_BUY){
if(OrderClose(ticket, OrderLots(), Bid, 0)){
//Print("success");
}
}
if(OrderType() == OP_SELL){
if(OrderClose(ticket, OrderLots(), Ask, 0)){
//Print("success");
}
}
}
}
Also check docs: OrderClose() and OrderType().
Update:
For different pairs running within 1 chart use close price from MarketInfo
Example:
MarketInfo("EURUSD",MODE_BID);
Check MarketInfo().
So you would suggest I make it: OrderClose(ticket, OrderLots(), MarketInfo("EURUSD",MODE_BID), 0); TicketA and TicketB are certainly not overriden. Lotsize is stable, good advise tho.
Yes, customize the example to your case. If it doesn't help, paste some more code to find out what's wrong.
Problem solved with some "lightening" and complete re-write. Also used all 6 ways to close order just in case. That results in 4 errors and 2 successful closes. So works.... Thanks!