OnExecutionUpdate

[delTik]

could someone check that I have this right ..

		protected override void OnExecutionUpdate(Execution execution, string executionId, double price, int quantity, MarketPosition marketPosition, string orderId, DateTime time)
		{
			if (execution.Order.Name.StartsWith("EnterLong"))
			{
				if (orderState == OrderState.Filled || orderState == OrderState.PartFilled)
		  		{
					//long position open
			 	}
			}
			else if (execution.Order.Name.StartsWith("ExitLong"))
			{
				if (orderState == OrderState.Filled || orderState == OrderState.PartFilled)
				{
					//long position closed				
				}
			}
			else if (execution.Order.Name.StartsWith("ExitLongLimit"))
			{
				if (orderState == OrderState.Accepted)
		  		{
					//ExitLongLimit accepted
				}
				else if (orderState == OrderState.Cancelled)
		  		{
					//ExtLongLimit cancelled e.g. CancelOrder()
				}
				else if (orderState == OrderState.Filled || orderState == OrderState.PartFilled)
		  		{
					//ExitLongLimit filled, long position closed
				}
			}
		}

I used the following order names e.g. EnterLong, ExitLong & ExitLongLimit

[NinjaTrader_ChelseaB]

Hello delTik,

I would not be able to say if this is correct as the code that places the entry orders is not included. The names you are looking for would need to match the signal names used for the order.

Using string.StartsWith is fine to use.

You can also use IOrders to match orders.

For the most part this looks fine.
The question is, when you test the code does it do what you are expecting?