separate actions from disable()

[deathstar]

How would one separate the disable() so it sends mail prior to disabling?


if ( x0 > y0)

{
SendMail());
}
{

Disable();

[NinjaTrader_Matthew]

You can create a custom bool to track when the condition where SendMail is being called has been ran

			bool mailSent = false;
			
			if( 1 > 2)
			{
				SendMail();
				mailSent = true;
			}

Then when your bool is true, you can call Disable()

			
			if(mailSent)
			{
				Disable();
			}

[koganam]

How would one separate the disable() so it sends mail prior to disabling?


if ( x0 > y0)

{
SendMail());
}
{

Disable();

I would have to assume that you want to be sure that the SendMail() function actually functioned, before you Disable(). Whether the end mail server actually sent the mail is a different issue.

However, the SendMail() method does not return a value, so you have to come at it laterally; checking instead if the method threw an exception, then Disable() only if the method did not throw an exeption.
Use a try ... catch block.

1. In the exception block, set a string variable
2. Compare the string variable to the value that would be set by the exception, and if it is not set, then Disable().

[deathstar]

You can create a custom bool to track when the condition where SendMail is being called has been ran

			bool mailSent = false;
			
			if( 1 > 2)
			{
				SendMail();
				mailSent = true;
			}

Then when your bool is true, you can call Disable()

			
			if(mailSent)
			{
				Disable();
			}

I tried this but it seems to disable prior to sending mail. I had tried previously with a if x = 0 /x = 1

It works great if I just remove disable() from it but then my email gets tons of emails.

koganam, I like this idea, I will put this in.

[deathstar]

This is my last attempt I can ether get it so send one on every bar update or it closes BEFORE sending. My test mail come though everytime.


protected override void OnBarUpdate()
{
// Condition set 1
if (x >= y)
{
Disable();
c0 = true;
}

}
protected override void OnTermination()
{if (c0 == true)
{
SendMail("email1", "email2", "543", "your breaking my balls here");
}
}

Any ideas are accepted.

[NinjaTrader_Matthew]

As a test, please try the following which I know will work

private bool mailSent = false;
private bool onlyOnce = false;

       protected override void OnBarUpdate()
        {
                        // sends mail when strategy is real-time, and will only fire once
			if(!Historical && !onlyOnce)
			{
				
				SendMail(fromMe, toYou, "disableTest", "sent!");
			
				mailSent = true;
				onlyOnce = true;
			}
			
			if (mailSent == true)
			{
				Disable();
			}
        }