RSI Strategy

[mccallum28]

Please can someone help me with this strategy, its a simple RSI, MA cross system but myprogramming capabilities are very basic and i am strugling to get this sorted.

I have the system entering long?short trades when the RSI crosses above or below the MA.

The problem is I only want the strategy to trade short when the cross happens above the 80 line and trade long when the crossover happens below the 20 line.

Please can someone help me with this code, everything i try comes up with compile errors...

[NinjaTrader_AdamP]

mccallum,

Please find an example below.

            // Condition set 1
            if ( CrossBelow(RSI_MA_350(3, 3).RSI_Plot, RSI_MA_350(3,3). RSI_MA_350_Plot, 1) && RSI(3,3)[0]>80 )							
            {
                EnterShort(DefaultQuantity, "");
            }

            // Condition set 2
            if ( CrossAbove(RSI_MA_350(3, 3).RSI_MA_350_Plot, RSI_MA_350(3,3).RSI_Plot, 1) && RSI(3,3)[0]<20 )
            {
                EnterLong(DefaultQuantity, "");
            }

I would suggest you watch the following video as it explains what the && and || operations are. They are ways of linking conditions based on logic.

http://www.youtube.com/watch?v=JZpo01eSO9c

[mccallum28]

Thank you very much, I was completely off. thank you...

[mccallum28]

Hi again,

I tried using the && function as suggested with RSI and and RSI_MA_350 but as soon as I add the code the strategy does not place any trades. I can see that the conditions have been met but no trades are placed.

I have attached the strategy and the indicator used, thanks for your help with this, I have been trying to get it right since 7am...

Ryan

mccallum,

Please find an example below.

            // Condition set 1
            if ( CrossBelow(RSI_MA_350(3, 3).RSI_Plot, RSI_MA_350(3,3). RSI_MA_350_Plot, 1) && RSI(3,3)[0]>80 )							
            {
                EnterShort(DefaultQuantity, "");
            }

            // Condition set 2
            if ( CrossAbove(RSI_MA_350(3, 3).RSI_MA_350_Plot, RSI_MA_350(3,3).RSI_Plot, 1) && RSI(3,3)[0]<20 )
            {
                EnterLong(DefaultQuantity, "");
            }

I would suggest you watch the following video as it explains what the && and || operations are. They are ways of linking conditions based on logic.

http://www.youtube.com/watch?v=JZpo01eSO9c

[NinjaTrader_AdamP]

mccallum,

It could be that the way I referenced the RSI is accessing a different plot than the one you wanted. I would suggest trying the following instead.

            // Condition set 1
            if ( CrossBelow(RSI_MA_350(3, 3).RSI_Plot, RSI_MA_350(3,3). RSI_MA_350_Plot, 1) && RSI(3,3).RSI_Plot[0]>80 )							
            {
                EnterShort(DefaultQuantity, "");
            }

            // Condition set 2
            if ( CrossAbove(RSI_MA_350(3, 3).RSI_MA_350_Plot, RSI_MA_350(3,3).RSI_Plot, 1) && RSI(3,3).RSI_Plot[0]<20 )
            {
                EnterLong(DefaultQuantity, "");
            }

[mccallum28]

Hi,

I get a CS106 compile error when I add that code.

I have No idea what to do to get this fixed..

[NinjaTrader_AdamP]

mccallum,

It looks like Joydeep has a ticket from you that you sent in to our support email. He will continue to work with you there.

[mccallum28]

Thanks Adam,