Auto refresh screen

[Benluk]

I am new to OOP language.

I download MACDUpDown from Ninjatrader script sharing file for 7.0.
Then I added a couple lines of coding at the bottome of OnBarUpdate() section as follows:

// My code - MACD main vs signal dots
if (Value[0] > Avg[0] && Value[1] < Avg[1] )
DrawDot("tag"+CurrentBar, true, Time[0], Low[0], Color.Lime);

if (Value[0] < Avg[0] && Value[1] > Avg[1] )
DrawDot("tag"+CurrentBar, true, Time[0], High[0], Color.Red);

The intention is to display a dot on the price chart when MACD main and signal line crosses over. It looks fine when I update my chart at the end of a day. I see green/red/green/red pattern.

When I applied it on a real-time chart, e.g. 3-min chart, during that bar interval, the condition may be met. A dot is generated. But when the condition was changed and the criteria was no longer met, the dot stays. Then I see pattern like green/green/red/red, etc.

When I hit F5 to refresh screen, the chart becomes correct and cleaned up.

How to have a clean up a chart in real-time without hitting F5? Before a price interval is completed, when the condition is changed, how to remove the display of a dot?

Thank you.

[koganam]

It sounds like you have "CalculateOnBarClose = false". Is that correct?

[Benluk]

Yes. It is correct.

[koganam]

In that case, you have to code around the specific scenario that you are trying to create. Essentially you are evaluating the condition on every tick, and then you want to draw or remove the dot as required.

Here is how I would go about it.

bool Condition1 = Value[0] > Avg[0] && Value[1] < Avg[1] ;
bool Condition2 = Value[0] < Avg[0] && Value[1] > Avg[1] ;

if (Condition1)
{
DrawDot("tag"+CurrentBar, true, Time[0], Low[0], Color.Lime);
}
else if (Condition2)
{
DrawDot("tag"+CurrentBar, true, Time[0], High[0], Color.Red);
}
else
{
RemoveDrawObject("tag"+CurrentBar);
}

I was too lazy to actually test the code (a no, no, I know, but hey?), so if you get any errors, let me know, and I will stop being lazy.

Of course, you can concatenate the code the way you have it. This just happens to be my coding style, because it will be so much easier for me to understand what the heck I was thinking if I look at this code again a few months from now, when I want to change the conditions or something else for that matter. I can just change the conditions code, and not bother with the rest of my code at all.

[Benluk]

Thank you for the suggestion. I will test it when I have chance again.

I assume that the removedrawobject will only remove the unnecessary dots created by this particular coding segment.

If it happens that, for the same price bar, it has another dots created by other coding segment, will this removedrawobject remove other dots too?

[koganam]

Thank you for the suggestion. I will test it when I have chance again.

I assume that the removedrawobject will only remove the unnecessary dots created by this particular coding segment.

If it happens that, for the same price bar, it has another dots created by other coding segment, will this removedrawobject remove other dots too?

RemoveDrawObject() is highly specific and targeted. It will remove ONLY the object that has the specified tag. If your DrawObjects have unique tags, then only one object can possibly be removed; the one that has the specified tag,