Adjusting this should stabilize the signal:
old:
for(i = 0; i <= limit; i++) {
new:
for(i = 2; i <= MathMax(limit, 2); i++) {
The stable signal occurs on the third bar of the chart (counting from the current bar = 1)
according to the logic of the signal code
old:
for(i = 0; i <= limit; i++) {
new:
for(i = 2; i <= MathMax(limit, 2); i++) {
The stable signal occurs on the third bar of the chart (counting from the current bar = 1)
according to the logic of the signal code
You are missing trading opportunities:
- Free trading apps
- Over 8,000 signals for copying
- Economic news for exploring financial markets
Registration
Log in
You agree to website policy and terms of use
If you do not have an account, please register
I found this moving average cross over indicator that draw arrow of buy and sell.
What I found that it is cheating 1 bar ahead and re-paint it.
The indicator is attached.
Please some one help me.
Below is the Code:::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
/*
+------------------------------------------------------------------+
| Allows you to enter two SMA periods and it will then show you at |
| Which point they crossed over. It is more usful on the shorter |
| periods that get obscured by the bars / candlesticks and when |
| the zoom level is out. Also allows you then to remove the SMAs |
| from the chart. (SMAs are initially set at 10 and 20) |
+------------------------------------------------------------------+
*/
#property indicator_chart_window
#property indicator_buffers 2
#property indicator_color1 Blue
#property indicator_color2 Red
double CrossUp[];
double CrossDown[];
extern int FasterSMA = 10;
extern int SlowerSMA = 20;
//+------------------------------------------------------------------+
//| Custom indicator initialization function |
//+------------------------------------------------------------------+
int init()
{
//---- indicators
SetIndexStyle(0, DRAW_ARROW, EMPTY);
SetIndexArrow(0, 233);
SetIndexBuffer(0, CrossUp);
SetIndexStyle(1, DRAW_ARROW, EMPTY);
SetIndexArrow(1, 234);
SetIndexBuffer(1, CrossDown);
//----
return(0);
}
//+------------------------------------------------------------------+
//| Custom indicator deinitialization function |
//+------------------------------------------------------------------+
int deinit()
{
//----
//----
return(0);
}
//+------------------------------------------------------------------+
//| Custom indicator iteration function |
//+------------------------------------------------------------------+
int start() {
int limit, i, counter;
double fasterSMAnow, slowerSMAnow, fasterSMAprevious, slowerSMAprevious, fasterSMAafter, slowerSMAafter;
double Range, AvgRange;
int counted_bars=IndicatorCounted();
//---- check for possible errors
if(counted_bars<0) return(-1);
//---- last counted bar will be recounted
if(counted_bars>0) counted_bars--;
limit=Bars-counted_bars;
for(i = 0; i <= limit; i++) {
counter=i;
Range=0;
AvgRange=0;
for (counter=i ;counter<=i+9;counter++) {
AvgRange=AvgRange+MathAbs(High[counter]-Low[counter]);
}
Range=AvgRange/10;
fasterSMAnow = iMA(NULL, 0, FasterSMA, 0, MODE_SMA, PRICE_TYPICAL, i);
fasterSMAprevious = iMA(NULL, 0, FasterSMA, 0, MODE_SMA, PRICE_TYPICAL, i+1);
fasterSMAafter = iMA(NULL, 0, FasterSMA, 0, MODE_SMA, PRICE_TYPICAL, i-1);
slowerSMAnow = iMA(NULL, 0, SlowerSMA, 0, MODE_SMA, PRICE_TYPICAL, i);
slowerSMAprevious = iMA(NULL, 0, SlowerSMA, 0, MODE_SMA, PRICE_TYPICAL, i+1);
slowerSMAafter = iMA(NULL, 0, SlowerSMA, 0, MODE_SMA, PRICE_TYPICAL, i-1);
if ((fasterSMAnow > slowerSMAnow) && (fasterSMAprevious < slowerSMAprevious) && (fasterSMAafter > slowerSMAafter)) {
CrossUp[i] = Low[i] - Range*0.5;
}
else if ((fasterSMAnow < slowerSMAnow) && (fasterSMAprevious > slowerSMAprevious) && (fasterSMAafter < slowerSMAafter)) {
CrossDown[i] = High[i] + Range*0.5;
}
}
return(0);
}
========================================================