closing open orders on terrmination

[dadarara]

Hi

I am using the bellow code.
and get an error of object reference is not set to an instance of an object

what am I not defining correctly ? do I need to specifically define the account ?

else if (State == State.Terminated)
{
foreach (Order order in Account.Orders)
{
if ( Position.MarketPosition == MarketPosition.Flat) CancelOrder(order);
}
}

also trying :

for( int i = 0; i < Account.Orders.Count; i++ )
{
if ( Position.MarketPosition == MarketPosition.Flat) CancelOrder(Account.Orders[i]);
}

thanks

[NinjaTrader_BrandonH]

Hello dadarara,

Thank you for your post.

I would not suggest checking PositionAccount when attempting to cancel all orders on the account as you would be canceling all orders, not just those of the instrument the strategy is applied to.

If you want to cancel all orders on the account you could do something like the following, this will also work in State.Terminated.

if (Account != null)
foreach (Order actOrdr in Account.Orders)
{
Account.Cancel(new[] { actOrdr });
}

If you are trying to have all strategy orders canceled when disabling the strategy, I would instead suggest checking the Cancel entry orders when a strategy is disabled and the Cancel exit orders when a strategy is disabled options in the Control Center's Tools > Options > Strategies menu

Here are a few Help Guide links for more information

Account class - https://ninjatrader.com/support/help...ount_class.htm

NinjaScript Properties - https://ninjatrader.com/support/help...riptProperties

Please let us know if we may further assist.