Hello guys I'm new!
I'm trying to code a strategy that includes ADX as a filter to determine a trending and a non trending market.
So I'm having issues expressing the ADX variable

I wrote it as


"vars ADX18 = series(ADX(Price,18));"


But I'm sure its wrong. I need help writing the code well. Thanks

First you need to define 'Price' by

Of course yes. I defined price already, just wanted to be sure of the code for expressing ADX because it doesn't seem to work


function run ()
{
vars Price = series(price());

vars ADX18 = series(ADX(Price,18));

}

Correct?

'doesn't seem to work' is't a clear explanation of your problem.

After you define the ADX18 you need to include a trade condition.

We can help code writing, but obviously you should first reveal the purpose of your code and what problem you had with it.

Okay lets say a simple moving average crossover system with ADX filter
The strategy longs when SMA5 is above SMA20 and ADX18 > 20. Close longs when SMA5 crossunder SMA20 . The opposite for shorts

{

function run()

vars Price = series(price());

SMA5 = series(SMA(seriesC(),5));
SMA20 = series(SMA(series(),20));
vars ADX18 = series(ADX(price,18));

if(SMA5[0] > SMA20[0] and ADX18[0]>20) enterLong();
else if(SMA5[0]< SMA20[0] and ADX18[0]>20) enterShort();

if(crossOver(SMA5, SMA20)) exitShort();
else if(crossUnder(SMA5, SMA20)) exitLong();

}


When I ran it, zorro says
"ADX(): wrong number of parameters"

Look up the ADX function in the manual and check what parameters it has.

The manual defines ADX Function as ADX(int TimePeriod)



function run()

{

vars Price = series(price());

vars SMA5 = series(SMA(seriesC(), 5));
vars SMA20 = series(SMA(seriesC(), 20));
int TimePeriod = 18;
ADX(TimePeriod);


if(SMA5[0]>SMA20[0] and ADX[0] >20) enterLong();
else if(SMA5[0]<SMA20[0] and ADX[0] >20) enterShort();

if(crossOver(SMA5,SMA20)) exitShort();
if(crossUnder(SMA5,SMA20)) exitLong();

}


Ran it on Zorro. Brought out
" Error in line 13:
subscript requires a array or pointer type
if(SMA5[0]>SMA20[0] and ADX[0] >20) enterLong(); "

Please help with the correct code

ADX is a function, not a vars.

Got it guys. Thanks