Help with indicator please

double ExtMapBuffer1[];
double RSI_Val[[color=DarkOrange]put a number here[/color]];
double RSI_Val_SMA[[color=DarkOrange]put a number here[/color]];

Quoting sangmane

Disliked

int init()
{
IndicatorBuffers(3);
SetIndexStyle(0,DRAW_LINE);
SetIndexBuffer(0,ExtMapBuffer1);
SetIndexBuffer(1,RSI_Val);
SetIndexBuffer(2,RSI_Val_SMA);
return;
}

Ignored

Quoting lmyyyks

Disliked

i guess i've figured out the problem you have to set the array size,

Inserted Code

double ExtMapBuffer1[];
double RSI_Val[[color=DarkOrange]put a number here[/color]];
double RSI_Val_SMA[[color=DarkOrange]put a number here[/color]];

but then another problem appears.
the calculated figure doesn't match the calculated figure displayed in the chart..........

Ignored

Quoting dawsonsg

Disliked

Hi lmyyyks,

Thanks for the email. I don't want to sound stupid but what else do I have to put in. I changed it to what I have below but there is still no output.

Essentially I want it to display a line with a value of either a 1 or a...

Ignored

SetArrayAsSeries(RSI_Val_SMA, true);
SetArrayAsSeries(RSI_Val, true);

Quoting lmyyyks

Disliked

put this

Inserted Code

SetArrayAsSeries(RSI_Val_SMA, true);
SetArrayAsSeries(RSI_Val, true);

maybe the functions are spelt wrong or parameters count is wrong. I don't have the reference right now. But it's the way I dealt with it.
DO REMEMBER to check the returned value, because it is not the same value as displayed both in the navigator window and the chart.

I don't remember having sent you any e-mail btw.

Ignored

Quoting lmyyyks

Disliked

rsival and rsivalsma are not to be drawn, is is necessary to set buffer for them to be able to be used by imaonarray?

Ignored

//+------------------------------------------------------------------+
//| iMAOnArray.mq4 |
//| Coders Guru |
//| http://www.metatrader.info |
//+------------------------------------------------------------------+

#property copyright "Coders Guru"
#property link "http://www.metatrader.info"


#property indicator_separate_window
#property indicator_color1 LawnGreen
#property indicator_color2 DarkBlue

extern int RSIPeriod = 14;
extern int RSISmooth = 5;

double ExtMapBuffer1[];
double RSI_Val[];
double RSI_Val_SMA[];

int init()
{
IndicatorBuffers(3);
SetIndexStyle(0,DRAW_LINE);
SetIndexBuffer(0,ExtMapBuffer1);
SetIndexBuffer(1,RSI_Val);
SetIndexBuffer(2,RSI_Val_SMA);
return;
}

int deinit()
{
return(0);
}

int start()
{
int bar, limit;

int counted_bars=IndicatorCounted();
if(counted_bars<0) return(-1);
if(counted_bars>0) counted_bars--;
limit=Bars-IndicatorCounted();


for(bar=0; bar<limit; bar++)
RSI_Val[bar] = iCustom(NULL, 0, "RSI",RSIPeriod,0,bar);

for(bar=0; bar<limit; bar++)
RSI_Val_SMA[bar]=iMAOnArray(RSI_Val,Bars,RSISmooth,0,MODE_SMA,bar);

for(bar=0; bar<limit; bar++){
if(RSI_Val[bar]>RSI_Val_SMA[bar])
ExtMapBuffer1[bar] = 1;
else
ExtMapBuffer1[bar] = 0;
}

return(0);
} 

Quoting sangmane

Disliked

no, it is not. but it will make arrays easy to use without have to worry about array initialization.

the main cause of the problem is this line:
RSI_Val[bar] = iCustom(NULL, 0, "RSI",0,0,bar);
using "0" means RSI Period = 0. It's better to use external parameter.
or use the iRSI() function

[/php]

Ignored