Simple if statement not working in NinjaScript 8

[ieee1294]

Hi there,

I'm trying to put a dot in the chart when a condition is met. For some reason the if statement is not working properly. I'm using NinjaTrader 8.

In the protected override void OnBarUpdate() function, the following works if it's the only statement:

Draw.Dot(this, "Bull" + CurrentBar, true, 0, Low[0] - 5, Brushes.Blue); // draw a dot under every price bar

But if I try to compare the current to the previous bar like this, it's not working:

if (Low[0] >= Low[1]) // draw a dot whenever the current low is above the previous low
{
Draw.Dot(this, "Bull" + CurrentBar, true, 0, Low[0] - 5, Brushes.Blue);
}

Any idea what's wrong here? Thanks in advance

[NinjaTrader_PaulH]

Hello ieee1294,

Thanks for your post and welcome to the NinjaTrader forums!

If your script is not working as expected, always check the "log" tab of the control center for any run time type error messages. If your script is not checking for the CurrentBar to be more than 1 then it is possible that your script would error on bar index error. Reference: https://ninjatrader.com/support/help...currentbar.htm
So you might want to use:

if (Currentbar < 1) return; // ensure there are two bars before going further

If you want your draw statement to work on other instruments I would suggest changing Low[0] - 5 to Low[0] - 5 * TickSize TickSize is an instrument property that represents the smallest movement by that instrument, so in the example the dot would always be 5 ticks below the Low regardless of the applied instrument. Reference: https://ninjatrader.com/support/help.../?ticksize.htm

[ieee1294]

Paul, thanks for the extremely prompt reply - did not expect that on a New Year's Eve!

Didn't know about the log tab until now, and thanks - it's like you said, I didn't check for CurrentBar. That fixed my issue. Thanks for the TickSize tip, I'm also using it already, just showing simplified code to get the point across easily.

Hope you have a happy new year! Cheers.