I noticed a problem immediately with this EA.

The "Close All" function shouldn't work properly as coded.

Reason being: MT4 will only execute one operation per tick.....

void CloseAll()
{
int total = OrdersTotal();
for(int i=total-1;i>=0;i--)
{
OrderSelect(i, SELECT_BY_POS);

It should be more like "while(OrdersTotal()>0)" etc......

That will force the program to loop instead of moving on after one order closes.

I could be wrong, but that is based on first glance.

__________________
Ideas are not trade advice.