ExitLongStop & ExitLongLimit

[nordseven]

Yes i see in column Action - "BuytoCover" limit orders, always cancelled.
SellShort limit orders works fine and this is surprice for me, I didn't expect.

[bltdavid]

When i'm use ExitLongStop() with ExitLongLimit(), ExitLongLimit() always have been Cancelled. Why?

My onOrderUpdate() code:

[LEFT]if (myEntryOrder != null && myEntryOrder == order) {
                if (order.OrderState == OrderState.Filled) {
                    if (myStopOrder != null) { CancelOrder(myStopOrder); }
                    if (myProfitOrder != null) { CancelOrder(myProfitOrder); }
                    myStopOrder = ExitLongStop(0, true, qty, stopPrice, "long [B][COLOR=Red]stop[/COLOR][/B]", "long position");
                    myProfitOrder = ExitLongLimit(0, true, qty, profitPrice, "long [COLOR=Red][B]target[/B][/COLOR]", "long position");
                    myEntryOrder = null;[/LEFT]
                }
            }

Don't do this from OnOrderUpdate, use OnExecution instead.

After moving this code to OnExecution, remove these lines,

if (myStopOrder != null) { CancelOrder(myStopOrder); }
if (myProfitOrder != null) { CancelOrder(myProfitOrder); }

Also recommend you use unique signal names, see changes in red above.

[nordseven]

I'm tried to use onExecution() before, but don't changing name (long stop, long profit). Now it works, thank you!

Little question!
If i'm move everything to onExecution() why i must remove lines

if (myStopOrder != null) { CancelOrder(myStopOrder); }
if (myProfitOrder != null) { CancelOrder(myProfitOrder); }

onExecution() do something for canceling my orders?

[NinjaTrader_PatrickH]

Hello nordseven,

Glad you go that working. Can you provide more detail on what you mean on your last question?

[bltdavid]

I'm tried to use onExecution() before, but don't changing name (long stop, long profit). Now it works, thank you!

Little question!
If i'm move everything to onExecution() why i must remove lines

if (myStopOrder != null) { CancelOrder(myStopOrder); }
if (myProfitOrder != null) { CancelOrder(myProfitOrder); }

onExecution() do something for canceling my orders?

Because they should already be null. Other code elsewhere is responsible for making sure they become null.

If they are not null at this point, inside OnExecution, you actually have a bigger problem.

Think about it carefully: why possibly would a leftover stop or target still exist such that you need to cancel it just before submitting a new stop and target order? That's not how it (**) works.

(**) By "it" I mean the flow of order management, or in other words: the dance between OnOrderUpdate and OnExecution and "who sets who to null, and when".