Announcement

Collapse
No announcement yet.

Partner 728x90

Collapse

Suggestion: move brush serialization into backround

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

    Suggestion: move brush serialization into backround

    Hi.

    1) Every BRUSH inputs have to be serialized inside code. That makes an extra code inside inis/strategies.

    Code:
    		[NinjaScriptProperty]
    		[XmlIgnore]
    		[Display(Name="Smth", Description="", Order=1, GroupName="Parameters")]
    		public Brush XYZ
    		{ get; set; }
    
    		[COLOR="Red"][Browsable(false)]
    		public string XYZSerializable
    		{
    			get { return Serialize.BrushToString(XYZ); }
    			set { XYZ= Serialize.StringToBrush(value); }
    		}	
    [/COLOR]
    Thus, to make the user-end code cleaner, i dont know, why it is not possible, that NT removed the need for that lines to be coded inside script, and did that standard serialization in the background. and if there is a need for explicitly do that in code, that user had an "optionally" to insert that code of block.


    2) also, the auto-generated code which is tied in the end of every script (ninjatrader code, neither change nor remove and below...) - i think that also makes the place heavier. Why just NT cant embed that code in the back-end itself, out of the editor/script? thus, script will be much cleaner for developers.

    3) will be excellent to add this feature: we could put the default value inside input, like:
    Code:
        [NinjaScriptProperty]
        [Display(Name = "Minutes To Expire", Default=5)]
        public int ExpireMinutes

    so, no need to additionally initialize that variable with:

    Code:
    public int ExpireMinutes = 5;
    will make the code much cleaner. (thus, there comes a possibility to move all user-input #region in the top of the script, which is excellent! like all platforms have user-input codes in the top !!



    =================================

    please send these suggestion to dev.team, hope you will see the profitability of these things.
    Last edited by ttodua; 09-28-2017, 04:10 AM.

    #2
    Hello,

    Thank you for the post.

    I will forward these items to development as a feature request.

    Please let me know if I may be of additional assistance.
    JesseNinjaTrader Customer Service

    Comment

    Latest Posts

    Collapse

    Topics Statistics Last Post
    Started by CortexZenUSA, Today, 12:53 AM
    0 responses
    1 view
    0 likes
    Last Post CortexZenUSA  
    Started by CortexZenUSA, Today, 12:46 AM
    0 responses
    1 view
    0 likes
    Last Post CortexZenUSA  
    Started by usazencortex, Today, 12:43 AM
    0 responses
    5 views
    0 likes
    Last Post usazencortex  
    Started by sidlercom80, 10-28-2023, 08:49 AM
    168 responses
    2,266 views
    0 likes
    Last Post sidlercom80  
    Started by Barry Milan, Yesterday, 10:35 PM
    3 responses
    13 views
    0 likes
    Last Post NinjaTrader_Manfred  
    Working...
    X