Experts: Lavika EURGBP/EURUSD

 

Lavika EURGBP/EURUSD:

The Expert Advisor uses four iMA (Moving Average, MA) indicators from two timeframes. Two indicators with averaging periods 1 and 8 are created on the H1 timeframe and two indicators with averaging periods 2 and 24 are also created on the H4 timeframe.

Author: Erin Lavika Tania

 

Nice work but I see this at the "CheckSignal" function and don't get it, are you sure that it's ok?


   if(RAVI0_2_24_H1>0.0)

      if(RAVI0_2_24_H4>RAVI0_2_24_H4_1 && RAVI0_2_24_H4_1<RAVI0_2_24_H4_2 && RAVI0_2_24_H4_2<RAVI0_2_24_H4_3)

         buy_signal=true;

   if(RAVI0_2_24_H1>0.0)

      if(RAVI0_2_24_H4<RAVI0_2_24_H4_1 && RAVI0_2_24_H4_1>RAVI0_2_24_H4_2 && RAVI0_2_24_H4_2>RAVI0_2_24_H4_3)

         buy_signal=true;

 
vagsag:

Nice work but I see this at the "CheckSignal" function and don't get it, are you sure that it's ok?


   if(RAVI0_2_24_H1>0.0)

      if(RAVI0_2_24_H4>RAVI0_2_24_H4_1 && RAVI0_2_24_H4_1<RAVI0_2_24_H4_2 && RAVI0_2_24_H4_2<RAVI0_2_24_H4_3)

         buy_signal=true;

   if(RAVI0_2_24_H1>0.0)

      if(RAVI0_2_24_H4<RAVI0_2_24_H4_1 && RAVI0_2_24_H4_1>RAVI0_2_24_H4_2 && RAVI0_2_24_H4_2>RAVI0_2_24_H4_3)

         buy_signal=true;

 

This EA is profitable but it opens ONLY SHORT positions. What is wrong and how to change the code?

I am not a programmer .

Hope that I get an answer very soon

Ralph from Berlin

 
rarigerm:

This EA is profitable but it opens ONLY SHORT positions. What is wrong and how to change the code?

I am not a programmer .

Hope that I get an answer very soon

Ralph from Berlin

Yes i think this EA have a little problem with command for BUY... i have try this with real account and so far Only Short position... I hope someone can fixed it too...
 

Im having issues with this EA, i get in the journal multiple errors with the following comment: m transaction confirmed: false

Does anyone knows why this is happening?

 
البرودي عبدالرحيم #:

It's a typo; the first two statements is for "buy", and the second two statements  is for "sell". So, change it to 

if(RAVI0_2_24_H1<0.0) and sell_signal=true. There is also a typo on the handle statement.