C++ MFC-приложение, ошибка при рисовании.

@PazDim · Регистрация: 08.08.2011

Author24 — интернет-сервис помощи студентам

Учусь программировать, изучаю MFC. Решил написать простенькое приложение. Суть в том, чтобы на форме отображался прямоугольник(создал для него класс, включающий цвет, координаты и т.д.). Он реагирует на нажатие клавиш, перемещаясь по окну(работает). Все это реализовано в функции, которая вызывается из таймера. Все работает, доволен, но где-то после четырех тысяч вызовов(таймер срабатывает каждые 100 миллисекунд) выскакивает ошибка:
http://imageshost.ru/photo/128018/id795763.html
Скореее всего ошибка проискходит в этом месте:

C++

#include "GenCode.h"
//////////GLOBAL VARIABLES
GRect MyRect(CPoint(200,200),50,50,RED);
void GenRun(CDC *pDC,CRect *WndRect)
{
    CKeys keys;
    if(keys.m_KeyA)
        MyRect.Go(-1,0,STEP);
    if(keys.m_KeyD)
        MyRect.Go(1,0,STEP);
    if(keys.m_KeyS)
        MyRect.Go(0,1,STEP);
    if(keys.m_KeyW)
        MyRect.Go(0,-1,STEP);
    //Закрасить фон
    CPen MyPen;
    MyPen.CreatePen(PS_NULL,1,RED);
    CPen *pOldPen=0;
    pOldPen=pDC->SelectObject(&MyPen);
    CBrush MyBrush;
    MyBrush.CreateSolidBrush(WHITE);
    CBrush *pOldBrush=0;
    pOldBrush=(CBrush*)pDC->SelectObject(&MyBrush);
    pDC->Rectangle(*WndRect);
    pDC->SelectObject(pOldBrush);
    pDC->SelectObject(pOldPen);
    /////////////////////
    MyRect.Draw(pDC);
};

Ошибка при попытке вогнать в *pDC свое перо. Проверял - отечки памяти нет.
http://narod.ru/disk/21182766001/MFC.rar.html
Это ссылка на файлы проекта, если нужны.
Как можно исправить ошибку? Заранее спасибо.

Jupiter · 08.08.2011, 16:25

через Ф10/Ф11 запускаетя без исключения

@PazDim · 08.08.2011, 17:01 **[ТС]**

Сообщение от Maxwe11

через Ф10/Ф11 запускаетя без исключения

А она и так запускается без исключения

. Исключение возникает где-то после 4000 вызовов функций.

Добавлено через 32 минуты
Пробовал вообще убрать операции замены пера - исключение при рисовании прямоугольника

@xAtom · 08.08.2011, 17:11

Сообщение от PazDim

CPen MyPen;
MyPen.CreatePen(PS_NULL,1,RED);
CPen *pOldPen=0;
pOldPen=pDC->SelectObject(&MyPen);
CBrush MyBrush;
MyBrush.CreateSolidBrush(WHITE);

C++

[b]PazDim[/b], а кто по-твоему удалять будет объекты GDI после их использования HBRUSH, HPEN, здесь классы CPen, CBrush, обёртки нужно вызывать метод DeleteObject. Вот так напиши

//...

Сообщение от PazDim

//Закрасить фон
CPen MyPen;
MyPen.CreatePen(PS_NULL,1,RED);
CPen *pOldPen=0;
pOldPen=pDC->SelectObject(&MyPen);
CBrush MyBrush;
MyBrush.CreateSolidBrush(WHITE);
CBrush *pOldBrush=0;
pOldBrush=(CBrush*)pDC->SelectObject(&MyBrush);
pDC->Rectangle(*WndRect);
/////////////////////
MyRect.Draw(pDC);

MyPen.DeleteObject();
MyBrush.DeleteObject();
pDC->SelectObject(pOldBrush);
pDC->SelectObject(pOldPen);
pDC->ReleaseOutputDC();

Не удивительно, что ошибка вызывалась.

Jupiter · 08.08.2011, 17:18

Сообщение от xAtom

PazDim, а кто по-твоему удалять будет объекты GDI после их использования HBRUSH, HPEN, здесь классы CPen, CBrush, обёртки нужно вызывать метод DeleteObject. Вот так напиши

ужос...они внутри используют динамическую память?

@PazDim · 08.08.2011, 20:27 **[ТС]**

Ну вот и первые грабли

. Спасибо.
ЗЫ. Пробовал вообще все связанное с рисованием удалить - исключение не вылезает, но приложение конкретно виснет. Сейчас попробую удалить перо и кисть.
...
ЗЗЫ. Не, все равно ошибка вылазит

. Вот что сделал:

C++

#include "GenCode.h"
//////////GLOBAL VARIABLES
GRect MyRect(CPoint(200,200),50,50,RED);
void GenRun(CDC *pDC,CRect *WndRect)
{
    CKeys keys;
    if(keys.m_KeyA)
        MyRect.Go(-1,0,STEP);
    if(keys.m_KeyD)
        MyRect.Go(1,0,STEP);
    if(keys.m_KeyS)
        MyRect.Go(0,1,STEP);
    if(keys.m_KeyW)
        MyRect.Go(0,-1,STEP);
    //Закрасить фон
    CPen MyPen;
    MyPen.CreatePen(PS_NULL,1,RED);
    CPen *pOldPen=0;
    pOldPen=pDC->SelectObject(&MyPen);
    CBrush MyBrush;
    MyBrush.CreateSolidBrush(WHITE);
    CBrush *pOldBrush=0;
    pOldBrush=(CBrush*)pDC->SelectObject(&MyBrush);
    pDC->Rectangle(*WndRect);
    pDC->SelectObject(pOldBrush);
    pDC->SelectObject(pOldPen);
    MyPen.DeleteObject();
    MyBrush.DeleteObject();
    /////////////////////
    MyRect.Draw(pDC);
};

Ну и в функции Draw тоже удаляю перо. Если перо/кисть не использовать, то исключение будет при рисовании прямоугольника...

Добавлено через 17 минут
Блин, pDC->ReleaseOutputDC() сразу не разглядел... Что это?

Добавлено через 17 минут
Исключение по-моему вызывается из-за того, что в какой-то момент указатель pDC становится равным NULL(и без ReleaseOutputDC()). Функцию вызываю из таймера:

C++

void CMFCWnd::OnTimer(UINT_PTR nIDEvent)
{
    CRect WndRect(0,0,0,0);
    this->GetWindowRect(WndRect);
    GenRun(this->GetDC(),&WndRect);
}

Немного напрягает то, что функция GetDC() вызывается каждый раз. Так должно быть?

Добавлено через 2 часа 34 минуты
Временное решение проблемы найдено: использовать CClientDC вместо CDC

. Уже сейчас, пока я пишу эти строки, зарегестрировано 41138 запусков функции рисования

(при использовании CDC их было около 5000). Интересно, чего они там такого понапихали в CClientDC, что не вызывается исключение? Может кто-нибудь объяснить?

Jupiter Каратель 6610 / 4029 / 401 Регистрация: 26.03.2010 Сообщений: 9,273 Записей в блоге: 1
	08.08.2011, 16:25	2
	через Ф10/Ф11 запускаетя без исключения 0

@PazDim 5 / 5 / 1 Регистрация: 08.08.2011 Сообщений: 118
	08.08.2011, 17:01 [ТС]	3
	Сообщение от Maxwe11 через Ф10/Ф11 запускаетя без исключения А она и так запускается без исключения . Исключение возникает где-то после 4000 вызовов функций. Добавлено через 32 минуты Пробовал вообще убрать операции замены пера - исключение при рисовании прямоугольника 0

Jupiter Каратель 6610 / 4029 / 401 Регистрация: 26.03.2010 Сообщений: 9,273 Записей в блоге: 1
	08.08.2011, 17:18	5
	Сообщение от xAtom PazDim, а кто по-твоему удалять будет объекты GDI после их использования HBRUSH, HPEN, здесь классы CPen, CBrush, обёртки нужно вызывать метод DeleteObject. Вот так напиши ужос...они внутри используют динамическую память? 0

	08.08.2011, 20:27