Unable to detect the color of ray

[dp2002]

I am trying to detect a color of a ray or line that is manually drawn. The following describes the code I am using to detect the color of a ray object. I am drawing the ray on the chart and setting it to "Dark Red". The conditional statement comparing the color of the ray to Brushes.DarkRed never evaluates to True even though the print statement shows the values to be the exactly the same, "#FF8B0000". Not sure what I am doing wrong, but any assistance would be appreciated.

protected override void OnBarUpdate()
{
	//Add your custom indicator logic here.
	foreach (DrawingTool draw in DrawObjects)
	{
		if (draw is DrawingTools.Ray)
		{
			DrawingTools.Ray myRay = draw as DrawingTools.Ray;
			Print(string.Format("line color: {0}, Brushes.DarkRed: {1}",  myRay.Stroke.Brush, Brushes.DarkRed));
			if (myRay.Stroke.Brush == Brushes.DarkRed)
			{
				myRay.Stroke.Brush = Brushes.Black;	
			}
					
		}
	}
}

[NinjaTrader_Jesse]

Hello,

Thank you for the post.

In this case, an == with the brush objects would likely result in an untended comparison. Instead you could compare the string of each brush:

if (Equals(myRay.Stroke.Brush.ToString(), Brushes.DarkRed.ToString()))
{
	myRay.Stroke.Brush = Brushes.Black;	
}

I look forward to being of further assistance.

[dp2002]

That worked. Thank you.

[dp2002]

Since were converting the values to strings, is there a benefit of using,

if (Equals(myRay.Stroke.Brush.ToString(), Brushes.DarkRed.ToString())) {...}

instead of,

if (myRay.Stroke.Brush.ToString() == Brushes.DarkRed.ToString()) {...}

[NinjaTrader_Jesse]

Hello,

For a string not necessarily, I prefer to use Equals syntax because it is overload style syntax. Both would be acceptable for a string.

I look forward to being of further assistance.