Help with displaying fixed text

[tkaboris]

ok but it still says tag1 doesnt exist in current context. i have all this code insdie OnMarketData.

[tkaboris]

Ok i removed remove object and it probably should work

[NinjaTrader_Jesse]

Hello tkaboris,

You need to surround strings with quotes in C#. "tag1".

We highly suggest using external C# learning resources if you plan on doing manual coding as that will greatly help with small hiccups like forgetting quotes around a string.

As a side note you don't need to use RemoveDrawObject if you are updating the text, you just call the method to draw the text again with the same tag. RemoveDrawObject is only used to permanently remove an object so its no longer displayed and is not used when updating an objects text or appearance.

[tkaboris]

ok. thank you
I dont understand what I do wrong but when in positive it diplays correct text, however when psition is in negative it doenst display correct ammount, nor ammount nor color. I have attached code before. is it something special to display negative number?

protected override void OnMarketData(MarketDataEventArgs marketDataUpdate)
        {
            if (marketDataUpdate.MarketDataType == MarketDataType.Last)
            {

                #region PNL on the chart
                NinjaTrader.Gui.Tools.SimpleFont fontNeutral = new NinjaTrader.Gui.Tools.SimpleFont("Verdana", 12) { Size = 20, Bold = true };
                NinjaTrader.Gui.Tools.SimpleFont fontPositive = new NinjaTrader.Gui.Tools.SimpleFont("Verdana", 12) { Size = 20, Bold = true };
                NinjaTrader.Gui.Tools.SimpleFont fontNegative = new NinjaTrader.Gui.Tools.SimpleFont("Verdana", 12) { Size = 20, Bold = true };

                if (Position.MarketPosition != MarketPosition.Flat){
                    if(Position.GetUnrealizedProfitLoss(PerformanceUnit.Currency, Close[0]) >= 0.1){

                    Draw.TextFixed(this, "tag1", "OPEN PNL $ " + Position.GetUnrealizedProfitLoss(PerformanceUnit.Currency, Close[0]).ToString(), TextPosition.BottomRight, Brushes.RoyalBlue,
                      fontPositive, Brushes.RoyalBlue, Brushes.RoyalBlue, 0);
                    }
                }
                else if (Position.MarketPosition != MarketPosition.Flat){
                    if (Position.GetUnrealizedProfitLoss(PerformanceUnit.Currency, Close[0]) <= 0.0){
                    Draw.TextFixed(this, "tag1", "OPEN PNL $ " + Position.GetUnrealizedProfitLoss(PerformanceUnit.Currency, Close[0]).ToString(), TextPosition.BottomRight, Brushes.Red,
                      fontNegative, Brushes.Red, Brushes.Red, 0);
                    }
            }
                else if (Position.MarketPosition == MarketPosition.Flat){                    
                    Draw.TextFixed(this, "tag1", "DAILY PNL $ " + Account.Get(AccountItem.GrossRealizedProfitLoss, Currency.UsDollar), TextPosition.BottomRight, Brushes.White,
                      fontNeutral, Brushes.White, Brushes.White, 0);
                }



                #endregion​
}}

[tkaboris]

Never mind I had to move th closing bracket to the end of the second condition. Thank you