Need Help on this Paint Bar Study

[shzhning]

Hello, I'm trying to create a simple paint bar study, based on ADX and DM value. It works as a strategy, but would not "paint" bar as an indicator. It compiles without any issue. Can someone please help?

Here's the indicator code:

// Wizard generated variables
private int level = 20;
// Default setting for Level
// User defined variables (add any user defined variables below)
private Color upcolor = Color.Blue;
private Color downcolor = Color.Red;
#endregion

/// <summary>
/// This method is used to configure the indicator and is called once before any bar data is loaded.
/// </summary>
protected override void Initialize()
{
Overlay = true;
PriceTypeSupported = false;
}

/// <summary>
/// Called on each bar update event (incoming tick)
/// </summary>
protected override void OnBarUpdate()
{

if (ADX(5)[0] > level
&& ADX(5)[0] > ADX(5)[1]
&& DM(5).DiPlus[0] > DM(5).DiMinus[0])
{
BarColor = upcolor;
}

if (ADX(5)[0] > level
&& ADX(5)[0] > ADX(5)[1]
&& DM(5).DiPlus[0] < DM(5).DiMinus[0])
{
BarColor = downcolor;
}

}

[NinjaTrader_PatrickH]

Hello shzhning,

Thank you for your note.

When enabling the indicator on the chart do you see any errors listed on the Log tab of the Control Center? If so, what do they report?

[shzhning]

This is the message: Error on calling "OnBarUpdate" method for indicator "ADXPaintBar" on bar 0: You are accessing an index with a value that is invalid since it's out of range, I.E. acessing a series [barsAgo] with a value of 5 when there are only 4 bars on the chart.

How can I correct this issue? I do have dozens of bars - it's a generic 10 minute bar chart.

FYI: this part of code is directly copy and pasted from the strategy generated by ninjatrader

if (ADX(5)[0] > level
&& ADX(5)[0] > ADX(5)[1]
&& DM(5).DiPlus[0] > DM(5).DiMinus[0])

Thanks

[koganam]

This is the message: Error on calling "OnBarUpdate" method for indicator "ADXPaintBar" on bar 0: You are accessing an index with a value that is invalid since it's out of range, I.E. acessing a series [barsAgo] with a value of 5 when there are only 4 bars on the chart.

How can I correct this issue? I do have dozens of bars - it's a generic 10 minute bar chart.

FYI: this part of code is directly copy and pasted from the strategy generated by ninjatrader

if (ADX(5)[0] > level
&& ADX(5)[0] > ADX(5)[1]
&& (CurrentBar > 0)
&& DM(5).DiPlus[0] > DM(5).DiMinus[0])

Thanks

Added, in blue, to your original text.

[shzhning]

thanks for the quick reply. I added "&& (CurrentBar > 0)" as you suggested, but for some reason it's still not working. Received same error message in the log.

Really at a loss...

[koganam]

thanks for the quick reply. I added "&& (CurrentBar > 0)" as you suggested, but for some reason it's still not working. Received same error message in the log.

Really at a loss...

That means that your error is coming from a different part of your code.

I have reread your first post. There are 2 different code blocks, each of which will require the necessary modification. I responded to your post that showed only one block.

The alternative would be to use a Bars escape at the start of your OnBarUpdate() code. Put this as the first statement:

if (CurrentBar < 1) return;

[shzhning]

It's working now

Thank you!