Announcement

Collapse

Looking for a User App or Add-On built by the NinjaTrader community?

Visit NinjaTrader EcoSystem and our free User App Share!

Have a question for the NinjaScript developer community? Open a new thread in our NinjaScript File Sharing Discussion Forum!
See more
See less

Partner 728x90

Collapse

Requesting clarification on SampleOnOrderUpdate code

Collapse
X
 
  • Filter
  • Time
  • Show
Clear All
new posts

  • NinjaTrader_BrandonH
    replied
    Hello markdshark,

    Thanks for your post.

    If you have an order quantity of two, one order with a stop and target and the second order will only a stop, you would need to create a separate Stop order object for the script that has only a stop and no target, such as stopOrder2, so that the separate Stop order object remains after the stopOrder and targetOrder were set to null.

    Let me know if I may assist further.

    Leave a comment:


  • Requesting clarification on SampleOnOrderUpdate code

    Hi there. I have a question about the very last few lines of this sample code provided. Here is the code:

    // Reset our stop order and target orders' Order objects after our position is closed. (1st Entry) <<<<<<<<<<<<<<<<< ?
    if ((stopOrder != null && stopOrder == execution.Order) || (targetOrder != null && targetOrder == execution.Order)){
    if (execution.Order.OrderState == OrderState.Filled || execution.Order.OrderState == OrderState.PartFilled){
    stopOrder = null;
    targetOrder = null;
    }
    }​

    My question is about the parenthesized bit at the end of the comment "(1st Entry)". I'm trying to understand how the code flows if say my order quantity is 2 orders, both with the same stoploss level set but for one of which I have set a target takeprofit.

    The second line appears to nullify the stopOrder object if the first takeprofit is hit but there's still a stop remaining.

Latest Posts

Collapse

Topics Statistics Last Post
Started by nitrojam, 08-17-2023, 04:26 AM
17 responses
228 views
0 likes
Last Post krish007  
Started by cay7man, Yesterday, 10:34 AM
17 responses
63 views
0 likes
Last Post cay7man
by cay7man
 
Started by SherwoodJP, 08-31-2023, 02:58 PM
2 responses
71 views
0 likes
Last Post LunaticEssence  
Started by habibalex, 01-10-2017, 01:02 PM
57 responses
8,639 views
0 likes
Last Post yeshe
by yeshe
 
Started by pennywise, 06-16-2023, 03:18 AM
77 responses
1,753 views
0 likes
Last Post GTBrooks  
Working...
X