C / C++ / MFC

leorex wrote:

Is there any common mistakes that people would make for drawing a lot of GDI objects?

leorex wrote:

Do I need to post my code to see which part went wrong?

leorex wrote:

Do I need to post my code to see which part went wrong?

leorex wrote:

for(i_row=0; i_row<17; i_row++)
{
for(j_column=0; j_column<7; j_column++)
{
hBrush_1 = CreateSolidBrush(color[i_row][j_column]);
holdBrush = SelectObject(hdc, hBrush_1);
Rectangle(hdc, 90*j_column+100, 50+30*i_row, 90*j_column+180, 30*i_row+75); //rember is x, y, x, y

}
}

hBrush_1 = CreateSolidBrush(color[i_row][j_column]);
SelectObject(hdc, hBrush_1);
Rectangle(hdc, 90*j_column+100, 50+30*i_row, 90*j_column+180, 30*i_row+75); //rember is x, y, x, y
DeleteObject(hBrush_1);

HBRUSH holdBrush = GetCurrentObject(hdc, OBJ_BRUSH);
...
hBrush_1 = CreateSolidBrush(color[i_row][j_column]);
SelectObject(hdc, hBrush_1);
Rectangle(hdc, 90*j_column+100, 50+30*i_row, 90*j_column+180, 30*i_row+75); //rember is x, y, x, y
SelectObject(hdc, holdBrush);
DeleteObject(hBrush_1);
...

//We make 3 brushes to play with
HBRUSH brush1    = CreateBrush...;
HBRUSH brush2    = CreateBrush...;
HBRUSH brush3    = CreateBrush...;

//Draw with the first brush
HBRUSH holdBrush = SelectObject(dc, brush1); //holdBrush now has the handle of the brush that was originally selected into the DC
...draw...

//Draw with the second brush
holdBrush = SelectObject(dc, brush2); //At this point, brush1 was previously selected into the DC, so now holdBrush equals brush1
...draw....

//Draw with the third brush
holdBrush = SelectObject(dc, brush3); //At this point, brush2 was previously selected into the DC, so now holdBrush equals brush2
...draw....

//We're done, cleanup (badly)
SelectObject(dc, holdBrush); //holdBrush has brush2 in it, so now we are simply selecting brush2 back into the DC
DeleteObject(brush1);
DeleteObject(brush2); //And here you delete the brush selected into the DC, bad move
DeleteObject(brush3);
...