EA close at profit Taregt. Been Trying to, but doesn't works

New comment
 

Hey guys, hope you could help me here. Been editing but the EA won't close/delete the Limit/Stop order. could anyone help me check what is wrong? thank you


int start()
{
if (AccountProfit()>= My_Money_Profit_Target)
   {
    for(i=OrdersTotal()-1;i>=0;i--)
       {
       OrderSelect(i,SELECT_BY_POS);
       int type   = OrderType();
               
       bool result = false;
              
       switch(type)
          {
          //Close opened long positions
          case OP_BUY  : result = OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_BID),Slippage,Pink);
                         break;
                         
          //Close pending long limit positions
          case OP_BUYLIMIT  : result = OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_BID),Slippage,Pink);
                         break;
                         
          //Close pending long stop positions
          case OP_BUYSTOP  : result = OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_BID),Slippage,Pink);
                         break;
               
          //Close opened short positions
          case OP_SELL : result = OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_ASK),Slippage,Pink);
                         break;
                         
          //Close pending short limit positions
          case OP_SELLLIMIT : result = OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_ASK),Slippage,Pink);
                         break;
                         
          //Close pending short stop positions
          case OP_SELLSTOP : result = OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_ASK),Slippage,Pink);
                                                   
          }
          
       if(result == false)
          {
            Sleep(0);
          }  
       }
      Print ("Account Profit Reached. All Open Trades Have Been Closed");
      return(0);
   }  
   
   Comment("Balance: ",AccountBalance(),", Account Equity: ",AccountEquity(),", Account Profit: ",AccountProfit(),
           "\nMy Account Profit Target: ",My_Money_Profit_Target);
   
  return(0);
}
 
Rizal:

Hey guys, hope you could help me here. Been editing but the EA won't close/delete the Limit/Stop order. could anyone help me check what is wrong? thank you


You can't close a pending order, you need to delete it : see documentation for OrderDelete().
New comment
Reason: