EA only makes sell trades? even when theres no OP_SELL?

New comment

Reilly Gailey 2016.11.24 09:42

Made a simple moving average cross over ea.

2 things;

It only makes sell trades.

I dont know how to add in a maximum open trades code.

Please help.

Here is the code for it only making sells.

can supply full ea if needed

void OnTick()
  {
//---

double previousfast = iMA(NULL,0,fastma,fastmashift,fastmamethod,fastmaappliedto,2);
double currentfast = iMA(NULL,0,fastma,fastmashift,fastmamethod,fastmaappliedto,1);
//^(NULL=moving average on the current currency pair. 0=the current timeframe. fastma= the moving average period.
double previousslow = iMA(NULL,0,slowma,slowmashift,slowmamethod,slowmaappliedto,2);
double currentslow = iMA(NULL,0,slowma,slowmashift,slowmamethod,slowmaappliedto,1);

if(previousfast<previousslow && currentfast>currentslow)
//^buy
ticket1 = OrderSend(Symbol(),OP_BUY,lotsize,Ask,3,Ask-(stoploss),Ask+(takeprofit),"EA Jim Dandy MA Trade",magicnumber,0,Green);
   if(ticket1<0)
   Print("Order Failed #",GetLastError());
else
   Print("Order success1");

if (previousfast>previousslow && currentfast<currentslow)

//^sell

ticket2 = OrderSend(Symbol(),OP_SELL,lotsize,Bid,3,Bid+(stoploss),Bid-(takeprofit),"EA Jim Dandy MA Trade",magicnumber,0,Red);

if(ticket2<0)

   Print("Order Failed #",GetLastError());
   else
   Print("Order success2");


  }

Keith Watford 2016.11.24 11:49 #1

I can imagine that you are getting a LOT of unwanted prints

if(previousfast<previousslow && currentfast>currentslow)
//^buy
ticket1 = OrderSend(Symbol(),OP_BUY,lotsize,Ask,3,Ask-(stoploss),Ask+(takeprofit),"EA Jim Dandy MA Trade",magicnumber,0,Green);
   if(ticket1<0)
   Print("Order Failed #",GetLastError());
   else
   Print("Order success1");

You have to put in the brackets

if(previousfast<previousslow && currentfast>currentslow)
   //^buy
  {
   ticket1=OrderSend(Symbol(),OP_BUY,lotsize,Ask,3,Ask-(stoploss),Ask+(takeprofit),"EA Jim Dandy MA Trade",magicnumber,0,Green);
   if(ticket1<0)
      Print("Order Failed #",GetLastError());
   else
      Print("Order success1");
  }

otherwise this will be executed every tick

   if(ticket1<0)
   Print("Order Failed #",GetLastError());
   else
   Print("Order success1");

Also you don't seem to have anything in your code to only check on a new bar.

return value of 'OrderSend' Translate piece of Mt4 How do I add

Reilly Gailey 2016.11.28 07:05 #2

Keith Watford:

I can imagine that you are getting a LOT of unwanted prints

You have to put in the brackets

otherwise this will be executed every tick

   if(ticket1<0)
   Print("Order Failed #",GetLastError());
   else
   Print("Order success1");

Also you don't seem to have anything in your code to only check on a new bar.

thanks man, could you suggest code to only check new bar?

Keith Watford 2016.11.28 07:13 #3

ReillyFox:
thanks man, could you suggest code to only check new bar?

   static datetime BarTime=0;
   datetime now_bar=Time[0];
   if(BarTime!=now_bar)
     {
      BarTime=now_bar;
      //new_bar=true;
      //Put code here that only needs to be executed when a new bar opens
     }

or have the bool new_bar and use as condition before executing code blocks

OrderModify CheckOpenLong CheckOpenShort

New comment