Announcement

Collapse
No announcement yet.

Partner 728x90

Collapse

Feature Request: How to make setting Time Properties more user friendly

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

    Feature Request: How to make setting Time Properties more user friendly

    Many places in Ninja Trader the GUI permits a trader to set the value of a TIME property. ie: Chart Properties: "Session Begins (local time)" & "Session Ends (local time)

    It is clear to me that you are using code similar to TimeSpan.Parse(txtSessionStart) to see if the characters the traders entered is a valid time. Unfortunately the TimeSpan Parse isn't great at recognising valid times. ie "6:00 AM" is valid but "6:00AM" & "6pm" are not valid, but they should be.

    In Ninja 7.0 please consider changing this.
    Use the DateTime.Parse() Method to parse the users entry & then extract the time portion with the TimeOfDay method. eg:

    private TimeSpan tsBoxBegin = DateTime.Parse(@"07:00 PM").TimeOfDay;

    It is way more robust & effective way to validate user entry.

    Hope this helps
    Dave

    #2
    Hello David Lean,

    Thank you for your post and suggestion. I will forward this to our development team for future considerations.

    Please let me know if I may be of further assistance.
    Christopher J.NinjaTrader Customer Service

    Comment

    Latest Posts

    Collapse

    Topics Statistics Last Post
    Started by DJ888, 04-16-2024, 06:09 PM
    4 responses
    11 views
    0 likes
    Last Post DJ888
    by DJ888
     
    Started by terofs, Today, 04:18 PM
    0 responses
    5 views
    0 likes
    Last Post terofs
    by terofs
     
    Started by nandhumca, Today, 03:41 PM
    0 responses
    5 views
    0 likes
    Last Post nandhumca  
    Started by The_Sec, Today, 03:37 PM
    0 responses
    3 views
    0 likes
    Last Post The_Sec
    by The_Sec
     
    Started by GwFutures1988, Today, 02:48 PM
    1 response
    9 views
    0 likes
    Last Post NinjaTrader_Clayton  
    Working...
    X