Method call problem within OnBarUpdate()

[Spiderbird]

Hi there,

I'm running into an issue where, within OnBarUpdate, I try to call the following code:

private void Stop_EndStrategy()
			{
				Stop_GoFlat();
				
				//  If no long/short positions, halt processing.
				if (Position.MarketPosition == MarketPosition.Flat)
				{	haltProcessing = true;	}
			}

Whenever I do this, the editor throws back both a "} expected" error and a "Type or namespace definition, or end of file expected" error.

If I take this bit of code out of the "protected override void OnBarUpdate()" section, everything compiles fine.

I checked the NT8 help files, but I'm wondering if I'm putting this method in the wrong place.

Can anyone help give some perspective?

[NinjaTrader_AlanP]

Hello Spiderbird,

Thank you for your note.

Without the full code I would be unable to know exactly what the issue is.

Below is a sample which demonstrates how to halt a strategy from processing.

https://ninjatrader.com/support/forum/showthread.php?t=4804

Please let us know if you need further assistance.

[koganam]

Hi there,

I'm running into an issue where, within OnBarUpdate, I try to call the following code:

private void Stop_EndStrategy()
			{
				Stop_GoFlat();
				
				//  If no long/short positions, halt processing.
				if (Position.MarketPosition == MarketPosition.Flat)
				{	haltProcessing = true;	}
			}

Whenever I do this, the editor throws back both a "} expected" error and a "Type or namespace definition, or end of file expected" error.

If I take this bit of code out of the "protected override void OnBarUpdate()" section, everything compiles fine.

I checked the NT8 help files, but I'm wondering if I'm putting this method in the wrong place.

Can anyone help give some perspective?

Whereas C#7 now does allow one to define "local functions' that are private and scoped to another method, I really find it hard to see why you need it here, and in any case, the syntax is not quite what you have shown. Example ref: https://blogs.msdn.microsoft.com/dot...in-csharp-7-0/

Much better, will be to define your method outside any other method, in class scope. I would suggest that you take the method code out of the OnBarUpdate() handler.

[Spiderbird]

Hi Koganam,

That did the trick (moving the code out of OnBarUpdate). Thanks for the C#7 reference as well.
Appreciate the help!

- R