Little issue using SendMail() to know when a Strategy is disabled

[sagetrade]

Thank you. Works well!

[pstrusi]

Would it also work if I use a bool? Like the above?

I don't see why not, Just make sure that variable's value does not change unwillingly in the code. That was just a simple example to do it but you can create a more refined one with more functions.

[sagetrade]

No problem, here you can see an easy workaround.
Previously declare an Integer variable called "a" in this example, then it follows like this:

private bool Realtime = false;

protected override void OnStateChange()
		{
			if (State == State.SetDefaults)
			{
				.......			     		
			}
			else if (State == State.Configure)
			{
				.......
			}
			else if (State == State.Realtime)
			{
			    [COLOR="Red"]Realtime = true[/COLOR];
			}
			else if (State == State.Terminated)
			{
				[COLOR="red"]if ( Realtime == true )[/COLOR]
				{
					SendMail ("[email protected]", "STRATEGY DISABLED ! ", " Review " );
				}
			}
		}

Would it also work if I use a bool? Like the above?

[pstrusi]

No problem, here you can see an easy workaround.
Previously declare an Integer variable called "a" in this example, then it follows like this:

protected override void OnStateChange()
		{
			if (State == State.SetDefaults)
			{
				.......				
			}
			else if (State == State.Configure)
			{
				.......
			}
			else if (State == State.Realtime)
			{
			    a=1;
			}
			else if (State == State.Terminated)
			{
				if ( a > 0 )
				{
					SendMail ("[email protected]", "STRATEGY DISABLED ! ", " Review " );
				}
			}
		}

[sagetrade]

Thanks Jim, excellent suggestion!

In fact, I can confirm to you that was the cause of the error in the log. With a few lines checking first the state was the solution.

Regards!

Dear pstrusi,

I face the same issue. Would you mind to share how you did rewrite the code?

[pstrusi]

Thanks Jim, excellent suggestion!

In fact, I can confirm to you that was the cause of the error in the log. With a few lines checking first the state was the solution.

Regards!

[NinjaTrader_Jim]

Hello pstrusi,

Thanks for opening the thread.

I am not sure if you are receiving a null reference exception elsewhere in OnStateChange(), but you should not have issue sending an email from State.Terminated.

I would suggest to add some additional code to control your email so it is triggered after State.Terminated happens after the strategy has reached State.Realtime or State.Historical, since State.Terminated is reached after you open a strategy dialog window. This is outlined in the documentation page linked below.

NinjaScript Lifecycle - https://ninjatrader.com/support/help...fecycle_of.htm

[pstrusi]

Since I like to know if my Strategy was disabled, I use the email sharing service; so I put this basic instruction:

protected override void OnStateChange()
{
if (State == State.SetDefaults)
{
........
}
else if (State == State.Configure)
{
........
}
else if (State == State.Terminated)
{
SendMail ("[email protected]", "STRATEGY DISABLED ! ", " Review " );
}
}

Whenever I set the Strategy from the Tab "Strategy", a "light error, non-disruptive" pops up in the log:
Strategy 'PRS': Error on calling 'OnStateChange' method: Object reference not set to an instance of an object.

It seems NOT to have serious consequences of any kind, it even works as thought but since I don't like to see errors, I'd like to know if perhaps there's some restriction to use sendmail() from state.Terminated, if so, please indicate me another way to send the alert.

Thanks