loop search for horizontal hand drawn lines by color --- what am I doing wrong ??

[llanqui]

Hello,

The code below finds the hand drawn horizontal lines, and the correct price...but it is not matching on the brush color!!! (black or cyan)

Anyone?


foreach (DrawingTool draw in DrawObjects.ToList())
{

// select User Drawn Horizontal or Ray Lines
if (draw.IsUserDrawn)
{

if (draw is DrawingTools.HorizontalLine)
{

DrawingTools.HorizontalLine thisLine = draw as DrawingTools.HorizontalLine;

Print ("Horizontal Line Found " + thisLine.StartAnchor.Price.ToString() + " ..... " + thisLine.Stroke.Brush.ToString() + " ..... " + Brushes.Cyan.ToString() + " ..... " + Brushes.Black.ToString());

if (thisLine.Stroke.Brush == Brushes.Black)

Print ("Matches BLACK");

if (thisLine.Stroke.Brush == Brushes.Cyan)

Print ("Matches CYAN");

)

)

)

[llanqui]

well, it is not pretty, but changing the Brush to a string and then comparing them works


private string thisColor;
private string myBlack = Brushes.Black.ToString();
private string myCyan = Brushes.Cyan.ToString();



DrawingTools.HorizontalLine thisLine = draw as DrawingTools.HorizontalLine;

thisColor = thisLine.Stroke.Brush.ToString();

if (thisColor == myBlack)
Print ("Matches BLACK");

if (thisColor == myCyan())
Print ("Matches CYAN");

if (thisColor == myBlack || thisColor == myCyan)


I really don't believe it has to be this convoluted...isn't there a more elegant way to compare two brushes/color?
.
(it works so unless I get a better answer will leave it that way)

[NinjaTrader_Jesse]

Hello llanqui,

What you have done is correct, brushes cannot be compared by using == you would have to get the brushes name and compare that instead which you are doing.