• If this is your first visit, you will have to register before you can post. To view messages, please scroll below and select the forum that you would like to visits. Questions? Be sure to check out the Forum FAQ.

Announcement

Collapse
No announcement yet.

Partner 728x90

Collapse

API Suggestion: remove slang namings in methods

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

  • koganam
    replied
    Originally posted by misova View Post
    Hi,
    to make new NT8 API more professional,
    I would suggest to deprecate slang sugar/shortcuts in names of the (public) methods like this:

    MasterInstrument.Round2TickSize(price);

    I don't find it professional and would prefer simple:

    MasterInstrument.RoundToTickSize(price);
    While we are about it, how about we rename these methods in the Serialize namespace?
    • Brush2String()
    • Pen2String()
    • String2Brush()
    • String2Pen()

    Leave a comment:


  • NinjaTrader_PatrickH
    replied
    Hey guys,

    Thank you for your suggestion.

    Leave a comment:


  • sledge
    replied
    Tried browsing the docs real quick... thought I came across more before.

    This is a new one:

    RoundDown2TickSize()

    There are other methods though that do use "To".

    Leave a comment:


  • sledge
    replied
    I agree.

    I think I saw another new method in nt8 with '2'.

    http://www.ninjatrader.com/support/f...round2ticksize

    Leave a comment:


  • misova
    started a topic API Suggestion: remove slang namings in methods

    API Suggestion: remove slang namings in methods

    Hi,
    to make new NT8 API more professional,
    I would suggest to deprecate slang sugar/shortcuts in names of the (public) methods like this:

    MasterInstrument.Round2TickSize(price);

    I don't find it professional and would prefer simple:

    MasterInstrument.RoundToTickSize(price);
    Last edited by misova; 05-23-2015, 03:21 AM.

Latest Posts

Collapse

Topics Statistics Last Post
Started by GeoOrtiz, 02-21-2020, 03:44 PM
1 response
10 views
0 likes
Last Post NinjaTrader_ChelseaB  
Started by KeenEdge, Today, 04:08 PM
0 responses
2 views
0 likes
Last Post KeenEdge  
Started by aipjcnow, Yesterday, 12:56 PM
2 responses
11 views
0 likes
Last Post NinjaTrader_ChelseaB  
Started by Nice-Spread, Yesterday, 02:02 AM
2 responses
14 views
0 likes
Last Post Nice-Spread  
Started by jaysinclair, Today, 01:04 PM
0 responses
3 views
0 likes
Last Post jaysinclair  
Working...
X