More Efficient - Check Account Position

[bc24fl]

I have the below code in OnBarUpdate and worried that it isn't the most efficient especially if it's being called multiple times per minute. Specifically I'm worried about the "LOCK". What is the impact of LOCK and is there a better way of dynamically getting the account's position.

// check if the account position is flat.

private bool isAccountPositionFlat()
{

int accountLength = getInstrumentName().Length;

lock (myAccount.Positions)
{
foreach (Position position in myAccount.Positions)
{
if (getInstrumentName() == position.Instrument.ToString )
{
return false;
}
}
}
return true;
}

[NinjaTrader_PaulH]

Hello bc24fl,

Thanks for your post.

The "lock" will prevent you from accessing that object until the lock is released. For example, NinjaTrader may lock this collection when adding to it or manipulating it, and lock would be used in the script to ensure that it is accessed when it is free.

Can you clarify what information you are looking for with your code and what you are trying to accomplish?