OrderClose(), requotes and time between orders

Point Zero 2015.10.28 12:54

Hello everyone,

I hope you are fine. I am having problems processing requotes in my CloseOrder() function. My function tries to close trades in a loop. If the broker sends a requote, the Ea tries again and again until closed.

In theory it should work, however sometimes the broker sends back an error "Many requests" and does not accept any other transaction requests. This is my code:

/**
* Closes desired orders 
* @param    int   Type
* @param    int   Token
* @param    int   ticket
* @return   bool
*/
bool CloseOrder(int Type, int Token = EMPTY_VALUE, int ticket = EMPTY_VALUE)
{
        for(int i = OrdersTotal()-1; i >= 0; i--)
        {
                if(OrderSelect(i, SELECT_BY_POS, MODE_TRADES) &&
                   OrderSymbol() == Symbol() &&
                   Type == OrderType() && 
                   (ticket == EMPTY_VALUE || ticket == OrderTicket()))
                { 
                   if(Type == OP_BUY && Bid > OrderOpenPrice()-(Ask-Bid) && Token == OnlyInLoss) continue;
                   if(Type == OP_SELL && Ask < OrderOpenPrice()+(Ask-Bid) && Token == OnlyInLoss) continue;
              if(Type == OP_BUY || Type == OP_SELL)  
              {
                 int closed = false;
                 while(!closed)
                 {
                    closed = OrderClose(OrderTicket(), OrderLots(), OrderClosePrice(), Slippage, Gold);
                    if(!closed) Print(ShortName +" (OrderClose Error) "+ ErrorDescription(GetLastError()) +". Trying again...");
                 }
         }
         if(Type == OP_BUYSTOP || Type == OP_SELLSTOP || Type == OP_BUYLIMIT || Type == OP_SELLLIMIT)  
              {
                 if(OrderMagicNumber() == MagicNumber)
                 {
               if(!OrderDelete(OrderTicket(), Gold))
               {
                  Print(ShortName +" (OrderDelete Error) "+ ErrorDescription(GetLastError()));
                  return(false);
               }
            }
         }
      }
   }
   return(true);
}

My question is, how can I keep this loop alive without the broker rejecting my orders for "too many requests"? Should I leave a sleep() period between the OrderClose() calls? If so, how much?

Thanks

qjol 2015.10.28 13:35 #1

i would try a Sleep() for a few seconds after 2-3 requotes and increasing it after 5-6 not to get "too many requests"

Point Zero 2015.10.28 14:05 #2

qjol:
i would try a Sleep() for a few seconds after 2-3 requotes and increasing it after 5-6 not to get "too many requests"

Hi Q! Thanks for your message. Are there are official mql4 guidelines for this? What is considered "too many" request per unit of time? Thanks!

Keith Watford 2015.10.28 20:54 #3

I don't know what is considered too many requests.

But when an OrderClose fails, you should re-select the order before trying again.

Especially after a re-quote, OrderClosePrice will not be current.

Closing all open trades can any one solve Function to know the

qjol 2015.10.29 02:29 #4

GumRai:

I don't know what is considered too many requests.

But when an OrderClose fails, you should re-select the order before trying again.

Especially after a re-quote, OrderClosePrice will not be current.

I'm not gonna argue with you on that because I never investigate it

but IMHO OrderSelect() olds only the index of order pool or order ticket, and changes that occur on OrderClosePrice() doesn't matter (my 2¢)

bool  OrderSelect(
   int     index,            // index or order ticket
   int     select,           // flag
   int     pool=MODE_TRADES  // mode
   );

but of course you have nothing to lose by selecting the order again

Any questions from newcomers Questions from Beginners MQL4 Detecting Stop Loss

Keith Watford 2015.10.29 04:22 #5

qjol:

but i think that OrderSelect() olds only the index of order pool or order ticket, and changes that occur on OrderClosePrice() doesn't matter (my 2¢)

from the documentation

The OrderSelect() function copies order data into program environment and all further calls of OrderClosePrice(), OrderCloseTime(), OrderComment(), OrderCommission(), OrderExpiration(), OrderLots(), OrderMagicNumber(), OrderOpenPrice(), OrderOpenTime(), OrderPrint(), OrderProfit(), OrderStopLoss(), OrderSwap(), OrderSymbol(), OrderTakeProfit(), OrderTicket(), OrderType() functions return the data, copied earlier.

It is possibly due to not re-selecting the order after a re-quote, that flaab gets so many additional re-quotes.

qjol 2015.10.29 05:18 #6

@ GumRai you were right, just checked it out (with a script i ran), Order must be re-selected, my mistake, sorry.

Keith Watford 2015.10.29 05:29 #7

qjol:
@ GumRai you were right, just checked it out (with a script i ran), Order must be re-selected, my mistake, sorry.

No need to apologise. We all learn from each other.

qjol 2015.10.29 05:58 #8

it's not just about the apology, it's also for the sake of the forum, if some day, someone is gonna search the forum about this...

best Regards

William Roeder 2015.10.30 12:02 #9

GumRai: But when an OrderClose fails, you should re-select the order before trying again. Especially after a re-quote, OrderClosePrice will not be current.	You must RefreshRates and then reselect the order for OrderClosePrice to be current.

Keith Watford 2015.10.30 19:05 #10

WHRoeder:

GumRai: But when an OrderClose fails, you should re-select the order before trying again. Especially after a re-quote, OrderClosePrice will not be current.	You must RefreshRates and then reselect the order for OrderClosePrice to be current.

RefreshRates updates the pre-defined variables and arrays

OrderClosePrice is not one of these.

1 2

New comment