MACD on 15 min charts

[Manuel17]

Hello,

I had a strategy working on daily bars and I am trying to convert it so that it works on 15 minutes bar at 3.45 pm.

However the following line of code is not working:

if (CrossAbove(MACD(macdfastactual, macdslowactual, macdsmoothactual), MACD(macdfastactual, macdslowactual, macdsmoothactual).Avg,macdlookbackactual)
				&& ToTime(Time[0]) == 154500
				)
            {
				
                EnterLong(1,"Buy MACD Crossover");				
            }

The following variables are initial declared( thats what I was using with a daily bar chart)

private int macdfast=12;
		private int macdslow=26;
		private int macdsmooth=9;
		private int macdlookback=3;

They are then transformed for a 15 minutes chart:

int macdfastactual=macdfast*periodsperday;
			int macdslowactual=macdslow*periodsperday;
			int macdsmoothactual=macdsmooth*periodsperday;
			int macdlookbackactual=macdlookback*periodsperday;

[NinjaTrader_ZacharyG]

Hello Manuel17,

Thank you for your inquiry.

Can you please clarify what you mean by the line of code not working?

Are you getting a compile error? Is the EnterLong() not executing?

Have you done any debugging to check if the conditions you have specified are true or not? You can do this by utilizing Prints.

We have a post on our support forum detailing debugging: http://ninjatrader.com/support/forum...58&postcount=1

[Manuel17]

Hello Zachary,

Sorry for the late reply, I was away. The EnterLong is not executing at all and I do not understand why. I tried to debug it and it does not work.

Manuel

[NinjaTrader_PaulH]

Hello Manuel17,

Thanks for your reply.

Please add a Print statement just above and just after the EnterLong() to see if you are getting the logic correct but the order is not being entered.

For example:
{
Print ("about to enterlong");
EnterLong(1,"Buy MACD Crossover");
Print ("Entered long");
}

If you do not get any print statements then you would need to look at the conditional statement:
if (CrossAbove(MACD(macdfastactual, macdslowactual, macdsmoothactual), MACD(macdfastactual, macdslowactual, macdsmoothactual).Avg,macdlookbackactual)
&& ToTime(Time[0]) == 154500)

Just above the condition you can add a print statement like: Print ("Bar: "+Time[0]); to verify the time. You can also add print statements to print out the values of Macd and it average so that you know going into the if statement what values it is evaluating. This is the only way you will know why it does or does not work.