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

Why duplicate declarations in built-in indicator?

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

    Why duplicate declarations in built-in indicator?

    in CurrentDayOHLC indicator's (built-in indicator) source we see, that the six fields (currentDate, currentOpen , etc...) are assigned value in class declaration:

    private DateTime currentDate = Core.Globals.MinDate;
    ...


    and also, in state.configure.

    currentDate = Core.Globals.MinDate;
    ...



    I'm interested, generally, why the value assignment is needed two times? isn't it enough, when in class declaration they have values assigned already?

    #2
    Hello TazoTodua,
    Thanks for your post.

    Generally you are correct that you would not need to assign a value twice like it is done in this indicator.

    Looks like the development team simply wanted to ensure that it was assigned once during OnStateChange. This would ensure that value is updated when the indicator is added to the list of applied objects.
    Josh G.NinjaTrader Customer Service

    Comment

    Latest Posts

    Collapse

    Topics Statistics Last Post
    Started by elirion, Today, 01:36 AM
    0 responses
    3 views
    0 likes
    Last Post elirion
    by elirion
     
    Started by gentlebenthebear, Today, 01:30 AM
    0 responses
    2 views
    0 likes
    Last Post gentlebenthebear  
    Started by samish18, Yesterday, 08:31 AM
    2 responses
    9 views
    0 likes
    Last Post elirion
    by elirion
     
    Started by Mestor, 03-10-2023, 01:50 AM
    16 responses
    389 views
    0 likes
    Last Post z.franck  
    Started by rtwave, 04-12-2024, 09:30 AM
    4 responses
    33 views
    0 likes
    Last Post rtwave
    by rtwave
     
    Working...
    X