Multiple Brushes in OnRenderTargetChanged()

[swcooke]

I am trying to understand brush concepts and I am looking at the documentation for OnRenderTargetChange() which gives this example:

public override void OnRenderTargetChanged()
{
  // if dxBrush exists on first render target change, dispose of it
  if (dxBrush != null)
  {
    dxBrush.Dispose();
  }
 
  // recalculate dxBrush from value caluled in OnBarUpdated when RenderTarget is recreated
  if (RenderTarget != null)
    dxBrush = brushColor.ToDxBrush(RenderTarget);
}

My question is, do I reproduce this code for each brush like this:

public override void OnRenderTargetChanged()
{
  //first brush
  if (dxBrush != null)  { dxBrush.Dispose(); }
  if (RenderTarget != null){dxBrush = brushColor.ToDxBrush(RenderTarget);}

    //second brush
  if (dxBrush2 != null)  { dxBrush2.Dispose(); }
  if (RenderTarget != null){dxBrush2 = brushColor2.ToDxBrush(RenderTarget);}

  //third brush
  if (dxBrush3 != null)  { dxBrush3.Dispose(); }
  if (RenderTarget != null){dxBrush3 = brushColor3.ToDxBrush(RenderTarget);}
}

Is this a common approach? Any pitfalls that have to do with the order of these?

[NinjaTrader_Jesse]

Hello swcooke,

Thank you for the post.

Yes, this is pretty standard and really shouldn't have any pitfalls. The problems would come when not correctly recreating brushes and you end up trying to use invalid objects. Recreating the brushes when the render target changes is needed so this type of logic is suggested to be used.

I do see you are using the same variable for each line of syntax in your sample unless that was intended:

[B]brushColor[/B].ToDxBrush(RenderTarget);

I look forward to being of further assistance.

[swcooke]

Thanks for pointing that out. I have edited my post and fixed that. One last question. Can you confirm that the order of each of these does not matter?

[NinjaTrader_Jesse]

Hello swcooke,

Yes, the order should not matter, mainly you want to include objects that do need to be recreated in this override. You can make the order inside of the method however it makes logical sense and looks good to you.

I look forward to being of further assistance.