В настоящее время я делаю обзор кода, и следующий код заставил меня прыгнуть. Я вижу несколько проблем с этим кодом. Согласен ли ты со мной? Если да, то как объяснить моему коллеге, что это неправильно (упрямый тип...)?
- Поймать общее исключение (Exception ex)
- Использование "if (ex-something)" вместо наличия другого блока catch
- Мы едим SoapException, HttpException и WebException. Но если веб-служба не удалась, делать нечего.
код:
try
{
// Call to a WebService
}
catch (Exception ex)
{
if (ex is SoapException || ex is HttpException || ex is WebException)
{
// Log Error and eat it.
}
else
{
throw;
}
}