Click a button and change color on rectangle?

[norwaymfc]

I have a problem. I need to change color to the rectangle when I click a button on the toolbar.
Here is the code for OnDraw:

void CObligatorisk_oppgaveView::OnDraw(CDC* pDC){

	CObligatorisk_oppgaveDoc* pDoc = GetDocument();
	ASSERT_VALID(pDoc);
	if (!pDoc)
		return;

	// Tegner ut rektangel A og rektangel B og resultatet C
	pDC->FrameRect(rectA, &blackBrush);
	pDC->FrameRect(rectB, &blackBrush);

	if(fargered=true)
	{
		pDC->FrameRect(rectC, &redBrush);
	}
	else 
	{
		CBrush blackHatchBrush;
		blackHatchBrush.CreateHatchBrush(HS_FDIAGONAL, RGB(0,0,0));
		CBrush* pOldBrush = pDC->SelectObject(&blackHatchBrush);

		CPen blackPen;
		blackPen.CreatePen(PS_SOLID, 3, RGB(0, 0, 0));
		CPen* pOldPen = pDC->SelectObject(&blackPen);

		pDC->Rectangle(rectC);

		pDC->SelectObject(pOldPen);
		pDC->SelectObject(pOldBrush);

	}

Here is the code for the button:

void CObligatorisk_oppgaveView::OnFargerR()
{
	fargered = true;
	Invalidate();

}

Can some one help me?

[mmesser]

if(fargered=true)

Perhaps this should be

if(fargered==true)

[cilu]

I have a problem. I need to change color to the rectangle when I click a button on the toolbar.
Here is the code for OnDraw:

So what is the problem? Should we guess?

if(fargered=true)

Perhaps this should be

if(fargered==true)

You should not use any of the two forms, but:

if(fargered)

and if you want to test it agains false then

if(!fargered)

PS: doesn't you compiler give you a warning here

if(fargered=true)

It should.

[norwaymfc]

No warning from the compiler, strange since you said that.

I'll check...

[yaros]

The compiler generates a warning on an assignment within conditional expression on Warning Level 4, looks like you didn't change the default level 3.