• 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

    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.

    #2
    I agree.

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

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

    Comment


      #3
      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".

      Comment


        #4
        Hey guys,

        Thank you for your suggestion.
        Patrick H.NinjaTrader Customer Service

        Comment


          #5
          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()

          Comment

          Latest Posts

          Collapse

          Topics Statistics Last Post
          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
          2 views
          0 likes
          Last Post jaysinclair  
          Started by markbb10, Today, 12:48 PM
          0 responses
          6 views
          0 likes
          Last Post markbb10  
          Started by Brillo, Today, 11:53 AM
          0 responses
          2 views
          0 likes
          Last Post Brillo
          by Brillo
           
          Working...
          X