Invalid ticket for OrderClose function

 

Здравствуйте!
Есть функция по закрытию все ордеров

void close_all_order()//закрыть все ордера
{
   int i;
   int t = OrdersTotal();
   for (i=0;i<t;i++) 
   {
      OrderSelect(i, SELECT_BY_POS, MODE_TRADES);
      if(OrderType()==OP_BUY)
      {
         OrderClose(OrderTicket(),OrderLots(),Bid,3,Violet);
      
      }
      if (OrderType()==OP_SELL)
      {
         OrderClose(OrderTicket(),OrderLots(),Ask,3,Violet);
      }
   } 
}

Но во время работы выдает ошибку сабжа! В чем проблема?
Или как еще можно закрыть все открытые ордера?

 
Wade писал (а):

Или как еще можно закрыть все открытые ордера?


Попробуй так..
int CloseAllOrders()
  {
   while (OrdersTotal()>0)
    {
      OrderSelect(0,SELECT_BY_POS);
      if (OrderType()==OP_BUY)       OrderClose(OrderTicket(),Lots,Bid,3,Green);
      else if (OrderType()==OP_SELL) OrderClose(OrderTicket(),Lots,Ask,3,Red);
    }
  
  }
 
Этот код зацикленный. Достаточно иметь один отложенный ордер, чтобы вся эта конструкция зациклилась намертво.
 

Спасибо, что так оперативно ответили)
Код Rosh помог...только я не очень понял в чем фактическая разница с моим?

 
Wade писал (а):

...только я не очень понял в чем фактическая разница с моим?

;-)

For едет с последнего ордера к первому.
 
А есть существенная разница - идти от начала к концу или наоборот?
 
"Операция Ы" смотрел? Как Вицин горшок потянул снизу - помнишь?
 

Please Help me? err msg : Invalid ticket for OrderClose function.

//+------------------------------------------------------------------+
//| slope line il.mq4.mq4
//| ilhan
//| //+----------------------------------------------- -------------------+
//+------------------------------------------------------------------+
#property copyright "ilhan"
#property link "//+----------------------------------------------- -------------------+"


//---- input parameters
extern int period=80;
extern int method=3; // MODE_SMA
extern int price=0; // PRICE_CLOSE
//---- buffers
double Uptrend[];
double Dntrend[];
double ExtMapBuffer[];
extern double Lots = 0.1;
extern int Slippage=3;
//+------------------------------------------------------------------+
//| expert initialization function |
//+------------------------------------------------------------------+
int init()
{
IndicatorBuffers(3);
SetIndexBuffer(0, Uptrend);
//ArraySetAsSeries(Uptrend, true);
SetIndexBuffer(1, Dntrend);
//ArraySetAsSeries(Dntrend, true);
SetIndexBuffer(2, ExtMapBuffer);
ArraySetAsSeries(ExtMapBuffer, true);

SetIndexStyle(0,DRAW_LINE,STYLE_SOLID,2);
SetIndexStyle(1,DRAW_LINE,STYLE_SOLID,2);

IndicatorShortName("Slope Direction Line("+period+")");
return(0);
}

//+------------------------------------------------------------------+
//| expert deinitialization function |
//+------------------------------------------------------------------+
int deinit()
{
//----

//----
return(0);
}

double WMA(int x, int p)
{
return(iMA(NULL, 0, p, 0, method, price, x));
}
//+------------------------------------------------------------------+
//| expert start function |
//+------------------------------------------------------------------+
int start()

{
bool Long=false;
bool Short=false;
bool Sideways=false;
int total;
total=OrdersTotal();


int counted_bars = 1; //IndicatorcCounted()

if(counted_bars < 0)
return(-1);

int x = 0;
int p = MathSqrt(period);
int e = Bars - counted_bars + period + 1;

double vect[], trend[];

if(e > Bars)
e = Bars;

ArrayResize(vect, e);
ArraySetAsSeries(vect, true);
ArrayResize(trend, e);
ArraySetAsSeries(trend, true);

for(x = 0; x < e; x++)
{
vect[x] = 2*WMA(x, period/2) - WMA(x, period);
// Print("Bar date/time: ", TimeToStr(Time[x]), " close: ", Close[x], " vect[", x, "] = ", vect[x], " 2*WMA(p/2) = ", 2*WMA(x, period/2), " WMA(p) = ", WMA(x, period));
}
for(x = 0; x < e-period; x++)

ExtMapBuffer[x] = iMAOnArray(vect, 0, p, 0, method, x);

for(x = e-period; x >= 0; x--)
{
trend[x] = trend[x+1];
if (ExtMapBuffer[x]> ExtMapBuffer[x+1]) trend[x] =1;
if (ExtMapBuffer[x]< ExtMapBuffer[x+1]) trend[x] =-1;
}



if (trend[x]>0)
Short=true;
Sideways=true;
{
Uptrend[x] = ExtMapBuffer[x];
if (trend[x+1]<0) Uptrend[x+1]=ExtMapBuffer[x+1];
Dntrend[x] = EMPTY_VALUE;
}
if( Short==true && OrdersTotal()==0)
OrderSend(Symbol(),OP_BUY,Lots,Ask,Slippage,Ask/3,Ask*3,0,0,Blue);

OrderSelect(0, SELECT_BY_POS, MODE_TRADES);
Comment("Sideways detected");
if(OrderType()==OP_BUY) OrderClose(OrderTicket(),1,Ask,3,Red);

else
if (trend[x]<0)
Long=true;
Sideways=true;
{

Dntrend[x] = ExtMapBuffer[x];
if (trend[x+1]>0) Dntrend[x+1]=ExtMapBuffer[x+1];
Uptrend[x] = EMPTY_VALUE;
}

if(Long==true && OrdersTotal()==0)
OrderSend(Symbol(),OP_SELL,Lots,Bid,Slippage,Bid*3,Bid/3,0,0,Red);

OrderSelect(0, SELECT_BY_POS, MODE_TRADES);
Comment("Sideways detected");
if(Sideways==true && OrdersTotal()!=0)
if(OrderType()==OP_SELL) OrderClose(OrderTicket(),1,Bid,3,Red);

}



//+------------------------------------------------------------------+

Причина обращения: