Download MetaTrader 5

Having problems with OrderClose

To add comments, please log in or register
arrn3g3r
10
arrn3g3r  

Hi,


I am new to programming MQL4, and have been looking at the MQL4 book examples. I am able to OrderSelect successfully, but when I get to the while function the EA freezes. It also says uninitialized variable for Ticket, and Lot. Any ideas?


bool res;

res = OrderSelect(1, SELECT_BY_TICKET);

if(res == false)
   {
   Alert("Error selecting order!");
   }
else
   {
   Alert("Order selected successfully");
   Ticket = OrderTicket();
   Tip    = OrderType();
   Lot    = OrderLots();
   }
while(res==true)
{
   if(Tip==0 && Cls_B==true)
   {
   ticket = OrderClose(Ticket, Lot, Ask, 10*10, 0);
   if(ticket <0)
  
   {
   Alert("Order did not close!");
   }
  
   else
{
if(Tip==1 && Cls_S==true)
   {
   ticket = OrderClose(Ticket, Lot, Ask, 10*10, 0);
   if(ticket <0)
  
   {
   Alert("Order did not close!");
   }
Keith Watford
Moderator
10570
Keith Watford  
res = OrderSelect(1, SELECT_BY_TICKET);

Why are you selecting ticket #1 ?

This will certainly only exist in the strategy tester



if(res == false)
   {
   Alert("Error selecting order!");
   }
else
   {
   Alert("Order selected successfully");
   Ticket =
OrderTicket();
   Tip    =
OrderType();
   Lot    =
OrderLots();
   }

The highlighted variables are only assigned values if the orderselect is successful


while(res==true)

where is res set to false or where is the break? You risk being stuck in a continuous loop


if(Tip==0 && Cls_B==true)
   {
   ticket = OrderClose(Ticket, Lot, Ask, 10*10, 0);
   if(ticket <0)
  
   {



if Tip==0, then it is a buy order. Buy orders are close at Bid, not Ask. Just use OrderClosePrice(), it is simpler and can be used for buy and sell orders.

OrderClose() returns a bool, so check whether it returns true or false, not numbers

arrn3g3r
10
arrn3g3r  

OrderSelect(1, SELECT_BY_TICKET)

I thought it selects the position of the order. For example: if there is 2 orders, it would take the order in position 1. Or 1/1.


I think I misunderstood the options here, does this reflect my thoughts?

res = OrderSelect(0, SELECT_BY_POS, MODE_TRADES);


while(res == true)

If I add a break; at the end, that'll work?


OrderClosePrice()

added this, and using it, although I am getting an error for variable CP - possible loss of data due to type conversion   


Here are the adjustments I've made. My EA is still getting frozen after the OrderSelect, and not even alerting me about OrderClose.


EDIT:


I decided to isolate the first order on my strategy tester, which was a sell order. It looks like the criteria for the Cls_S isn't being met, although the same criteria is used earlier in my code to activate a Opn_B order. Any ideas why this might be happening?

bool res;

res = OrderSelect(0, SELECT_BY_POS, MODE_TRADES);

if(res == true)
   {
   Alert("Order selected successfully");
   Ticket = OrderTicket();
   Tip    = OrderType();
   Lot    = OrderLots();
   CP     = OrderClosePrice();
   }
else
{
   Alert("Order Not Selected");
}
   
while(res == true)
{
   if(Tip==0 && Cls_B==true)
   {
   closeticket = OrderClose(Ticket, Lot, CP, 10*10, 0);
   if(closeticket == false)
   
   {
   Alert("Order Did Not Close!");

   }
   else 
   {
   

if(Tip==1 && Cls_S==true)
   {
   closeticket = OrderClose(Ticket, Lot, CP, 10*10, 0);
   if(closeticket == false)
   
   {
   Alert("Order Did Not Close!");

   }
break;
Keith Watford
Moderator
10570
Keith Watford  

We don't know what the conditions for Cls_S to be true are. We don't know whether Cls_S is a bool or not.

We don't know what these variables are


   Ticket = OrderTicket();
   Tip    = OrderType();
   Lot    = OrderLots();
   CP     = OrderClosePrice();
if CP and/or Lot are integers, they will be incorrect values
To add comments, please log in or register