К примеру надо написать такоe:

Код

HKEY h; ::RegOpenKey( ..., &hkey ); HANDLE hfile = ::CreateFile( ... ) HANDLE hmap = ::CreateFileMapping void *ptr = MapViewOfFile( hmap, ... ) ::RegQueryValueEx( hkey, ...ptr, ... ); ::UnmapViewOfFile( ptr ) ::CloseHandle( hmap ); ::CloseHandle( hfile ) ::RegCloseKey( hkey );

Очевидно, что большинство этих функций могут вернуть ошибку.

Вопрос: как вы пишите код чтобы эти ошибки корректно обработать?

try
{
Твой проблемный код
}
catch (CFileException *e) или другой клас смотри мсдн
{
Обработка ошибки
}

Добавлено @ 11:57
Пользуйтесь поиском и факю

вы используете для каждого объекта (блока памяти, ключа реестра, файла итд) класс - обертку или просто удаляете их после catch (...)?

подчеркну, что мне интересны способы, которые на практике используют участники форума, а не канонические (описанные в msdn). Кстати лично я exception недолюбливаю.

AndyY
Что-тоя не въехал , но есть GetLastError

bel_nikita прав.
Все WinAPI-шные функции сохраняют последнюю ошибку в особой глобальной переменной, которую можно получить с пом. функции GetLastError(). А уже расшифровать смысл значения кода ошибки можно с пом. ряда действий, как именно - см. в нашем FAQ.

GetLastError() хороша, но зачастую в ней меняется значение ошибки, хотя таковая не на что и не повлияла. Это черевато тем, что ошибка вроде бы появилась, пользовотелю мы об этом сообщели, а что может быть хуже вобще прекратили выполнение какой-либо операции, а при детальом анализе окажется, что дело то и выеденного яйца не стоит. Просто пример. Поробуйте помотреть как изменется значение возвращаемой ошибки до и после вызова CreateWindow(...), при этом просто создайте окно(практика показывает, что создание предопределнных классов большей часть проходит успешно).

AndyY, в ответ на твой вопрос, я вот например, написал пару-тройку функций для работы как с GetLastError, так и со всем остальным и заключил все в namespace, а после к ним можно обращаться откуда угодно, могу весь код положить сюда.

Это сообщение отредактировал(а) Royan - 10.5.2004, 19:50

хм, обсуждение на мой взгляд несколько ушло в сторону.
меня интересовал вопрос, кто как справляется с проблемой - что-то внутри сложной функции не получилось сделать - как организовать код, чтобы освободить все занятые ресурсы и при этом код остался легко модифицируемым и читаемым.
диагностика причины облома - тоже безусловно полезная штука, на мой взгляд больше информации чем пара GetLastError/обломившаяся функция с параметрами/стек вызова в общем случае не получить (особенно-удаленно).

AndyY, как вариант - можно все функции, возвращающие NULL/не NULL в случае успеха или наоборот, заключать в макросы типа ASSERT, которые проверяют условие на истинность и вызывают исключение в случае неудачи. Пример (с отключаемыми в Release-версии проверками):

Цитата

class CMyException { public:     char err_msg[1000];     CMyException(const char *msg, int iLine, const char *file_name)     {         sprintf(err_msg, "Ошибка в %d-ой строке файла %s. Текст выражения, вызвавшего ошибку: %s",             iLine, file_name, msg);     } }; #ifdef _DEBUG     #define MY_ASSERT(expr) {if(!(expr)) throw new CMyException(#expr, __LINE__, __FILE__);} #else     #define MY_ASSERT(expr) expr #endif ////////////////////////////////////////////////////////////////////////// // тест int main(int argc, char* argv[]) {     try     {         MY_ASSERT(true); // сработает         MY_ASSERT(false); // вызовет исключение     }     catch (CMyException *ex)     {         printf(ex->err_msg);         delete ex;     }     return 0; }

В качестве практического примера можно заключить в MY_ASSERT вызов функций RegOpenKey, UnmapViewOfFile и пр. -- все они возвращают значение, по которому можно определить, выполнена ли операция успешно; проверив это значение макросом MY_ASSERT, можно выполнить некие действия при возникновении ошибки - закрыть файл, освободить память, выдать сообщение с информацией об ошибке из объекта CMyException *ex (это сообщение содержит номер строки, имя файла и выражение, вызвавшее ошибку -- вполне исчерпывающая информация).

mr.DUDA спасибо, ваш метод понятен.

Значит, ваш код выглядит примерно так:

Код

HANDLE hfile = 0; HANDLE hmap =0; try{ MY_ASSERT(::RegOpenKey( ..., &hkey )!=0); hfile = ::CreateFile( ... ) MY_ASSERT(hfile!=INVALID_HANDLE_VALUE); ... } catch( CMyException *pe ) { ... pe->Delete(); } if( hmap )::CloseHandle( hmap ); if( hfile )::CloseHandle( hfile );

Жду еще мнений.
Если интересно, свой взгляд на проблему я изложил тут