Hi Andy, if its ok I'll add some code below (will edit this post when done) as a tide-over until you're back & can do the job properly. Don't worry mate, enjoy your holiday - we'll keep the fires burning until you're back
Edit: Ok firstly you'll need to add the following to the EA's code up the top - in v1.09 its at line 83 under the // candletiger comment.
Code: Select all
extern bool useAO = true;
extern bool useStoch = true;
extern bool useRSI = true;Then line 2310-ish you'll see the function trendDirect() replace it with the following:-
Code: Select all
bool trendDirect(string symbol, int type)
{
double mA = iMA(symbol, PERIOD_H4, 240, 0, MODE_LWMA, PRICE_OPEN, 0);
double sT = iStochastic(symbol, PERIOD_H4, 5, 3, 3, MODE_LWMA, 1, MODE_SIGNAL, 0);
double aO = GetAo(symbol, PERIOD_H4, 0);
double aO1 = GetAo(symbol, PERIOD_H4, 1);
double lO = iLow(symbol, PERIOD_H4, 0);
double hO = iHigh(symbol, PERIOD_H4, 0);
double gRSI = GetSlopeRSI(symbol, PERIOD_H4, 0);
if (useStoch)
{
if (type == OP_BUY)
{
if (useBothForPullback)
{
if (lO > mA && sT < 20 && aO < 0 && aO1 < aO)
return(true);
}
else if (lO > mA && sT < 20)
return(true);
}
if (type == OP_SELL)
{
if (useBothForPullback)
{
if (hO < mA && sT > 80 && aO > 0 && aO1 > aO)
return(true);
}
else if (hO < mA && sT > 80)
return(true);
}
}
if (useRSI)
{
if (type == OP_BUY)
{
if (useBothForPullback)
{
if (lO > mA && gRSI < RSITMAlowlevel && aO < 0 && aO1 < aO)
return(true);
}
else if (lO > mA && gRSI < RSITMAlowlevel)
return(true);
}
if (type == OP_SELL)
{
if (useBothForPullback)
{
if (hO < mA && gRSI > RSITMAhighlevel && aO > 0 && aO1 > aO)
return(true);
}
else if (hO < mA && gRSI > RSITMAhighlevel)
return(true);
}
}
if (useAO)
{
if (type == OP_BUY && lO > mA && aO < 0 && aO1 < aO)
return(true);
else if (type == OP_SELL && hO < mA && aO > 0 && aO1 > aO)
return(true);
}
return(false);
}Its not a choice whether to use Stoch or RSI. You can use both or either. Just make sure one of them at least is set to true.
Awesome Oscillator is used as a confirming measure for Stoch/RSI trades if you set useBothForPullback to true, or as an entry in its own right if you set useAO to true. Set both those to false and it won't be used at all. The AO needs to be in positive territory for a sell trade or negative for a buy trade, and it now needs to have moved closer to the other direction than the previous H4 candle as that's how I read Bob's rule in the manual.
Lastly if I've stuffed something up here by all means don't hesitate to point it out!
Edit: improved the logic a bit, separating out the useAO parts from the useStoch and useRSI. Should work exactly the same.
Thanks for the signature.