Как решить, где обрабатывать исключение - в рамках функции, вызванной или в глобальной?
Я пишу клиент-серверное приложение для Windows, используя WinSock, и у меня есть класс для сервера. при инициализации сервера у меня есть такой код:
class Server {
static const int MaxClients = 10;
std::vector connections;
CRITICAL_SECTION cs;
int port;
SOCKET ServerSocket;
sockaddr_in ServerAddress;
void init();
public:
Server(int Port);
void addConnection(const Client& newClient);
void closeConnection(int index);
void Listen();
int size();
/*virtual void ClientService();
virtual void SendMsg(const std::string& msg);*/
virtual ~Server();
};
void Server::init() {
WSADATA wsaData;
if (WSAStartup(MAKEWORD(2, 2), &wsaData))
throw Exception("WinSock init failed");
ServerSocket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
if (ServerSocket == INVALID_SOCKET)
throw Exception("Socket failed to create!");
ServerAddress.sin_family = AF_INET;
ServerAddress.sin_addr.s_addr = htonl(INADDR_ANY);
ServerAddress.sin_port = htons(port);
if (bind(ServerSocket,(sockaddr*)&ServerAddress,sizeof(ServerAddress)) == SOCKET_ERROR)
{
throw Exception("Binding failed");
closesocket(ServerSocket);
}
}
где мне нужно обрабатывать исключения? Здесь, в частном методе init () (который вызывается из конструктора) или в основной функции? Есть какие-то правила, где размещать обработчики исключений и от чего это зависит?
Ответы 3
Это зависит от того, действительно ли вы собираетесь исключить ручка, например повторите попытку с немного другим вводом или решите проигнорировать проблему и все равно продолжить (редко бывает целесообразно, но может может быть полезным). В этом случае вы вполне можете захотеть перехватить исключение рядом с его источником.
В большинстве случаев единственное, что вы действительно можете сделать с исключением, - это зарегистрировать его и прервать операцию (например, предоставить пользователю страницу с ошибкой в веб-приложении или открыть дружественное диалоговое окно «Эта программа разбилась, извините» и выйти. ). В этом случае ваш блок catch, скорее всего, будет довольно близко к вершине стека - например, в основном методе.
Основное правило состоит в том, что вы перехватываете исключения только в той части кода, где вы можете фактически ручка. Под обработкой я подразумеваю, что вы можете предпринять некоторые действия, основанные на сгенерированном исключении, например, попытаться создать сокет в другой раз или попытаться использовать другую конфигурацию.
Кроме того, необходимо рассмотреть исключения, которые пересекают границы библиотеки. В зависимости от контекста вы можете захотеть их перехватить и повторно вызвать другой тип исключения, например, чтобы скрыть детали реализации от клиента библиотеки.
Что касается вопроса о перехвате исключения в области видимости функции, в которой оно возникло (т.е. в функции init()), то редко имеет смысл перехватить исключение в области видимости, в которой оно сгенерировано. По сути, это означало бы, что вы обнаружили ошибку и знаете, как ее обрабатывать (и собираетесь ее обрабатывать), но что вы используете исключение, чтобы выполнить не что иное, как передачу управления другой части вашей функции. .
Хотя могут быть случаи, когда это может иметь смысл (хотя это будет запах кода, который указывает на то, что функция достаточно сложна, чтобы ее нужно было реорганизовать), логика для обработки этой ошибки (поскольку обрабатывается является) должна использовать более общий элемент управления. потоковые методы, такие как операторы if или break. Даже крайне непопулярный goto может быть предпочтительнее выдачи исключения только для того, чтобы обработать его в той же функции.
Обратите внимание, что ваш вызов closesocket () сразу после выброса исключения никогда не будет выполнен.