Announcement

Collapse
No announcement yet.

Partner 728x90

Collapse

Requesting clarification on SampleOnOrderUpdate code

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

    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.

    #2
    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.
    <span class="name">Brandon H.</span><span class="title">NinjaTrader Customer Service</span><iframe name="sig" id="sigFrame" src="/support/forum/core/clientscript/Signature/signature.php" frameborder="0" border="0" cellspacing="0" style="border-style: none;width: 100%; height: 120px;"></iframe>

    Comment

    Latest Posts

    Collapse

    Topics Statistics Last Post
    Started by ttrader23, Today, 04:48 PM
    0 responses
    2 views
    0 likes
    Last Post ttrader23  
    Started by wisdomspoon, Today, 03:50 PM
    3 responses
    14 views
    0 likes
    Last Post rockmanx00  
    Started by Mountain_cast, 02-10-2025, 11:14 PM
    2 responses
    19 views
    0 likes
    Last Post Mountain_cast  
    Started by Robotman, Today, 01:06 PM
    2 responses
    37 views
    0 likes
    Last Post Robotman  
    Started by PH_GMT, Today, 12:40 PM
    4 responses
    19 views
    0 likes
    Last Post PH_GMT
    by PH_GMT
     
    Working...
    X