Unlimited order execution for my EA?

My ea can execute max 2 orders at time.....
I want EA that can execute unlimited number of orders!
What do I need to change here!??


void orders()
{
int totalorders = OrdersTotal();
for(int i=totalorders-1;i>=0;i--)
{
OrderSelect(i, SELECT_BY_POS);
bool result = false;
if ( OrderSymbol()==Symbol())
{
if ( OrderType() == OP_SELL ) {sells=sells+1;}
if ( OrderType() == OP_BUY ) {buys=buys+1;}
}
}
return;
}

That just looks like an order tracking piece of code...

I dont get it....why is my ea limiting trades to 2max!???




//+-------------------------------------------------------------------------------------------+
extern int MagicNumber = 1803;
extern int Risk_percent = 1;
extern double minLots = 0.1;
extern double maxLots = 9;
extern double TP = 100;
//+-------------------------------------------------------------------------------------------+
double sells=0;
double buys=0;
double Lots=0.1;
double BuyLots=0.1;
double SellLots=0.1;
double Poin;
//+-------------------------------------------------------------------------------------------+
void orders()
{
int totalorders = OrdersTotal();
for(int i=totalorders-1;i>=0;i--)
{
OrderSelect(i, SELECT_BY_POS);
bool result = false;
if ( OrderSymbol()==Symbol())
{
if ( OrderType() == OP_SELL ) {sells=sells+1;}
if ( OrderType() == OP_BUY ) {buys=buys+1;}
}
}
return;
}
//+-------------------------------------------------------------------------------------------+
//| expert initialization function |
//+-------------------------------------------------------------------------------------------+
int init()
{
//----
//Checking for unconvetional Point digits number
if (Point == 0.00001) Poin = 0.0001; //5 digits
else if (Point == 0.001) Poin = 0.01; //3 digits
else Poin = Point; //Normal
//----
return(0);
}

//+-------------------------------------------------------------------------------------------+
//| expert deinitialization function |
//+-------------------------------------------------------------------------------------------+
int deinit()
{
//----
//----
return(0);
}

//+-------------------------------------------------------------------------------------------+
//| expert start function |
//+-------------------------------------------------------------------------------------------+
int start()
{
sells=0;
buys=0;
orders();
Call_MM();
Trade1();
Trade2();
Trade3();
}

//+-------------------------------------------------------------------------------------------+
//| expert trades |
//+-------------------------------------------------------------------------------------------+
void Trade1()
{
if (sells==0 && buys==0)
{
OrderSend(Symbol(),OP_BUY, Lots, Ask, 200, 0, Ask+TP*Poin," ", MagicNumber, 236, Green);
OrderSend(Symbol(),OP_SELL,Lots, Bid, 200, 0, Bid-TP*Poin," ", MagicNumber, 236, Red );
}
}
//+-------------------------------------------------------------------------------------------+
void Trade2()
{
double BuyLots=(buys*Lots);
double SellLots=Lots;
if (sells==1 && buys==0)
{
OrderSend(Symbol(),OP_BUY, BuyLots, Ask, 200, 0, Ask+TP*Poin," ", MagicNumber, 236, Green);
OrderSend(Symbol(),OP_SELL,SellLots, Bid, 200, 0, Bid-TP*Poin," ", MagicNumber, 236, Red );
}
}
//+-------------------------------------------------------------------------------------------+
void Trade3()
{
double SellLots=(sells*Lots);
double BuyLots=Lots;
if (sells==0 && buys==1)
{
OrderSend(Symbol(),OP_BUY, BuyLots, Ask, 200, 0, Ask+TP*Poin," ", MagicNumber, 236, Green);
OrderSend(Symbol(),OP_SELL,SellLots, Bid, 200, 0, Bid-TP*Poin," ", MagicNumber, 236, Red );
}
}
//+-------------------------------------------------------------------------------------------+
//+-------------------------------------------------------------------------------------------+
//| call moneymanagment |
//+-------------------------------------------------------------------------------------------+
void Call_MM()
{
Lots=AccountFreeMargin()/100000*Risk_percent;
Lots=MathMin(maxLots,MathMax(minLots,Lots));
if(minLots<0.1)
Lots=NormalizeDouble(Lots,2);
else
{
if(minLots<1) Lots=NormalizeDouble(Lots,1);
else Lots=NormalizeDouble(Lots,0);
}
return(0);
}
//+-------------------------------------------------------------------------------------------+

Hey RR...Do you like my complex coding

Well, frankly, my code doesn't use functions much.

But I think the reason why your EA is only opening two trades is this:

if (sells==0 && buys==0)
if (sells==1 && buys==0)
if (sells==0 && buys==1)


Trade 1 opens a trade, let's say it's short: (Sells = 1, Buys = 0)
Trade 2 opens a trade, let's say it's long: (Sella = 1, Buys = 1)

Now what? Instead of using ==, try using >=. That'll open up the limit.

Theres no one specific reason, its just the way the code is written. You basically have three functions that open orders and they are called in sequence. If the number of orders don't meet the criteria for these then no new orders are processed. Basically, its the IF statements that are not being met and therefore no new orders are opened.

its a wacky piece of code!

If I post a full code Im sure you will LOL!

Thanks guys

This is what I see...

When Trade1() is executed for the first time two orders will be open - BUY and SELL. If you execute the function orders() then both variables sells and buys will be equal to 1.

It's because of this code:

if ( OrderSymbol()==Symbol())
{
if ( OrderType() == OP_SELL ) {sells=sells+1;}
if ( OrderType() == OP_BUY ) {buys=buys+1;}
}

It will find a SELL order and increment the sells variable (sells = 1). It will also find a BUY order and increment the buys value (buys = 1).

With sells = 1 and buys = 1 both Trade1(), Trade2() and Trade3() won't open new orders.

Thanks again....its working!

Respect