Execution Interface

[dibDab]

Re. https://ninjatrader.com/support/help...er_methods.htm

On the OnExecutionUpdate() event, is it correct to use

If (execution.MarketPosition == MarketPosition.Long) ?

[NinjaTrader_Jesse]

Hello

You can use that, that is a property of the event and the execution object.

https://ninjatrader.com/support/helpGuides/nt8//onexecutionupdate.htm?zoom_highlightsub=onexecution

https://ninjatrader.com/support/helpGuides/nt8//execution.htm?zoom_highlightsub=execution

[dibDab]

ok thanks, do you see anything wrong with this statement ...

if (execution.MarketPosition == MarketPosition.Long && (execution.Name == "long stop" || execution.Name == "long target" || execution.Name == "Exit on session close"))

sample population (from visual studio debug) is:

{execution='NT-00001-67' instrument='MES 12-22' account='Sim101' exchange=Default price=3906.5 quantity=1 marketPosition=Long orderId='NT-00002-67' time='2022-10-31 04:25:19' sod=False statementDate='0001-01-01'}

[NinjaTrader_Jesse]

Hello dibDab,

That's not really something I could answer as I don't have all the details, you can use Prints to identify if the values you are using are correct for the situation. You would need to output the variables you used there and see if that matches the condition you made.

[dibDab]

doesn't like the "==" operator, same problem using execution.Order == OrderAction.BuyToCover


not sure what else to use?

[NinjaTrader_Jesse]

Hello dibDab,

The error is saying you are trying to compare an order vs an int which wouldn't be a comparison that you can do. An enum such as OrderAction.BuyToCover equates to an integer when used in a condition which is why you see that in the error. You are trying to compare an enum against an order object.

The enum is named after the property it represents which is OrderAction, you would have to use execution.Order.OrderAction if you wanted to make that type of comparison.

[dibDab]

apologies cut & paste error, I get "==" operator error with

execution.Order.OrderAction == OrderAction.Sell & execution.MarketPosition == MarketPosition.Long

in protected override void OnExecutionUpdate

[NinjaTrader_Jesse]

Hello dibDab,

In what you provided you only used one &, for a condition you need &&.