Exponential MA Cross EA Help | Forex Factory

Options

Permalinks
Print Thread

Similar Threads

Thoughts on Compounding/Exponential systems 5 replies

no-lag-Triple exponential Moving average(t3) based on heiken ashi values 9 replies

spread monitor - Help with exponential calculation needed 1 reply

Bid Ask To 50 Day Exponential MA 0 replies

Simple vs exponential MAs? 2 replies

Attachments: Exponential MA Cross EA Help
Exit Attachments

Exponential MA Cross EA Help

  • #1
  • First Post: Nov 1, 2012 2:58pm Nov 1, 2012 2:58pm
Built from http://sufx.core.t3-ism.net/ExpertAdvisorBuilder/.

EA specs
20 exponential cross 50 exponential buy above sell below.
Trailing stop for exit

Seems to not take the trade as intended, can anyone see the error?
Much appreciated
Attached File(s)
File Type: mq4 50cross20MAExpert.mq4   9 KB | 363 downloads
  • #2
  • Nov 1, 2012 7:32pm Nov 1, 2012 7:32pm
Quoting Armst227
Built from http://sufx.core.t3-ism.net/ExpertAdvisorBuilder/.

EA specs
20 exponential cross 50 exponential buy above sell below.
Trailing stop for exit

Seems to not take the trade as intended, can anyone see the error?
Much appreciated
There are two problems:
First, the EA's generated from that site are not 5-digit compatible, so you're getting an OrderSend Error 130. The easy fix to this is to add an extra zero to your Stop Loss and Take Profit, the 60 becomes 600 and 30 becomes 300.

Second, your trade logic is incomplete. In order to trade the cross you must also take in consideration the previous bars value.
Inserted Code
Replace:
double Buy1_1 = iMA(NULL, 0, 20, 0, MODE_EMA, PRICE_CLOSE, Current + 0);
double Buy1_2 = iMA(NULL, 0, 50, 0, MODE_EMA, PRICE_CLOSE, Current + 0);

double Sell1_1 = iMA(NULL, 0, 20, 0, MODE_EMA, PRICE_CLOSE, Current + 0);
double Sell1_2 = iMA(NULL, 0, 50, 0, MODE_EMA, PRICE_CLOSE, Current + 0);


with:
double Sell1_2 = iMA(NULL, 0, 50, 0, MODE_EMA, PRICE_CLOSE, Current + 1);
double Buy1_1  = iMA(NULL, 0, 20, 0, MODE_EMA, PRICE_CLOSE, Current + 1);
double Buy1_2  = iMA(NULL, 0, 50, 0, MODE_EMA, PRICE_CLOSE, Current + 0);
double Sell1_1 = iMA(NULL, 0, 20, 0, MODE_EMA, PRICE_CLOSE, Current + 0);


Inserted Code
Replace:
   if (Buy1_1 > Buy1_2) Order = SIGNAL_BUY;

   if (Sell1_1 < Sell1_2) Order = SIGNAL_SELL;

with:
   if (Buy1_1 < Buy1_2 && Sell1_1 > Sell1_2) Order = SIGNAL_BUY;

   if (Buy1_1 > Buy1_2 && Sell1_1 < Sell1_2) Order = SIGNAL_SELL;
  • #3
  • Nov 1, 2012 7:56pm Nov 1, 2012 7:56pm
Ok I think I understand, andt hanks for your reply. From my understanding we are adding another parameter shifted back a bar or two. Do you find the +1 shift is enough?
  • #4
  • Last Post: Nov 1, 2012 8:09pm Nov 1, 2012 8:09pm
Quoting Armst227
Ok I think I understand, andt hanks for your reply. From my understanding we are adding another parameter shifted back a bar or two. Do you find the +1 shift is enough?
The parameter is already there you just need to shift it back one bar.
And yes one is usually enough.
0 traders viewing now