iMA on array

Hello all,

I'm trying to put a moving average on an indicator I made but seem to have some trouble with it. For some reason it draws a straight line accros the indicator chart. I have placed the following code at the end of the indicator to use iMA on array calculation, but for some reason i can't get it to run the line correctly.

//----caluclate trigger MA
ArraySetAsSeries(force, true);
triggerline[i] = iMAOnArray(force,0,MA,0,MODE_EMA,0);

Can anyone help? The rest of the indicator is cacluating correctly and drawing the histogram I asked it to do, but for some reason the above code only draws a straight line.

it needs to be a part of a loop so it steps through the array calculating the ma. Currently you are only calculating based on the last element. change the last 0 you have to be the variable that tracks your position in the loop.

limit = bars-counted_bars;
for(int i=limit;i>=0;i--){
triggerline[i] = iMAOnArray(force,0,MA,0,MODE_EMA,i) ;
}

Magnum,

THANKS so much...that one little detail worked. Now the indicator works as I imagined it would. Now all I have to do is test it for accuracy.

THANK YOU AGAIN VERY MUCH.

Hi there, can somebody help ? I don't seem to be able to get this to work:
I just want an arrow on chart when RSI crosses on its EMA

//---- 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++)



double Rsi = iRSI(Symbol(), 0, RSI_Period, PRICE_CLOSE, i);
double RsiEMA=iMAOnArray(Rsi,0,EMA,0,MODE_EMA,i);

if (
Rsi > RsiEMA

)
{
CrossUp[i] = Low[i] - Range*Position;
CrossDown[i] = EMPTY_VALUE;
}
else
if (
Rsi < RsiEMA

)

from what i can tell just by eyeballing this for a moment is that this whole thing is wrong so you are basically asking for someone to do a total rewrite for you.

example of a very minor issue amongst greater problems... where is the 'array' for your imaonarray?

double Rsi = iRSI(Symbol(), 0, RSI_Period, PRICE_CLOSE, i);
double RsiEMA=iMAOnArray(Rsi,0,EMA,0,MODE_ EMA,i);


after you get that sorted you then have to look at where your loops are for setting up the array and then see if the logic still works how you intend.