Removing plots

[keepsimple]

Hi,
I have a strategy where i have a button, which includes the following code.
It is supposed to remove certain DrawObjects.
foreach(IDrawObject draw in DrawObjects)
{
if(draw.Tag.StartsWith("Name"))
{
Print(draw.Tag);
// RemoveDrawObject(draw.Tag);
}
ChartControl.ChartPanel.Refresh();
}

if I run the code like that without the RemoveDrawObject statement, it prints all the desired DrawObjects to the Output Window.
If I uncomment the RemoveDrawObject statement,
it prints only one tag and removes only one Object.
Why does the code not remove all objects which were printed to the Output window, when the statement was commented out?

[NinjaTrader_CalH]

Hello Keepsimple,

Thank you for your note.

You might be getting an Object handling error in either the Log tab or the Trace file.

You can check the Trace file which is located in (My) Documents -> NinjaTrader 7 -> Trace

You will see a line with **************** Handled Exception trapped************

You can also send me the file to platformsupport [at] ninjatrader [dot] com
Put ATTN Cal in the subject and reference this thread in the body

[keepsimple]

ys, think you are right, here is the log.

2014-10-14 15:36:02:823 in OnUnhandledThreadException
2014-10-14 15:36:02:823 *************** unhandled exception trapped ***************
2014-10-14 15:36:02:823 Die Auflistung wurde geändert. Der Enumerationsvorgang kann möglicherweise nicht ausgeführt werden.
2014-10-14 15:36:02:823 bei System.Collections.Hashtable.HashtableEnumerator.M oveNext()
bei NinjaTrader.Gui.Chart.DrawObjectsEnumerator.MoveNe xt()
bei NinjaTrader.Strategy.MyOFTBeta01.Split_Click(Objec t sender, EventArgs e)
bei System.Windows.Forms.ToolStripItem.RaiseEvent(Obje ct key, EventArgs e)
bei System.Windows.Forms.ToolStripButton.OnClick(Event Args e)
bei System.Windows.Forms.ToolStripItem.HandleClick(Eve ntArgs e)
bei System.Windows.Forms.ToolStripItem.HandleMouseUp(M ouseEventArgs e)
bei System.Windows.Forms.ToolStripItem.FireEventIntera ctive(EventArgs e, ToolStripItemEventType met)
bei System.Windows.Forms.ToolStripItem.FireEvent(Event Args e, ToolStripItemEventType met)
bei System.Windows.Forms.ToolStrip.OnMouseUp(MouseEven tArgs mea)
bei System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
bei System.Windows.Forms.Control.WndProc(Message& m)
bei System.Windows.Forms.ScrollableControl.WndProc(Mes sage& m)
bei System.Windows.Forms.ToolStrip.WndProc(Message& m)
bei System.Windows.Forms.Control.ControlNativeWindow.O nMessage(Message& m)
bei System.Windows.Forms.Control.ControlNativeWindow.W ndProc(Message& m)
bei System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

Do not know, what causes the error.

[NinjaTrader_CalH]

Keepsimple,

What is occuring is the foreach is getting the number of objects in the collection of DrawObjects, but since we are removing an object from that collection we get the enumeration error.

You will need to convert the foreach over to a for loop. This should work around the enumeration error that you are getting.

for(int i = 0; i < DrawObjects.Count; i++)
{

}

Let me know if I can be of further assistance.

[keepsimple]

Thanks,

yes that might work, but how do I acces the DrawObject to remove then?

RemoveDrawObject(???????), what do I put in the brackets?

[NinjaTrader_CalH]

Keepsimple,

You would want to use the same logic but slightly modified -

for(int i = 0; i < DrawObjects.Count; i++)
{
     if(DrawObjects[i].Tag.StartsWith("Name"))
     { 
          Print(DrawObjects[i].Tag);
          RemoveDrawObject(DrawObjects[i].Tag);
     }
     ChartControl.ChartPanel.Refresh();
}

[keepsimple]

Hi Cal,

thought about something like that.
I copied and pasted your code.
But unfortunately it does not compile.
I get several error messages,
1. int cannot be converted into string
2. this[string] method has invalid arguments

Really appreciate your help, which was great so far.

[NinjaTrader_CalH]

Keepsimple,

Can you please give me a visual of those compile error messages or attach your script so that I can look into it?

[keepsimple]

see Attachment

[NinjaTrader_CalH]

Keepsimple,

Can you send me the script so that I can test this on my end?

[keepsimple]

Thanks Cal,

for your help, which was really great.

Solved the problem.

Regards