Aaaaaa.....help me with code pls

I code an EA that sends 4 orders at the time, one buy and 3 buystops.
Now I want EA to close and delete all orders if number of orders is less
than 3(lets say buy order hits sl, and now i have 3 pending orders).

At the end of the code i have:
--------------------------------------------------
void Close_all()
{
int totalorders = OrdersTotal();
for(int i=totalorders-1;i>=0;i--)
{
OrderSelect(i, SELECT_BY_POS);
bool result = false;
if (OrderSymbol()==Symbol() )
if (OrderMagicNumber()==MagicNumber )
{


if (OrderType()==OP_BUY) result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_BID), 5, Red );
if (OrderType()==OP_SELL) result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_ASK), 5, Red );
if (OrderType()==OP_BUYSTOP) result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_BID), 5, Red );
if (OrderType()==OP_SELLSTOP) result = OrderClose( OrderTicket(), OrderLots(), MarketInfo(OrderSymbol(), MODE_ASK), 5, Red );

}
}
return;
}
----------------------------------------------------

Than at the expert start function I have:

int start()
{
if(OrdersTotal()<=3) Close_all();
-------------------------------------------------------

Now this works great when I start backtest, but after sometime,
orders just get closed at some minimal loss constantly, and then later
after some few houndreds closed orders, start to work normaly again!!!

Im lost, any ideas!

have you looked at the area where the problem seems to occur?

Problem is within the code.......
Is there some simple way to close trades if OrderTotal is less than 3!

use OrderDelete() for pending orders instead of OrderClose()

I tried with this.....
int start()
{
int totalorders = OrdersTotal();
for(int j=0; j<totalorders;j++)
{
OrderSelect(j,SELECT_BY_POS,MODE_TRADES);
if (OrderSymbol() == Symbol() && OrderType() > 1)
{
if(OrdersTotal()<=3)EndSession();
if(OrdersTotal()>4)EndSession();
}
}
---------------------------------------------------------

bool EndSession()
{
int totalorders = OrdersTotal();
for(int j=0; j<totalorders;j++)
{
OrderSelect(j,SELECT_BY_POS,MODE_TRADES);
if (OrderSymbol() == Symbol() && OrderType() > 1)
{
OrderDelete(OrderTicket());
}
else if (OrderSymbol() == Symbol() && OrderType() == OP_BUY)
{
OrderClose(OrderTicket(), OrderLots(), Bid, 3, Red);
}
else if (OrderSymbol() == Symbol() && OrderType() == OP_SELL)
{
OrderClose(OrderTicket(), OrderLots(), Ask, 3, Red);
}
}
return(true);
}

Same problem.......

i meant area on the chart where this problem occurs, like the date. Sounds like there is a conflict somwhere.

I just run test with different symbol....same problem!

But the difference is that now it doesnt work at the start of test, but works later????

Here is my solution first part of the code is ok I rewrote the function this is a standard boilerplate type close function. I think I first saw it in one of Ronald Raygun's EAs so I would guess he wrote it originally, don't know for sure though

You have 4 orders placed all at same time yeah? And when one gets closed it closes all other trades.
Maybe the problem is with the opening of orders?

ThatWasMe, big THANKS goes to you.........
and to others that tried to help......you suck

Sorry, I didnt mean that....big thanks to all !!!
Respect

lol. Dont mention it!
Yeah u have used else if statement when u actually wanted to close all trades anyway. U also suck.

im suprised tho flash, i have seen u put quite a few eas out there. How are they workin out btw? Dnt u find ea's to be problematic? as in a little too robotic?

Not this ea......the size of code is a little less than 36GB!!!
To run it, I use 64 x quad core PC with liquid nitrogen cooling,
and a direct access to nuclear power plant reactor
for some serious power....(altrough its not enough)
When I start to trade this EA, sometimes the whole city
loose power for some few minutes until my ea calculate the size of
optimal lots...... now, imagine what it does when it calculates the TP and SL!!!