Обработка исключений

@Феанор · Регистрация: 08.09.2011

Добрый день!
Столкнулся с такой проблемой. У меня есть класс

C++

class MyClass
{
public:
     MyClass(/* какие то параметры*/);
     void OnMouseMove(int X, int Y);
     ~MyClass();
private:
     int T;
};
 
MyClass::MyClass(/* какие то параметры*/)
{
     // инициализирую T исходя из параметров, переданных в конструктор
 
     if (/*условие на параметры*/)   // При определенных параметрах
          throw 1;     // Выбрасываю исключение
}
 
void MyClass::OnMouseMove(int X, int Y)
{
     // Тут я использую переменную T
}

Я создаю объект этого класса в функции CreateWindow (в блоке try), и вызываю метод класса OnMouseMove в WndProc (при сообщении WM_MOUSEMOVE).
Если при создании объекта возбуждается исключение, то должен выполняться код из блока catch (как я понимаю), но вместо этого в функции OnMouseMove возникает ошибка, когда я пытаюсь обратиться к переменной T. Я так понимаю, что когда возникает исключение, то объект класса MyClass не создается, и в WndProc я пытаюсь обратиться к несуществующему объекту. Но я не понимаю, почему не выполняется код из блока catch! (кстати, если в catch поставить точку останова, то все прекрасно работает)

@cppGhost · 16.07.2012, 18:02

приложите код блока catch, может не так ловите исключение?

@Феанор · 16.07.2012, 22:28 **[ТС]**

C++

catch(...)
{
     MessageBox(NULL, L"ERROR", L"ERROR", MB_OK | MB_ICONINFORMATION);
     PostQuitMessage(0);
}

Я не обрабатываю исключения. Просто вывожу сообщение об ошибке и пытаюсь закрыть приложение.

Jupiter · 16.07.2012, 22:34

Феанор, без всего кода, друдно что либо сказать

@Феанор · 17.07.2012, 10:31 **[ТС]**

Если проверять при обработке сообщения WM_MOUSEMOVE

C++

1 2	if (p_class) p_class->OnMouseMove(...)

то все хорошо работает.
Однако если пытаться все тело функции WndProc обернуть в try, то он не ловит эту ошибку.

@cppGhost · 17.07.2012, 13:28

Я правда поудалял части OpenGl в приложенном проекте, но у меня catch поймал. Студия у меня 2008. Как вариант, попробуйте сделать тот же механизм в простом проекте. А так все верно и должно работать.

@Феанор · 17.07.2012, 15:01 **[ТС]**

Действительно. Если без OpenGL все прекрасно ловит. Интересно, из-за чего тогда так может быть? И как быть, если мне нужно юзать OpenGL? Каждый раз перед обращением к p_class проверять его на существование...смысл тогда вообще исключений?

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

Я правда поудалял части OpenGl в приложенном проекте, но у меня catch поймал. Студия у меня 2008. Как вариант, попробуйте сделать тот же механизм в простом проекте. А так все верно и должно работать.

а с OpenGL вообще не пробовали запускать?

@Nukutu · 17.07.2012, 15:28

Сообщение от Феанор

Интересно, из-за чего тогда так может быть?

Внимательно почитайте как правильно применять конструкцию try throw catch.
И еще:
The PostQuitMessage function posts a WM_QUIT message to the thread's message queue and returns immediately; the function simply indicates to the system that the thread is requesting to quit at some time in the future.
Т.е на PostQuitMessage программа не будет завершаться, а будет продолжаться до твоего return MainLoop();
И в Init() она после catch доберется до return InitGL();
Вобщем, ошибка на ошибке.

@Феанор · 17.07.2012, 16:01 **[ТС]**

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

The PostQuitMessage function posts a WM_QUIT message to the thread's message queue and returns immediately; the function simply indicates to the system that the thread is requesting to quit at some time in the future.
Т.е на PostQuitMessage программа не будет завершаться, а будет продолжаться до твоего return MainLoop();
И в Init() она после catch доберется до return InitGL();

Да, про PostQuitMessage я понял, спасибо.

Тогда такой код:

C++

int Init()
{
    try
    {
        p_class = new CMyClass();
    }
    catch(...)
    {
        MessageBox(NULL, L"ERROR", L"ERROR", MB_OK | MB_ICONINFORMATION);
        return false;
    }
 
    return InitGL();
}

Разве в этом случае не должно быть уничтожено окно и завершиться работа приложения?

@cppGhost · 17.07.2012, 17:32

С OpenGL я не пробовал, так как никогда с ним дело не имел и что там нужно подключать не в курсе (приложенный проект был MSVS 2010, поэтому я просто создал свой пустой проект и накидал туда файлы). А по поводу верности использования try catch, я ничего неверного не вижу.

@Nukutu · 17.07.2012, 21:09

Феанор,
В этом случае приложение должно постоянно завершаться.

@Феанор · 18.07.2012, 00:16 **[ТС]**

Постоянно? Даже когда нет исключений?
Я еще раз посмотрел про исключения, но так и не понимаю, что не так с try/throw/catch.

И кстати, даже в таком случае приложение не завершается, а возникает та же ошибка в OnMouseMove.

@DU · 18.07.2012, 00:44

WndProc может позватся еще до создания MyClass. В частности она может позваться и с сообщением от мышки. Поэтому краш происходит не из-за бросания исключения и не из-за некорректного отлова этого исключения, а из-за обращения к несозданному объекту. Как это правильно лечить - хз. Самое простое - при обработке проверять, а можно ли обращатся к p_class.

@Nukutu · 18.07.2012, 01:29

Сообщение от Феанор

И кстати, даже в таком случае приложение не завершается, а возникает та же ошибка в OnMouseMove.

Что то ты не так делаешь, у меня твой код работает и приложение завершается без ошибки.
Экземпляр класса действительно не создается, так как конструктор класса до конца не отрабатывается, а прерывается throw.
Применение в даном случае throw и есть не правильное использование конструкции try, throw, catch.

@Феанор · 18.07.2012, 11:25 **[ТС]**

DU, почему то если исключения не возникает, то WndProc не вызывается до создания p_class (по крайне мере, ошибок нет, если нет исключений, и при обработке сообщений мы обращаемся уже к созданному объекту).

Nukutu, может это зависит от настроек среды?

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

Применение в даном случае throw и есть не правильное использование конструкции try, throw, catch.

Я все равно не понимаю. Если я вызываю throw в конструкторе, то память же должна освободиться?

@Nukutu 179 / 105 / 13 Регистрация: 22.03.2010 Сообщений: 456
	18.07.2012, 01:29	14
	Сообщение от Феанор И кстати, даже в таком случае приложение не завершается, а возникает та же ошибка в OnMouseMove. Что то ты не так делаешь, у меня твой код работает и приложение завершается без ошибки. Экземпляр класса действительно не создается, так как конструктор класса до конца не отрабатывается, а прерывается throw. Применение в даном случае throw и есть не правильное использование конструкции try, throw, catch. 0

@cppGhost 32 / 30 / 6 Регистрация: 21.06.2012 Сообщений: 91
	16.07.2012, 18:02	2
	приложите код блока catch, может не так ловите исключение? 0

Jupiter Каратель 6593 / 4014 / 401 Регистрация: 26.03.2010 Сообщений: 9,273 Записей в блоге: 1
	16.07.2012, 22:34	4
	Феанор, без всего кода, друдно что либо сказать 0

@cppGhost 32 / 30 / 6 Регистрация: 21.06.2012 Сообщений: 91
	17.07.2012, 13:28	6
	Я правда поудалял части OpenGl в приложенном проекте, но у меня catch поймал. Студия у меня 2008. Как вариант, попробуйте сделать тот же механизм в простом проекте. А так все верно и должно работать. 0

@Феанор 0 / 0 / 1 Регистрация: 08.09.2011 Сообщений: 30
	17.07.2012, 15:01 [ТС]	7
	Действительно. Если без OpenGL все прекрасно ловит. Интересно, из-за чего тогда так может быть? И как быть, если мне нужно юзать OpenGL? Каждый раз перед обращением к p_class проверять его на существование...смысл тогда вообще исключений? Сообщение от cppGhost Я правда поудалял части OpenGl в приложенном проекте, но у меня catch поймал. Студия у меня 2008. Как вариант, попробуйте сделать тот же механизм в простом проекте. А так все верно и должно работать. а с OpenGL вообще не пробовали запускать? 0

@Nukutu 179 / 105 / 13 Регистрация: 22.03.2010 Сообщений: 456
	17.07.2012, 15:28	8
	Сообщение от Феанор Интересно, из-за чего тогда так может быть? Внимательно почитайте как правильно применять конструкцию try throw catch. И еще: The PostQuitMessage function posts a WM_QUIT message to the thread's message queue and returns immediately; the function simply indicates to the system that the thread is requesting to quit at some time in the future. Т.е на PostQuitMessage программа не будет завершаться, а будет продолжаться до твоего return MainLoop(); И в Init() она после catch доберется до return InitGL(); Вобщем, ошибка на ошибке. 0

@cppGhost 32 / 30 / 6 Регистрация: 21.06.2012 Сообщений: 91
	17.07.2012, 17:32	10
	С OpenGL я не пробовал, так как никогда с ним дело не имел и что там нужно подключать не в курсе (приложенный проект был MSVS 2010, поэтому я просто создал свой пустой проект и накидал туда файлы). А по поводу верности использования try catch, я ничего неверного не вижу. 0

@Nukutu 179 / 105 / 13 Регистрация: 22.03.2010 Сообщений: 456
	17.07.2012, 21:09	11
	Феанор, В этом случае приложение должно постоянно завершаться. 0

@Феанор 0 / 0 / 1 Регистрация: 08.09.2011 Сообщений: 30
	18.07.2012, 00:16 [ТС]	12
	Постоянно? Даже когда нет исключений? Я еще раз посмотрел про исключения, но так и не понимаю, что не так с try/throw/catch. И кстати, даже в таком случае приложение не завершается, а возникает та же ошибка в OnMouseMove. 0

@DU 1492 / 1138 / 165 Регистрация: 05.12.2011 Сообщений: 2,279
	18.07.2012, 00:44	13
	WndProc может позватся еще до создания MyClass. В частности она может позваться и с сообщением от мышки. Поэтому краш происходит не из-за бросания исключения и не из-за некорректного отлова этого исключения, а из-за обращения к несозданному объекту. Как это правильно лечить - хз. Самое простое - при обработке проверять, а можно ли обращатся к p_class. 0

@Феанор 0 / 0 / 1 Регистрация: 08.09.2011 Сообщений: 30
	18.07.2012, 11:25 [ТС]	15
	DU, почему то если исключения не возникает, то WndProc не вызывается до создания p_class (по крайне мере, ошибок нет, если нет исключений, и при обработке сообщений мы обращаемся уже к созданному объекту). Nukutu, может это зависит от настроек среды? Сообщение от Nukutu Применение в даном случае throw и есть не правильное использование конструкции try, throw, catch. Я все равно не понимаю. Если я вызываю throw в конструкторе, то память же должна освободиться? 0

Опции темы